Fix callback handling in the Skyframe-based target pattern eval.

Add a unit test - there is test coverage for this in RunCommandTest, but this
runs much faster.

--
MOS_MIGRATED_REVID=114302062

1 files changed, 3 insertions, 0 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeExecutor.java b/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeExecutor.java
index 291730adba..12c38a8cba 100644
--- a/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeExecutor.java
+++ b/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeExecutor.java
@@ -1789,6 +1789,9 @@ public abstract class SkyframeExecutor implements WalkableGraphFactory {
       eventBus.post(new TargetParsingCompleteEvent(patternParsingValue.getTargets(),
           patternParsingValue.getFilteredTargets(), patternParsingValue.getTestFilteredTargets(),
           time));
+      if (callback != null) {
+        callback.notifyTargets(patternParsingValue.getTargets());
+      }
       eventBus.post(new LoadingPhaseCompleteEvent(
           /*was expandedTargetsToLoad*/patternParsingValue.getTargets(),
           // TODO(ulfjack): Should be: Sets.difference(originalTargetsToLoad, expandedTargetsToLoad)