Post PatternExpandingError from the skyframe target pattern evaluator as needed.

This fixes https://github.com/bazelbuild/bazel/issues/4731.

Also, add test coverage for posting of ParsingFailedEvent and handling of cycles in LoadingPhaseRunnerTest.

Change-Id: I88c9d33417b9c3c7a06c92a6137d58f37b991b0c
PiperOrigin-RevId: 188138972

1 files changed, 8 insertions, 0 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/PatternExpandingError.java b/src/main/java/com/google/devtools/build/lib/skyframe/PatternExpandingError.java
index 1d94871501..f24f095ef2 100644
--- a/src/main/java/com/google/devtools/build/lib/skyframe/PatternExpandingError.java
+++ b/src/main/java/com/google/devtools/build/lib/skyframe/PatternExpandingError.java
@@ -35,6 +35,14 @@ public final class PatternExpandingError implements BuildEvent {
     this.skipped = skipped;
   }
 
+  public List<String> getPattern() {
+    return pattern;
+  }
+
+  public boolean getSkipped() {
+    return skipped;
+  }
+
   public static PatternExpandingError failed(List<String> pattern, String message) {
     return new PatternExpandingError(pattern, message, false);
   }