Reviewers: skybrian,

Description:
Make CoverageTest less dependent on execution order.


Please review this at http://gwt-code-reviews.appspot.com/1741804/

Affected files:
  M user/test/com/google/gwt/dev/jjs/test/CoverageTest.java


Index: user/test/com/google/gwt/dev/jjs/test/CoverageTest.java
===================================================================
--- user/test/com/google/gwt/dev/jjs/test/CoverageTest.java     (revision 11054)
+++ user/test/com/google/gwt/dev/jjs/test/CoverageTest.java     (working copy)
@@ -154,6 +154,7 @@
     }

     private void testArrayReference() {
+      ia = new int[] {i, j};
       // ArrayReference
       i = ia[0];
       assertEquals(ia[0], i);
@@ -415,6 +416,7 @@
     }

     private void testForeachStatement() {
+      ia = new int[] {i, j};
       // Array of primitive.
       for (int q : ia) {
         i = q;
@@ -557,6 +559,7 @@

     private void testPostfixExpression() {
       // PostfixExpression
+      i = 1;
       assertEquals(1, i++);
       assertEquals(2, i--);
     }
@@ -822,6 +825,22 @@
     return false;
   }

+  @Override
+  public void gwtSetUp() throws Exception {
+    super.gwtSetUp();
+    d = 0;
+    f = 0;
+    i = 1 + 2 + 3;
+    ia = null;
+    iaa = null;
+    iaaa = null;
+    l = 0;
+    o = null;
+    s = "foo";
+    sa = new String[]{"foo", "bar", "bar"};
+    z = false;
+  }
+
   protected static void sfoo() {
   }



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

Reply via email to