HHH-19516 Upgrade to JUnit 5.13.0 for 6.6

Author: beikov

hibernate-testing/src/main/java/org/hibernate/testing/bytecode/enhancement/extension/engine/BytecodeEnhancedTestEngine.java

@@ -24,14 +24,15 @@
 import java.util.function.Supplier;
 
 import org.hibernate.testing.bytecode.enhancement.extension.BytecodeEnhanced;
-import org.hibernate.testing.junit4.BaseUnitTestCase;
 import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.ClassOrderer;
 import org.junit.jupiter.api.DisplayNameGenerator;
 import org.junit.jupiter.api.MethodOrderer;
 import org.junit.jupiter.api.TestInstance;
 import org.junit.jupiter.api.extension.ExecutionCondition;
+import org.junit.jupiter.api.extension.Extension;
 import org.junit.jupiter.api.extension.ExtensionContext;
+import org.junit.jupiter.api.extension.TestInstantiationAwareExtension;
 import org.junit.jupiter.api.io.CleanupMode;
 import org.junit.jupiter.api.io.TempDirFactory;
 import org.junit.jupiter.api.parallel.ExecutionMode;
@@ -48,6 +49,7 @@
 import org.junit.platform.engine.ExecutionRequest;
 import org.junit.platform.engine.TestDescriptor;
 import org.junit.platform.engine.UniqueId;
+import org.junit.platform.engine.reporting.OutputDirectoryProvider;
 import org.junit.platform.engine.support.hierarchical.EngineExecutionContext;
 import org.junit.platform.engine.support.hierarchical.HierarchicalTestEngine;
 import org.junit.platform.engine.support.hierarchical.ThrowableCollector;
@@ -118,7 +120,7 @@ public TestDescriptor discover(EngineDiscoveryRequest discoveryRequest, UniqueId
 	}
 
 	private void addEnhancementCheck(boolean enhance, String[] testEnhancedClasses,
-			ClassBasedTestDescriptor descriptor, JupiterConfiguration jc) {
+									 ClassBasedTestDescriptor descriptor, JupiterConfiguration jc) {
 		if ( testEnhancedClasses.length > 0 ) {
 			descriptor.addChild( new EnhancementWorkedCheckMethodTestDescriptor(
 					UniqueId.forEngine( getId() )
@@ -127,6 +129,7 @@ private void addEnhancementCheck(boolean enhance, String[] testEnhancedClasses,
 									descriptor.getTestClass().getName()
 							),
 					descriptor.getTestClass(),
+					descriptor::getEnclosingTestClasses,
 					jc,
 					enhance,
 					testEnhancedClasses
@@ -135,14 +138,14 @@ private void addEnhancementCheck(boolean enhance, String[] testEnhancedClasses,
 	}
 
 	private void replaceWithEnhanced(Class<?> enhanced, ClassBasedTestDescriptor descriptor, JupiterConfiguration jc,
-			Set<? extends TestDescriptor> children, TestDescriptor parent, String[] testEnhancedClasses)
+									 Set<? extends TestDescriptor> children, TestDescriptor parent, String[] testEnhancedClasses)
 			throws NoSuchMethodException {
 		replaceWithEnhanced( enhanced, descriptor, jc, children, parent, testEnhancedClasses, null );
 	}
 
 	private void replaceWithEnhanced(Class<?> enhanced, ClassBasedTestDescriptor descriptor, JupiterConfiguration jc,
-			Set<? extends TestDescriptor> children, TestDescriptor parent, String[] testEnhancedClasses,
-			Object enhancementContextId)
+									 Set<? extends TestDescriptor> children, TestDescriptor parent, String[] testEnhancedClasses,
+									 Object enhancementContextId)
 			throws NoSuchMethodException {
 		DelegatingJupiterConfiguration configuration = new DelegatingJupiterConfiguration( jc, enhancementContextId );
 
@@ -162,6 +165,7 @@ private void replaceWithEnhanced(Class<?> enhanced, ClassBasedTestDescriptor des
 								convertUniqueId( child.getUniqueId(), enhancementContextId ),
 								updated.getTestClass(),
 								findMethodReplacement( updated, testMethod ),
+								updated::getEnclosingTestClasses,
 								configuration
 						)
 				);
@@ -173,6 +177,7 @@ private void replaceWithEnhanced(Class<?> enhanced, ClassBasedTestDescriptor des
 						convertUniqueId( child.getUniqueId(), enhancementContextId ),
 						updated.getTestClass(),
 						findMethodReplacement( updated, testMethod ),
+						updated::getEnclosingTestClasses,
 						configuration
 				) );
 			}
@@ -251,6 +256,11 @@ private DelegatingJupiterConfiguration(JupiterConfiguration configuration, Objec
 			);
 		}
 
+		@Override
+		public Predicate<Class<? extends Extension>> getFilterForAutoDetectedExtensions() {
+			return configuration.getFilterForAutoDetectedExtensions();
+		}
+
 		@Override
 		public Optional<String> getRawConfigurationParameter(String s) {
 			return configuration.getRawConfigurationParameter( s );
@@ -276,6 +286,11 @@ public boolean isExtensionAutoDetectionEnabled() {
 			return configuration.isExtensionAutoDetectionEnabled();
 		}
 
+		@Override
+		public boolean isThreadDumpOnTimeoutEnabled() {
+			return configuration.isThreadDumpOnTimeoutEnabled();
+		}
+
 		@Override
 		public ExecutionMode getDefaultExecutionMode() {
 			return configuration.getDefaultExecutionMode();
@@ -320,6 +335,16 @@ public CleanupMode getDefaultTempDirCleanupMode() {
 		public Supplier<TempDirFactory> getDefaultTempDirFactorySupplier() {
 			return configuration.getDefaultTempDirFactorySupplier();
 		}
+
+		@Override
+		public TestInstantiationAwareExtension.ExtensionContextScope getDefaultTestInstantiationExtensionContextScope() {
+			return configuration.getDefaultTestInstantiationExtensionContextScope();
+		}
+
+		@Override
+		public OutputDirectoryProvider getOutputDirectoryProvider() {
+			return configuration.getOutputDirectoryProvider();
+		}
 	}
 
 	private static class DelegatingDisplayNameGenerator implements DisplayNameGenerator {
@@ -342,13 +367,13 @@ private String prefix() {
 		}
 
 		@Override
-		public String generateDisplayNameForNestedClass(Class<?> aClass) {
-			return prefix() + delegate.generateDisplayNameForNestedClass( aClass );
+		public String generateDisplayNameForNestedClass(List<Class<?>> enclosingInstanceTypes, Class<?> nestedClass) {
+			return prefix() + delegate.generateDisplayNameForNestedClass( enclosingInstanceTypes, nestedClass );
 		}
 
 		@Override
-		public String generateDisplayNameForMethod(Class<?> aClass, Method method) {
-			return prefix() + delegate.generateDisplayNameForMethod( aClass, method );
+		public String generateDisplayNameForMethod(List<Class<?>> enclosingInstanceTypes, Class<?> testClass, Method testMethod) {
+			return prefix() + delegate.generateDisplayNameForMethod( enclosingInstanceTypes, testClass, testMethod );
 		}
 	}
 
@@ -357,12 +382,11 @@ private static class EnhancementWorkedCheckMethodTestDescriptor extends TestMeth
 		private final boolean enhanced;
 		private final String[] classes;
 
-		public EnhancementWorkedCheckMethodTestDescriptor(UniqueId uniqueId, Class<?> testClass,
-				JupiterConfiguration configuration,
-				boolean enhanced, String[] classes) {
+		public EnhancementWorkedCheckMethodTestDescriptor(UniqueId uniqueId, Class<?> testClass, Supplier<List<Class<?>>> enclosingInstanceTypes, JupiterConfiguration configuration, boolean enhanced, String[] classes) {
 			super(
 					prepareId( uniqueId, testMethod( enhanced ) ),
 					testClass, testMethod( enhanced ),
+					enclosingInstanceTypes,
 					configuration
 			);
 			this.enhanced = enhanced;
@@ -375,7 +399,7 @@ private static Method testMethod(boolean enhanced) {
 
 		@Override
 		public JupiterEngineExecutionContext execute(JupiterEngineExecutionContext context,
-				DynamicTestExecutor dynamicTestExecutor) {
+													 DynamicTestExecutor dynamicTestExecutor) {
 			ExtensionContext extensionContext = context.getExtensionContext();
 			ThrowableCollector throwableCollector = context.getThrowableCollector();