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

All links on trend graph map are wrong on "testngreports" subpage

    XMLWordPrintable

Details

    Description

      The TestNG Plugin for Jenkins adds a new menu entrry to all projects called "TestNG Results" which, when clicked, leads to a page called "TestNG Results Trends" with the URL "/job/projectname/testngreports/".

      On that page a HTML image map shows the trend and offers links to the single builds. However, the anchors in the imagemap are all like "BUILDNUMBER/testngreports/". What they should point to instead is "../BUILDNUMBER/testngreports" - note the "../".

      I guess the devs simply reused code from the project overview widget, where the URLs work together nicely.
      As it is now, however, the testng report page contains links to non-existent pages, leading to 404.

      Attachments

        Activity

          Code changed in jenkins
          User: Nalin Makar
          Path:
          README
          src/main/java/hudson/plugins/testng/util/GraphHelper.java
          src/main/resources/hudson/plugins/testng/TestNGProjectAction/index.groovy
          http://jenkins-ci.org/commit/testng-plugin-plugin/e8153d09dd5ceb42b7de8d10881c8ee2b048b961
          Log:
          Fixed: JENKINS-23285 - All links on trend graph map are wrong on "testngreports" subpage

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Nalin Makar Path: README src/main/java/hudson/plugins/testng/util/GraphHelper.java src/main/resources/hudson/plugins/testng/TestNGProjectAction/index.groovy http://jenkins-ci.org/commit/testng-plugin-plugin/e8153d09dd5ceb42b7de8d10881c8ee2b048b961 Log: Fixed: JENKINS-23285 - All links on trend graph map are wrong on "testngreports" subpage
          nullin Nalin Makar added a comment -

          Fixed. will be available once v1.7 is released.

          nullin Nalin Makar added a comment - Fixed. will be available once v1.7 is released.

          Thanks! Very appreciated.

          huuugo Hubert Grzeskowiak added a comment - Thanks! Very appreciated.

          People

            nullin Nalin Makar
            huuugo Hubert Grzeskowiak
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: