• Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • maven-plugin
    • None

      The current implementation of Hudson's Maven support needlessly requires the use of maven-surefire-plugin for executing tests. If an alternative test runner plugin is used, Hudson's maven-plugin will not recognize test results, meaning that UNSTABLE builds are wrongly considered to be FAILED builds, and also that build status does not include test result reports.

      This behavior is due to the fact that the SurefireArchiver class includes conditional logic that insists that the test runner plugin is Surefire. If this conditional check is removed, then test results are correctly found and interpreted, and unstable builds are no longer considered failed builds.

      I've attached a simple patch that demonstrates this. Even though the contents of the patch may not be the correct fix, it at least illustrates the nature of the problem.

          [JENKINS-8334] maven-plugin insists on Surefire to run tests

          jvoegele created issue -
          kutzi made changes -
          Assignee New: kutzi [ kutzi ]
          kutzi made changes -
          Link New: This issue is duplicated by JENKINS-13188 [ JENKINS-13188 ]
          Alexander Bertram made changes -
          Attachment New: maven-plugin-kutzi.hpi [ 23042 ]
          SCM/JIRA link daemon made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]
          kutzi made changes -
          Link New: This issue is blocking JENKINS-16121 [ JENKINS-16121 ]
          kutzi made changes -
          Link New: This issue is related to JENKINS-16776 [ JENKINS-16776 ]
          Jenkins IRC Bot made changes -
          Component/s New: maven-plugin [ 16033 ]
          Component/s Original: maven2 [ 15487 ]
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 138444 ] New: JNJira + In-Review [ 188006 ]

            kutzi kutzi
            jvoegele jvoegele
            Votes:
            2 Vote for this issue
            Watchers:
            8 Start watching this issue

              Created:
              Updated:
              Resolved: