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

Cobertura misconfiguration destroys 'publishers' node

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      The following things happen when the 'sourceEncoding' value is misconfigured in your DSL script (AKA assign a value that isn't in the dropdown):

      When creating the job, the 'publishers' node is generated empty, no matter if other publishers are also defined.
      When regenerating the job, the 'publishers' node is generated properly, but none of the publishers show up in the UI.

      Console output of the seed job doesn't complain about anything.

      Example:

      job("RIP_publishers") {
          publishers {
              cobertura('**/target/site/cobertura/coverage.xml') {
                  sourceEncoding('UTF-8') // Invalid
              }
              git {
                  pushOnlyIfSuccess()
              }
          }
      }
      

        Attachments

          Activity

          thi Thierry Lacour created issue -
          Hide
          daspilker Daniel Spilker added a comment -

          Unfortunately the Cobertura plugin maintains it's own collection of valid encodings, see https://github.com/jenkinsci/cobertura-plugin/blob/cobertura-1.9.7/src/main/java/hudson/plugins/cobertura/renderers/SourceEncoding.java. I'm not going to replicate that in Job DSL. I will probably move the DSL to the Cobertura plugin.

          Show
          daspilker Daniel Spilker added a comment - Unfortunately the Cobertura plugin maintains it's own collection of valid encodings, see https://github.com/jenkinsci/cobertura-plugin/blob/cobertura-1.9.7/src/main/java/hudson/plugins/cobertura/renderers/SourceEncoding.java . I'm not going to replicate that in Job DSL. I will probably move the DSL to the Cobertura plugin.
          daspilker Daniel Spilker made changes -
          Field Original Value New Value
          Component/s cobertura-plugin [ 15500 ]
          Hide
          thi Thierry Lacour added a comment -

          That's understandable.
          I just wanted to get this out in the open since it's a pain to find out what's going wrong in bigger scripts.

          Show
          thi Thierry Lacour added a comment - That's understandable. I just wanted to get this out in the open since it's a pain to find out what's going wrong in bigger scripts.
          rtyler R. Tyler Croy made changes -
          Workflow JNJira [ 168223 ] JNJira + In-Review [ 183019 ]
          jglick Jesse Glick made changes -
          Labels robustness

            People

            • Assignee:
              daspilker Daniel Spilker
              Reporter:
              thi Thierry Lacour
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated: