diff --git a/src/main/java/edu/hm/hafner/grading/TestMarkdown.java b/src/main/java/edu/hm/hafner/grading/TestMarkdown.java index e4b107f6..6f9721cb 100644 --- a/src/main/java/edu/hm/hafner/grading/TestMarkdown.java +++ b/src/main/java/edu/hm/hafner/grading/TestMarkdown.java @@ -2,6 +2,8 @@ import java.util.function.Function; +import org.apache.commons.lang3.StringUtils; + import edu.hm.hafner.coverage.TestCase; /** @@ -89,13 +91,22 @@ else if (!stringBuilder.toString().endsWith(TRUNCATED_MESSAGE)) { } private String renderFailure(final TestCase issue) { - return String.format("
%n" - + "%s:%s" + return String.format("__%s:%s__%n" + + getMessage(issue) + + "
%n" + + "Stack Trace" + "%n%n" + "```text%n" + "%s%n" + "```" + "%n" - + "
%n", issue.getClassName(), issue.getMessage(), issue.getDescription()); + + "
%n%n", issue.getClassName(), issue.getTestName(), issue.getDescription()); + } + + private String getMessage(final TestCase issue) { + if (issue.getMessage().isBlank()) { + return StringUtils.EMPTY; + } + return issue.getMessage() + "%n"; } } diff --git a/src/test/java/edu/hm/hafner/grading/TestMarkdownTest.java b/src/test/java/edu/hm/hafner/grading/TestMarkdownTest.java index 2f855766..19defdfe 100644 --- a/src/test/java/edu/hm/hafner/grading/TestMarkdownTest.java +++ b/src/test/java/edu/hm/hafner/grading/TestMarkdownTest.java @@ -184,9 +184,13 @@ void shouldShowScoreWithTwoResults() { "|Modultests|0|0|10|-30", "*:moneybag:*|*1*|*2*|*3*|*:ledger:*", "*:moneybag:*|*-1*|*-2*|*-3*|*:ledger:*", - "test-class-failed-0:failed-message-0", - "test-class-failed-1:failed-message-1", - "test-class-failed-2:failed-message-2", + "__test-class-failed-0:test-failed-0__", + "__test-class-failed-1:test-failed-1__", + "__test-class-failed-2:test-failed-2__", + "failed-message-0", + "failed-message-1", + "failed-message-2", + "Stack Trace", "```text\nStackTrace-0\n```", "```text\nStackTrace-1\n```", "```text\nStackTrace-2\n```")