diff options
author | 2017-11-30 05:03:24 -0800 | |
---|---|---|
committer | 2017-11-30 05:05:43 -0800 | |
commit | 3549d761b3ed2120510b316a98fb0a9779feed12 (patch) | |
tree | 6fc09d6b4cdd44d72bdc0316ef05cb43139eaf5e /src/test/java/com/google/devtools/build | |
parent | bdf8354790a23884c8e888c6b158d73f8d57e461 (diff) |
remote: don't hide non-test failures behind test failures. Fixes #4082
Bazel should display the root cause of a test failure to the user. For
example, if a test could not be executed on a remote executor due to
there being no network connection, then it shouldn't display the test
as failed but tell the user about the network error.
RELNOTES:
PiperOrigin-RevId: 177439578
Diffstat (limited to 'src/test/java/com/google/devtools/build')
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/exec/StandaloneTestStrategyTest.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/exec/StandaloneTestStrategyTest.java b/src/test/java/com/google/devtools/build/lib/exec/StandaloneTestStrategyTest.java index 04ef2f1f20..c429e13b64 100644 --- a/src/test/java/com/google/devtools/build/lib/exec/StandaloneTestStrategyTest.java +++ b/src/test/java/com/google/devtools/build/lib/exec/StandaloneTestStrategyTest.java @@ -194,7 +194,7 @@ public final class StandaloneTestStrategyTest extends BuildViewTestCase { when(actionExecutionContext.getFileOutErr()).thenReturn(outErr); SpawnResult expectedSpawnResult = - new SpawnResult.Builder().setStatus(Status.EXECUTION_FAILED).setExitCode(1).build(); + new SpawnResult.Builder().setStatus(Status.NON_ZERO_EXIT).setExitCode(1).build(); when(spawnActionContext.exec(any(), any())) .thenThrow( new SpawnExecException( |