Disables testTimeLimit until we can figure out why its hanging some browsers (Safari in particular).
Time limits seem to work in general, but there's something special about this particular test method.
Review by: knorton (TBR)
git-svn-id: https://google-web-toolkit.googlecode.com/svn/trunk@1790 8db76d5a-ed1c-0410-87a9-c151d255dfc7
diff --git a/user/test/com/google/gwt/junit/client/BenchmarkTest.java b/user/test/com/google/gwt/junit/client/BenchmarkTest.java
index cbf8466..54cb922 100644
--- a/user/test/com/google/gwt/junit/client/BenchmarkTest.java
+++ b/user/test/com/google/gwt/junit/client/BenchmarkTest.java
@@ -20,7 +20,6 @@
import com.google.gwt.junit.client.annotations.RangeEnum;
import com.google.gwt.junit.client.annotations.Setup;
import com.google.gwt.junit.client.annotations.Teardown;
-import com.google.gwt.junit.client.annotations.IterationTimeLimit;
/**
* Basic Benchmark testing.
@@ -106,16 +105,19 @@
*
* @param numIterations
*/
- @IterationTimeLimit(1L)
- public void testTimeLimit(@RangeField("veryLargeRange")
- Integer numIterations) {
-
- somethingExpensive();
-
- // Make sure we hit the time limit, instead of running through all
- // iterations.
- assertTrue(numIterations < Integer.MAX_VALUE);
- }
+// TODO(tobyr) Disabled, because it can hang some browsers (Safari at least)
+// TimeLimits work in general (as evidenced by working benchmarks), but there's
+// something peculiar about this test causing problems.
+// @IterationTimeLimit(1L)
+// public void testTimeLimit(@RangeField("veryLargeRange")
+// Integer numIterations) {
+//
+// somethingExpensive();
+//
+// // Make sure we hit the time limit, instead of running through all
+// // iterations.
+// assertTrue(numIterations < Integer.MAX_VALUE);
+// }
public void testTwoParameterField() {
}
