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

Column "Test Result" has incorrect link

    XMLWordPrintable

Details

    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

          depends on

          Bug - A problem which impairs or prevents the functions of the product. JENKINS-25113 Column extensions: job.url in column.jelly is wrong in non-default views

          • Major - Major loss of function.
          • Resolved
          is duplicated by

          Bug - A problem which impairs or prevents the functions of the product. JENKINS-25274 The link to "Failed Tests" is invalid in custom views

          • Major - Major loss of function.
          • Resolved

          Bug - A problem which impairs or prevents the functions of the product. JENKINS-25425 Wrong link in Test Result column in custom view

          • Major - Major loss of function.
          • Resolved

          Bug - A problem which impairs or prevents the functions of the product. JENKINS-26325 Invalid test results link

          • Major - Major loss of function.
          • Resolved

          Bug - A problem which impairs or prevents the functions of the product. JENKINS-23624 Incorrect Test Result link

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. JENKINS-26526 Test results link only works from "All" tab

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Closed
          is related to

          Bug - A problem which impairs or prevents the functions of the product. JENKINS-25334 Percentage link does not work in non-default views

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Closed

          Activity

            Ascending order - Click to sort in descending order
            centic centic added a comment -

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

            centic centic added a comment - I have reported a general bug-report for this at JENKINS-25113
            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.

            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.
            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:

            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
            fredg Fred G added a comment -

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

            fredg Fred G added a comment - Thanks Ulli for the heads-up. I will take a look.
            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.

            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.
            igorkostenko Igor Kostenko added a comment -

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

            igorkostenko Igor Kostenko added a comment - Matthieu, could you upload plugin jar with PR 11 somewhere?
            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

            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
            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

            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

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

              Dates

                Created:
                Updated:
                Resolved: