diff options
author | Elliott Hughes <enh@google.com> | 2010-02-18 17:20:15 -0800 |
---|---|---|
committer | Elliott Hughes <enh@google.com> | 2010-02-19 10:35:06 -0800 |
commit | 7ee3a061452c5a7e5c8e661219a1f08a14171858 (patch) | |
tree | f995fb4c053e4c68a86909aa72aee284cb39133a /math/src | |
parent | 1d1f9244611a8ba3be65795d4ea6e84bcecdb89d (diff) | |
download | libcore-7ee3a061452c5a7e5c8e661219a1f08a14171858.zip libcore-7ee3a061452c5a7e5c8e661219a1f08a14171858.tar.gz libcore-7ee3a061452c5a7e5c8e661219a1f08a14171858.tar.bz2 |
Resync a load of tests with upstream, make our build faster.
I started off with a mission to remove uses of dalvik.annotation.* (stuff
like @TestTargetNew and other useless junk that just makes it harder to
stay in sync with upstream). I wrote a script to go through tests showing
me the diff between what we have and what upstream has, thinking that in
cases where upstream has also added tests, I may as well pull them in at
the same time...
...but I didn't realize how close we were to having dx fill its 1.5GiB heap.
After trying various alternatives, I decided to bite the bullet and break
core-tests up into one .jar per module. This adds parallelism back into this,
the slowest part of our build. (I can do even better, but I'll do that in a
separate patch, preferably after we've merged recent changes from master.)
Only a couple of dependencies were problematic: the worthless TestSuiteFactory
which already contained a comment suggesting we get rid of it, and the fact
that some tests -- most notably the concurrent ones -- also contained main
methods that started the JUnit tty-based TestRunner.
(In the long run, we want to be running the harmony tests directly from a
pristine "svn co" of upstream, using DalvikRunner. But this will be a big
help in the meantime, and starts the work of getting our current copy of
the tests into a state where we can start to extract any meaningful
changes/additions we've made.)
Diffstat (limited to 'math/src')
3 files changed, 3 insertions, 18 deletions
diff --git a/math/src/test/java/org/apache/harmony/math/tests/java/math/AllTests.java b/math/src/test/java/org/apache/harmony/math/tests/java/math/AllTests.java index 318e4b6..21f2ec5 100644 --- a/math/src/test/java/org/apache/harmony/math/tests/java/math/AllTests.java +++ b/math/src/test/java/org/apache/harmony/math/tests/java/math/AllTests.java @@ -24,13 +24,8 @@ import junit.framework.TestSuite; */ public class AllTests { - - public static void main(String[] args) { - junit.textui.TestRunner.run(AllTests.suite()); - } - public static Test suite() { - TestSuite suite = tests.TestSuiteFactory.createTestSuite("All tests for package org.apache.harmony.tests.java.math;"); + TestSuite suite = new TestSuite("All tests for package org.apache.harmony.tests.java.math;"); // $JUnit-BEGIN$ suite.addTestSuite(BigDecimalArithmeticTest.class); diff --git a/math/src/test/java/tests/api/java/math/AllTests.java b/math/src/test/java/tests/api/java/math/AllTests.java index 483e08a..a3fb664 100644 --- a/math/src/test/java/tests/api/java/math/AllTests.java +++ b/math/src/test/java/tests/api/java/math/AllTests.java @@ -23,13 +23,8 @@ import junit.framework.TestSuite; * Test suite that includes all tests for the Math project. */ public class AllTests { - - public static void main(String[] args) { - junit.textui.TestRunner.run(suite()); - } - public static Test suite() { - TestSuite suite = tests.TestSuiteFactory.createTestSuite("Tests for java.math"); + TestSuite suite = new TestSuite("Tests for java.math"); // $JUnit-BEGIN$ suite.addTestSuite(BigDecimalTest.class); suite.addTestSuite(BigIntegerTest.class); diff --git a/math/src/test/java/tests/math/AllTests.java b/math/src/test/java/tests/math/AllTests.java index 2773ad6..64fb60d 100644 --- a/math/src/test/java/tests/math/AllTests.java +++ b/math/src/test/java/tests/math/AllTests.java @@ -23,13 +23,8 @@ import junit.framework.TestSuite; * Test suite that includes all tests for the Math project. */ public class AllTests { - - public static void main(String[] args) { - junit.textui.TestRunner.run(AllTests.suite()); - } - public static Test suite() { - TestSuite suite = tests.TestSuiteFactory.createTestSuite("All Math test suites"); + TestSuite suite = new TestSuite("All Math test suites"); suite.addTest(tests.api.java.math.AllTests.suite()); suite.addTest(org.apache.harmony.math.tests.java.math.AllTests.suite()); |