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

Column "Test Result" has incorrect link

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      After upgrading from Jenkins 1.544, the "Test Result" column appears to have an incorrect link for each job, doubling up on the view path where we only expect one, i.e.:
      http://jenkinsSite:8080/ view/ViewName / view/viewName /job/jobName/lastCompletedBuild/testReport

      Looking at the "TestResultColumn/column.jelly" file, I can see the link as:
      <a href="${job.url}lastCompletedBuild/${tr.urlName}/">

      It would appear that a change made as part of JENKINS-19310, changes the value of ${job.url} thus probably causing this problem.

        Attachments

          Issue Links

            Activity

            Hide
            centic centic added a comment -

            I have reported a general bug-report for this at JENKINS-25113

            Show
            centic centic added a comment - I have reported a general bug-report for this at JENKINS-25113
            Hide
            campos_ddc Diogo Campos added a comment -

            It seems that this only happens for views using a regex to match jobs.

            Jobs selected individually from checkboxes do not have this problem.

            Show
            campos_ddc Diogo Campos added a comment - It seems that this only happens for views using a regex to match jobs. Jobs selected individually from checkboxes do not have this problem.
            Show
            drulli Ulli Hafner added a comment - Fred, I think you need to check all your columns. job.getUrl() now returns a url containing the view part. See discussions: https://github.com/jenkinsci/cobertura-plugin/commit/1a899111888f458109b47758a0c667c07c3995b8#commitcomment-8302902 https://github.com/jenkinsci/cobertura-plugin/commit/7fd072ed198217974ef275d5662bf882f92147dd JENKINS-24436
            Hide
            fredg Fred G added a comment -

            Thanks Ulli for the heads-up.
            I will take a look.

            Show
            fredg Fred G added a comment - Thanks Ulli for the heads-up. I will take a look.
            Hide
            maoueh Matthieu Ouellette-Vachon added a comment -

            @stephenthornhill made a PR on github to fix this issue PR #11.

            I tested it in my instance and it works like a charm.

            Show
            maoueh Matthieu Ouellette-Vachon added a comment - @stephenthornhill made a PR on github to fix this issue PR #11 . I tested it in my instance and it works like a charm.
            Hide
            igorkostenko Igor Kostenko added a comment -

            Matthieu, could you upload plugin jar with PR 11 somewhere?

            Show
            igorkostenko Igor Kostenko added a comment - Matthieu, could you upload plugin jar with PR 11 somewhere?
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: stephenthornhill
            Path:
            src/main/resources/jenkins/plugins/extracolumns/TestResultColumn/column.jelly
            http://jenkins-ci.org/commit/extra-columns-plugin/8fee58bbe7f2e6c496f8f64bee3dd23a18a69669
            Log:
            [FIXED JENKINS-23583] Test result url made absolute by prepending rootURL

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: stephenthornhill Path: src/main/resources/jenkins/plugins/extracolumns/TestResultColumn/column.jelly http://jenkins-ci.org/commit/extra-columns-plugin/8fee58bbe7f2e6c496f8f64bee3dd23a18a69669 Log: [FIXED JENKINS-23583] Test result url made absolute by prepending rootURL
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Fred G
            Path:
            src/main/resources/jenkins/plugins/extracolumns/TestResultColumn/column.jelly
            http://jenkins-ci.org/commit/extra-columns-plugin/3ec36ffb2e303757f9033bba9b8a5df9a6c6dc0b
            Log:
            Merge pull request #11 from stephenthornhill/master

            [FIXED JENKINS-23583] Test result url made absolute by prepending rootUR...

            Compare: https://github.com/jenkinsci/extra-columns-plugin/compare/90db80f86d0f...3ec36ffb2e30

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Fred G Path: src/main/resources/jenkins/plugins/extracolumns/TestResultColumn/column.jelly http://jenkins-ci.org/commit/extra-columns-plugin/3ec36ffb2e303757f9033bba9b8a5df9a6c6dc0b Log: Merge pull request #11 from stephenthornhill/master [FIXED JENKINS-23583] Test result url made absolute by prepending rootUR... Compare: https://github.com/jenkinsci/extra-columns-plugin/compare/90db80f86d0f...3ec36ffb2e30

              People

              • Assignee:
                fredg Fred G
                Reporter:
                amolago Chris Lake
              • Votes:
                8 Vote for this issue
                Watchers:
                17 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: