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

Link available in the Test report analyzer plugin is not correct if the testcases name has '/' character in it

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      My testcase name have '/' character in it. Example: "general_fs/extended_fd".The test report is in junit xml format. In the Junit test result link it replaces the '/' character by '_'. But in test result analyzer it coninuses to use '/' character, which tries to go to an unknown place. Please correct this issue asap as its blocking our automation process.

      Ex;

      As per the Junit test reporting the link goes to the location :. job/test1/37/testReport/Advfsqarun/ade/general_fs_extended_fd
      But as per the test result analyzer it goes to : job/test1/37/testReport/Advfsqarun/ade/general_fs/extended_fd

        Attachments

          Activity

          Hide
          bestonia best best added a comment - - edited

          It is not just the / character that needs to be replaced. We have the same issue with brackets (), @ symbol, comma and spaces.
          ie
          Results Analyzer links look like:
          ...1/testReport/(root)/TestClass/MyTest(george.bob@gmail.com,%20sometext,%20False)
          should be
          ...1/testReport/(root)/TestClass/MyTest_george_bob_gmail_com__sometext__False_/

          There is probably a list of other characters that needs to be replaced as well...

          Show
          bestonia best best added a comment - - edited It is not just the / character that needs to be replaced. We have the same issue with brackets (), @ symbol, comma and spaces. ie Results Analyzer links look like: ...1/testReport/(root)/TestClass/MyTest(george.bob@gmail.com,%20sometext,%20False) should be ...1/testReport/(root)/TestClass/MyTest_george_bob_gmail_com__sometext__False_/ There is probably a list of other characters that needs to be replaced as well...
          Hide
          hitha Hitha Jose added a comment -

          Yes, the Junit plugin will replace all the special chatracters with "_". So the same need to be updated in the test result analyzer aswell

          Show
          hitha Hitha Jose added a comment - Yes, the Junit plugin will replace all the special chatracters with "_". So the same need to be updated in the test result analyzer aswell
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Varun Menon
          Path:
          src/main/java/org/jenkinsci/plugins/testresultsanalyzer/result/info/ClassInfo.java
          src/main/java/org/jenkinsci/plugins/testresultsanalyzer/result/info/PackageInfo.java
          src/main/java/org/jenkinsci/plugins/testresultsanalyzer/result/info/ResultInfo.java
          http://jenkins-ci.org/commit/test-results-analyzer-plugin/9e934e4ca6c9e9d29c0095b871f79d585c96d15c
          Log:
          JENKINS-32990 - URL mapping is wrong when names contains special characters. Trying to get the safe name from junit plugin for url identification

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Varun Menon Path: src/main/java/org/jenkinsci/plugins/testresultsanalyzer/result/info/ClassInfo.java src/main/java/org/jenkinsci/plugins/testresultsanalyzer/result/info/PackageInfo.java src/main/java/org/jenkinsci/plugins/testresultsanalyzer/result/info/ResultInfo.java http://jenkins-ci.org/commit/test-results-analyzer-plugin/9e934e4ca6c9e9d29c0095b871f79d585c96d15c Log: JENKINS-32990 - URL mapping is wrong when names contains special characters. Trying to get the safe name from junit plugin for url identification
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Varun Menon
          Path:
          src/main/java/org/jenkinsci/plugins/testresultsanalyzer/result/info/ClassInfo.java
          src/main/java/org/jenkinsci/plugins/testresultsanalyzer/result/info/PackageInfo.java
          src/main/java/org/jenkinsci/plugins/testresultsanalyzer/result/info/ResultInfo.java
          http://jenkins-ci.org/commit/test-results-analyzer-plugin/ff46cd604091fb3076ba72c0bb53a139f8910a1c
          Log:
          Merge pull request #32 from menonvarun/JENKINS-32990

          JENKINS-32990 - URL mapping is wrong when names contains special cha…

          Compare: https://github.com/jenkinsci/test-results-analyzer-plugin/compare/d61cdd33fb39...ff46cd604091

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Varun Menon Path: src/main/java/org/jenkinsci/plugins/testresultsanalyzer/result/info/ClassInfo.java src/main/java/org/jenkinsci/plugins/testresultsanalyzer/result/info/PackageInfo.java src/main/java/org/jenkinsci/plugins/testresultsanalyzer/result/info/ResultInfo.java http://jenkins-ci.org/commit/test-results-analyzer-plugin/ff46cd604091fb3076ba72c0bb53a139f8910a1c Log: Merge pull request #32 from menonvarun/ JENKINS-32990 JENKINS-32990 - URL mapping is wrong when names contains special cha… Compare: https://github.com/jenkinsci/test-results-analyzer-plugin/compare/d61cdd33fb39...ff46cd604091
          Hide
          menonvarun Varun Menon added a comment -

          This issue is fixed in 0.3.3 release done today.
          Please update the bug in case you still see the issue.

          In case of reopening please let me know the following things:

          • A sample junit report xml
          • Coding language used
          • Unit test framework used
          Show
          menonvarun Varun Menon added a comment - This issue is fixed in 0.3.3 release done today. Please update the bug in case you still see the issue. In case of reopening please let me know the following things: A sample junit report xml Coding language used Unit test framework used
          Hide
          hitha Hitha Jose added a comment -

          I have verified iin 0.3.3 and the issue seems to be fixed. Thanks you.

          Show
          hitha Hitha Jose added a comment - I have verified iin 0.3.3 and the issue seems to be fixed. Thanks you.

            People

            Assignee:
            menonvarun Varun Menon
            Reporter:
            hitha Hitha Jose
            Votes:
            1 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: