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

TestNG 0.31 does not show test results when tests failed

    Details

    • Similar Issues:

      Description

      Whenever I use the TestNG-Plugin 0.31 it only shows reports if the tests passed. But if the failey there are no test results. If I downgrade to 0.26 for example everthing works fine. Both the failed and passed test results are displayed. Please fix this, would be very important. Thank you!

        Attachments

          Issue Links

            Activity

            Hide
            nalin_makar nalin_makar added a comment -

            kutzi,

            When user aborts a bit, the current executing step should stop asap, but because testng plugin only reports results and doesnt really execute any build steps or modify the workspace, it might be beneficial to run it. Simple use case would be where there are multiple calls to running tests and even though we want to abort the build, we want to make sure that results that were already published are parsed and displayed.

            Show
            nalin_makar nalin_makar added a comment - kutzi, When user aborts a bit, the current executing step should stop asap, but because testng plugin only reports results and doesnt really execute any build steps or modify the workspace, it might be beneficial to run it. Simple use case would be where there are multiple calls to running tests and even though we want to abort the build, we want to make sure that results that were already published are parsed and displayed.
            Hide
            gurkensalat Peter Lustig added a comment -

            Okay, so my testNG-Tests fail and this makes the whole build to fail. And that is the reason why no results are parsed and displayed? I definitely think this is not the right behaviour. I mean, it makes sense, to mark a build as fail if the tests failed. And it definitely makes sense to display test results if the tests failed. But nothing of this behaviour is implemented at the current moment. I would really appreciate it if one can see the results if tests fail.

            Show
            gurkensalat Peter Lustig added a comment - Okay, so my testNG-Tests fail and this makes the whole build to fail. And that is the reason why no results are parsed and displayed? I definitely think this is not the right behaviour. I mean, it makes sense, to mark a build as fail if the tests failed. And it definitely makes sense to display test results if the tests failed. But nothing of this behaviour is implemented at the current moment. I would really appreciate it if one can see the results if tests fail.
            Hide
            nalin_makar nalin_makar added a comment -

            Fix is to revert the change made for fixing JENKINS-12046.

            Show
            nalin_makar nalin_makar added a comment - Fix is to revert the change made for fixing JENKINS-12046 .
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Nalin Makar
            Path:
            README
            src/main/java/hudson/plugins/testng/Publisher.java
            http://jenkins-ci.org/commit/testng-plugin-plugin/523dfea36f91a1425d1f377101e9207d02a48afc
            Log:
            Fixed: JENKINS-12648 - Results should be reported for failed/aborted builds as well (Reverted fix for JENKINS-12046 in v0.30)

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Nalin Makar Path: README src/main/java/hudson/plugins/testng/Publisher.java http://jenkins-ci.org/commit/testng-plugin-plugin/523dfea36f91a1425d1f377101e9207d02a48afc Log: Fixed: JENKINS-12648 - Results should be reported for failed/aborted builds as well (Reverted fix for JENKINS-12046 in v0.30)
            Hide
            nalin_makar nalin_makar added a comment -

            Fix will be available as part of v0.32

            Show
            nalin_makar nalin_makar added a comment - Fix will be available as part of v0.32

              People

              • Assignee:
                nalin_makar nalin_makar
                Reporter:
                gurkensalat Peter Lustig
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: