diff options
| author |  gregce <gregce@google.com> | 2018-01-18 12:46:04 -0800 |
|---|---|---|
| committer |  Copybara-Service <copybara-piper@google.com> | 2018-01-18 12:48:05 -0800 |
| commit | 7fa23eaf4617431875acbb63052f7349546343ba (patch) | |
| tree | 6b77ab236733751b62c29d92069e4eab0a6dc216 /src/test/java/com/google/devtools/build/lib/packages/ConfigurationFragmentPolicyTest.java | |
| parent | 56567bbb56ab2f0d1423121abea4af2373acc8aa (diff) | |
Remove ConfigurationTransitionProxy.NONE and BaseRuleClasses.DYNAMIC_TRANSITION_MAP.
This leaves DATA as the last remaining legacy transition.
PiperOrigin-RevId: 182422552
Diffstat (limited to 'src/test/java/com/google/devtools/build/lib/packages/ConfigurationFragmentPolicyTest.java')
| -rw-r--r-- | src/test/java/com/google/devtools/build/lib/packages/ConfigurationFragmentPolicyTest.java | 17 |
1 files changed, 7 insertions, 10 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/packages/ConfigurationFragmentPolicyTest.java b/src/test/java/com/google/devtools/build/lib/packages/ConfigurationFragmentPolicyTest.java index 0e7de91a7e..7324889e46 100644 --- a/src/test/java/com/google/devtools/build/lib/packages/ConfigurationFragmentPolicyTest.java +++ b/src/test/java/com/google/devtools/build/lib/packages/ConfigurationFragmentPolicyTest.java @@ -16,7 +16,7 @@ package com.google.devtools.build.lib.packages; import static com.google.common.truth.Truth.assertThat; import com.google.common.collect.ImmutableSet; -import com.google.devtools.build.lib.analysis.config.transitions.ConfigurationTransitionProxy; +import com.google.devtools.build.lib.analysis.config.transitions.NoTransition; import com.google.devtools.build.lib.analysis.config.transitions.Transition; import com.google.devtools.build.lib.packages.ConfigurationFragmentPolicy.MissingFragmentPolicy; import com.google.devtools.build.lib.skylarkinterface.SkylarkModule; @@ -92,7 +92,7 @@ public final class ConfigurationFragmentPolicyTest { assertThat(policy.isLegalConfigurationFragment(Integer.class)).isTrue(); assertThat( - policy.isLegalConfigurationFragment(Integer.class, ConfigurationTransitionProxy.NONE)) + policy.isLegalConfigurationFragment(Integer.class, NoTransition.INSTANCE)) .isTrue(); // TODO(mstaib): .isFalse() when dynamic configurations care which configuration a fragment was // specified for @@ -102,13 +102,13 @@ public final class ConfigurationFragmentPolicyTest { assertThat(policy.isLegalConfigurationFragment(Long.class)).isTrue(); // TODO(mstaib): .isFalse() when dynamic configurations care which configuration a fragment was // specified for - assertThat(policy.isLegalConfigurationFragment(Long.class, ConfigurationTransitionProxy.NONE)) + assertThat(policy.isLegalConfigurationFragment(Long.class, NoTransition.INSTANCE)) .isTrue(); assertThat(policy.isLegalConfigurationFragment(Long.class, TEST_HOST_TRANSITION)) .isTrue(); assertThat(policy.isLegalConfigurationFragment(String.class)).isFalse(); - assertThat(policy.isLegalConfigurationFragment(String.class, ConfigurationTransitionProxy.NONE)) + assertThat(policy.isLegalConfigurationFragment(String.class, NoTransition.INSTANCE)) .isFalse(); assertThat(policy.isLegalConfigurationFragment(String.class, TEST_HOST_TRANSITION)) .isFalse(); @@ -128,8 +128,7 @@ public final class ConfigurationFragmentPolicyTest { assertThat(policy.isLegalConfigurationFragment(TestFragment.class)).isTrue(); assertThat( - policy.isLegalConfigurationFragment(TestFragment.class, - ConfigurationTransitionProxy.NONE)) + policy.isLegalConfigurationFragment(TestFragment.class, NoTransition.INSTANCE)) .isTrue(); assertThat( policy.isLegalConfigurationFragment(TestFragment.class, TEST_HOST_TRANSITION)) @@ -137,8 +136,7 @@ public final class ConfigurationFragmentPolicyTest { assertThat(policy.isLegalConfigurationFragment(OtherFragment.class)).isTrue(); assertThat( - policy.isLegalConfigurationFragment(OtherFragment.class, - ConfigurationTransitionProxy.NONE)) + policy.isLegalConfigurationFragment(OtherFragment.class, NoTransition.INSTANCE)) .isFalse(); assertThat( policy.isLegalConfigurationFragment(OtherFragment.class, TEST_HOST_TRANSITION)) @@ -146,8 +144,7 @@ public final class ConfigurationFragmentPolicyTest { assertThat(policy.isLegalConfigurationFragment(UnknownFragment.class)).isFalse(); assertThat( - policy.isLegalConfigurationFragment( - UnknownFragment.class, ConfigurationTransitionProxy.NONE)) + policy.isLegalConfigurationFragment(UnknownFragment.class, NoTransition.INSTANCE)) .isFalse(); assertThat( policy.isLegalConfigurationFragment( |