Use @RunWith(Suite.class) for emul test suites. Change-Id: I2fc94ccf4298714dfee8ba9628f9137e30cdb8a0

diff --git a/user/test/com/google/gwt/emultest/BigDecimalSuite.java b/user/test/com/google/gwt/emultest/BigDecimalSuite.java
index b0479a3..210a47e 100644
--- a/user/test/com/google/gwt/emultest/BigDecimalSuite.java
+++ b/user/test/com/google/gwt/emultest/BigDecimalSuite.java

@@ -20,23 +20,20 @@
 import com.google.gwt.emultest.java.math.BigDecimalConstructorsTest;
 import com.google.gwt.emultest.java.math.BigDecimalConvertTest;
 import com.google.gwt.emultest.java.math.BigDecimalScaleOperationsTest;
-import com.google.gwt.junit.tools.GWTTestSuite;
 
-import junit.framework.Test;
+import org.junit.runner.RunWith;
+import org.junit.runners.Suite;
+import org.junit.runners.Suite.SuiteClasses;
 
 /**
  * Test JRE emulation of BigDecimal.
  */
-public class BigDecimalSuite {
-
-  /** Note: due to compiler error, only can use one Test Case at a time. */
-  public static Test suite() {
-    GWTTestSuite suite = new GWTTestSuite("Tests for BigDecimal");
-    suite.addTestSuite(BigDecimalArithmeticTest.class);
-    suite.addTestSuite(BigDecimalCompareTest.class);
-    suite.addTestSuite(BigDecimalConstructorsTest.class);
-    suite.addTestSuite(BigDecimalConvertTest.class);
-    suite.addTestSuite(BigDecimalScaleOperationsTest.class);
-    return suite;
-  }
-}
+@RunWith(Suite.class)
+@SuiteClasses({
+  BigDecimalArithmeticTest.class,
+  BigDecimalCompareTest.class,
+  BigDecimalConstructorsTest.class,
+  BigDecimalConvertTest.class,
+  BigDecimalScaleOperationsTest.class,
+})
+public class BigDecimalSuite { }

diff --git a/user/test/com/google/gwt/emultest/BigIntegerSuite.java b/user/test/com/google/gwt/emultest/BigIntegerSuite.java
index 35e3ab0..c44c0ba 100644
--- a/user/test/com/google/gwt/emultest/BigIntegerSuite.java
+++ b/user/test/com/google/gwt/emultest/BigIntegerSuite.java

@@ -29,32 +29,29 @@
 import com.google.gwt.emultest.java.math.BigIntegerSubtractTest;
 import com.google.gwt.emultest.java.math.BigIntegerToStringTest;
 import com.google.gwt.emultest.java.math.BigIntegerXorTest;
-import com.google.gwt.junit.tools.GWTTestSuite;
 
-import junit.framework.Test;
+import org.junit.runner.RunWith;
+import org.junit.runners.Suite;
+import org.junit.runners.Suite.SuiteClasses;
 
 /**
  * Test JRE emulation of BigInteger.
  */
-public class BigIntegerSuite {
-
-  /** Note: due to compiler error, only can use one Test Case at a time. */
-  public static Test suite() {
-    GWTTestSuite suite = new GWTTestSuite("Tests for BigInteger");
-    suite.addTestSuite(BigIntegerAddTest.class);
-    suite.addTestSuite(BigIntegerAndTest.class);
-    suite.addTestSuite(BigIntegerCompareTest.class);
-    suite.addTestSuite(BigIntegerConstructorsTest.class);
-    suite.addTestSuite(BigIntegerDivideTest.class);
-    suite.addTestSuite(BigIntegerHashCodeTest.class);
-    suite.addTestSuite(BigIntegerModPowTest.class);
-    suite.addTestSuite(BigIntegerMultiplyTest.class);
-    suite.addTestSuite(BigIntegerNotTest.class);
-    suite.addTestSuite(BigIntegerOperateBitsTest.class);
-    suite.addTestSuite(BigIntegerOrTest.class);
-    suite.addTestSuite(BigIntegerSubtractTest.class);
-    suite.addTestSuite(BigIntegerToStringTest.class);
-    suite.addTestSuite(BigIntegerXorTest.class);
-    return suite;
-  }
-}
+@RunWith(Suite.class)
+@SuiteClasses({
+  BigIntegerAddTest.class,
+  BigIntegerAndTest.class,
+  BigIntegerCompareTest.class,
+  BigIntegerConstructorsTest.class,
+  BigIntegerDivideTest.class,
+  BigIntegerHashCodeTest.class,
+  BigIntegerModPowTest.class,
+  BigIntegerMultiplyTest.class,
+  BigIntegerNotTest.class,
+  BigIntegerOperateBitsTest.class,
+  BigIntegerOrTest.class,
+  BigIntegerSubtractTest.class,
+  BigIntegerToStringTest.class,
+  BigIntegerXorTest.class,
+})
+public class BigIntegerSuite { }

diff --git a/user/test/com/google/gwt/emultest/CollectionsSuite.java b/user/test/com/google/gwt/emultest/CollectionsSuite.java
index 6d69a38..e078443 100644
--- a/user/test/com/google/gwt/emultest/CollectionsSuite.java
+++ b/user/test/com/google/gwt/emultest/CollectionsSuite.java

@@ -35,45 +35,49 @@
 import com.google.gwt.emultest.java.util.PriorityQueueTest;
 import com.google.gwt.emultest.java.util.RandomTest;
 import com.google.gwt.emultest.java.util.StackTest;
+import com.google.gwt.emultest.java.util.TreeMapIntegerDoubleTest;
+import com.google.gwt.emultest.java.util.TreeMapIntegerDoubleWithComparatorTest;
+import com.google.gwt.emultest.java.util.TreeMapStringStringTest;
+import com.google.gwt.emultest.java.util.TreeMapStringStringWithComparatorTest;
+import com.google.gwt.emultest.java.util.TreeSetIntegerTest;
+import com.google.gwt.emultest.java.util.TreeSetIntegerWithComparatorTest;
 import com.google.gwt.emultest.java.util.VectorTest;
-import com.google.gwt.junit.tools.GWTTestSuite;
 
-import junit.framework.Test;
+import org.junit.runner.RunWith;
+import org.junit.runners.Suite;
+import org.junit.runners.Suite.SuiteClasses;
 
-/**
- * Test JRE Collections emulation.
- */
-public class CollectionsSuite {
+/** Test JRE Collections emulation. */
+@RunWith(Suite.class)
+@SuiteClasses({
+  ArrayDequeTest.class,
+  ArrayListTest.class,
+  ArraysTest.class,
+  BitSetTest.class,
+  CollectionsTest.class,
+  ComparatorTest.class,
+  DateTest.class,
+  EnumMapTest.class,
+  EnumSetTest.class,
+  HashMapTest.class,
+  HashSetTest.class,
+  IdentityHashMapTest.class,
+  LinkedHashMapTest.class,
+  LinkedHashSetTest.class,
+  LinkedListTest.class,
+  ObjectsTest.class,
+  PriorityQueueTest.class,
+  RandomTest.class,
+  StackTest.class,
+  VectorTest.class,
+  TreeMapStringStringTest.class,
+  TreeMapStringStringWithComparatorTest.class,
+  TreeMapIntegerDoubleTest.class,
+  TreeMapIntegerDoubleWithComparatorTest.class,
+  TreeSetIntegerTest.class,
+  TreeSetIntegerWithComparatorTest.class,
 
-  public static Test suite() {
-    GWTTestSuite suite = new GWTTestSuite("Tests for emulation of Java Collections");
-
-    // $JUnit-BEGIN$
-    suite.addTestSuite(ArrayDequeTest.class);
-    suite.addTestSuite(ArrayListTest.class);
-    suite.addTestSuite(ArraysTest.class);
-    suite.addTestSuite(BitSetTest.class);
-    suite.addTestSuite(CollectionsTest.class);
-    suite.addTestSuite(ComparatorTest.class);
-    suite.addTestSuite(DateTest.class);
-    suite.addTestSuite(EnumMapTest.class);
-    suite.addTestSuite(EnumSetTest.class);
-    suite.addTestSuite(HashMapSmokeTest.class);
-    suite.addTestSuite(HashMapTest.class);
-    suite.addTestSuite(HashSetTest.class);
-    suite.addTestSuite(IdentityHashMapTest.class);
-    suite.addTestSuite(LinkedHashMapTest.class);
-    suite.addTestSuite(LinkedHashSetTest.class);
-    suite.addTestSuite(LinkedListTest.class);
-    suite.addTestSuite(ObjectsTest.class);
-    suite.addTestSuite(PriorityQueueTest.class);
-    suite.addTestSuite(RandomTest.class);
-    suite.addTestSuite(StackTest.class);
-    suite.addTestSuite(VectorTest.class);
-    suite.addTest(TreeMapSuiteSub.suite());
-    suite.addTest(TreeSetSuiteSub.suite());
-    // $JUnit-END$
-
-    return suite;
-  }
-}
+  // Put last to reduce number of times the test framework switches modules
+  HashMapSmokeTest.class,
+})
+public class CollectionsSuite { }

