Details
-
Type:
Bug
-
Status:
Closed
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: 2.0 Report Plugin
-
Fix Version/s: 2.7.2
-
Component/s: Maven Surefire Report Plugin
-
Labels:None
-
Number of attachments :
Description
toggleDisplay uses only the name of the test method for identifying the tests, it should use the full class name+method name
This patch should make it work but it does not.
Index: src/main/java/org/apache/maven/plugins/surefire/report/SurefireReportGenerator.java =================================================================== — src/main/java/org/apache/maven/plugins/surefire/report/SurefireReportGenerator.java (revision 452343) +++ src/main/java/org/apache/maven/plugins/surefire/report/SurefireReportGenerator.java (working copy) @@ -407,10 +407,10 @@ sink.rawText( " <div class=\"detailToggle\" style=\"display:inline\">" ); - sink.link( "javascript:toggleDisplay('" + testCase.getName() + "');" ); + sink.link( "javascript:toggleDisplay('" + testCase.getFullName() + "');" ); - sink.rawText( "<span style=\"display: inline;\" " + "id=\"" + testCase.getName() + - "off\">+</span><span id=\"" + testCase.getName() + "on\" " + + sink.rawText( "<span style=\"display: inline;\" " + "id=\"" + testCase.getFullName() + + "off\">+</span><span id=\"" + testCase.getFullName() + "on\" " + "style=\"display: none;\">-</span> " ); sink.text( "[ Detail ]" ); sink.link_();