From 6f9d7d1c05a0dbb6db6bf058d01f7a05a14b4d91 Mon Sep 17 00:00:00 2001 From: Janak Ramakrishnan Date: Tue, 9 Aug 2016 08:45:34 +0000 Subject: Error out if loading phase error encountered even if we skip analysis. Also change error messages in these cases to not assume there was an execution phase. -- MOS_MIGRATED_REVID=129723717 --- src/test/java/com/google/devtools/build/lib/analysis/BuildViewTest.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/test/java/com/google') 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 8d45c93362..b1539f8766 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 @@ -1022,7 +1022,7 @@ public class BuildViewTest extends BuildViewTestBase { 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"); + .contains("command succeeded, but there were loading phase errors"); } @Test -- cgit v1.2.3