Always create a `CrossServerUnshareableActionExecutionValue` if the `ActionLookupData` key is marked as never shareable.
It does not make sense for an unshareable key to produce a shareable value. I will enforce this in an upcoming change.
PiperOrigin-RevId: 416328205
diff --git a/src/test/java/com/google/devtools/build/lib/skyframe/FilesystemValueCheckerTest.java b/src/test/java/com/google/devtools/build/lib/skyframe/FilesystemValueCheckerTest.java
index fb7fc00..856a10c 100644
--- a/src/test/java/com/google/devtools/build/lib/skyframe/FilesystemValueCheckerTest.java
+++ b/src/test/java/com/google/devtools/build/lib/skyframe/FilesystemValueCheckerTest.java
@@ -938,11 +938,11 @@
}
}
return ActionExecutionValue.create(
- artifactData,
+ ImmutableMap.copyOf(artifactData),
/*treeArtifactData=*/ ImmutableMap.of(),
/*outputSymlinks=*/ null,
/*discoveredModules=*/ null,
- /*actionDependsOnBuildId=*/ false);
+ /*shareable=*/ true);
}
private static ActionExecutionValue actionValueWithTreeArtifact(
@@ -952,7 +952,7 @@
ImmutableMap.of(output, tree),
/*outputSymlinks=*/ null,
/*discoveredModules=*/ null,
- /*actionDependsOnBuildId=*/ false);
+ /*shareable=*/ true);
}
private static ActionExecutionValue actionValueWithRemoteArtifact(