Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[FIXED JENKINS-17941] Added handy parseErrors link to the build page.…
… Fixed skipped counter. Added more awesomeness to the plug-in, thanks to Sebastian Schuberth
  • Loading branch information
kinow committed Jul 19, 2013
1 parent 7cab81f commit a0cf096
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/main/java/org/tap4j/plugin/TapResult.java
Expand Up @@ -163,7 +163,7 @@ public void tally() {
Plan plan = realTestSet.getPlan();

if (plan.isSkip()) {
this.skipped = testResults.size();
this.skipped += testResults.size();
} else {
for (TestResult testResult : testResults) {
if (isSkipped(testResult)) {
Expand Down
Expand Up @@ -6,6 +6,6 @@
xmlns:i="jelly:fmt">
<t:summary icon="/plugin/tap/icons/tap-48.png">
<p><a href="tapResults">TAP Extended Test Results</a></p>
<p>This build contains ${it.result.testSets.size()} TAP test set(s), and ${it.result.parseErrorTestSets.size()} parse error(s).</p>
<p>This build contains ${it.result.testSets.size()} TAP test set(s), and <a href='tapResults/#parseErrors'>${it.result.parseErrorTestSets.size()} parse error(s)</a>.</p>
</t:summary>
</j:jelly>
1 change: 1 addition & 0 deletions src/main/resources/org/tap4j/plugin/TapResult/index.jelly
Expand Up @@ -64,6 +64,7 @@
</j:choose>

<j:choose>
<a name="parseErrors" />
<j:when test="${it.hasParseErrors() == false}">
No parse errors found.
</j:when>
Expand Down

0 comments on commit a0cf096

Please sign in to comment.