Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 2.0 (2.2 plugin)
    • Fix Version/s: 2.3.1
    • Component/s: JUnit 3.x support
    • Labels:
      None
    • Environment:
      maven2.0.4, sun-jdk-1.5.0.09, maven-surefire-plugin 2.2, surefire 2.0, gentoo linux x86
    • Flags:
      Patch

      Description

      Surefire incorrectly interprets classpath ordering.
      Steps to reproduce:
      1. unzip my-app.zip - it's a simple mvn project created with
      mvn archetype:create -DgroupId=com.mycompany.app -DartifactId=my-app
      and lightly patched
      2. mvn test
      in my case, it prints out
      jar:file:/home/vyzivus/.m2/repository/jxta/jxta/2.0/jxta-2.0.jar!/log4j.properties
      jar:file:/home/vyzivus/.m2/repository/jxta/jxta/2.0/jxta-2.0.jar!/log4j.properties
      which is incorrect. log4j.properties is located both in jxta.jar and src/test/resources, but I think that src/test/resources takes precedence over jxta. This ordering is set correctly in surefire36745tmp file I think, but surefire seems to ignore the ordering.

      1. my-app.zip
        5 kB
        Martin Vysny
      2. output.log
        22 kB
      3. SUREFIRE61_barrettas_surefire_surefire-booter_for_rev_489098.patch
        2 kB
        Barrett Snyder
      4. SUREFIRE61-surefire-booter-r558713.patch
        2 kB
        Paul Gier

        Issue Links

          Activity

          Martin Vysny created issue -
          Barrett Snyder made changes -
          Field Original Value New Value
          Attachment SUREFIRE61_barrettas_surefire_surefire-booter_for_rev_489098.patch [ 25037 ]
          Brett Porter made changes -
          Fix Version/s 2.4 [ 13243 ]
          kenneyw made changes -
          Complexity Intermediate
          Testcase included yes
          Description Surefire incorrectly interprets classpath ordering.
          Steps to reproduce:

          1. unzip my-app.zip - it's a simple mvn project created with
             mvn archetype:create -DgroupId=com.mycompany.app -DartifactId=my-app
             and lightly patched

          2. mvn test
             in my case, it prints out
          jar:file:/home/vyzivus/.m2/repository/jxta/jxta/2.0/jxta-2.0.jar!/log4j.properties
          jar:file:/home/vyzivus/.m2/repository/jxta/jxta/2.0/jxta-2.0.jar!/log4j.properties
             which is incorrect. log4j.properties is located both in jxta.jar and src/test/resources, but I think that src/test/resources takes precedence over jxta. This ordering is set correctly in surefire36745tmp file I think, but surefire seems to ignore the ordering.
          Surefire incorrectly interprets classpath ordering.
          Steps to reproduce:
          1. unzip my-app.zip - it's a simple mvn project created with
          mvn archetype:create -DgroupId=com.mycompany.app -DartifactId=my-app
          and lightly patched
          2. mvn test
          in my case, it prints out
          jar:file:/home/vyzivus/.m2/repository/jxta/jxta/2.0/jxta-2.0.jar!/log4j.properties
          jar:file:/home/vyzivus/.m2/repository/jxta/jxta/2.0/jxta-2.0.jar!/log4j.properties
          which is incorrect. log4j.properties is located both in jxta.jar and src/test/resources, but I think that src/test/resources takes precedence over jxta. This ordering is set correctly in surefire36745tmp file I think, but surefire seems to ignore the ordering.
          mread made changes -
          Attachment output.log [ 27820 ]
          Michael Joyce made changes -
          Link This issue is duplicated by SUREFIRE-289 [ SUREFIRE-289 ]
          Paul Gier made changes -
          Link This issue is related to MNG-3118 [ MNG-3118 ]
          Paul Gier made changes -
          Attachment SUREFIRE61-surefire-booter-r558713.patch [ 28569 ]
          Brett Porter made changes -
          Fix Version/s 2.3.1 [ 13251 ]
          Patch Submitted [Yes]
          Fix Version/s 2.4 [ 13243 ]
          Brett Porter made changes -
          Resolution Fixed [ 1 ]
          Status Open [ 1 ] Closed [ 6 ]
          Assignee Brett Porter [ brettporter ]
          Benjamin Bentmann made changes -
          Link This issue relates to SUREFIRE-318 [ SUREFIRE-318 ]
          Mark Thomas made changes -
          Project Import Sun Apr 05 13:23:32 UTC 2015 [ 1428240212200 ]
          Mark Thomas made changes -
          Workflow jira [ 12727531 ] Default workflow, editable Closed status [ 12758759 ]
          Mark Thomas made changes -
          Flags Patch [ 10430 ]
          Patch Submitted Yes [ 10763 ]
          Mark Thomas made changes -
          Project Import Mon Apr 06 01:36:33 UTC 2015 [ 1428284193036 ]
          Mark Thomas made changes -
          Workflow jira [ 12965657 ] Default workflow, editable Closed status [ 13003397 ]

            People

            • Assignee:
              Brett Porter
              Reporter:
              Martin Vysny
            • Votes:
              5 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development