Maven Surefire
  1. Maven Surefire
  2. SUREFIRE-239

surefire plugin needs to be able to fork tests

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Number of attachments :
      2

      Description

      they can leak memory, so a "fork once for all" option would be good.

      1. pom.xml
        2 kB
        Andy Glick
      2. SurefirePlugin.java
        13 kB
        Andy Glick

        Issue Links

          Activity

          Hide
          Joakim Erdfelt added a comment -

          Additional note.
          This is required for the Cobertura plugin to work properly.

          Show
          Joakim Erdfelt added a comment - Additional note. This is required for the Cobertura plugin to work properly.
          Hide
          John Casey added a comment -

          I looked through surefire today, but couldn't find a place in the booter where it forked. Can you close this out if it's already in there, or when it does land there?

          Show
          John Casey added a comment - I looked through surefire today, but couldn't find a place in the booter where it forked. Can you close this out if it's already in there, or when it does land there?
          Hide
          Jason van Zyl added a comment -

          The basic forking is in. There are a couple options that should be added like jvmargs and envars, but i will enter those as separate issues and release what we have.

          Show
          Jason van Zyl added a comment - The basic forking is in. There are a couple options that should be added like jvmargs and envars, but i will enter those as separate issues and release what we have.
          Hide
          Jason van Zyl added a comment -

          Forking is not working on Windows.

          Show
          Jason van Zyl added a comment - Forking is not working on Windows.
          Hide
          Jason van Zyl added a comment -

          Forking is now working in general.

          Show
          Jason van Zyl added a comment - Forking is now working in general.

            People

            • Assignee:
              Jason van Zyl
              Reporter:
              Brett Porter
            • Votes:
              12 Vote for this issue
              Watchers:
              17 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: