Reinstate previous behavior of notifying the graph listener of `GET_B…

…ATCH` for every `DEP_REQUESTED` call.

Recent optimizations in 62d1b0b and 243c2b7 mean that deps can now be requested via `get()` instead of `getBatch()`. It is possible that existing regression tests are no longer WAI because they are not getting the `GET_BATCH` event (i.e. they are passing but not testing what they intend to).

Restoring the previous behavior is much easier than hunting down every test that is affected.

PiperOrigin-RevId: 479416036
Change-Id: Iea3ac0a2ad02313dc8e61084fa5f585763a5967f

src/test/java/com/google/devtools/build/skyframe/NotifyingHelper.java

@@ -80,6 +80,12 @@ static class NotifyingProcessableGraph implements ProcessableGraph {
  @Override
  public NodeEntry get(@Nullable SkyKey requestor, Reason reason, SkyKey key)
  throws InterruptedException {
+ // Maintains behavior for tests written when all DEP_REQUESTED calls were made as batch
+ // requests. Now there are optimizations in SkyFunctionEnvironment for looking up deps
+ // individually, but older tests may be written to listen for a GET_BATCH event.
+ if (reason == Reason.DEP_REQUESTED) {
+ notifyingHelper.graphListener.accept(key, EventType.GET_BATCH, Order.BEFORE, reason);
+ }
  return notifyingHelper.wrapEntry(key, delegate.get(requestor, reason, key));
  }