From dfcb955095a3f4378a0dec5a4ac9bcd8c890c6a3 Mon Sep 17 00:00:00 2001
From: Falko Modler <famod@users.noreply.github.com>
Date: Fri, 21 Jan 2022 23:46:02 +0100
Subject: [PATCH] Load MockitoConfiguration parent-first to make it work in
 continuous testing

Moves the class to a separate new module to reduce the blast radius.
---
 bom/application/pom.xml                       |  5 ++++
 core/runtime/pom.xml                          |  1 +
 test-framework/junit5-mockito-config/pom.xml  | 27 +++++++++++++++++++
 .../configuration/MockitoConfiguration.java   |  4 +--
 test-framework/junit5-mockito/pom.xml         |  7 +++++
 test-framework/junit5-properties/pom.xml      |  2 +-
 test-framework/pom.xml                        |  1 +
 7 files changed, 43 insertions(+), 4 deletions(-)
 create mode 100644 test-framework/junit5-mockito-config/pom.xml
 rename test-framework/{junit5-mockito => junit5-mockito-config}/src/main/java/org/mockito/configuration/MockitoConfiguration.java (86%)

diff --git a/bom/application/pom.xml b/bom/application/pom.xml
index 29eef6fd5f484..9e16eb02e80a7 100644
--- a/bom/application/pom.xml
+++ b/bom/application/pom.xml
@@ -2576,6 +2576,11 @@
                 <artifactId>quarkus-junit5-mockito</artifactId>
                 <version>${project.version}</version>
             </dependency>
+            <dependency>
+                <groupId>io.quarkus</groupId>
+                <artifactId>quarkus-junit5-mockito-config</artifactId>
+                <version>${project.version}</version>
+            </dependency>
             <dependency>
                 <groupId>io.quarkus</groupId>
                 <artifactId>quarkus-junit5-vertx</artifactId>
diff --git a/core/runtime/pom.xml b/core/runtime/pom.xml
index c7962c99bd504..582b4e42fa4ae 100644
--- a/core/runtime/pom.xml
+++ b/core/runtime/pom.xml
@@ -170,6 +170,7 @@
                         <parentFirstArtifact>io.quarkus:quarkus-class-change-agent</parentFirstArtifact>
                         <parentFirstArtifact>org.jacoco:org.jacoco.agent:runtime</parentFirstArtifact>
                         <parentFirstArtifact>io.quarkus:quarkus-bootstrap-gradle-resolver</parentFirstArtifact>
+                        <parentFirstArtifact>io.quarkus:quarkus-junit5-mockito-config</parentFirstArtifact>
 
                         <!-- RestAssured uses groovy, which seems to do some things with soft references that
                         prevent the ClassLoader from being GC'ed, see https://github.com/quarkusio/quarkus/issues/12498 -->
diff --git a/test-framework/junit5-mockito-config/pom.xml b/test-framework/junit5-mockito-config/pom.xml
new file mode 100644
index 0000000000000..505bb46ed6138
--- /dev/null
+++ b/test-framework/junit5-mockito-config/pom.xml
@@ -0,0 +1,27 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<project xmlns="http://maven.apache.org/POM/4.0.0"
+         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
+    <modelVersion>4.0.0</modelVersion>
+
+    <parent>
+        <groupId>io.quarkus</groupId>
+        <artifactId>quarkus-test-framework</artifactId>
+        <version>999-SNAPSHOT</version>
+    </parent>
+
+    <artifactId>quarkus-junit5-mockito-config</artifactId>
+    <name>Quarkus - Test framework - JUnit 5 - Mockito Config</name>
+    <description>
+        Contains a MockitoConfiguration that has to be loaded parent-first to work in continuous testing.
+        It is separated from junit5-mockito to minimize the blast radius.
+    </description>
+
+    <dependencies>
+        <dependency>
+            <groupId>org.mockito</groupId>
+            <artifactId>mockito-core</artifactId>
+        </dependency>
+    </dependencies>
+
+</project>
diff --git a/test-framework/junit5-mockito/src/main/java/org/mockito/configuration/MockitoConfiguration.java b/test-framework/junit5-mockito-config/src/main/java/org/mockito/configuration/MockitoConfiguration.java
similarity index 86%
rename from test-framework/junit5-mockito/src/main/java/org/mockito/configuration/MockitoConfiguration.java
rename to test-framework/junit5-mockito-config/src/main/java/org/mockito/configuration/MockitoConfiguration.java
index fdd103e884d23..df02392f8ed76 100644
--- a/test-framework/junit5-mockito/src/main/java/org/mockito/configuration/MockitoConfiguration.java
+++ b/test-framework/junit5-mockito-config/src/main/java/org/mockito/configuration/MockitoConfiguration.java
@@ -4,8 +4,6 @@
 
 import org.mockito.stubbing.Answer;
 
-import io.quarkus.test.junit.mockito.internal.MutinyAnswer;
-
 public class MockitoConfiguration extends DefaultMockitoConfiguration {
 
     @SuppressWarnings("unchecked")
@@ -14,7 +12,7 @@ public Answer<Object> getDefaultAnswer() {
         ClassLoader cl = Thread.currentThread().getContextClassLoader();
         try {
             // we need to load it from the TCCL (QuarkusClassLoader) instead of our class loader (JUnit CL)
-            Class<?> mutinyAnswer = cl.loadClass(MutinyAnswer.class.getName());
+            Class<?> mutinyAnswer = cl.loadClass("io.quarkus.test.junit.mockito.internal.MutinyAnswer");
             return (Answer<Object>) mutinyAnswer.getDeclaredConstructor().newInstance();
         } catch (ClassNotFoundException | SecurityException | IllegalArgumentException | IllegalAccessException
                 | InstantiationException | NoSuchMethodException | InvocationTargetException e) {
diff --git a/test-framework/junit5-mockito/pom.xml b/test-framework/junit5-mockito/pom.xml
index 337bbb7347f96..aed8ac6101148 100644
--- a/test-framework/junit5-mockito/pom.xml
+++ b/test-framework/junit5-mockito/pom.xml
@@ -18,6 +18,13 @@
             <groupId>io.quarkus</groupId>
             <artifactId>quarkus-junit5</artifactId>
         </dependency>
+        <!-- Contains the MockitoConfiguration that adds MutinyAnswer as a default answer.
+             The dependency is in this direction so that users don't need to worry about it
+             and only have to add this dependency (quarkus-junit5-mockito), but not ...-config. -->
+        <dependency>
+            <groupId>io.quarkus</groupId>
+            <artifactId>quarkus-junit5-mockito-config</artifactId>
+        </dependency>
         <dependency>
             <groupId>io.quarkus</groupId>
             <artifactId>quarkus-arc-deployment</artifactId>
diff --git a/test-framework/junit5-properties/pom.xml b/test-framework/junit5-properties/pom.xml
index a7a604a25caba..63fb5bf6fe370 100644
--- a/test-framework/junit5-properties/pom.xml
+++ b/test-framework/junit5-properties/pom.xml
@@ -13,7 +13,7 @@
     <artifactId>quarkus-junit5-properties</artifactId>
     <name>Quarkus - Test Framework - JUnit 5 - Properties</name>
     <description>
-        Contains junit-platform.properties in a "use-excludable" way
+        Contains junit-platform.properties in a "user-excludable" way
         until https://github.com/junit-team/junit5/issues/2794 is available.
     </description>
 
diff --git a/test-framework/pom.xml b/test-framework/pom.xml
index 9e30d6d7b1edf..1eef291e6d199 100644
--- a/test-framework/pom.xml
+++ b/test-framework/pom.xml
@@ -28,6 +28,7 @@
         <module>junit5-properties</module>
         <module>junit5</module>
         <module>junit5-mockito</module>
+        <module>junit5-mockito-config</module>
         <module>junit5-vertx</module>
         <module>amazon-lambda</module>
         <module>arquillian</module>