Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(34)

Issue 11901003: [Android] Reland 'Keep track of unknown test results at the TestRunner layer.' (Closed)

Created:
7 years, 11 months ago by frankf
Modified:
7 years, 11 months ago
Reviewers:
craigdh
CC:
chromium-reviews, klundberg+watch_chromium.org, frankf+watch_chromium.org, bulach+watch_chromium.org, yfriedman+watch_chromium.org, ilevy+watch_chromium.org
Visibility:
Public.

Description

[Android] Reland 'Keep track of unknown test results at the TestRunner layer.' Original CL https://codereview.chromium.org/11896050/ exposed a seperate issue fixed by https://codereview.chromium.org/12033066/ If the entire gtest run crashes/failes/times out, the tests that were not run are marked as 'unknown'. These tests are reported by GetAllBroken() and are retried by BaseTestSharder. Get rid of overall_* flags which are now redundant. TBR=craigdh@chromium.org BUG=171466 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=178617

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+46 lines, -69 lines) Patch
M build/android/pylib/base/base_test_sharder.py View 4 chunks +10 lines, -7 lines 0 comments Download
M build/android/pylib/base/test_result.py View 7 chunks +11 lines, -33 lines 0 comments Download
M build/android/pylib/gtest/single_test_runner.py View 1 1 chunk +19 lines, -17 lines 0 comments Download
M build/android/pylib/gtest/test_package.py View 5 chunks +1 line, -9 lines 0 comments Download
M build/android/pylib/utils/flakiness_dashboard_results_uploader.py View 1 chunk +4 lines, -1 line 0 comments Download
M build/android/run_tests.py View 2 chunks +1 line, -2 lines 0 comments Download

Powered by Google App Engine
This is Rietveld 408576698