Polish dynamic test examples in the User Guide

documentation/src/docs/asciidoc/writing-tests.adoc

@@ -310,9 +310,8 @@ however, this might be complemented by a registration facility in a later releas
 
 The following `DynamicTestsDemo` class demonstrates several examples of test factories and dynamic tests.
 
-The first method returns an invalid return type.
-Since invalid return types cannot be detected at compile-time
-a `JUnitException` is thrown when it is detected at run-time.
+The first method returns an invalid return type. Since an invalid return type cannot be
+detected at compile-time, a `JUnitException` is thrown when it is detected at run-time.
 
 The next five methods are very simple examples that demonstrate the generation of a
 `Collection`, `Iterable`, `Iterator`, or `Stream` of `DynamicTest` instances.
@@ -322,7 +321,7 @@ However, `dynamicTestsFromStream()` and `dynamicTestsFromIntStream()` demonstrat
 easy it is to generate dynamic tests for a given set of strings or a range of input numbers.
 
 The last method is truly dynamic in nature.
-`generateRandomNumberOfTests()` implements an `Iterator` that generates random numbers, a custom
+`generateRandomNumberOfTests()` implements an `Iterator` that generates random numbers, a
 display name generator, and a test executor and then provides all three to `DynamicTest.stream()`.
 Although the non-deterministic behavior of `generateRandomNumberOfTests()` is of course in conflict with
 test repeatability and should thus be used with care, it serves to demonstrate the expressiveness

documentation/src/test/java/example/DynamicTestsDemo.java

@@ -87,8 +87,8 @@ Stream<DynamicTest> dynamicTestsFromStream() {
 		// end::user_guide[]
 		// @formatter:off
 		// tag::user_guide[]
-		return Stream.of("test1", "test2", "test3")
-			.map(displayName -> dynamicTest(displayName, () -> { /* ... */ }));
+		return Stream.of("A", "B", "C").map(
+			str -> dynamicTest("test" + str, () -> { /* ... */ }));
 		// end::user_guide[]
 		// @formatter:on
 		// tag::user_guide[]
@@ -99,8 +99,8 @@ Stream<DynamicTest> dynamicTestsFromIntStream() {
 		// end::user_guide[]
 		// @formatter:off
 		// tag::user_guide[]
-		return IntStream.range(1, 100).mapToObj(n ->
-			dynamicTest("test" + n, () -> assertTrue(n % 11 != 0)));
+		return IntStream.range(1, 100).mapToObj(
+			n -> dynamicTest("test" + n, () -> assertTrue(n % 11 != 0)));
 		// end::user_guide[]
 		// @formatter:on
 		// tag::user_guide[]
@@ -129,12 +129,12 @@ public Integer next() {
 		};
 
 		// Generates display names like: input:5, input:37, input:85, etc.
-		Function<? super Integer, String> displayNameGenerator = (input) -> "input:" + input;
+		Function<Integer, String> displayNameGenerator = (input) -> "input:" + input;
 
 		// Executes tests based on the current input value.
-		Consumer<? super Integer> testExecutor = (input) -> assertTrue(input % 3 == 0);
+		Consumer<Integer> testExecutor = (input) -> assertTrue(input % 3 == 0);
 
-		// Creates a stream of dynamic tests.
+		// Returns a stream of dynamic tests.
 		return DynamicTest.stream(inputGenerator, displayNameGenerator, testExecutor);
 	}