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(