diff --git a/user/test/com/google/gwt/emultest/EmulJava8Suite.java b/user/test/com/google/gwt/emultest/EmulJava8Suite.java
index 572eb86..abab98a 100644
--- a/user/test/com/google/gwt/emultest/EmulJava8Suite.java
+++ b/user/test/com/google/gwt/emultest/EmulJava8Suite.java

@@ -48,59 +48,53 @@
 import com.google.gwt.emultest.java8.util.stream.LongStreamTest;
 import com.google.gwt.emultest.java8.util.stream.StreamSupportTest;
 import com.google.gwt.emultest.java8.util.stream.StreamTest;
-import com.google.gwt.junit.tools.GWTTestSuite;
 
-import junit.framework.Test;
+import org.junit.runner.RunWith;
+import org.junit.runners.Suite;
+import org.junit.runners.Suite.SuiteClasses;
 
-/**
- * Test Java8 JRE emulations.
- */
-public class EmulJava8Suite {
+/** Test JRE emulations. */
+@RunWith(Suite.class)
+@SuiteClasses({
+  //-- java.lang
+  DoubleTest.class,
+  FloatTest.class,
+  MathTest.class,
+  StringTest.class,
 
-  public static Test suite() {
-    GWTTestSuite suite = new GWTTestSuite("Tests for com.google.gwt.emul.java8");
+  //-- java.math
+  BigIntegerConvertTest.class,
 
-    //-- java.lang
-    suite.addTestSuite(DoubleTest.class);
-    suite.addTestSuite(FloatTest.class);
-    suite.addTestSuite(MathTest.class);
-    suite.addTestSuite(StringTest.class);
+  //-- java.util
+  ArraysTest.class,
+  ArrayListTest.class,
+  LinkedListTest.class,
+  ListTest.class,
+  VectorTest.class,
+  ComparatorTest.class,
+  MapTest.class,
+  MapEntryTest.class,
+  HashMapTest.class,
+  IdentityHashMapTest.class,
+  LinkedHashMapTest.class,
+  TreeMapTest.class,
+  OptionalTest.class,
+  OptionalIntTest.class,
+  OptionalLongTest.class,
+  OptionalDoubleTest.class,
+  PrimitiveIteratorTest.class,
+  SpliteratorsTest.class,
+  StringJoinerTest.class,
+  DoubleSummaryStatisticsTest.class,
+  IntSummaryStatisticsTest.class,
+  LongSummaryStatisticsTest.class,
 
-    //-- java.math
-    suite.addTestSuite(BigIntegerConvertTest.class);
-
-    //-- java.util
-    suite.addTestSuite(ArraysTest.class);
-    suite.addTestSuite(ArrayListTest.class);
-    suite.addTestSuite(LinkedListTest.class);
-    suite.addTestSuite(ListTest.class);
-    suite.addTestSuite(VectorTest.class);
-    suite.addTestSuite(ComparatorTest.class);
-    suite.addTestSuite(MapTest.class);
-    suite.addTestSuite(MapEntryTest.class);
-    suite.addTestSuite(HashMapTest.class);
-    suite.addTestSuite(IdentityHashMapTest.class);
-    suite.addTestSuite(LinkedHashMapTest.class);
-    suite.addTestSuite(TreeMapTest.class);
-    suite.addTestSuite(OptionalTest.class);
-    suite.addTestSuite(OptionalIntTest.class);
-    suite.addTestSuite(OptionalLongTest.class);
-    suite.addTestSuite(OptionalDoubleTest.class);
-    suite.addTestSuite(PrimitiveIteratorTest.class);
-    suite.addTestSuite(SpliteratorsTest.class);
-    suite.addTestSuite(StringJoinerTest.class);
-    suite.addTestSuite(DoubleSummaryStatisticsTest.class);
-    suite.addTestSuite(IntSummaryStatisticsTest.class);
-    suite.addTestSuite(LongSummaryStatisticsTest.class);
-
-    //-- java.util.stream
-    suite.addTestSuite(CollectorsTest.class);
-    suite.addTestSuite(DoubleStreamTest.class);
-    suite.addTestSuite(IntStreamTest.class);
-    suite.addTestSuite(LongStreamTest.class);
-    suite.addTestSuite(StreamTest.class);
-    suite.addTestSuite(StreamSupportTest.class);
-
-    return suite;
-  }
-}
+  //-- java.util.stream
+  CollectorsTest.class,
+  DoubleStreamTest.class,
+  IntStreamTest.class,
+  LongStreamTest.class,
+  StreamTest.class,
+  StreamSupportTest.class,
+})
+public class EmulJava8Suite { }

