Add experimental flag, that partially disables loading phase (pattern evaluation, test_suite expansion and configuration creation is still there). Also remove some unused code.

--
MOS_MIGRATED_REVID=103177839

3 files changed, 28 insertions, 6 deletions

diff --git a/src/test/java/com/google/devtools/build/lib/analysis/AspectTest.java b/src/test/java/com/google/devtools/build/lib/analysis/AspectTest.java
index b051db1a8c..bb9bf229f4 100644
--- a/src/test/java/com/google/devtools/build/lib/analysis/AspectTest.java
+++ b/src/test/java/com/google/devtools/build/lib/analysis/AspectTest.java
@@ -174,4 +174,14 @@ public class AspectTest extends AnalysisTestCase {
     assertThat(a.getProvider(TestAspects.RuleInfo.class).getData())
         .containsExactly("rule //a:a", "aspect //a:b data hello");
   }
+
+  @RunWith(JUnit4.class)
+  public static class AspectTestWithoutLoading extends AspectTest {
+    @Override
+    @Before
+    public void setUp() throws Exception {
+      disableLoading();
+      super.setUp();
+    }
+  }
 }
diff --git a/src/test/java/com/google/devtools/build/lib/analysis/util/AnalysisTestCase.java b/src/test/java/com/google/devtools/build/lib/analysis/util/AnalysisTestCase.java
index 4e6e041f69..6feb7bc933 100644
--- a/src/test/java/com/google/devtools/build/lib/analysis/util/AnalysisTestCase.java
+++ b/src/test/java/com/google/devtools/build/lib/analysis/util/AnalysisTestCase.java
@@ -126,6 +126,7 @@ public abstract class AnalysisTestCase extends FoundationTestCase {
 
   protected AnalysisTestUtil.DummyWorkspaceStatusActionFactory workspaceStatusActionFactory;
   private PathPackageLocator pkgLocator;
+  protected boolean enableLoading = true;
 
   @Override
   protected void setUp() throws Exception {
@@ -256,8 +257,8 @@ public abstract class AnalysisTestCase extends FoundationTestCase {
 
     LoadingResult loadingResult = loadingPhaseRunner
         .execute(reporter, eventBus, ImmutableList.copyOf(labels), loadingOptions,
-            buildOptions.getAllLabels(), viewOptions.keepGoing, /*determineTests=*/false,
-            /*callback=*/null);
+            buildOptions.getAllLabels(), viewOptions.keepGoing, enableLoading,
+            /*determineTests=*/false, /*callback=*/null);
 
     BuildRequestOptions requestOptions = optionsParser.getOptions(BuildRequestOptions.class);
     ImmutableSortedSet<String> multiCpu = ImmutableSortedSet.copyOf(requestOptions.multiCpus);
@@ -271,7 +272,8 @@ public abstract class AnalysisTestCase extends FoundationTestCase {
             viewOptions,
             AnalysisTestUtil.TOP_LEVEL_ARTIFACT_CONTEXT,
             reporter,
-            eventBus);
+            eventBus,
+            enableLoading);
   }
 
   protected void update(FlagBuilder config, String... labels) throws Exception {
@@ -363,4 +365,8 @@ public abstract class AnalysisTestCase extends FoundationTestCase {
   protected void clearAnalysisResult() {
     analysisResult = null;
   }
+
+  protected void disableLoading() {
+    enableLoading = false;
+  }
 }
diff --git a/src/test/java/com/google/devtools/build/lib/analysis/util/BuildViewTestCase.java b/src/test/java/com/google/devtools/build/lib/analysis/util/BuildViewTestCase.java
index ac9311d793..396f415230 100644
--- a/src/test/java/com/google/devtools/build/lib/analysis/util/BuildViewTestCase.java
+++ b/src/test/java/com/google/devtools/build/lib/analysis/util/BuildViewTestCase.java
@@ -173,6 +173,7 @@ public abstract class BuildViewTestCase extends FoundationTestCase {
   protected WorkspaceStatusAction.Factory workspaceStatusActionFactory;
 
   private MutableActionGraph mutableActionGraph;
+  protected boolean enableLoading = true;
 
   @Override
   protected void setUp() throws Exception {
@@ -1300,8 +1301,8 @@ public abstract class BuildViewTestCase extends FoundationTestCase {
     LoadingPhaseRunner runner = new LoadingPhaseRunner(getPackageManager(),
         Collections.unmodifiableSet(ruleClassProvider.getRuleClassMap().keySet()));
     LoadingResult loadingResult = runner.execute(reporter, eventBus, targets, loadingOptions,
-        getTargetConfiguration().getAllLabels(),
-        viewOptions.keepGoing, /*determineTests=*/false, /*callback=*/null);
+        getTargetConfiguration().getAllLabels(), viewOptions.keepGoing,
+        enableLoading, /*determineTests=*/false, /*callback=*/null);
     if (!doAnalysis) {
       // TODO(bazel-team): What's supposed to happen in this case?
       return null;
@@ -1313,7 +1314,8 @@ public abstract class BuildViewTestCase extends FoundationTestCase {
         viewOptions,
         AnalysisTestUtil.TOP_LEVEL_ARTIFACT_CONTEXT,
         reporter,
-        eventBus);
+        eventBus,
+        enableLoading);
   }
 
   protected static Predicate<Artifact> artifactNamed(final String name) {
@@ -1605,4 +1607,8 @@ public abstract class BuildViewTestCase extends FoundationTestCase {
 
     return result.build();
   }
+
+  protected void disableLoading() {
+    enableLoading = false;
+  }
 }