[SUREFIRE-791] JUnit47 provider reports incorrect elapsed time on test failure.

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.11
• Fix Version/s: 2.11
• Component/s: Junit 4.7+ (parallel) support
• Environment: all
• Assignee: Kristian Rosenvold
• Reporter: Nicolas Liochon
• Created: 16/Nov/11 03:27
• Updated: 16/Nov/11 13:57
• Resolved: 16/Nov/11 13:57

Description

A test like this one:

public class Test0 {

  @Test
  public void testT0() throws Exception {
    Assert.assertTrue(false);
  }
}

will report this with JUnit47:

Running Test0
Tests run: 1, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 1,414,930.38 sec <<< FAILURE!

This fix is in TestMethod#testFailure, adding "setEndTime();". It becomes

    public void testFailure( ReportEntry failure )
    {
        this.testFailure = failure;
        setEndTime();
    }

I don't think there is any side effect.