Revision: 9499
Author: [email protected]
Date: Wed Jan  5 14:37:41 2011
Log: Re-enable a few style tests now that old linux hosted mode is gone

Review at http://gwt-code-reviews.appspot.com/1228801

Review by: [email protected]
http://code.google.com/p/google-web-toolkit/source/detail?r=9499

Modified:
 /trunk/user/test/com/google/gwt/dom/client/StyleTest.java

=======================================
--- /trunk/user/test/com/google/gwt/dom/client/StyleTest.java Wed Oct 28 09:10:53 2009 +++ /trunk/user/test/com/google/gwt/dom/client/StyleTest.java Wed Jan 5 14:37:41 2011
@@ -36,6 +36,8 @@
 import com.google.gwt.dom.client.Style.TextDecoration;
 import com.google.gwt.dom.client.Style.VerticalAlign;
 import com.google.gwt.dom.client.Style.Visibility;
+import com.google.gwt.junit.DoNotRunWith;
+import com.google.gwt.junit.Platform;
 import com.google.gwt.junit.client.GWTTestCase;

 /**
@@ -85,14 +87,19 @@
     style.setCursor(Cursor.HELP);
     assertEquals(Cursor.HELP, style.getCursor());

-    // These aren't supported on old mozilla, so testing them will break.
-    // TODO: re-enable these cases when we finally drop linux hosted mode.
     /*
+     * Note, this will test fail on old mozilla (prior to gecko 1.8) due to
+     * unsupported style property
+     */
     style.setCursor(Cursor.COL_RESIZE);
     assertEquals(Cursor.COL_RESIZE, style.getCursor());
+
+    /*
+     * Note, this will test fail on old mozilla (prior to gecko 1.8) due to
+     * unsupported style property
+     */
     style.setCursor(Cursor.ROW_RESIZE);
     assertEquals(Cursor.ROW_RESIZE, style.getCursor());
-    */
   }

   public void testDisplay() {
@@ -106,12 +113,12 @@
     style.setDisplay(Display.INLINE);
     assertEquals(Display.INLINE, style.getDisplay());

-    // Not supported on old mozilla, so testing it will break.
-    // TODO: re-enable these cases when we finally drop linux hosted mode.
     /*
+     * Note, this will test fail on old mozilla (prior to gecko 1.8) due to
+     * unsupported style property
+     */
     style.setDisplay(Display.INLINE_BLOCK);
     assertEquals(Display.INLINE_BLOCK, style.getDisplay());
-    */
   }

   public void testFontStyle() {
@@ -241,6 +248,7 @@
     assertEquals(Visibility.HIDDEN, style.getVisibility());
   }

+  @DoNotRunWith({Platform.HtmlUnitUnknown})
   public void testUnits() {
     DivElement div = Document.get().createDivElement();
     Style style = div.getStyle();

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

Reply via email to