diff options
author | 2017-08-17 23:23:47 +0200 | |
---|---|---|
committer | 2017-08-18 09:02:10 +0200 | |
commit | 5bae50aa8479096a2038cdf7b3a4cf744a84de6a (patch) | |
tree | f8c9932132c0fbfdcf80639230290f2e4d512a15 /src/test/java/com/google/devtools/build/lib/skyframe/ConfigurationsForTargetsTest.java | |
parent | 03732205970679f757e6bb82f98c15607db17aa4 (diff) |
Remove BuildConfiguration.useDynamicConfigurations.
This is always true.
Part of the static config cleanup effort.
PiperOrigin-RevId: 165628823
Diffstat (limited to 'src/test/java/com/google/devtools/build/lib/skyframe/ConfigurationsForTargetsTest.java')
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/skyframe/ConfigurationsForTargetsTest.java | 20 |
1 files changed, 6 insertions, 14 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/skyframe/ConfigurationsForTargetsTest.java b/src/test/java/com/google/devtools/build/lib/skyframe/ConfigurationsForTargetsTest.java index 46c1cf99ca..0ea433908a 100644 --- a/src/test/java/com/google/devtools/build/lib/skyframe/ConfigurationsForTargetsTest.java +++ b/src/test/java/com/google/devtools/build/lib/skyframe/ConfigurationsForTargetsTest.java @@ -315,20 +315,12 @@ public class ConfigurationsForTargetsTest extends AnalysisTestCase { dep1.getConfiguration().getCpu(), dep2.getConfiguration().getCpu())) .containsExactly("armeabi-v7a", "k8"); - // We don't care what order split deps are listed, but it must be deterministic. Static and - // dynamic configurations happen to apply different orders (static: same order as the split - // transition definition, dynamic: ConfiguredTargetFunction.DYNAMIC_SPLIT_DEP_ORDERING). - // That's okay because of the "we don't care what order" principle. The primary value of this - // test is to check against the new dynamic code, which will soon replace the static code - // anyway. And the static code is already well-tested through all other Blaze tests. And - // checking its order would be a lot uglier. So we only worry about the dynamic case here. - if (getTargetConfiguration().useDynamicConfigurations()) { - assertThat( - ConfiguredTargetFunction.DYNAMIC_SPLIT_DEP_ORDERING.compare( - Dependency.withConfiguration(dep1.getLabel(), dep1.getConfiguration()), - Dependency.withConfiguration(dep2.getLabel(), dep2.getConfiguration()))) - .isLessThan(0); - } + // We don't care what order split deps are listed, but it must be deterministic. + assertThat( + ConfiguredTargetFunction.DYNAMIC_SPLIT_DEP_ORDERING.compare( + Dependency.withConfiguration(dep1.getLabel(), dep1.getConfiguration()), + Dependency.withConfiguration(dep2.getLabel(), dep2.getConfiguration()))) + .isLessThan(0); } } } |