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

    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

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

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

          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

          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

          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

          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

          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
          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
          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
          hitha Hitha Jose added a comment -

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

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

          People

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

            Dates

              Created:
              Updated:
              Resolved: