Skip to content

Commit 536e9c7

Browse files
committed
Print Diffs (Markdown) only once when needed
1 parent 82f723a commit 536e9c7

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

junit-platform-console/src/main/java/org/junit/platform/console/tasks/ConsoleTestExecutor.java

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -184,9 +184,13 @@ private void printSummary(TestExecutionSummary summary, PrintWriter out) {
184184
// Otherwise the failures have already been printed in detail
185185
if (EnumSet.of(Details.NONE, Details.SUMMARY, Details.TREE).contains(outputOptions.getDetails())) {
186186
summary.printFailuresTo(out);
187+
boolean[] diffFlag = {true};
187188
//adding diff code here
188-
out.printf("%nDiffs (Markdown):%n");
189189
summary.getFailures().forEach(failure -> {
190+
if(diffFlag[0]){
191+
out.printf("%nDiffs (Markdown):%n");
192+
diffFlag[0] = false;
193+
}
190194
//get AssertionFailedError
191195
if (failure.getException() instanceof AssertionFailedError) {
192196
AssertionFailedError assertionFailedError = (AssertionFailedError) failure.getException();
@@ -197,7 +201,6 @@ private void printSummary(TestExecutionSummary summary, PrintWriter out) {
197201
new DiffPrinter(testPlanListeners).printDiff(out, expected.getStringRepresentation(),
198202
actual.getStringRepresentation(), failure.getTestIdentifier());
199203
}
200-
201204
}
202205
});
203206
}

0 commit comments

Comments
 (0)