diff options
| author |  Brian Silverman <bsilver16384@gmail.com> | 2015-10-30 15:38:16 +0000 |
|---|---|---|
| committer |  Florian Weikert <fwe@google.com> | 2015-11-02 16:52:45 +0000 |
| commit | 6daaf768577d5cc4fc290403e06ea8ba9a2b9f63 (patch) | |
| tree | 2f65a4e2f9c1b67ce55bbf6520e3371db781b273 /src/main/java/com/google | |
| parent | c27dc1997f4f474f8533440c0ee4f2e19155af0e (diff) | |
Deflake --runs_per_test_detects_flakes.
Previously, if the first run failed (in iteration order, which I don't
think is necessarily execution order) then
--runs_per_test_detects_flakes would report FAILED instead of FLAKY.
--
Change-Id: Ice7889d46203e1598d94a4e3c0bcbe13a45b0fe1
Reviewed-on: https://bazel-review.googlesource.com/#/c/2210/
MOS_MIGRATED_REVID=106693725
Diffstat (limited to 'src/main/java/com/google')
| -rw-r--r-- | src/main/java/com/google/devtools/build/lib/runtime/TestResultAnalyzer.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/runtime/TestResultAnalyzer.java b/src/main/java/com/google/devtools/build/lib/runtime/TestResultAnalyzer.java index 9d1cca6ce9..185df95792 100644 --- a/src/main/java/com/google/devtools/build/lib/runtime/TestResultAnalyzer.java +++ b/src/main/java/com/google/devtools/build/lib/runtime/TestResultAnalyzer.java @@ -217,7 +217,7 @@ public class TestResultAnalyzer { int passes = 0; for (BlazeTestStatus runStatusForShard : singleShardStatuses) { shardStatus = aggregateStatus(shardStatus, runStatusForShard); - if (TestResult.isBlazeTestStatusPassed(shardStatus)) { + if (TestResult.isBlazeTestStatusPassed(runStatusForShard)) { passes++; } } |