Restore trimming of display names

marcphilipp · marcphilipp · commit 6db49a2542d7 · 2025-04-08T14:30:21.000+02:00
diff --git a/junit-jupiter-engine/src/main/java/org/junit/jupiter/engine/descriptor/DisplayNameUtils.java b/junit-jupiter-engine/src/main/java/org/junit/jupiter/engine/descriptor/DisplayNameUtils.java
@@ -77,6 +77,7 @@ static String determineDisplayName(AnnotatedElement element, Supplier<String> di
 		return findAnnotation(element, DisplayName.class) //
 				.map(DisplayName::value) //
 				.filter(StringUtils::isNotBlank) //
+				.map(String::trim) //
 				.orElseGet(displayNameSupplier);
 	}
 
diff --git a/jupiter-tests/src/test/java/org/junit/jupiter/engine/descriptor/DisplayNameUtilsTests.java b/jupiter-tests/src/test/java/org/junit/jupiter/engine/descriptor/DisplayNameUtilsTests.java
@@ -185,7 +185,7 @@ private LogRecord firstWarningLogRecord(LogRecordListener listener) throws Asser
 			() -> new AssertionError("Failed to find warning log record"));
 	}
 
-	@DisplayName("my-test-case")
+	@DisplayName("my-test-case\t")
 	@DisplayNameGeneration(value = CustomDisplayNameGenerator.class)
 	static class MyTestCase {
 

Original file line number	Diff line number	Diff line change
`@@ -77,6 +77,7 @@ static String determineDisplayName(AnnotatedElement element, Supplier<String> di`
`77`	`77`	`return findAnnotation(element, DisplayName.class) //`
`78`	`78`	`.map(DisplayName::value) //`
`79`	`79`	`.filter(StringUtils::isNotBlank) //`
	`80`	`+ .map(String::trim) //`
`80`	`81`	`.orElseGet(displayNameSupplier);`
`81`	`82`	`}`
`82`	`83`
Original file line number	Diff line number	Diff line change
`@@ -185,7 +185,7 @@ private LogRecord firstWarningLogRecord(LogRecordListener listener) throws Asser`
`185`	`185`	`() -> new AssertionError("Failed to find warning log record"));`
`186`	`186`	`}`
`187`	`187`
`188`		`- @DisplayName("my-test-case")`
	`188`	`+ @DisplayName("my-test-case\t")`
`189`	`189`	`@DisplayNameGeneration(value = CustomDisplayNameGenerator.class)`
`190`	`190`	`static class MyTestCase {`
`191`	`191`