Skip to content

Commit 16427c9

Browse files
fmeumcopybara-github
authored andcommitted
Do not count tests as failed that have not started
Before this commit, when the user interrupted a test run, tests that had not started until that point were counted as failed, whereas tests that were already running were counted as skipped. With this commmit, both types of tests are counted as skipped. Closes bazelbuild#17160. PiperOrigin-RevId: 501495265 Change-Id: I4dd157c32d70eb46a02070251684899b84c9802e
1 parent f0ed5ca commit 16427c9

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

src/main/java/com/google/devtools/build/lib/runtime/TerminalTestResultNotifier.java

+2-1
Original file line numberDiff line numberDiff line change
@@ -216,7 +216,8 @@ public void notify(Set<TestSummary> summaries, int numberOfExecutedTargets) {
216216
for (TestSummary summary : summaries) {
217217
if (TestResult.isBlazeTestStatusPassed(summary.getStatus())) {
218218
stats.passCount++;
219-
} else if (summary.getStatus() == BlazeTestStatus.NO_STATUS) {
219+
} else if (summary.getStatus() == BlazeTestStatus.NO_STATUS
220+
|| summary.getStatus() == BlazeTestStatus.BLAZE_HALTED_BEFORE_TESTING) {
220221
stats.noStatusCount++;
221222
} else if (summary.getStatus() == BlazeTestStatus.FAILED_TO_BUILD) {
222223
stats.failedToBuildCount++;

0 commit comments

Comments
 (0)