diff --git a/user/test/com/google/gwt/emultest/EmulSuite.java b/user/test/com/google/gwt/emultest/EmulSuite.java
index 1e82a93..483ae7a 100644
--- a/user/test/com/google/gwt/emultest/EmulSuite.java
+++ b/user/test/com/google/gwt/emultest/EmulSuite.java

@@ -50,70 +50,66 @@
 import com.google.gwt.emultest.java.sql.SqlDateTest;
 import com.google.gwt.emultest.java.sql.SqlTimeTest;
 import com.google.gwt.emultest.java.sql.SqlTimestampTest;
-import com.google.gwt.junit.tools.GWTTestSuite;
 
-import junit.framework.Test;
+import org.junit.runner.RunWith;
+import org.junit.runners.Suite;
+import org.junit.runners.Suite.SuiteClasses;
 
 /**
  * Test JRE emulations.
  */
-public class EmulSuite {
+@RunWith(Suite.class)
+@SuiteClasses({
+  CoercionsTest.class,
 
-  /**
-   * Note: due to compiler error, only can use one Test Case at a time.
-   */
-  public static Test suite() {
-    GWTTestSuite suite = new GWTTestSuite("Tests for com.google.gwt.emul.java");
+  //-- java.io
+  ByteArrayInputStreamTest.class,
+  ByteArrayOutputStreamTest.class,
+  FilterInputStreamTest.class,
+  FilterOutputStreamTest.class,
+  InputStreamTest.class,
+  OutputStreamTest.class,
 
-    // $JUnit-BEGIN$
-    suite.addTestSuite(CoercionsTest.class);
-    //-- java.io
-    suite.addTestSuite(ByteArrayInputStreamTest.class);
-    suite.addTestSuite(ByteArrayOutputStreamTest.class);
-    suite.addTestSuite(FilterInputStreamTest.class);
-    suite.addTestSuite(FilterOutputStreamTest.class);
-    suite.addTestSuite(InputStreamTest.class);
-    suite.addTestSuite(OutputStreamTest.class);
-    //-- java.lang
-    suite.addTestSuite(BooleanTest.class);
-    suite.addTestSuite(ByteTest.class);
-    suite.addTestSuite(CharacterTest.class);
-    suite.addTestSuite(CompilerConstantStringTest.class);
-    suite.addTestSuite(DoubleTest.class);
-    suite.addTestSuite(FloatTest.class);
-    suite.addTestSuite(IntegerTest.class);
-    suite.addTestSuite(JsExceptionTest.class);
-    suite.addTestSuite(LongTest.class);
-    suite.addTestSuite(MathTest.class);
-    suite.addTestSuite(NullPointerExceptionTest.class);
-    suite.addTestSuite(ObjectTest.class);
-    suite.addTestSuite(ShortTest.class);
-    suite.addTestSuite(StringBufferTest.class);
-    suite.addTestSuite(StringTest.class);
-    suite.addTestSuite(SystemTest.class);
-    suite.addTestSuite(ThrowableTest.class);
-    suite.addTestSuite(ThrowableStackTraceEmulTest.class);
-    suite.addTestSuite(TypeTest.class);
-    //-- java.math
-    // BigDecimal is tested in {@link BigDecimalSuite}
-    // BigInteger is tested in {@link BigIntegerSuite}
-    suite.addTestSuite(RoundingModeTest.class);
-    suite.addTestSuite(MathContextTest.class);
-    suite.addTestSuite(MathContextWithObfuscatedEnumsTest.class);
+  //-- java.lang
+  BooleanTest.class,
+  ByteTest.class,
+  CharacterTest.class,
+  CompilerConstantStringTest.class,
+  DoubleTest.class,
+  FloatTest.class,
+  IntegerTest.class,
+  JsExceptionTest.class,
+  LongTest.class,
+  MathTest.class,
+  NullPointerExceptionTest.class,
+  ObjectTest.class,
+  ShortTest.class,
+  StringBufferTest.class,
+  StringTest.class,
+  SystemTest.class,
+  ThrowableTest.class,
+  ThrowableStackTraceEmulTest.class,
+  TypeTest.class,
 
-    //-- java.nio
-    suite.addTestSuite(CharsetTest.class);
-    suite.addTestSuite(StandardCharsetsTest.class);
+  //-- java.math
+  // BigDecimal is tested in {@link BigDecimalSuite}
+  // BigInteger is tested in {@link BigIntegerSuite}
+  RoundingModeTest.class,
+  MathContextTest.class,
 
-    //-- java.security
-    suite.addTestSuite(MessageDigestTest.class);
+  //-- java.nio
+  CharsetTest.class,
+  StandardCharsetsTest.class,
 
-    //-- java.sql
-    suite.addTestSuite(SqlDateTest.class);
-    suite.addTestSuite(SqlTimeTest.class);
-    suite.addTestSuite(SqlTimestampTest.class);
-    // $JUnit-END$
+  //-- java.security
+  MessageDigestTest.class,
 
-    return suite;
-  }
-}
+  //-- java.sql
+  SqlDateTest.class,
+  SqlTimeTest.class,
+  SqlTimestampTest.class,
+
+  // Put last to reduce number of times the test framework switches modules
+  MathContextWithObfuscatedEnumsTest.class,
+})
+public class EmulSuite { }

