Fix local execution to not throw an exception

The SpawnInputExpander returns null for empty files, but the
ActionInputPrefetcher does not expect null values, and implementations may
throw NPE.

PiperOrigin-RevId: 158960425

1 files changed, 31 insertions, 0 deletions

diff --git a/src/test/java/com/google/devtools/build/lib/exec/local/LocalSpawnRunnerTest.java b/src/test/java/com/google/devtools/build/lib/exec/local/LocalSpawnRunnerTest.java
index 29e46e9ba4..d2370b819f 100644
--- a/src/test/java/com/google/devtools/build/lib/exec/local/LocalSpawnRunnerTest.java
+++ b/src/test/java/com/google/devtools/build/lib/exec/local/LocalSpawnRunnerTest.java
@@ -27,6 +27,7 @@ import com.google.common.collect.ImmutableList;
 import com.google.common.io.ByteStreams;
 import com.google.devtools.build.lib.actions.ActionInput;
 import com.google.devtools.build.lib.actions.ActionInputFileCache;
+import com.google.devtools.build.lib.actions.ActionInputHelper;
 import com.google.devtools.build.lib.actions.ExecutionRequirements;
 import com.google.devtools.build.lib.actions.ResourceManager;
 import com.google.devtools.build.lib.actions.ResourceSet;
@@ -428,6 +429,36 @@ public class LocalSpawnRunnerTest {
     runner.exec(spawn, policy);
   }
 
+  /**
+   * Regression test: the SpawnInputExpander can return null values for empty files, but the
+   * ActionInputPrefetcher expects no null values.
+   */
+  @SuppressWarnings("unchecked")
+  @Test
+  public void checkPrefetchCalledNonNull() throws Exception {
+    Subprocess.Factory factory = mock(Subprocess.Factory.class);
+    when(factory.create(any())).thenReturn(new FinishedSubprocess(0));
+    SubprocessBuilder.setSubprocessFactory(factory);
+    ActionInputPrefetcher mockPrefetcher = mock(ActionInputPrefetcher.class);
+    @SuppressWarnings("rawtypes")
+    ArgumentCaptor<Iterable> captor = ArgumentCaptor.forClass(Iterable.class);
+
+    LocalExecutionOptions options = Options.getDefaults(LocalExecutionOptions.class);
+    LocalSpawnRunner runner = new LocalSpawnRunner(
+        logger, execCount, fs.getPath("/execroot"), mockPrefetcher, options, resourceManager,
+        USE_WRAPPER, "product-name", LocalEnvProvider.UNMODIFIED);
+
+    policy.inputMapping.put(PathFragment.create("relative/path"), null);
+    policy.inputMapping.put(
+        PathFragment.create("another/relative/path"), ActionInputHelper.fromPath("/absolute/path"));
+    timeoutMillis = 123 * 1000L;
+    outErr = new FileOutErr(fs.getPath("/out/stdout"), fs.getPath("/out/stderr"));
+    runner.exec(SIMPLE_SPAWN, policy);
+    verify(mockPrefetcher).prefetchFiles(captor.capture());
+    assertThat(captor.getValue()).doesNotContain(null);
+    assertThat(captor.getValue()).containsExactly(ActionInputHelper.fromPath("/absolute/path"));
+  }
+
   @Test
   public void checkLocalEnvProviderCalled() throws Exception {
     Subprocess.Factory factory = mock(Subprocess.Factory.class);