Fix console launcher's --uid option (#4589)

Prior to this commit, unique id selectors weren't
included in the list of explicit selectors - which
resulted a false error message:
```
    Please specify an explicit selector option or
    use `--scan-class-path` or `--scan-modules`
```

Fixes #4587

---------

Co-authored-by: Marc Philipp <[email protected]>

Author: sormuras

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

@@ -52,6 +52,7 @@ repository on GitHub.
   the `ConsoleLauncher`. Prior to this release, the created class loader was closed prior
   to JVM shutdown hooks being invoked, which caused hooks to fail with a
   `ClassNotFoundException` when loading classes during shutdown.
+* Fix support of `--uid` and `--select-unique-id` options in the console launcher.
 
 [[release-notes-5.13.0-junit-platform-deprecations-and-breaking-changes]]
 ==== Deprecations and Breaking Changes

documentation/src/docs/asciidoc/user-guide/running-tests.adoc

@@ -861,7 +861,7 @@ The following discovery selectors are provided out of the box:
 | `{NestedClassSelector}`       | `{DiscoverySelectors_selectNestedClass}`       | `{Select}("<identifier>")`  | `--select <identifier>`                          | `nested-class:com.acme.Foo/Bar`
 | `{NestedMethodSelector}`      | `{DiscoverySelectors_selectNestedMethod}`      | `{Select}("<identifier>")`  | `--select <identifier>`                          | `nested-method:com.acme.Foo/Bar#m`
 | `{PackageSelector}`           | `{DiscoverySelectors_selectPackage}`           | `{SelectPackages}`          | `--select-package com.acme.foo`                  | `package:com.acme.foo`
-| `{UniqueIdSelector}`          | `{DiscoverySelectors_selectUniqueId}`          | `{Select}("<identifier>")`  | `--select <identifier>`                          | `uid:...`
+| `{UniqueIdSelector}`          | `{DiscoverySelectors_selectUniqueId}`          | `{Select}("<identifier>")`  | `--select-unique-id <identifier>`                | `uid:[engine:Foo]/[segment:Bar]`
 | `{UriSelector}`               | `{DiscoverySelectors_selectUri}`               | `{SelectUris}`              | `--select-uri \file:///foo.txt`                  | `uri:file:///foo.txt`
 |===
 

junit-platform-console/src/main/java/org/junit/platform/console/options/TestDiscoveryOptions.java

@@ -204,6 +204,7 @@ public void setSelectorIdentifiers(List<DiscoverySelectorIdentifier> selectorIde
 
 	public List<DiscoverySelector> getExplicitSelectors() {
 		List<DiscoverySelector> selectors = new ArrayList<>();
+		selectors.addAll(getSelectedUniqueIds());
 		selectors.addAll(getSelectedUris());
 		selectors.addAll(getSelectedFiles());
 		selectors.addAll(getSelectedDirectories());

platform-tests/src/test/java/org/junit/platform/console/tasks/DiscoveryRequestCreatorTests.java

@@ -22,6 +22,7 @@
 import static org.junit.platform.engine.discovery.DiscoverySelectors.selectIteration;
 import static org.junit.platform.engine.discovery.DiscoverySelectors.selectMethod;
 import static org.junit.platform.engine.discovery.DiscoverySelectors.selectPackage;
+import static org.junit.platform.engine.discovery.DiscoverySelectors.selectUniqueId;
 import static org.junit.platform.engine.discovery.DiscoverySelectors.selectUri;
 
 import java.io.File;
@@ -36,6 +37,7 @@
 import org.junit.platform.commons.PreconditionViolationException;
 import org.junit.platform.console.options.TestDiscoveryOptions;
 import org.junit.platform.engine.Filter;
+import org.junit.platform.engine.UniqueId;
 import org.junit.platform.engine.discovery.ClassNameFilter;
 import org.junit.platform.engine.discovery.ClassSelector;
 import org.junit.platform.engine.discovery.ClasspathResourceSelector;
@@ -46,6 +48,7 @@
 import org.junit.platform.engine.discovery.MethodSelector;
 import org.junit.platform.engine.discovery.PackageNameFilter;
 import org.junit.platform.engine.discovery.PackageSelector;
+import org.junit.platform.engine.discovery.UniqueIdSelector;
 import org.junit.platform.engine.discovery.UriSelector;
 import org.junit.platform.launcher.LauncherDiscoveryRequest;
 
@@ -218,6 +221,21 @@ void convertsEngineOptions() {
 		assertThat(engineFilters.get(1).toString()).contains("excludes", "[engine2]");
 	}
 
+	@Test
+	void propagatesUniqueIdSelectors() {
+		options.setSelectedUniqueId(List.of(selectUniqueId("[engine:a]/[1:1]"), selectUniqueId("[engine:b]/[2:2]")));
+
+		var request = convert();
+		var uriSelectors = request.getSelectorsByType(UniqueIdSelector.class);
+
+		assertThat(uriSelectors) //
+				.extracting(UniqueIdSelector::getUniqueId) //
+				.containsExactly( //
+					UniqueId.parse("[engine:a]/[1:1]"), //, //
+					UniqueId.parse("[engine:b]/[2:2]") //
+				);
+	}
+
 	@Test
 	void propagatesUriSelectors() {
 		options.setSelectedUris(List.of(selectUri("a"), selectUri("b")));