Polishing

Author: mpkorstanje

platform-tests/src/test/java/org/junit/platform/StackTracePruningTests.java

@@ -76,7 +76,6 @@ void shouldNotPruneStackTraceWhenDisabled() {
 
 		assertStackTraceMatch(stackTrace, """
 				\\Qorg.junit.jupiter.api.Assertions.fail(Assertions.java:\\E.+
-				\\Qorg.junit.platform.StackTracePruningTests$FailingTestTestCase.fail(StackTracePruningTests.java:\\E.+
 				>>>>
 				\\Qorg.junit.platform.commons.util.ReflectionUtils.invokeMethod(ReflectionUtils.java:\\E.+
 				>>>>
@@ -94,7 +93,6 @@ void shouldAlwaysKeepJupiterAssertionStackTraceElement() {
 
 		assertStackTraceMatch(stackTrace, """
 				\\Qorg.junit.jupiter.api.Assertions.fail(Assertions.java:\\E.+
-				\\Qorg.junit.platform.StackTracePruningTests$FailingTestTestCase.fail(StackTracePruningTests.java:\\E.+
 				>>>>
 				""");
 	}
@@ -126,7 +124,6 @@ void shouldKeepExactlyEverythingBetweenTestCallAndFirstAssertionCall() {
 		assertStackTraceMatch(stackTrace,
 			"""
 					\\Qorg.junit.jupiter.api.Assertions.fail(Assertions.java:\\E.+
-					\\Qorg.junit.platform.StackTracePruningTests$FailingTestTestCase.fail(StackTracePruningTests.java:\\E.+
 					\\Qorg.junit.platform.StackTracePruningTests$FailingTestTestCase.failingAssertion(StackTracePruningTests.java:\\E.+
 					""");
 	}
@@ -146,7 +143,6 @@ void shouldKeepExactlyEverythingAfterLifecycleMethodCall(Class<?> methodClass) {
 		assertStackTraceMatch(stackTrace,
 			"""
 					\\Qorg.junit.jupiter.api.Assertions.fail(Assertions.java:\\E.+
-					\\Qorg.junit.platform.StackTracePruningTests$FailingBeforeEachTestCase.fail(StackTracePruningTests.java:\\E.+
 					\\Qorg.junit.platform.StackTracePruningTests$FailingBeforeEachTestCase.setUp(StackTracePruningTests.java:\\E.+
 					""");
 	}
@@ -198,10 +194,6 @@ static class FailingTestTestCase {
 
 		@Test
 		void failingAssertion() {
-			fail();
-		}
-
-		private static void fail() {
 			Assertions.fail();
 		}
 
@@ -211,7 +203,7 @@ void multipleFailingAssertions() {
 		}
 
 		private void failMultiple() {
-			Assertions.assertAll(FailingTestTestCase::fail, FailingTestTestCase::fail);
+			Assertions.assertAll(Assertions::fail, Assertions::fail);
 		}
 
 		@Test
@@ -227,10 +219,6 @@ static class FailingBeforeEachTestCase {
 
 		@BeforeEach
 		void setUp() {
-			fail();
-		}
-
-		private static void fail() {
 			Assertions.fail();
 		}