Apply comment

Issue: #2816

Author: YongGoose

junit-platform-launcher/src/main/java/org/junit/platform/launcher/Launcher.java

@@ -13,8 +13,6 @@
 import static org.apiguardian.api.API.Status.STABLE;
 
 import org.apiguardian.api.API;
-import org.junit.platform.engine.Namespace;
-import org.junit.platform.engine.support.store.NamespacedHierarchicalStore;
 
 /**
  * The {@code Launcher} API is the main entry point for client code that
@@ -129,5 +127,4 @@ public interface Launcher {
 	@API(status = STABLE, since = "1.4")
 	void execute(TestPlan testPlan, TestExecutionListener... listeners);
 
-	NamespacedHierarchicalStore<Namespace> getStore(LauncherDiscoveryRequest launcherDiscoveryRequest);
 }

junit-platform-launcher/src/main/java/org/junit/platform/launcher/core/DefaultLauncher.java

@@ -102,11 +102,6 @@ public void execute(TestPlan testPlan, TestExecutionListener... listeners) {
 		execute((InternalTestPlan) testPlan, listeners);
 	}
 
-	@Override
-	public NamespacedHierarchicalStore<Namespace> getStore(LauncherDiscoveryRequest launcherDiscoveryRequest) {
-		return new NamespacedHierarchicalStore<>(this.sessionStore);
-	}
-
 	private LauncherDiscoveryResult discover(LauncherDiscoveryRequest discoveryRequest,
 			EngineDiscoveryOrchestrator.Phase phase) {
 		return discoveryOrchestrator.discover(discoveryRequest, phase);

junit-platform-launcher/src/main/java/org/junit/platform/launcher/core/DefaultLauncherSession.java

@@ -118,11 +118,6 @@ public void execute(LauncherDiscoveryRequest launcherDiscoveryRequest, TestExecu
 		public void execute(TestPlan testPlan, TestExecutionListener... listeners) {
 			throw new PreconditionViolationException("Launcher session has already been closed");
 		}
-
-		@Override
-		public NamespacedHierarchicalStore<Namespace> getStore(LauncherDiscoveryRequest launcherDiscoveryRequest) {
-			throw new PreconditionViolationException("Launcher session has already been closed");
-		}
 	}
 
 	private static LauncherInterceptor composite(List<LauncherInterceptor> interceptors) {

junit-platform-launcher/src/main/java/org/junit/platform/launcher/core/DelegatingLauncher.java

@@ -10,8 +10,6 @@
 
 package org.junit.platform.launcher.core;
 
-import org.junit.platform.engine.Namespace;
-import org.junit.platform.engine.support.store.NamespacedHierarchicalStore;
 import org.junit.platform.launcher.Launcher;
 import org.junit.platform.launcher.LauncherDiscoveryListener;
 import org.junit.platform.launcher.LauncherDiscoveryRequest;
@@ -54,9 +52,4 @@ public void execute(TestPlan testPlan, TestExecutionListener... listeners) {
 		delegate.execute(testPlan, listeners);
 	}
 
-	@Override
-	public NamespacedHierarchicalStore<Namespace> getStore(LauncherDiscoveryRequest launcherDiscoveryRequest) {
-		return delegate.getStore(launcherDiscoveryRequest);
-	}
-
 }

junit-platform-launcher/src/main/java/org/junit/platform/launcher/core/SessionPerRequestLauncher.java

@@ -13,8 +13,6 @@
 import java.util.List;
 import java.util.function.Supplier;
 
-import org.junit.platform.engine.Namespace;
-import org.junit.platform.engine.support.store.NamespacedHierarchicalStore;
 import org.junit.platform.launcher.Launcher;
 import org.junit.platform.launcher.LauncherDiscoveryListener;
 import org.junit.platform.launcher.LauncherDiscoveryRequest;
@@ -73,13 +71,6 @@ public void execute(TestPlan testPlan, TestExecutionListener... listeners) {
 		}
 	}
 
-	@Override
-	public NamespacedHierarchicalStore<Namespace> getStore(LauncherDiscoveryRequest launcherDiscoveryRequest) {
-		try (LauncherSession session = createSession()) {
-			return session.getLauncher().getStore(launcherDiscoveryRequest);
-		}
-	}
-
 	private LauncherSession createSession() {
 		LauncherSession session = new DefaultLauncherSession(interceptorFactory.get(), sessionListenerSupplier,
 			launcherSupplier);