Skip to content

Commit dcabef0

Browse files
committed
Allow performance tests to throw general Exception
The PerformanceTestRunner unnecessarily restricts exceptions that can be thrown by the test() method to CoreException. The test runner can handle exceptions of every type anyway. Some performance tests in Equinox need to throw IOExceptions, thus this change generalizes the exception thrown by the test() method.
1 parent 605ef63 commit dcabef0

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

runtime/tests/org.eclipse.core.tests.harness/src/org/eclipse/core/tests/harness/PerformanceTestRunner.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@
1616
import static org.junit.Assert.fail;
1717

1818
import junit.framework.TestCase;
19-
import org.eclipse.core.runtime.CoreException;
2019
import org.eclipse.test.performance.Dimension;
2120
import org.eclipse.test.performance.Performance;
2221
import org.eclipse.test.performance.PerformanceMeter;
@@ -37,7 +36,7 @@ public void setRegressionReason(String comment) {
3736
/**
3837
* Implemented by subclasses to perform the work to be measured.
3938
*/
40-
protected abstract void test() throws CoreException;
39+
protected abstract void test() throws Exception;
4140

4241
/**
4342
* Executes the performance test the given number of times. Use the outer time

0 commit comments

Comments
 (0)