Allow @ResourceLock on @ClassTemplate classes (#5176)

Fixes #5155.

---------

Signed-off-by: raccoonback <kosb15@naver.com>
Co-authored-by: M.P. Korstanje <mpkorstanje@users.noreply.github.com>
Co-authored-by: Marc Philipp <mail@marcphilipp.de>

Author: 3 people

documentation/src/docs/asciidoc/release-notes/release-notes-6.0.2.adoc

@@ -35,7 +35,8 @@ repository on GitHub.
 [[release-notes-6.0.2-junit-jupiter-bug-fixes]]
 ==== Bug Fixes
 
-* ❓
+* Allow using `@ResourceLock` on classes annotated with `@ClassTemplate` (or
+  `@ParameterizedClass`).
 
 [[release-notes-6.0.2-junit-jupiter-deprecations-and-breaking-changes]]
 ==== Deprecations and Breaking Changes

junit-jupiter-engine/src/main/java/org/junit/jupiter/engine/descriptor/ClassTemplateInvocationTestDescriptor.java

@@ -10,6 +10,7 @@
 
 package org.junit.jupiter.engine.descriptor;
 
+import static java.util.Collections.emptySet;
 import static java.util.Objects.requireNonNull;
 import static org.apiguardian.api.API.Status.INTERNAL;
 import static org.junit.jupiter.engine.descriptor.CallbackSupport.invokeAfterCallbacks;
@@ -36,6 +37,7 @@
 import org.junit.jupiter.engine.extension.MutableExtensionRegistry;
 import org.junit.platform.engine.TestSource;
 import org.junit.platform.engine.UniqueId;
+import org.junit.platform.engine.support.hierarchical.ExclusiveResource;
 import org.junit.platform.engine.support.hierarchical.ThrowableCollector;
 
 /**
@@ -112,6 +114,12 @@ public Function<ResourceLocksProvider, Set<ResourceLocksProvider.Lock>> getResou
 
 	// --- Node ----------------------------------------------------------------
 
+	@Override
+	public Set<ExclusiveResource> getExclusiveResources() {
+		// Resources are already collected and returned by the enclosing ClassTemplateTestDescriptor
+		return emptySet();
+	}
+
 	@Override
 	public JupiterEngineExecutionContext prepare(JupiterEngineExecutionContext context) {
 		MutableExtensionRegistry registry = context.getExtensionRegistry();

jupiter-tests/src/test/java/org/junit/jupiter/engine/ClassTemplateInvocationTests.java

@@ -79,6 +79,7 @@
 import org.junit.jupiter.api.extension.ParameterResolutionException;
 import org.junit.jupiter.api.extension.ParameterResolver;
 import org.junit.jupiter.api.extension.RegisterExtension;
+import org.junit.jupiter.api.parallel.ResourceLock;
 import org.junit.jupiter.engine.descriptor.ClassTemplateInvocationTestDescriptor;
 import org.junit.jupiter.engine.descriptor.ClassTemplateTestDescriptor;
 import org.junit.jupiter.engine.descriptor.ClassTestDescriptor;
@@ -95,6 +96,7 @@
 import org.junit.platform.engine.UniqueId;
 import org.junit.platform.engine.discovery.DiscoverySelectors;
 import org.junit.platform.engine.reporting.ReportEntry;
+import org.junit.platform.engine.support.hierarchical.ExclusiveResource;
 import org.junit.platform.testkit.engine.EngineExecutionResults;
 import org.opentest4j.AssertionFailedError;
 import org.opentest4j.TestAbortedException;
@@ -1004,6 +1006,24 @@ void ignoresComposedAnnotations() {
 		assertThat(engineDescriptor.getDescendants()).isEmpty();
 	}
 
+	@Test
+	void classTemplateWithResourceLockCollectsExclusiveResources() {
+		var results = discoverTestsForClass(ClassTemplateWithResourceLockTestCase.class);
+		var classTemplateDescriptor = (ClassTemplateTestDescriptor) getOnlyElement(
+			results.getEngineDescriptor().getChildren());
+
+		assertThat(classTemplateDescriptor.getExclusiveResources()).extracting(
+			ExclusiveResource::getKey).containsExactly("test-resource");
+	}
+
+	@Test
+	void classTemplateWithResourceLockExecutesSuccessfully() {
+		var results = executeTestsForClass(ClassTemplateWithResourceLockTestCase.class);
+
+		results.testEvents().assertStatistics(stats -> stats.started(2).succeeded(2));
+		results.containerEvents().assertStatistics(stats -> stats.started(4).succeeded(4));
+	}
+
 	// -------------------------------------------------------------------
 
 	private static Stream<String> allReportEntryValues(EngineExecutionResults results) {
@@ -1567,4 +1587,15 @@ void test() {
 		}
 	}
 
+	@SuppressWarnings("JUnitMalformedDeclaration")
+	@ClassTemplate
+	@ExtendWith(TwoInvocationsClassTemplateInvocationContextProvider.class)
+	@ResourceLock("test-resource")
+	static class ClassTemplateWithResourceLockTestCase {
+		@Test
+		void test() {
+			// This test verifies that @ResourceLock works with @ClassTemplate (issue #5155)
+		}
+	}
+
 }