From b46de754aba0f24d67cd9c882f8a82428915fae5 Mon Sep 17 00:00:00 2001 From: Chi Wang Date: Thu, 2 Dec 2021 16:33:54 +0100 Subject: [PATCH] Automated rollback of commit d84f7998ef8f15e27376a0c8f25b320145c4ba9e. (#14358) *** Reason for rollback *** Breaks some targets. PiperOrigin-RevId: 412968486 Co-authored-by: dacek --- .../devtools/build/lib/remote/RemoteSpawnCacheTest.java | 2 +- .../devtools/build/lib/remote/RemoteSpawnRunnerTest.java | 7 ------- 2 files changed, 1 insertion(+), 8 deletions(-) diff --git a/src/test/java/com/google/devtools/build/lib/remote/RemoteSpawnCacheTest.java b/src/test/java/com/google/devtools/build/lib/remote/RemoteSpawnCacheTest.java index 61ce8de1bd9734..4171d01fe5b763 100644 --- a/src/test/java/com/google/devtools/build/lib/remote/RemoteSpawnCacheTest.java +++ b/src/test/java/com/google/devtools/build/lib/remote/RemoteSpawnCacheTest.java @@ -143,7 +143,7 @@ public MetadataProvider getMetadataProvider() { @Override public ArtifactExpander getArtifactExpander() { - return SIMPLE_ARTIFACT_EXPANDER; + throw new UnsupportedOperationException(); } @Override diff --git a/src/test/java/com/google/devtools/build/lib/remote/RemoteSpawnRunnerTest.java b/src/test/java/com/google/devtools/build/lib/remote/RemoteSpawnRunnerTest.java index 81bbc05a481c43..68fce99846b68b 100644 --- a/src/test/java/com/google/devtools/build/lib/remote/RemoteSpawnRunnerTest.java +++ b/src/test/java/com/google/devtools/build/lib/remote/RemoteSpawnRunnerTest.java @@ -74,7 +74,6 @@ import com.google.devtools.build.lib.exec.RemoteLocalFallbackRegistry; import com.google.devtools.build.lib.exec.SpawnCheckingCacheEvent; import com.google.devtools.build.lib.exec.SpawnExecutingEvent; -import com.google.devtools.build.lib.exec.SpawnInputExpander; import com.google.devtools.build.lib.exec.SpawnRunner; import com.google.devtools.build.lib.exec.SpawnRunner.SpawnExecutionContext; import com.google.devtools.build.lib.exec.SpawnSchedulingEvent; @@ -1268,7 +1267,6 @@ public void shouldReportCheckingCacheBeforeScheduling() throws Exception { Spawn spawn = newSimpleSpawn(); SpawnExecutionContext policy = mock(SpawnExecutionContext.class); - when(policy.getSpawnInputExpander()).thenReturn(mock(SpawnInputExpander.class)); when(policy.getTimeout()).thenReturn(Duration.ZERO); when(executor.executeRemotely( @@ -1312,7 +1310,6 @@ public void shouldReportExecutingStatusWithoutMetadata() throws Exception { Spawn spawn = newSimpleSpawn(); SpawnExecutionContext policy = mock(SpawnExecutionContext.class); - when(policy.getSpawnInputExpander()).thenReturn(mock(SpawnInputExpander.class)); when(policy.getTimeout()).thenReturn(Duration.ZERO); when(executor.executeRemotely( @@ -1356,7 +1353,6 @@ public void shouldReportExecutingStatusAfterGotExecutingStageFromMetadata() thro Spawn spawn = newSimpleSpawn(); SpawnExecutionContext policy = mock(SpawnExecutionContext.class); - when(policy.getSpawnInputExpander()).thenReturn(mock(SpawnInputExpander.class)); when(policy.getTimeout()).thenReturn(Duration.ZERO); when(executor.executeRemotely( @@ -1417,7 +1413,6 @@ public void shouldIgnoreInvalidMetadata() throws Exception { Spawn spawn = newSimpleSpawn(); SpawnExecutionContext policy = mock(SpawnExecutionContext.class); - when(policy.getSpawnInputExpander()).thenReturn(mock(SpawnInputExpander.class)); when(policy.getTimeout()).thenReturn(Duration.ZERO); when(executor.executeRemotely( @@ -1466,7 +1461,6 @@ public void shouldReportExecutingStatusIfNoExecutingStatusFromMetadata() throws Spawn spawn = newSimpleSpawn(); SpawnExecutionContext policy = mock(SpawnExecutionContext.class); - when(policy.getSpawnInputExpander()).thenReturn(mock(SpawnInputExpander.class)); when(policy.getTimeout()).thenReturn(Duration.ZERO); when(executor.executeRemotely( @@ -1516,7 +1510,6 @@ public void shouldReportExecutingStatusEvenNoOperationFromServer() throws Except Spawn spawn = newSimpleSpawn(); SpawnExecutionContext policy = mock(SpawnExecutionContext.class); - when(policy.getSpawnInputExpander()).thenReturn(mock(SpawnInputExpander.class)); when(policy.getTimeout()).thenReturn(Duration.ZERO); when(executor.executeRemotely(