Uploaded image for project: 'Maven Surefire'
  1. Maven Surefire
  2. SUREFIRE-54

Remove use of parent classloader in surefirebooter but keep TestNG support working

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Critical
    • Resolution: Fixed
    • 2.3
    • 2.3
    • None
    • None

    Description

      The removal of the parent system classloader while running tests in surefire booter totally broke testNG support.

      I am going to revert the commit:
      http://svn.apache.org/viewvc/maven/surefire/trunk/surefire-booter/src/main/java/org/apache/maven/surefire/booter/SurefireBooter.java?r1=427040&r2=438999&diff_format=h
      ... in order to keep testng working for now, but we should find a way to solve both problems soon.

      The removal of the parent classloader is needed, according to Kenney for the following reason:
      If this is not done, the System classloader is added, in this case an AppClassloader containing everything in the root classpath. For instance, in maven, everything in core/ is available.This can cause clashes with the plexus-utils used in maven itself.

      Attachments

        Activity

          People

            kenneyw kenneyw
            fgiust Fabrizio Giustina
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: