Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-6545

JUnit report parsing cannot handle nested suites

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

      JUnit allows to nest testsuites indefinetly (using TestSuite.addTestSuite()) which results in xml files that have <testsuites> element as root and under that a hierarchy of <testsuite> elements. Currently the SuiteResult class in Hudson can only handle a single level of nesting, which results in the "None of the test reports contained any result" error in the build.

      So my junit result looks like this:

      <?xml version="1.0" encoding="UTF-8"?>
      <testsuites name="Automated Tests" tests="186" errors="0" failures="0" ignored="0">
        <testsuite name="test.AutomatedTests" time="4.486">
          <testsuite name="test.fs.FileSystemTest" time="0.014">
            <testcase name="testPrefix" classname="test.fs.FileSystemTest" time="0.003"/>
          </testsuite>
          <testsuite name="test.db.DatabaseTest" time="0.014">
            <testcase name="testTable" classname="test.db.DatabaseTest" time="0.003"/>
          </testsuite>
        </testsuite>
      </testsuites
      

          [JENKINS-6545] JUnit report parsing cannot handle nested suites

          nukleon created issue -

          ewysp added a comment -

          Try the xUnit plug in. I ran into the same issue as you, and found this plugin to suit our needs ... for now.

          http://wiki.jenkins-ci.org/display/JENKINS/xUnit+Plugin

          ewysp added a comment - Try the xUnit plug in. I ran into the same issue as you, and found this plugin to suit our needs ... for now. http://wiki.jenkins-ci.org/display/JENKINS/xUnit+Plugin

          MichaelBreu added a comment -

          Hello ewvsg,

          It would be helpful if you could include here your XSLT-file needed for the xUnit-Workaround.

          Michael

          MichaelBreu added a comment - Hello ewvsg, It would be helpful if you could include here your XSLT-file needed for the xUnit-Workaround. Michael

          Here is a description of the XSL needed to transform the nested junit xml output into something which can be parsed by Jenkins.

          http://stackoverflow.com/questions/4640300/hudson-ci-and-phpunit-none-of-the-test-reports-contained-any-result

          Russell Francis added a comment - Here is a description of the XSL needed to transform the nested junit xml output into something which can be parsed by Jenkins. http://stackoverflow.com/questions/4640300/hudson-ci-and-phpunit-none-of-the-test-reports-contained-any-result

          Code changed in jenkins
          User: Kohsuke Kawaguchi
          Path:
          changelog.html
          core/src/main/java/hudson/tasks/junit/SuiteResult.java
          http://jenkins-ci.org/commit/jenkins/4129ac50a39ae2522c20ce73e2b10872a07c04be
          Log:
          [FIXED JENKINS-6545] support nested test suites

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Kohsuke Kawaguchi Path: changelog.html core/src/main/java/hudson/tasks/junit/SuiteResult.java http://jenkins-ci.org/commit/jenkins/4129ac50a39ae2522c20ce73e2b10872a07c04be Log: [FIXED JENKINS-6545] support nested test suites
          SCM/JIRA link daemon made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]

          dogfood added a comment -

          Integrated in jenkins_main_trunk #648
          [FIXED JENKINS-6545] support nested test suites

          Kohsuke Kawaguchi : 4129ac50a39ae2522c20ce73e2b10872a07c04be
          Files :

          • core/src/main/java/hudson/tasks/junit/SuiteResult.java
          • changelog.html

          dogfood added a comment - Integrated in jenkins_main_trunk #648 [FIXED JENKINS-6545] support nested test suites Kohsuke Kawaguchi : 4129ac50a39ae2522c20ce73e2b10872a07c04be Files : core/src/main/java/hudson/tasks/junit/SuiteResult.java changelog.html

          Code changed in jenkins
          User: Stephen Connolly
          Path:
          test/src/main/java/org/jvnet/hudson/test/TestExtensionLoader.java
          http://jenkins-ci.org/commit/jenkins/97f1d02cbdc0aa85d2e907d878b126e80fd0880b
          Log:
          Tsk Tsk! Kohsuke you missed a possible NPE for people running hpi:run on plugins in fixing JENKINS-6545

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Stephen Connolly Path: test/src/main/java/org/jvnet/hudson/test/TestExtensionLoader.java http://jenkins-ci.org/commit/jenkins/97f1d02cbdc0aa85d2e907d878b126e80fd0880b Log: Tsk Tsk! Kohsuke you missed a possible NPE for people running hpi:run on plugins in fixing JENKINS-6545

          dogfood added a comment -

          Integrated in jenkins_main_trunk #649
          Tsk Tsk! Kohsuke you missed a possible NPE for people running hpi:run on plugins in fixing JENKINS-6545

          Stephen Connolly : 97f1d02cbdc0aa85d2e907d878b126e80fd0880b
          Files :

          • test/src/main/java/org/jvnet/hudson/test/TestExtensionLoader.java

          dogfood added a comment - Integrated in jenkins_main_trunk #649 Tsk Tsk! Kohsuke you missed a possible NPE for people running hpi:run on plugins in fixing JENKINS-6545 Stephen Connolly : 97f1d02cbdc0aa85d2e907d878b126e80fd0880b Files : test/src/main/java/org/jvnet/hudson/test/TestExtensionLoader.java

          This is my fix, at least you could mention my name in the changeset... since you copied it from the Hudson changeset i made few days ago

          https://github.com/hudson/hudson/commit/3d79ac89df3a3107904fa3b91b5d86075c86a5dd

          I wanted to commit the fix to Jenkins but nobody answer me about how was the procedure in the developers list...

          Victor Garcia Perez added a comment - This is my fix, at least you could mention my name in the changeset... since you copied it from the Hudson changeset i made few days ago https://github.com/hudson/hudson/commit/3d79ac89df3a3107904fa3b91b5d86075c86a5dd I wanted to commit the fix to Jenkins but nobody answer me about how was the procedure in the developers list...

            Unassigned Unassigned
            nukleon nukleon
            Votes:
            3 Vote for this issue
            Watchers:
            7 Start watching this issue

              Created:
              Updated:
              Resolved: