[gwt-contrib] [google-web-toolkit] r9518 committed - TreeItemTest got upset by Safari 3....

[codesite-noreply]

Revision: 9518
Author: rj...@google.com
Date: Fri Jan  7 13:48:26 2011
Log: TreeItemTest got upset by Safari 3.
Review at http://gwt-code-reviews.appspot.com/1270801
Review by: con...@google.com
http://code.google.com/p/google-web-toolkit/source/detail?r=9518

Modified:
 /trunk/user/test/com/google/gwt/user/client/ui/TreeItemTest.java

=======================================
--- /trunk/user/test/com/google/gwt/user/client/ui/TreeItemTest.java	Fri  
Jan  7 08:26:38 2011
+++ /trunk/user/test/com/google/gwt/user/client/ui/TreeItemTest.java	Fri  
Jan  7 13:48:26 2011
@@ -23,7 +23,7 @@
  */
 public class TreeItemTest extends GWTTestCase {

-  private static final String html = "<b>hello</b><i>world</i>";
+  private static final String HTML = "<b>hello</b><i>world</i>";

   @Override
   public String getModuleName() {
@@ -60,8 +60,8 @@

   public void testAddItemSafeHtml() {
     TreeItem item = new TreeItem("foo");
-    TreeItem child = item.addItem(SafeHtmlUtils.fromSafeConstant(html));
-    assertEquals(html, child.getHTML().toLowerCase());
+    TreeItem child = item.addItem(SafeHtmlUtils.fromSafeConstant(HTML));
+    assertEquals(HTML, child.getHTML().toLowerCase());
   }

   /**
@@ -72,7 +72,9 @@
     String text = "Some<br>text";
     TreeItem item = root.addTextItem(text);
     assertEquals(text, item.getText());
-    assertEquals("Some&lt;br&gt;text", item.getHTML());
+    // Normalize the html for ancient safari 3
+    String html = item.getHTML().replace(">", "&gt;");
+    assertEquals("Some&lt;br&gt;text", html);
   }

   public void testAsTreeItem() {
@@ -159,8 +161,8 @@

   public void testInsertItemSafeHtml() {
     TreeItem item = new TreeItem("foo");
-    TreeItem child = item.insertItem(0,  
SafeHtmlUtils.fromSafeConstant(html));
-    assertEquals(html, child.getHTML().toLowerCase());
+    TreeItem child = item.insertItem(0,  
SafeHtmlUtils.fromSafeConstant(HTML));
+    assertEquals(HTML, child.getHTML().toLowerCase());
   }

   /**
@@ -199,15 +201,15 @@
   }

   public void testSafeHtmlConstructor() {
-    TreeItem item = new TreeItem(SafeHtmlUtils.fromSafeConstant(html));
-
-    assertEquals(html, item.getHTML().toLowerCase());
+    TreeItem item = new TreeItem(SafeHtmlUtils.fromSafeConstant(HTML));
+
+    assertEquals(HTML, item.getHTML().toLowerCase());
   }

   public void testSetSafeHtml() {
     TreeItem item = new TreeItem("foo");
-    item.setHTML(SafeHtmlUtils.fromSafeConstant(html));
-    assertEquals(html, item.getHTML().toLowerCase());
+    item.setHTML(SafeHtmlUtils.fromSafeConstant(HTML));
+    assertEquals(HTML, item.getHTML().toLowerCase());
   }

   /**

--
http://groups.google.com/group/Google-Web-Toolkit-Contributors