Protect against potential problems when converting selectors to sources

Author: marcphilipp

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

@@ -67,7 +67,7 @@ public void selectorProcessed(UniqueId engineId, DiscoverySelector selector, Sel
 		if (result.getStatus() == FAILED) {
 			this.issues.add(DiscoveryIssue.builder(Severity.ERROR, selector + " resolution failed") //
 					.cause(result.getThrowable()) //
-					.source(toSource(selector)) //
+					.source(toSourceSafely(selector)) //
 					.build());
 		}
 		else if (result.getStatus() == UNRESOLVED && selector instanceof UniqueIdSelector uniqueIdSelector) {
@@ -78,7 +78,17 @@ else if (result.getStatus() == UNRESOLVED && selector instanceof UniqueIdSelecto
 		}
 	}
 
-	static @Nullable TestSource toSource(DiscoverySelector selector) {
+	private static @Nullable TestSource toSourceSafely(DiscoverySelector selector) {
+		try {
+			return toSource(selector);
+		}
+		catch (Exception e) {
+			logger.error(e, () -> "Failed to convert DiscoverySelector [%s] into TestSource".formatted(selector));
+			return null;
+		}
+	}
+
+	private static @Nullable TestSource toSource(DiscoverySelector selector) {
 		if (selector instanceof ClassSelector classSelector) {
 			return ClassSource.from(classSelector.getClassName());
 		}