Delete tests for legacy semantics for field and method searches

I'm intentionally leaving this commit in the Git history, in case we later decide to re-introduce these tests. See #3553
junit-team · Apr 7, 2024 · ef17fc0 · ef17fc0
1 parent 64b697f
commit ef17fc0
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 74 deletions.
diff --git a/...ava/org/junit/jupiter/engine/extension/OrderedProgrammaticExtensionRegistrationTests.java b/...ava/org/junit/jupiter/engine/extension/OrderedProgrammaticExtensionRegistrationTests.java
@@ -21,7 +21,6 @@
 
 import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.BeforeEach;
-import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Order;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.TestInstance;
@@ -168,20 +167,6 @@ void classLevelWithDefaultOrderDoesNotShadowExtensionFromSuperclass() {
 		);
 	}
 
-	@Disabled("Disabled until legacy search mode is supported")
-	@Test
-	void classLevelWithDefaultOrderShadowingOrderFromSuperclassInLegacyMode() {
-		Class<?> testClass = DefaultOrderShadowingDefaultOrderAndExplicitOrderClassLevelExtensionRegistrationTestCase.class;
-		String testClassName = testClass.getSimpleName();
-		Class<?> parent = DefaultOrderAndExplicitOrderClassLevelExtensionRegistrationTestCase.class;
-		String parentName = parent.getSimpleName();
-		assertOutcome(testClass, //
-			parentName + " :: extension1 :: before test", //
-			parentName + " :: extension2 :: before test", //
-			testClassName + " :: extension3 :: before test" //
-		);
-	}
-
 	@Test
 	void classLevelWithExplicitOrderDoesNotShadowExtensionFromSuperclass() {
 		Class<?> testClass = ExplicitOrderShadowingDefaultOrderAndExplicitOrderClassLevelExtensionRegistrationTestCase.class;
@@ -196,20 +181,6 @@ void classLevelWithExplicitOrderDoesNotShadowExtensionFromSuperclass() {
 		);
 	}
 
-	@Disabled("Disabled until legacy search mode is supported")
-	@Test
-	void classLevelWithExplicitOrderShadowingOrderFromSuperclassInLegacyMode() {
-		Class<?> testClass = ExplicitOrderShadowingDefaultOrderAndExplicitOrderClassLevelExtensionRegistrationTestCase.class;
-		String testClassName = testClass.getSimpleName();
-		Class<?> parent = DefaultOrderAndExplicitOrderClassLevelExtensionRegistrationTestCase.class;
-		String parentName = parent.getSimpleName();
-		assertOutcome(testClass, //
-			parentName + " :: extension3 :: before test", //
-			testClassName + " :: extension2 :: before test", //
-			parentName + " :: extension1 :: before test" //
-		);
-	}
-
 	@Test
 	void classLevelWithDefaultOrderAndExplicitOrderFromInterface() {
 		Class<?> testClass = DefaultOrderAndExplicitOrderExtensionRegistrationFromInterfaceTestCase.class;

diff --git a/.../test/java/org/junit/jupiter/engine/extension/ProgrammaticExtensionRegistrationTests.java b/.../test/java/org/junit/jupiter/engine/extension/ProgrammaticExtensionRegistrationTests.java
@@ -32,7 +32,6 @@
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.BeforeEach;
-import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Nested;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.TestInstance;
@@ -120,28 +119,6 @@ void instanceLevelWithInheritedExtensions() {
 		);
 	}
 
-	@Disabled("Disabled until legacy search mode is supported")
-	@Test
-	void instanceLevelWithInheritedAndHiddenExtensionsInLegacyMode() {
-		Class<?> testClass = InstanceLevelExtensionRegistrationParentTestCase.class;
-		String parent = testClass.getSimpleName();
-		assertOneTestSucceeded(testClass);
-		assertThat(callSequence).containsExactly( //
-			parent + " :: extension1 :: before test", //
-			parent + " :: extension2 :: before test" //
-		);
-
-		callSequence.clear();
-		testClass = InstanceLevelExtensionRegistrationChildTestCase.class;
-		String child = testClass.getSimpleName();
-		assertOneTestSucceeded(testClass);
-		assertThat(callSequence).containsExactly( //
-			parent + " :: extension1 :: before test", //
-			child + " :: extension2 :: before test", //
-			child + " :: extension3 :: before test" //
-		);
-	}
-
 	@Test
 	void classLevelWithInheritedExtensions() {
 		Class<?> testClass = ClassLevelExtensionRegistrationParentTestCase.class;
@@ -164,28 +141,6 @@ void classLevelWithInheritedExtensions() {
 		);
 	}
 
-	@Disabled("Disabled until legacy search mode is supported")
-	@Test
-	void classLevelWithInheritedAndHiddenExtensionsInLegacyMode() {
-		Class<?> testClass = ClassLevelExtensionRegistrationParentTestCase.class;
-		String parent = testClass.getSimpleName();
-		assertOneTestSucceeded(testClass);
-		assertThat(callSequence).containsExactly( //
-			parent + " :: extension1 :: before test", //
-			parent + " :: extension2 :: before test" //
-		);
-
-		callSequence.clear();
-		testClass = ClassLevelExtensionRegistrationChildTestCase.class;
-		String child = testClass.getSimpleName();
-		assertOneTestSucceeded(testClass);
-		assertThat(callSequence).containsExactly( //
-			parent + " :: extension1 :: before test", //
-			child + " :: extension2 :: before test", //
-			child + " :: extension3 :: before test" //
-		);
-	}
-
 	/**
 	 * @since 5.5
 	 */