Skip to content

Commit 13ec14a

Browse files
authored
Reuse checks title information as summary (#675)
1 parent 233585b commit 13ec14a

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

src/main/java/io/jenkins/plugins/junit/checks/JUnitChecksPublisher.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,9 +44,10 @@ public void publishChecks(TaskListener listener) {
4444
@VisibleForTesting
4545
ChecksDetails extractChecksDetails() {
4646
String testsURL = DisplayURLProvider.get().getTestsURL(run);
47+
String resultDetails = extractChecksTitle();
4748
ChecksOutput output = new ChecksOutput.ChecksOutputBuilder()
48-
.withTitle(extractChecksTitle())
49-
.withSummary("<sub>Send us [feedback](https://github.com/jenkinsci/junit-plugin/issues)")
49+
.withTitle(resultDetails)
50+
.withSummary(resultDetails)
5051
.withText(extractChecksText(testsURL))
5152
.build();
5253

src/test/java/io/jenkins/plugins/junit/checks/JUnitChecksPublisherTest.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,7 @@ public void extractChecksDetailsPassingTestResults() throws Exception {
7070
ChecksOutput output = checksDetails.getOutput().get();
7171

7272
assertThat(output.getTitle().get(), is("passed: 6"));
73+
assertThat(output.getSummary().get(), is("passed: 6"));
7374
assertThat(output.getText().get(), is(""));
7475
}
7576

@@ -98,6 +99,7 @@ public void extractChecksDetailsFailingSingleTestResult() throws Exception {
9899
ChecksOutput output = checksDetails.getOutput().get();
99100

100101
assertThat(output.getTitle().get(), is("some.package.somewhere.WhooHoo.testHudsonReporting failed"));
102+
assertThat(output.getSummary().get(), is("some.package.somewhere.WhooHoo.testHudsonReporting failed"));
101103
}
102104

103105
@Test
@@ -125,6 +127,7 @@ public void extractChecksDetailsFailingMultipleTests() throws Exception {
125127
ChecksOutput output = checksDetails.getOutput().get();
126128

127129
assertThat(output.getTitle().get(), is("failed: 3, passed: 5"));
130+
assertThat(output.getSummary().get(), is("failed: 3, passed: 5"));
128131
}
129132

130133
@Test

0 commit comments

Comments
 (0)