Don't mention that a test passes in its name (that's the default, right?!)

Author: Nicolai Parlog

src/test/java/org/codefx/demo/junit5/basics/AssertTest.java

@@ -21,7 +21,7 @@
 class AssertTest {
 
 	@Test
-	void assertWithBoolean_pass() {
+	void assertWithBoolean() {
 		assertTrue(true);
 		assertTrue(this::truism);
 
@@ -33,7 +33,7 @@ private boolean truism() {
 	}
 
 	@Test
-	void assertWithComparison_pass() {
+	void assertWithComparison() {
 		List<String> expected = asList("element");
 		List<String> actual = new LinkedList<>(expected);
 
@@ -74,7 +74,7 @@ private Address(String city, String street, String number) {
 	}
 
 	@Test
-	void assertExceptions_pass() {
+	void assertExceptions() {
 		Exception exception = assertThrows(Exception.class, this::throwing);
 		assertEquals("I'm failing on purpose.", exception.getMessage());
 	}

src/test/java/org/codefx/demo/junit5/dynamic/ArithmeticTreeTest.java

@@ -41,7 +41,7 @@ private static DynamicNode generateTestTreeFor(
 		else {
 			var testsForChildren = generateTestsFor(arithmeticNode.operands(), treeTestData);
 			var expected = treeTestData.resultFor(arithmeticNode);
-			var testName = arithmeticNode + " should evaluate to " + expected;
+			var testName = arithmeticNode + " should evaluate to " + expected + " (ops '+3' and '*10' fail)";
 			return dynamicContainer(testName, concat(of(testForNode), testsForChildren));
 		}
 	}

src/test/java/org/codefx/demo/junit5/dynamic/ArithmeticTreeTestData.java

@@ -73,8 +73,6 @@ static ArithmeticTreeTestData generateRandom() {
 	}
 
 	private static ArithmeticNode[] pickRandomNodes(List<ArithmeticNode> nodes, Random random) {
-
-
 		return IntStream.range(0, random.nextInt(3) + 2)
 				// compute a random index
 				.map(n -> random.nextInt(nodes.size()))

src/test/java/org/codefx/demo/junit5/extensions/ExpectedExceptionTest.java

@@ -5,7 +5,7 @@
 class ExpectedExceptionTest {
 
 	@Test
-	void noExceptionExpected_throwsNoException_passes() {
+	void noExceptionExpected_throwsNoException() {
 		// do nothing
 	}
 
@@ -20,7 +20,7 @@ void exceptionExpected_throwsNoException_fails() {
 	}
 
 	@Test(expected = IndexOutOfBoundsException.class)
-	void exceptionExpected_throwsException_passes() {
+	void exceptionExpected_throwsException() {
 		throw new IndexOutOfBoundsException("I'm failing on purpose.");
 	}
 

src/test/java/org/codefx/demo/junit5/extensions/TimeoutTest.java

@@ -7,17 +7,17 @@
 class TimeoutTest {
 
 	@Test
-	void quickTestWithoutTimeout_testPasses() {
+	void quickTestWithoutTimeout() {
 		assertTrue(true);
 	}
 
 	@Test(timeout = 0)
-	void quickTestWithoutZeroTimeout_testPasses() {
+	void quickTestWithoutZeroTimeout() {
 		assertTrue(true);
 	}
 
 	@Test(timeout = 10_000)
-	void quickTestWithoutVeryLargeTimeout_testPasses() {
+	void quickTestWithoutVeryLargeTimeout() {
 		assertTrue(true);
 	}