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

Compatibility for tap plugin with pipeline plugin

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      Using pipeline plugin https://wiki.jenkins-ci.org/display/JENKINS/Pipeline+Multibranch+Plugin I cannot see TAP options in left pane

      Are them compatible?

       

        Attachments

          Activity

          Hide
          albfan Alberto Fanjul Alonso added a comment -

          Trying step([$class: "TapPublisher", testResults: "test-suite.log"])

          Show
          albfan Alberto Fanjul Alonso added a comment - Trying step( [$class: "TapPublisher", testResults: "test-suite.log"] )
          Hide
          oleg_nenashev Oleg Nenashev added a comment -

          Which version of Tap plugin do you use?

          Show
          oleg_nenashev Oleg Nenashev added a comment - Which version of Tap plugin do you use?
          Hide
          oleg_nenashev Oleg Nenashev added a comment -

          I am not sure what is missing there, we are using TAP plugin with Pipeline on our instance, it works well.
          One thing is missing though: Descriptor in Pipeline Publisher does not have a @Symbol annotation: https://github.com/jenkinsci/tap-plugin/blob/master/src/main/java/org/tap4j/plugin/TapPublisher.java#L668-L669

          If it is implemented, the call pattern could be publishTap(testResults: "test-suite.log").

          What needs to be done:

          • Add a symbol
          • Add a unit test, which would confirm that the new feature works correctly

          It is a newbie-friendly issue IMO

          Show
          oleg_nenashev Oleg Nenashev added a comment - I am not sure what is missing there, we are using TAP plugin with Pipeline on our instance, it works well. One thing is missing though: Descriptor in Pipeline Publisher does not have a @Symbol annotation: https://github.com/jenkinsci/tap-plugin/blob/master/src/main/java/org/tap4j/plugin/TapPublisher.java#L668-L669 If it is implemented, the call pattern could be publishTap(testResults: "test-suite.log"). What needs to be done: Add a symbol Add a unit test, which would confirm that the new feature works correctly It is a newbie-friendly issue IMO

            People

            Assignee:
            Unassigned Unassigned
            Reporter:
            albfan Alberto Fanjul Alonso
            Votes:
            1 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated: