diff options
Diffstat (limited to 'src/test')
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/skylark/SkylarkStringRepresentationsTest.java | 11 | ||||
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/syntax/EvaluationTest.java | 19 |
2 files changed, 13 insertions, 17 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/skylark/SkylarkStringRepresentationsTest.java b/src/test/java/com/google/devtools/build/lib/skylark/SkylarkStringRepresentationsTest.java index 8db33169eb..82e2dfed90 100644 --- a/src/test/java/com/google/devtools/build/lib/skylark/SkylarkStringRepresentationsTest.java +++ b/src/test/java/com/google/devtools/build/lib/skylark/SkylarkStringRepresentationsTest.java @@ -389,17 +389,6 @@ public class SkylarkStringRepresentationsTest extends SkylarkTestCase { } @Test - public void testStringRepresentationsOfUnknownObjects() throws Exception { - update("mock", new Object()); - - assertThat(eval("str(mock)")).isEqualTo("<unknown object java.lang.Object>"); - assertThat(eval("repr(mock)")).isEqualTo("<unknown object java.lang.Object>"); - assertThat(eval("'{}'.format(mock)")).isEqualTo("<unknown object java.lang.Object>"); - assertThat(eval("'%s' % mock")).isEqualTo("<unknown object java.lang.Object>"); - assertThat(eval("'%r' % mock")).isEqualTo("<unknown object java.lang.Object>"); - } - - @Test public void testLegacyStringRepresentations_Labels() throws Exception { setSkylarkSemanticsOptions("--incompatible_descriptive_string_representations=false"); diff --git a/src/test/java/com/google/devtools/build/lib/syntax/EvaluationTest.java b/src/test/java/com/google/devtools/build/lib/syntax/EvaluationTest.java index 77f627fd65..fe16f1b195 100644 --- a/src/test/java/com/google/devtools/build/lib/syntax/EvaluationTest.java +++ b/src/test/java/com/google/devtools/build/lib/syntax/EvaluationTest.java @@ -611,6 +611,15 @@ public class EvaluationTest extends EvaluationTestCase { }; } + private Object createUnknownObj() { + return new Object() { + @Override + public String toString() { + return "<unknown object>"; + } + }; + } + @Test public void testPercOnObject() throws Exception { newTest("--incompatible_descriptive_string_representations=true") @@ -620,8 +629,8 @@ public class EvaluationTest extends EvaluationTestCase { .update("obj", createObjWithStr()) .testStatement("'%s' % obj", "<str legacy marker>"); newTest() - .update("unknown", new Object()) - .testStatement("'%s' % unknown", "<unknown object java.lang.Object>"); + .update("unknown", createUnknownObj()) + .testStatement("'%s' % unknown", "<unknown object>"); } @Test @@ -633,10 +642,8 @@ public class EvaluationTest extends EvaluationTestCase { .update("obj", createObjWithStr()) .testStatement("'%s %s' % (obj, obj)", "<str legacy marker> <str legacy marker>"); newTest() - .update("unknown", new Object()) - .testStatement( - "'%s %s' % (unknown, unknown)", - "<unknown object java.lang.Object> <unknown object java.lang.Object>"); + .update("unknown", createUnknownObj()) + .testStatement("'%s %s' % (unknown, unknown)", "<unknown object> <unknown object>"); } @Test |