From 3b038b9f7657f2d4c77700964ca06d92b3a6db12 Mon Sep 17 00:00:00 2001
From: brandjon <brandjon@google.com>
Date: Tue, 11 Jul 2017 21:43:28 +0200
Subject: Automated rollback of commit
 0d2aa6369b09940d584b544fb10796387e0fafa0.

*** Reason for rollback ***

PiperOrigin-RevId: 161567104
---
 .../lib/skylark/SkylarkStringRepresentationsTest.java | 11 -----------
 .../devtools/build/lib/syntax/EvaluationTest.java     | 19 +++++++++++++------
 2 files changed, 13 insertions(+), 17 deletions(-)

(limited to 'src/test/java/com/google')

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
@@ -388,17 +388,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
-- 
cgit v1.2.3