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

Combining with xUnit Plugin cause "Failed to instantiate class org.jenkins_ci.plugins.flexible_publish.ConditionalPublisher"

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Won't Fix
    • Labels:
      None
    • Environment:
      Jenkins 1.532.1
      Flexible Publish plugin 0.12
      Maven Integration plugin 2.0.1
      xUnit plugin 1.80
    • Similar Issues:

      Description

      How to recreate:

      • Create a new Maven 2/3 job.
      • Add the flexible publish post-build action
      • Add one condition action (I used xUnit with jUnit on Files Match)
      • Save
      • Edit again, and add another conditional action (I used the JaCoCo publisher with Files Match).
      • Save

      I get: Failed to instantiate class org.jenkins_ci.plugins.flexible_publish.ConditionalPublisher

      I expected it to save.

      Ciao!

        Attachments

          Issue Links

            Activity

            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: ikedam
            Path:
            src/main/java/org/jenkins_ci/plugins/flexible_publish/ConditionalPublisher.java
            http://jenkins-ci.org/commit/flexible-publish-plugin/9145b7c4efeed18c0df027f8e378ce099edda8e4
            Log:
            JENKINS-21497 Fixed a wrong error message when failed to instantiate a publisher.

            Compare: https://github.com/jenkinsci/flexible-publish-plugin/compare/f60940f04492...9145b7c4efee

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: ikedam Path: src/main/java/org/jenkins_ci/plugins/flexible_publish/ConditionalPublisher.java http://jenkins-ci.org/commit/flexible-publish-plugin/9145b7c4efeed18c0df027f8e378ce099edda8e4 Log: JENKINS-21497 Fixed a wrong error message when failed to instantiate a publisher. Compare: https://github.com/jenkinsci/flexible-publish-plugin/compare/f60940f04492...9145b7c4efee
            Hide
            ikedam ikedam added a comment -

            xunit-plugin generates broken HTML like this:

                    <tr>
                      <td class="setting-leftspace"> </td>
                      <td class="setting-name">Choose your threshold mode</td>
                      <td class="setting-main">
                       <input name="thresholdMode" value="1" type="radio" checked="true" />
                       <label class="attach-previous">Use a number of tests</label>
                       <br />
                       <input name="thresholdMode" value="2" type="radio" /><label class="attach-previous">Use a percent of tests</label>
                      </td>
                    </tr>
                    <tr class="validation-error-area"><td colspan="2"></td><td></td></tr>
                  </tbody></table>
                </td></tr>
              </td></tr>
              <br /><br />  <!-- This is wrong HTML!!!! -->
              <tr><td colspan="3">
                <tr>
                  <td></td>
                  <td></td>
                  <td>
            

            Web browsers converts this HTML like this:

            ...
            <tr class="dropdownList-container">
              <td colspan="2"></td>
              <td colspan="2">
                <br /><br />  <!-- wrong HTML comes here -->
                <table name="publisher" width="100%">
                <tbody>
                  <tr class="dropdownList-start rowvg-start">
                  <td style="display:none">
                     <input name="stapler-class" value="hudson.tasks.test.AggregatedTestResultPublisher" type="hidden">
                  </td>
            ...
            

            This cause dropdownList work wrong as invalid "br" tags prevents it find out "tr.dropdownList-start".

            Show
            ikedam ikedam added a comment - xunit-plugin generates broken HTML like this: <tr> <td class="setting-leftspace"> </td> <td class="setting-name">Choose your threshold mode</td> <td class="setting-main"> <input name="thresholdMode" value="1" type="radio" checked="true" /> <label class="attach-previous">Use a number of tests</label> <br /> <input name="thresholdMode" value="2" type="radio" /><label class="attach-previous">Use a percent of tests</label> </td> </tr> <tr class="validation-error-area"><td colspan="2"></td><td></td></tr> </tbody></table> </td></tr> </td></tr> <br /><br /> <!-- This is wrong HTML!!!! --> <tr><td colspan="3"> <tr> <td></td> <td></td> <td> Web browsers converts this HTML like this: ... <tr class="dropdownList-container"> <td colspan="2"></td> <td colspan="2"> <br /><br /> <!-- wrong HTML comes here --> <table name="publisher" width="100%"> <tbody> <tr class="dropdownList-start rowvg-start"> <td style="display:none"> <input name="stapler-class" value="hudson.tasks.test.AggregatedTestResultPublisher" type="hidden"> </td> ... This cause dropdownList work wrong as invalid "br" tags prevents it find out "tr.dropdownList-start".
            Hide
            ikedam ikedam added a comment -

            I created a ticket for xunit plugin: JENKINS-21532
            This would be fixed in that ticket. Won't fix in this ticket.

            Please contact to the maintainer of xunit plugin if you want to hurry that fix.

            Show
            ikedam ikedam added a comment - I created a ticket for xunit plugin: JENKINS-21532 This would be fixed in that ticket. Won't fix in this ticket. Please contact to the maintainer of xunit plugin if you want to hurry that fix.
            Hide
            docwhat Christian Höltje added a comment -

            Fast work! You guys rock!

            Show
            docwhat Christian Höltje added a comment - Fast work! You guys rock!
            Hide
            ikedam ikedam added a comment -

            Fixed in 0.13.
            It will be available in a day.
            Please try that.

            Show
            ikedam ikedam added a comment - Fixed in 0.13. It will be available in a day. Please try that.

              People

              • Assignee:
                ikedam ikedam
                Reporter:
                docwhat Christian Höltje
              • Votes:
                4 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: