Prevent simultaneous use of @InjectMock and @InjectSpy

Closes: #50163

Author: geoand

test-framework/junit5-mockito/src/main/java/io/quarkus/test/junit/mockito/internal/CreateMockitoSpiesCallback.java

@@ -12,6 +12,7 @@
 import io.quarkus.arc.ClientProxy;
 import io.quarkus.arc.InjectableContext;
 import io.quarkus.arc.InstanceHandle;
+import io.quarkus.test.InjectMock;
 import io.quarkus.test.junit.callback.QuarkusTestAfterAllCallback;
 import io.quarkus.test.junit.callback.QuarkusTestAfterConstructCallback;
 import io.quarkus.test.junit.callback.QuarkusTestContext;
@@ -35,7 +36,14 @@ public void afterConstruct(Object testInstance) {
         while (current.getSuperclass() != null) {
             for (Field field : current.getDeclaredFields()) {
                 InjectSpy injectSpyAnnotation = field.getAnnotation(InjectSpy.class);
+                InjectMock injectMockAnnotation = field.getAnnotation(InjectMock.class);
                 if (injectSpyAnnotation != null) {
+                    if (injectMockAnnotation != null) {
+                        cleanup();
+                        throw new IllegalArgumentException(
+                                "Fields cannot be annotated with both @InjectSpy and @InjectMock. Offending field is '"
+                                        + field.getDeclaringClass().getName() + "#" + field.getName() + "'");
+                    }
                     InstanceHandle<?> beanHandle = CreateMockitoMocksCallback.getBeanHandle(testInstance, field,
                             InjectSpy.class);
                     Object spy = createSpyAndSetTestField(testInstance, field, beanHandle,
@@ -72,6 +80,10 @@ private Object createSpyAndSetTestField(Object testInstance, Field field, Instan
 
     @Override
     public void afterAll(QuarkusTestContext context) {
+        cleanup();
+    }
+
+    private static void cleanup() {
         if (!statesToDestroy.isEmpty()) {
             for (InjectableContext.ContextState state : statesToDestroy) {
                 Arc.container().requestContext().destroy(state);