diff --git a/user/test/com/google/gwt/emultest/TreeMapSuiteSub.java b/user/test/com/google/gwt/emultest/TreeMapSuiteSub.java
deleted file mode 100644
index 2f0e804..0000000
--- a/user/test/com/google/gwt/emultest/TreeMapSuiteSub.java
+++ /dev/null

@@ -1,38 +0,0 @@
-/*
- * Copyright 2008 Google Inc.
- * 
- * Licensed under the Apache License, Version 2.0 (the "License"); you may not
- * use this file except in compliance with the License. You may obtain a copy of
- * the License at
- * 
- * http://www.apache.org/licenses/LICENSE-2.0
- * 
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
- * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
- * License for the specific language governing permissions and limitations under
- * the License.
- */
-package com.google.gwt.emultest;
-
-import com.google.gwt.emultest.java.util.TreeMapIntegerDoubleTest;
-import com.google.gwt.emultest.java.util.TreeMapIntegerDoubleWithComparatorTest;
-import com.google.gwt.emultest.java.util.TreeMapStringStringTest;
-import com.google.gwt.emultest.java.util.TreeMapStringStringWithComparatorTest;
-import com.google.gwt.junit.tools.GWTTestSuite;
-
-import junit.framework.Test;
-
-/**
- * Tests <code>TreeMap</code>.
- */
-public class TreeMapSuiteSub {
-  public static Test suite() {
-    GWTTestSuite suite = new GWTTestSuite("Tests for com.google.gwt.emul.java.util.TreeMap");
-    suite.addTestSuite(TreeMapStringStringTest.class);
-    suite.addTestSuite(TreeMapStringStringWithComparatorTest.class);
-    suite.addTestSuite(TreeMapIntegerDoubleTest.class);
-    suite.addTestSuite(TreeMapIntegerDoubleWithComparatorTest.class);
-    return suite;
-  }
-}

diff --git a/user/test/com/google/gwt/emultest/TreeSetSuiteSub.java b/user/test/com/google/gwt/emultest/TreeSetSuiteSub.java
deleted file mode 100644
index a7cf7d1..0000000
--- a/user/test/com/google/gwt/emultest/TreeSetSuiteSub.java
+++ /dev/null

@@ -1,34 +0,0 @@
-/*
- * Copyright 2008 Google Inc.
- * 
- * Licensed under the Apache License, Version 2.0 (the "License"); you may not
- * use this file except in compliance with the License. You may obtain a copy of
- * the License at
- * 
- * http://www.apache.org/licenses/LICENSE-2.0
- * 
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
- * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
- * License for the specific language governing permissions and limitations under
- * the License.
- */
-package com.google.gwt.emultest;
-
-import com.google.gwt.emultest.java.util.TreeSetIntegerTest;
-import com.google.gwt.emultest.java.util.TreeSetIntegerWithComparatorTest;
-import com.google.gwt.junit.tools.GWTTestSuite;
-
-import junit.framework.Test;
-
-/**
- * Tests <code>TreeSet</code>.
- */
-public class TreeSetSuiteSub {
-  public static Test suite() {
-    GWTTestSuite suite = new GWTTestSuite("Tests for com.google.gwt.emul.java.util.TreeSet");
-    suite.addTestSuite(TreeSetIntegerTest.class);
-    suite.addTestSuite(TreeSetIntegerWithComparatorTest.class);
-    return suite;
-  }
-}