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

Do not fail when no Nunit test reports found

    XMLWordPrintable

    Details

    • Type: New Feature
    • Status: Resolved (View Workflow)
    • Priority: Major
    • Resolution: Fixed
    • Component/s: nunit-plugin
    • Labels:
      None
    • Environment:
      Jenkins 1.585, NUnuit plugin 0.16
    • Similar Issues:

      Description

      Currently when no report files are found we get the following console output and a failed build:

      FATAL: No NUnit test report files were found. Configuration error?

      However because our build job is parameterised, we do not always generate test results, and there is no way to parameterise the "Public NUnit test result report" Post-build Action.

      It would make sense for there to be an advanced option that does not fail the build if no reports were found, much like for the "Archive the artifacts" Post-build Action.

        Attachments

          Activity

          Hide
          garrettendi Dale Hards added a comment -

          Reassigning to redsolo who is the maintainer for the NUnit plugin

          Show
          garrettendi Dale Hards added a comment - Reassigning to redsolo who is the maintainer for the NUnit plugin
          Hide
          garrettendi Dale Hards added a comment -

          I'd like to check this bug has been noticed? It is causing a fairly major headache on our build system.

          Show
          garrettendi Dale Hards added a comment - I'd like to check this bug has been noticed? It is causing a fairly major headache on our build system.
          Hide
          garrettendi Dale Hards added a comment -

          Reassigning back to Bruno Kinoshita who I notice is the lead on the nunit-plugin

          Show
          garrettendi Dale Hards added a comment - Reassigning back to Bruno Kinoshita who I notice is the lead on the nunit-plugin
          Hide
          kinow Bruno P. Kinoshita added a comment -

          Hello Dale.

          Doesn't sound so hard to implement this new feature. IIRC I had to do something similar in TAP Plug-in, since I mimicked Jenkins JUnit reports behaviour there.

          I'll try to tackle this issue next Sunday, but drop a note here if you have a pull request before that. I can merge it and cut a release right away too.

          Thanks
          Bruno

          Show
          kinow Bruno P. Kinoshita added a comment - Hello Dale. Doesn't sound so hard to implement this new feature. IIRC I had to do something similar in TAP Plug-in, since I mimicked Jenkins JUnit reports behaviour there. I'll try to tackle this issue next Sunday, but drop a note here if you have a pull request before that. I can merge it and cut a release right away too. Thanks Bruno
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Bruno P. Kinoshita
          Path:
          .gitignore
          src/main/java/hudson/plugins/nunit/NUnitArchiver.java
          src/main/java/hudson/plugins/nunit/NUnitPublisher.java
          src/main/resources/hudson/plugins/nunit/NUnitPublisher/config.jelly
          src/test/java/hudson/plugins/nunit/NUnitArchiverTest.java
          src/test/java/hudson/plugins/nunit/NUnitPublisherTest.java
          http://jenkins-ci.org/commit/nunit-plugin/e6e5f01c60476b6949b823e14522d4a0c15ea780
          Log:
          [FIXED JENKINS-25295] Do not fail when no Nunit test reports found

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Bruno P. Kinoshita Path: .gitignore src/main/java/hudson/plugins/nunit/NUnitArchiver.java src/main/java/hudson/plugins/nunit/NUnitPublisher.java src/main/resources/hudson/plugins/nunit/NUnitPublisher/config.jelly src/test/java/hudson/plugins/nunit/NUnitArchiverTest.java src/test/java/hudson/plugins/nunit/NUnitPublisherTest.java http://jenkins-ci.org/commit/nunit-plugin/e6e5f01c60476b6949b823e14522d4a0c15ea780 Log: [FIXED JENKINS-25295] Do not fail when no Nunit test reports found
          Hide
          kinow Bruno P. Kinoshita added a comment -

          Done, but I couldn't extensively test it. Thus, I'll wait till someone else can `mvn clean hpi:run` and test it, or `mvn clean package` and manually install the plug-in in some test instance and give some positive feedback. Then we can cut a new release with this fix.

          Thanks!
          Bruno

          Show
          kinow Bruno P. Kinoshita added a comment - Done, but I couldn't extensively test it. Thus, I'll wait till someone else can `mvn clean hpi:run` and test it, or `mvn clean package` and manually install the plug-in in some test instance and give some positive feedback. Then we can cut a new release with this fix. Thanks! Bruno
          Hide
          garrettendi Dale Hards added a comment -

          Hi Bruno, thanks for fixing the issue so quickly. I'm afraid we won't be able to test it in the way you describe as ours is a live system that is in constant use and we can't really risk upsetting things.

          Would there be someone else in the project who could test?

          Show
          garrettendi Dale Hards added a comment - Hi Bruno, thanks for fixing the issue so quickly. I'm afraid we won't be able to test it in the way you describe as ours is a live system that is in constant use and we can't really risk upsetting things. Would there be someone else in the project who could test?

            People

            Assignee:
            kinow Bruno P. Kinoshita
            Reporter:
            garrettendi Dale Hards
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: