Allow the EventHandler returned by ActionExecutionContext to be different from the global one in BlazeExecutor. Instead, it can be one from SkyframeActionExecutor that suppresses ProgressLike messages. This allows an action that was rewound to execute without emitting any progress-like events.
Deprecate ActionExecutionContext#getEventBus: posts should go over the handler, allowing them to be easily filtered.
Also do this for ActionCachedContext, although I suspect that most (not all) of the uses of ActionCachedContext could be dealt with by using Skyframe-level caching, rather than a NotifyOnActionCacheHit action.
PiperOrigin-RevId: 225861266
diff --git a/src/test/java/com/google/devtools/build/lib/exec/StandaloneTestStrategyTest.java b/src/test/java/com/google/devtools/build/lib/exec/StandaloneTestStrategyTest.java
index e643331..b499f26 100644
--- a/src/test/java/com/google/devtools/build/lib/exec/StandaloneTestStrategyTest.java
+++ b/src/test/java/com/google/devtools/build/lib/exec/StandaloneTestStrategyTest.java
@@ -93,6 +93,7 @@
new ActionKeyContext(),
/* metadataHandler= */ null,
fileOutErr,
+ /*eventHandler=*/ null,
/* clientEnv= */ ImmutableMap.of(),
/* topLevelFilesets= */ ImmutableMap.of(),
/* artifactExpander= */ null,
