Add functionality to make certain SkyValues unshareable, meaning they are not serialized. Tag TestCompletionValue and any ActionExecutionValue coming from a NotifyOnActionCacheHit (i.e., tests) like that. To make such values really not shared, request the ActionExecutionValue from TestCompletionFunction as opposed to the ArtifactValue (propagating the unshareable bit up seemed like too much fuss, and I have a dream of getting rid of ArtifactValue anyway).

PiperOrigin-RevId: 200504358
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 385b738..5b1305c 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
@@ -745,22 +745,24 @@
         throw new IllegalStateException(e);
       }
     }
-    return new ActionExecutionValue(
+    return ActionExecutionValue.create(
         artifactData,
         ImmutableMap.<Artifact, TreeArtifactValue>of(),
         ImmutableMap.<Artifact, FileArtifactValue>of(),
-        /*outputSymlinks=*/ null);
+        /*outputSymlinks=*/ null,
+        /*notifyOnActionCacheHitAction=*/ false);
   }
 
   private ActionExecutionValue actionValueWithEmptyDirectory(Artifact emptyDir) {
     TreeArtifactValue emptyValue = TreeArtifactValue.create
         (ImmutableMap.<TreeFileArtifact, FileArtifactValue>of());
 
-    return new ActionExecutionValue(
+    return ActionExecutionValue.create(
         ImmutableMap.<Artifact, FileValue>of(),
         ImmutableMap.of(emptyDir, emptyValue),
         ImmutableMap.<Artifact, FileArtifactValue>of(),
-        /*outputSymlinks=*/ null);
+        /*outputSymlinks=*/ null,
+        /*notifyOnActionCacheHitAction=*/ false);
   }
 
   private ActionExecutionValue actionValueWithTreeArtifacts(List<TreeFileArtifact> contents) {
@@ -789,11 +791,12 @@
       treeArtifactData.put(dirDatum.getKey(), TreeArtifactValue.create(dirDatum.getValue()));
     }
 
-    return new ActionExecutionValue(
+    return ActionExecutionValue.create(
         fileData,
         treeArtifactData,
         ImmutableMap.<Artifact, FileArtifactValue>of(),
-        /*outputSymlinks=*/ null);
+        /*outputSymlinks=*/ null,
+        /*notifyOnActionCacheHitAction=*/ false);
   }
 
   @Test