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

…racters. Trying to get the safe name from junit plugin for url identification
jenkinsci · Feb 19, 2016 · 9e934e4 · 9e934e4
1 parent d61cdd3
commit 9e934e4
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src/main/java/org/jenkinsci/plugins/testresultsanalyzer/result/info/ClassInfo.java b/src/main/java/org/jenkinsci/plugins/testresultsanalyzer/result/info/ClassInfo.java
@@ -40,7 +40,7 @@ private void addTests(Integer buildNumber, TabulatedResult classResult, String u
 				testCaseInfo.setName(testCaseName);
 			}
 
-			testCaseInfo.putTestCaseResult(buildNumber, testCaseResult, url + "/" + testCaseResult.getName());
+			testCaseInfo.putTestCaseResult(buildNumber, testCaseResult, url + "/" + testCaseResult.getSafeName());
 			tests.put(testCaseName, testCaseInfo);
 		}
 	}

diff --git a/src/main/java/org/jenkinsci/plugins/testresultsanalyzer/result/info/PackageInfo.java b/src/main/java/org/jenkinsci/plugins/testresultsanalyzer/result/info/PackageInfo.java
@@ -45,7 +45,7 @@ public void addClasses(Integer buildNumber, TabulatedResult packageResult, Strin
 				classInfo = new ClassInfo();
 				classInfo.setName(className);
 			}
-			classInfo.putBuildClassResult(buildNumber, (TabulatedResult) classResult, url + "/" + classResult.getName());
+			classInfo.putBuildClassResult(buildNumber, (TabulatedResult) classResult, url + "/" + classResult.getSafeName());
 			classes.put(className, classInfo);
 		}
 	}

diff --git a/src/main/java/org/jenkinsci/plugins/testresultsanalyzer/result/info/ResultInfo.java b/src/main/java/org/jenkinsci/plugins/testresultsanalyzer/result/info/ResultInfo.java
@@ -21,7 +21,7 @@ public void addPackage(Integer buildNumber, TabulatedResult packageResult, Strin
 			packageInfo = new PackageInfo();
 			packageInfo.setName(packageName);
 		}
-		packageInfo.putPackageResult(buildNumber, packageResult, url + getResultUrl(packageResult) +"/" + packageResult.getName());
+		packageInfo.putPackageResult(buildNumber, packageResult, url + getResultUrl(packageResult) +"/" + packageResult.getSafeName());
 		packageResults.put(packageName, packageInfo);
 	}