From a3b8e3972379f7e32f7c72c298ee20d8127f9aff Mon Sep 17 00:00:00 2001 From: dslomov Date: Wed, 13 Dec 2017 05:55:07 -0800 Subject: Implement --(no)expand_test_suites flag. --noexpand_test_suites flag will suppress expansion of test_suite targets into constituent tests, so that command-line aspects can analyze test_suite targets. RELNOTES: Added --(no)expand_test_suites flag. PiperOrigin-RevId: 178892829 --- .../lib/skyframe/TargetPatternPhaseKeyTest.java | 33 ++++++++++++++-------- 1 file changed, 21 insertions(+), 12 deletions(-) (limited to 'src/test/java/com/google/devtools/build/lib/skyframe') diff --git a/src/test/java/com/google/devtools/build/lib/skyframe/TargetPatternPhaseKeyTest.java b/src/test/java/com/google/devtools/build/lib/skyframe/TargetPatternPhaseKeyTest.java index 70ba224165..e9243751e7 100644 --- a/src/test/java/com/google/devtools/build/lib/skyframe/TargetPatternPhaseKeyTest.java +++ b/src/test/java/com/google/devtools/build/lib/skyframe/TargetPatternPhaseKeyTest.java @@ -49,47 +49,56 @@ public class TargetPatternPhaseKeyTest { .addEqualityGroup(of(ImmutableList.of("c"), "")) .addEqualityGroup(of(ImmutableList.of(), "")) .addEqualityGroup(of( - ImmutableList.of(), "", ImmutableList.of(), false, null, + ImmutableList.of(), "", ImmutableList.of(), false, true, null, COMPILE_ONE_DEPENDENCY)) .addEqualityGroup(of( - ImmutableList.of(), "", ImmutableList.of(), true, null, + ImmutableList.of(), "", ImmutableList.of(), false, false, null, COMPILE_ONE_DEPENDENCY)) - .addEqualityGroup(of( + .addEqualityGroup(of( + ImmutableList.of(), "", ImmutableList.of(), true, true, null, + COMPILE_ONE_DEPENDENCY)) + .addEqualityGroup(of( + ImmutableList.of(), "", ImmutableList.of(), true, false, null, + COMPILE_ONE_DEPENDENCY)) + .addEqualityGroup(of( ImmutableList.of(), "", ImmutableList.of(), false, - emptyTestFilter(), + true, emptyTestFilter(), BUILD_TESTS_ONLY)) .addEqualityGroup(of( ImmutableList.of(), "", ImmutableList.of(), true, - emptyTestFilter(), + true, emptyTestFilter(), BUILD_TESTS_ONLY)) .addEqualityGroup(of( ImmutableList.of(), "", ImmutableList.of(), false, - emptyTestFilter(), + true, emptyTestFilter(), DETERMINE_TESTS)) .addEqualityGroup(of( ImmutableList.of(), "", ImmutableList.of(), true, - emptyTestFilter(), + true, emptyTestFilter(), DETERMINE_TESTS)) .addEqualityGroup(of( - ImmutableList.of(), "", ImmutableList.of("a"), false, null)) + ImmutableList.of(), "", ImmutableList.of("a"), false, true, + null)) .addEqualityGroup(of( - ImmutableList.of(), "", ImmutableList.of("a"), true, null)) + ImmutableList.of(), "", ImmutableList.of("a"), true, true, + null)) .testEquals(); } private TargetPatternPhaseKey of(ImmutableList targetPatterns, String offset, ImmutableList buildTagFilter, - boolean includeManualTests, @Nullable TestFilter testFilter, Flag... flags) { + boolean includeManualTests, boolean expandTestSuites, @Nullable TestFilter testFilter, + Flag... flags) { ImmutableSet set = ImmutableSet.copyOf(flags); boolean compileOneDependency = set.contains(Flag.COMPILE_ONE_DEPENDENCY); boolean buildTestsOnly = set.contains(Flag.BUILD_TESTS_ONLY); boolean determineTests = set.contains(Flag.DETERMINE_TESTS); return new TargetPatternPhaseKey(targetPatterns, offset, compileOneDependency, buildTestsOnly, - determineTests, buildTagFilter, includeManualTests, testFilter); + determineTests, buildTagFilter, includeManualTests, expandTestSuites, testFilter); } private TargetPatternPhaseKey of(ImmutableList targetPatterns, String offset) { - return of(targetPatterns, offset, ImmutableList.of(), false, null); + return of(targetPatterns, offset, ImmutableList.of(), false, true, null); } private TestFilter emptyTestFilter() { -- cgit v1.2.3