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

Single malformed junit result causes failure to import rest of junit results.

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Component/s: junit-plugin
    • Labels:
      None
    • Environment:
      Platform: PC, OS: Windows XP

      Description

      Background:

      I am using Hudson for a C++ project. I have created a job which acts as a
      regression test (via some scripting). This script creates several XML files in
      the junit report format--it does not actually use Junit.

      Problem:

      If one of the junit results files is malformed enough to cause the XML parser
      to fail (for example, a segfault in the script causes the XML file to be
      incomplete), none of the remaining (or already parsed) results files, which may
      be completely valid, are imported into Hudson. The build results will not show
      any test results.

      I would expect only the test results from the malformed results file to be
      missing from the build status--not all of test results.

        Issue Links

          Activity

          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in hudson
          User: : huybrechts
          Path:
          branches/junit/main/core/src/main/java/hudson/tasks/junit/TestResult.java
          http://fisheye4.cenqua.com/changelog/hudson/?cs=20536
          Log:
          tentative fix for JENKINS-3149 – if file is .xml but does not parse, record a fake failing test

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in hudson User: : huybrechts Path: branches/junit/main/core/src/main/java/hudson/tasks/junit/TestResult.java http://fisheye4.cenqua.com/changelog/hudson/?cs=20536 Log: tentative fix for JENKINS-3149 – if file is .xml but does not parse, record a fake failing test
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in hudson
          User: : huybrechts
          Path:
          trunk/hudson/main/core/src/main/java/hudson/tasks/junit/CaseResult.java
          trunk/hudson/main/core/src/main/java/hudson/tasks/junit/ClassResult.java
          trunk/hudson/main/core/src/main/java/hudson/tasks/junit/JUnitResultArchiver.java
          trunk/hudson/main/core/src/main/java/hudson/tasks/junit/PackageResult.java
          trunk/hudson/main/core/src/main/java/hudson/tasks/junit/SuiteResult.java
          trunk/hudson/main/core/src/main/java/hudson/tasks/junit/TabulatedResult.java
          trunk/hudson/main/core/src/main/java/hudson/tasks/junit/TestObject.java
          trunk/hudson/main/core/src/main/java/hudson/tasks/junit/TestResult.java
          trunk/hudson/main/core/src/main/java/hudson/tasks/junit/TestResultAction.java
          trunk/hudson/main/core/src/main/resources/hudson/model/AbstractBuild/sidepanel.jelly
          trunk/hudson/main/core/src/main/resources/hudson/tasks/junit/CaseResult/index.jelly
          trunk/hudson/main/core/src/main/resources/hudson/tasks/junit/ClassResult/body.jelly
          trunk/hudson/main/core/src/main/resources/hudson/tasks/junit/JUnitResultArchiver/config.jelly
          trunk/hudson/main/core/src/main/resources/hudson/tasks/junit/MetaTabulatedResult/body.jelly
          trunk/hudson/main/core/src/main/resources/hudson/tasks/junit/TabulatedResult/index.jelly
          trunk/hudson/main/core/src/main/resources/hudson/tasks/test/MatrixTestResult/index.jelly
          trunk/hudson/plugins/claim/pom.xml
          trunk/hudson/plugins/claim/src/main/java/hudson/plugins/claim/ClaimBuildAction.java
          trunk/hudson/plugins/claim/src/main/java/hudson/plugins/claim/ClaimPublisher.java
          trunk/hudson/plugins/claim/src/main/webapp/help-sticky.html
          trunk/hudson/pom.xml
          http://fisheye4.cenqua.com/changelog/hudson/?cs=20580
          Log:
          Merging in the junit branch. Lots of new features and bugfixes, including:

          • support for TestAction that can contribute various things to any TestObject (packages, classes and methods), using the TestDataPublisher extension point.
            Features include UI on the TestObject page, badges on the overview page, tasks in the side panel and annotating test output.
          • a junit-attachments plugin that uses this to record test attachments (files)
          • an extension to the claim plugin that offers claims on failing tests
          • editable test descriptions (on any TestObject)
          • a history page for all TestObjects (including test count and duration graph)
          • error details for failed tests are shown in a collapsed panel in test overview pages (package+class)
          • [FIXED JENKINS-3149] if file is .xml but does not parse, record a fake failing test
          • [FIXED JENKINS-1820] extra column displaying skipped tests
          • JENKINS-2046 new test packages, classes and methods are shown in bold
          • [FIXED JENKINS-2228] test duration charts on all levels
          • [FIXED JENKINS-3451] expand environment variables in test results path
          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in hudson User: : huybrechts Path: trunk/hudson/main/core/src/main/java/hudson/tasks/junit/CaseResult.java trunk/hudson/main/core/src/main/java/hudson/tasks/junit/ClassResult.java trunk/hudson/main/core/src/main/java/hudson/tasks/junit/JUnitResultArchiver.java trunk/hudson/main/core/src/main/java/hudson/tasks/junit/PackageResult.java trunk/hudson/main/core/src/main/java/hudson/tasks/junit/SuiteResult.java trunk/hudson/main/core/src/main/java/hudson/tasks/junit/TabulatedResult.java trunk/hudson/main/core/src/main/java/hudson/tasks/junit/TestObject.java trunk/hudson/main/core/src/main/java/hudson/tasks/junit/TestResult.java trunk/hudson/main/core/src/main/java/hudson/tasks/junit/TestResultAction.java trunk/hudson/main/core/src/main/resources/hudson/model/AbstractBuild/sidepanel.jelly trunk/hudson/main/core/src/main/resources/hudson/tasks/junit/CaseResult/index.jelly trunk/hudson/main/core/src/main/resources/hudson/tasks/junit/ClassResult/body.jelly trunk/hudson/main/core/src/main/resources/hudson/tasks/junit/JUnitResultArchiver/config.jelly trunk/hudson/main/core/src/main/resources/hudson/tasks/junit/MetaTabulatedResult/body.jelly trunk/hudson/main/core/src/main/resources/hudson/tasks/junit/TabulatedResult/index.jelly trunk/hudson/main/core/src/main/resources/hudson/tasks/test/MatrixTestResult/index.jelly trunk/hudson/plugins/claim/pom.xml trunk/hudson/plugins/claim/src/main/java/hudson/plugins/claim/ClaimBuildAction.java trunk/hudson/plugins/claim/src/main/java/hudson/plugins/claim/ClaimPublisher.java trunk/hudson/plugins/claim/src/main/webapp/help-sticky.html trunk/hudson/pom.xml http://fisheye4.cenqua.com/changelog/hudson/?cs=20580 Log: Merging in the junit branch. Lots of new features and bugfixes, including: support for TestAction that can contribute various things to any TestObject (packages, classes and methods), using the TestDataPublisher extension point. Features include UI on the TestObject page, badges on the overview page, tasks in the side panel and annotating test output. a junit-attachments plugin that uses this to record test attachments (files) an extension to the claim plugin that offers claims on failing tests editable test descriptions (on any TestObject) a history page for all TestObjects (including test count and duration graph) error details for failed tests are shown in a collapsed panel in test overview pages (package+class) [FIXED JENKINS-3149] if file is .xml but does not parse, record a fake failing test [FIXED JENKINS-1820] extra column displaying skipped tests JENKINS-2046 new test packages, classes and methods are shown in bold [FIXED JENKINS-2228] test duration charts on all levels [FIXED JENKINS-3451] expand environment variables in test results path
          Hide
          mindless Alan Harder added a comment -

          marking fixed as per above commit comment

          Show
          mindless Alan Harder added a comment - marking fixed as per above commit comment

            People

            • Assignee:
              huybrechts huybrechts
              Reporter:
              dangthill dangthill
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: