Fix integration tests

Author: marcphilipp

junit-platform-reporting/src/main/java/org/junit/platform/reporting/open/xml/OpenTestReportGeneratingListener.java

@@ -10,6 +10,7 @@
 
 package org.junit.platform.reporting.open.xml;
 
+import static org.apiguardian.api.API.Status.EXPERIMENTAL;
 import static org.junit.platform.reporting.open.xml.JUnitFactory.legacyReportingName;
 import static org.junit.platform.reporting.open.xml.JUnitFactory.type;
 import static org.junit.platform.reporting.open.xml.JUnitFactory.uniqueId;
@@ -51,6 +52,7 @@
 import java.util.concurrent.ConcurrentHashMap;
 import java.util.concurrent.atomic.AtomicInteger;
 
+import org.apiguardian.api.API;
 import org.junit.platform.commons.JUnitException;
 import org.junit.platform.commons.util.StringUtils;
 import org.junit.platform.engine.ConfigurationParameters;
@@ -77,6 +79,12 @@
 import org.opentest4j.reporting.events.root.Events;
 import org.opentest4j.reporting.schema.Namespace;
 
+/**
+ * Open Test Reporting events XML generating test execution listener.
+ *
+ * @since 1.9
+ */
+@API(status = EXPERIMENTAL, since = "1.9")
 public class OpenTestReportGeneratingListener implements TestExecutionListener {
 
 	static final String ENABLED_PROPERTY_NAME = "junit.platform.reporting.open.xml.enabled";
@@ -106,7 +114,7 @@ public void testPlanExecutionStarted(TestPlan testPlan) {
 				reportInfrastructure();
 			}
 			catch (Exception e) {
-				throw new RuntimeException("Failed to initialize XML events file: " + eventsXml, e);
+				throw new JUnitException("Failed to initialize XML events file: " + eventsXml, e);
 			}
 		}
 	}

platform-tooling-support-tests/projects/jar-describe-module/junit-platform-reporting.expected.txt

@@ -1,9 +1,11 @@
 org.junit.platform.reporting@${platformVersion} jar:file:.+/junit-platform-reporting-\d.+\.jar..module-info\.class
 exports org.junit.platform.reporting.legacy
 exports org.junit.platform.reporting.legacy.xml
+exports org.junit.platform.reporting.open.xml
 requires java.base mandated
 requires java.xml
 requires org.apiguardian.api static transitive
 requires org.junit.platform.commons
 requires org.junit.platform.engine transitive
 requires org.junit.platform.launcher transitive
+provides org.junit.platform.launcher.TestExecutionListener with org.junit.platform.reporting.open.xml.OpenTestReportGeneratingListener

platform-tooling-support-tests/src/test/java/platform/tooling/support/tests/ArchUnitTests.java

@@ -20,6 +20,7 @@
 import static com.tngtech.archunit.core.domain.JavaModifier.PUBLIC;
 import static com.tngtech.archunit.core.domain.properties.HasModifiers.Predicates.modifier;
 import static com.tngtech.archunit.core.domain.properties.HasName.Predicates.name;
+import static com.tngtech.archunit.core.domain.properties.HasName.Predicates.nameContaining;
 import static com.tngtech.archunit.lang.conditions.ArchPredicates.are;
 import static com.tngtech.archunit.lang.conditions.ArchPredicates.have;
 import static com.tngtech.archunit.lang.syntax.ArchRuleDefinition.classes;
@@ -73,7 +74,8 @@ void avoidJavaUtilLogging(JavaClasses classes) {
 	@ArchTest
 	void avoidThrowingGenericExceptions(JavaClasses classes) {
 		// LoggerFactory.java:155 -> new Throwable()
-		var subset = classes.that(are(not(name("org.junit.platform.commons.logging.LoggerFactory$DelegatingLogger"))));
+		var subset = classes.that(are(not(
+			name("org.junit.platform.commons.logging.LoggerFactory$DelegatingLogger").or(nameContaining(".shadow.")))));
 		GeneralCodingRules.NO_CLASSES_SHOULD_THROW_GENERIC_EXCEPTIONS.check(subset);
 	}