Browse Source

For JUnitTestRunnerTest: testInvalidTestCase and testNoTestCase

JUnit4 adaption returns an ERROR wheres JUnit3 returns a FAILURE.
Modify testcase to reflect this.



git-svn-id: https://svn.apache.org/repos/asf/ant/core/trunk@454207 13f79535-47bb-0310-9956-ffa450edef68
master
Peter Reilly 18 years ago
parent
commit
3b6aedbd21
1 changed files with 15 additions and 3 deletions
  1. +15
    -3
      src/tests/junit/org/apache/tools/ant/taskdefs/optional/junit/JUnitTestRunnerTest.java

+ 15
- 3
src/tests/junit/org/apache/tools/ant/taskdefs/optional/junit/JUnitTestRunnerTest.java View File

@@ -61,15 +61,27 @@ public class JUnitTestRunnerTest extends TestCase {
public void testNoTestCase(){ public void testNoTestCase(){
TestRunner runner = createRunner(NoTestCase.class); TestRunner runner = createRunner(NoTestCase.class);
runner.run(); runner.run();
assertEquals(runner.getFormatter().getError(), JUnitTestRunner.FAILURES, runner.getRetCode());
// On junit3 this is a FAILURE, on junit4 this is an ERROR
int ret = runner.getRetCode();
if (ret != JUnitTestRunner.FAILURES && ret != JUnitTestRunner.ERRORS) {
fail("Unexpected result " + ret + " from junit runner");
}
// JUnit3 test
//assertEquals(runner.getFormatter().getError(), JUnitTestRunner.FAILURES, runner.getRetCode());
} }


// check that an exception in the constructor is noticed // check that an exception in the constructor is noticed
public void testInvalidTestCase(){ public void testInvalidTestCase(){
TestRunner runner = createRunner(InvalidTestCase.class); TestRunner runner = createRunner(InvalidTestCase.class);
runner.run(); runner.run();
String error = runner.getFormatter().getError();
assertEquals(error, JUnitTestRunner.FAILURES, runner.getRetCode());
// On junit3 this is a FAILURE, on junit4 this is an ERROR
int ret = runner.getRetCode();
if (ret != JUnitTestRunner.FAILURES && ret != JUnitTestRunner.ERRORS) {
fail("Unexpected result " + ret + " from junit runner");
}
// JUNIT3 test
//assertEquals(error, JUnitTestRunner.FAILURES, runner.getRetCode());
//@fixme as of now does not report the original stacktrace. //@fixme as of now does not report the original stacktrace.
//assertTrue(error, error.indexOf("thrown on purpose") != -1); //assertTrue(error, error.indexOf("thrown on purpose") != -1);
} }


Loading…
Cancel
Save