Maven Surefire
  1. Maven Surefire
  2. SUREFIRE-730

JUnit4RunListener does not report results from concurrently running tests correctly

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 2.5, 2.8.1
    • Fix Version/s: 2.9
    • Component/s: Junit 4.x support
    • Labels:
      None
    • Flags:
      Patch

      Description

      JUnit4Listener determines succesful pass of a test via a single boolean flag on the class that is set when a test to "true" when a test is started, set to "false" when a test failed, and checked after a test finishes. If all the tests run in serial, this scheme works, but when the tests are all run in parallel, after the first failed test no other successful tests will be reported as having been run (because after the flag is set to "fail" nothing ever resets it - all the tests have already started).

      It would be simple to fix this by changing the single failure flag to a hash map of tests to pass/fail flags, and such a change would greatly improve my quality of life.

      1. sure.dif
        2 kB
        James Neal

        Activity

        James Neal created issue -
        Kristian Rosenvold made changes -
        Field Original Value New Value
        Assignee Kristian Rosenvold [ krosenvold ]
        Status Open [ 1 ] Closed [ 6 ]
        Fix Version/s 2.8.2 [ 17312 ]
        Resolution Fixed [ 1 ]
        Mark Thomas made changes -
        Project Import Sun Apr 05 13:23:32 UTC 2015 [ 1428240212200 ]
        Mark Thomas made changes -
        Workflow jira [ 12727976 ] Default workflow, editable Closed status [ 12758398 ]
        Mark Thomas made changes -
        Patch Submitted Yes [ 10763 ]
        Flags Patch [ 10430 ]
        Mark Thomas made changes -
        Project Import Mon Apr 06 01:36:33 UTC 2015 [ 1428284193036 ]
        Mark Thomas made changes -
        Workflow jira [ 12965343 ] Default workflow, editable Closed status [ 13002794 ]

          People

          • Assignee:
            Kristian Rosenvold
            Reporter:
            James Neal
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development