Skip to content

Commit

Permalink
[FIXED JENKINS-25280] Failed tests do not get displayed properly in t…
Browse files Browse the repository at this point in the history 
…he overview

there is no such mothod getFullDisplayName in a TestObject - only getDisplayName

Change-Id: Idd454be00a22ee3c93f661a3ee3ae322a25c271d
  • Loading branch information
@jtnord
jtnord committed Oct 23, 2014
1 parent 90292ed commit e135651
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion core/src/main/resources/lib/hudson/failed-test.jelly
View file
Expand Up @@ -91,7 +91,7 @@ THE SOFTWARE.
<l:icon class="icon-document-delete icon-sm"/>
</a>
<st:nbsp/>
<a href="${url}" class="model-link inside"><st:out value="${result.fullDisplayName}"/></a>
<a href="${url}" class="model-link inside"><st:out value="${result.displayName}"/></a>
<j:forEach var="badge" items="${result.testActions}">
<st:include it="${badge}" page="badge.jelly" optional="true"/>
</j:forEach>
Expand Down

3 comments on commit e135651

@jtnord
Copy link
Member Author

@jtnord jtnord commented on e135651 Oct 23, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jglick - not sure why this file is still in core and not junit?

@jglick
Copy link
Member

@jglick jglick commented on e135651 Oct 25, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops! Indeed it should be in junit-plugin. I will try to fix that.

@jglick
Copy link
Member

@jglick jglick commented on e135651 Oct 25, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cf. jenkinsci/junit-plugin#3

Please sign in to comment.