1 files changed, 12 insertions, 0 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/analysis/BuildViewTest.java b/src/test/java/com/google/devtools/build/lib/analysis/BuildViewTest.java
index cd7ec79b0d..ac289bb5ca 100644
--- a/src/test/java/com/google/devtools/build/lib/analysis/BuildViewTest.java
+++ b/src/test/java/com/google/devtools/build/lib/analysis/BuildViewTest.java
@@ -999,6 +999,18 @@ public class BuildViewTest extends BuildViewTestBase {
     assertThat(loadingFailureRecorder.events).isEmpty();
   }
 
+  @Test
+  public void testLoadingErrorReportedCorrectly() throws Exception {
+    scratch.file("a/BUILD", "cc_library(name='a')");
+    scratch.file("b/BUILD", "cc_library(name='b', deps = ['//missing:lib'])");
+
+    reporter.removeHandler(failFastHandler);
+    AnalysisResult result = update(defaultFlags().with(Flag.KEEP_GOING), "//a", "//b");
+    assertThat(result.hasError()).isTrue();
+    assertThat(result.getError())
+        .contains("execution phase succeeded, but there were loading phase errors");
+  }
+
   /** Runs the same test with the reduced loading phase. */
   @TestSpec(size = Suite.SMALL_TESTS)
   @RunWith(JUnit4.class)