diff options
| author |  vladmos <vladmos@google.com> | 2017-06-30 14:01:45 +0200 |
|---|---|---|
| committer |  Marcel Hlopko <hlopko@google.com> | 2017-07-03 09:04:59 +0200 |
| commit | 469079377a9561a7c2cc7a46492c44e012b9fb33 (patch) | |
| tree | 7b9baaf83c2a9ee2c9a57862d833ae72d5ae9305 /src/test/java/com/google/devtools/build/lib/rules/config | |
| parent | 86c9d942452d82a479d499ffe61695a983f16bba (diff) | |
Refactor Printer
It's now easier to customize Printer if in different situations objects should
be printed differently. Also its API is cleaner now. Names of methods of SkylarkValue objects now reflect names of Skylark functions: SkylarkValue#repr and SkylarkPrintableValue#str.
PiperOrigin-RevId: 160635154
Diffstat (limited to 'src/test/java/com/google/devtools/build/lib/rules/config')
| -rw-r--r-- | src/test/java/com/google/devtools/build/lib/rules/config/ConfigFeatureFlagTest.java | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/rules/config/ConfigFeatureFlagTest.java b/src/test/java/com/google/devtools/build/lib/rules/config/ConfigFeatureFlagTest.java index b649872487..0d87489daf 100644 --- a/src/test/java/com/google/devtools/build/lib/rules/config/ConfigFeatureFlagTest.java +++ b/src/test/java/com/google/devtools/build/lib/rules/config/ConfigFeatureFlagTest.java @@ -237,7 +237,7 @@ public final class ConfigFeatureFlagTest extends SkylarkTestCase { assertThat(getConfiguredTarget("//test:flag")).isNull(); assertContainsEvent( "in allowed_values attribute of config_feature_flag rule //test:flag: " - + "cannot contain duplicates, but contained multiple of ['double']"); + + "cannot contain duplicates, but contained multiple of [\"double\"]"); } @Test @@ -260,7 +260,7 @@ public final class ConfigFeatureFlagTest extends SkylarkTestCase { assertThat(getConfiguredTarget("//test:top")).isNull(); assertContainsEvent( "in default_value attribute of config_feature_flag rule //test:flag: " - + "must be one of ['eagle', 'legal'], but was 'beagle'"); + + "must be one of [\"eagle\", \"legal\"], but was \"beagle\""); } @Test @@ -284,7 +284,7 @@ public final class ConfigFeatureFlagTest extends SkylarkTestCase { // TODO(mstaib): when configurationError is implemented, switch to testing for that assertContainsEvent( "in config_feature_flag rule //test:flag: " - + "value must be one of ['configured', 'default', 'other'], but was 'invalid'"); + + "value must be one of [\"configured\", \"default\", \"other\"], but was \"invalid\""); } @Test |