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

Broken link on Tap Results

    XMLWordPrintable

Details

    Description

      Tap results generate broken links. I don't know exactly de content is expected to be on those links

      Related code is here:

      https://github.com/jenkinsci/tap-plugin/blob/master/src/main/resources/org/tap4j/plugin/model/TapStreamResult/body.jelly#L66
      https://github.com/jenkinsci/tap-plugin/blob/65b57b2cbbea64c033cd07511ad71f46a370bcee/src/main/java/org/tap4j/plugin/model/TapTestResultResult.java#L171

      e.g.

      Task for https://github.com/albfan/git-ignore

      test stream are generated with

      $ git clone https://github.com/albfan/git-ignore
      $ cd t/
      $ make DEFAULT_TEST_TARGET=jenkins

      test stream outputs on tapout/ dir.

      Here they are:

      t/tapout/t0001-exclude.sh

      ok 1 - exclude a single file
      ok 2 - exclude a bunch of files

      1. passed all 2 test(s)
        1..2

      t/tapout/t0002-ignore.sh

      ok 1 - Ignoring a single file
      ok 2 - Ignoring a bunch of files

      1. passed all 2 test(s)
        1..2

      Here is a generated link for one of these 4 test

      <jenkins-url>/job/git-ignore/10/tapTestReport/t%2Ftapout%2Ft0001-exclude.sh-1/

      Attachments

        Activity

          Fixed in 1.23

          kinow Bruno P. Kinoshita added a comment - Fixed in 1.23

          Had to fix the URL prefix in Jenkins. Also, updating the minimum version to check compatibility. Also had to change a few parts of the code to be compatible with API changes in Jenkins.

          Let me know if that works. Thanks for reporting the issue.

          Bruno

          kinow Bruno P. Kinoshita added a comment - Had to fix the URL prefix in Jenkins. Also, updating the minimum version to check compatibility. Also had to change a few parts of the code to be compatible with API changes in Jenkins. Let me know if that works. Thanks for reporting the issue. Bruno

          Code changed in jenkins
          User: Bruno P. Kinoshita
          Path:
          pom.xml
          src/main/java/org/tap4j/plugin/TapProjectAction.java
          src/main/java/org/tap4j/plugin/TapPublisher.java
          src/main/java/org/tap4j/plugin/TapResult.java
          src/main/java/org/tap4j/plugin/model/TapTestResultResult.java
          src/main/resources/org/tap4j/plugin/TapBuildAction/summary.jelly
          src/main/resources/org/tap4j/plugin/TapResult/index.jelly
          http://jenkins-ci.org/commit/tap-plugin/a410f4c079e1062bf09f7a55f225ffe728cc581a
          Log:
          [FIXED JENKINS-28508] Fix broken links on Tap Results

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Bruno P. Kinoshita Path: pom.xml src/main/java/org/tap4j/plugin/TapProjectAction.java src/main/java/org/tap4j/plugin/TapPublisher.java src/main/java/org/tap4j/plugin/TapResult.java src/main/java/org/tap4j/plugin/model/TapTestResultResult.java src/main/resources/org/tap4j/plugin/TapBuildAction/summary.jelly src/main/resources/org/tap4j/plugin/TapResult/index.jelly http://jenkins-ci.org/commit/tap-plugin/a410f4c079e1062bf09f7a55f225ffe728cc581a Log: [FIXED JENKINS-28508] Fix broken links on Tap Results

          People

            kinow Bruno P. Kinoshita
            albfan Alberto Fanjul Alonso
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: