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

TAP Plugin generates bad detail links on "tapTestReport" page

    XMLWordPrintable

Details

    Description

      (See this page for the original report of this bug in context)

      The fix for JENKINS-16971 (e8dcf5c) breaks my Jenkins TAP plugin installation. Specifically, the detail links on the http://jenkins.example.org/job/JOBNAME/BUILDNUM/tapTestReport/ page are incorrect. They should read something like http://jenkins.example.org/job/JOBNAME/BUILDNUM/tapTestReport/summary.tap-1/. Instead they read something like http://jenkins.example.org/var/lib/jenkins/jobs/JOBNAME/builds/2013-04-04_12-43-45/tap-master-files/test/mpi/summary.tap-1/. Obviously this URL is invalid and yields an HTTP 404 from the web server.
      I suspect the problem is caused by the replacement of getName() with getAbsolutePath(). Reverting commit e8dcf5c fixes the error in my otherwise clean build of 80a01b1. I made no attempt to test or understand the issue actually reported in and/or addressed by JENKINS-16971.

      Attachments

        Activity

          Fixed in 1.12

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

          Fixed on issues 18885 and 17855, with commit http://jenkins-ci.org/commit/tap-plugin/21e7af07d5ae18327eb7a2626184970a6ff1d995

          Sorry for mixing issues in the same commit, but I started working on 18885 this morning and it included this issue too.

          Cutting a new release tonight, tomorrow morning/noon you should be able to give 1.12 a try.

          Thanks!

          kinow Bruno P. Kinoshita added a comment - Fixed on issues 18885 and 17855, with commit http://jenkins-ci.org/commit/tap-plugin/21e7af07d5ae18327eb7a2626184970a6ff1d995 Sorry for mixing issues in the same commit, but I started working on 18885 this morning and it included this issue too. Cutting a new release tonight, tomorrow morning/noon you should be able to give 1.12 a try. Thanks!

          No problem. Let me know if you need any more info. I've gone back to my custom 1.11 snapshot with the troublesome commit Dave cited git revert'ed out.

          raffenet Ken Raffenetti added a comment - No problem. Let me know if you need any more info. I've gone back to my custom 1.11 snapshot with the troublesome commit Dave cited git revert'ed out.

          Reopening due to regression

          kinow Bruno P. Kinoshita added a comment - Reopening due to regression

          @Ken, weird. I'll take a look tomorrow Ken. Sorry for that :-/

          kinow Bruno P. Kinoshita added a comment - @Ken, weird. I'll take a look tomorrow Ken. Sorry for that :-/

          People

            kinow Bruno P. Kinoshita
            goodell Dave Goodell
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: