Merge pull request quarkusio#41958 from holly-cummins/correct-overamb…

…itious-test

Change assertions in templated normal test to more realistic ones

integration-tests/test-extension/tests/src/test/java/io/quarkus/it/extension/it/TestTemplateCanSeeByteCodeChangesDevModeIT.java

@@ -59,7 +59,7 @@ public void testThatTheTestsPassed() throws MavenInvocationException, IOExceptio
         // This is a bit brittle when we add tests, but failures are often so catastrophic they're not even reported as failures,
         // so we need to check the pass count explicitly
         Assertions.assertEquals(0, results.getTestsFailed());
-        Assertions.assertEquals(9, results.getTestsPassed());
+        Assertions.assertEquals(8, results.getTestsPassed());
     }
 
 }

integration-tests/test-extension/tests/src/test/java/io/quarkus/it/extension/it/TestTemplateCanSeeByteCodeChangesTestModeIT.java

@@ -69,7 +69,7 @@ public void testThatTheTestsPassed() throws MavenInvocationException, IOExceptio
         // This is a bit brittle when we add tests, but failures are often so catastrophic they're not even reported as failures,
         // so we need to check the pass count explicitly
         Assertions.assertEquals(0, results.getTestsFailed());
-        Assertions.assertEquals(9, results.getTestsPassed());
+        Assertions.assertEquals(8, results.getTestsPassed());
     }
 
 }

integration-tests/test-extension/tests/src/test/resources-filtered/projects/project-using-test-template-from-extension-with-bytecode-changes/src/test/java/org/acme/TemplatedNormalTest.java

@@ -11,7 +11,7 @@
 // No QuarkusTest annotation
 
 /**
- * It's likely we would never expect this to work; unit tests which don't have a @QuarkusTest
+ * It's likely we would never expect Quarkus bytecode changes to be visible in this kind of test; unit tests which don't have
  * annotation would not be able to
  * benefit from bytecode manipulations from extensions.
  */
@@ -46,22 +46,8 @@ void classloaderIntrospectionTestTemplate(ExtensionContext context) {
     @TestTemplate
     @ExtendWith(MyContextProvider.class)
     void contextAnnotationCheckingTestTemplate(ExtensionContext context) {
-        Annotation[] contextAnnotations = context.getRequiredTestClass()
-                .getAnnotations();
-        Assertions.assertTrue(Arrays.toString(contextAnnotations)
-                .contains("AnnotationAddedByExtension"),
-                "The JUnit extension context does not see the annotation, only sees " + Arrays.toString(
-                        contextAnnotations));
+        // We don't expect to see the annotations because we don't have a @QuarkusTest annotation, but the basic test should work
+        Assertions.assertEquals(true, true);
     }
 
-    @TestTemplate
-    @ExtendWith(MyContextProvider.class)
-    void executionAnnotationCheckingTestTemplate(ExtensionContext context) {
-        Annotation[] myAnnotations = this.getClass()
-                .getAnnotations();
-        Assertions.assertTrue(Arrays.toString(myAnnotations)
-                .contains("AnnotationAddedByExtension"),
-                "The test execution does not see the annotation, only sees " + Arrays.toString(
-                        myAnnotations));
-    }
 }

integration-tests/test-extension/tests/src/test/resources-filtered/projects/project-using-test-template-from-extension/src/test/java/org/acme/TemplatedNormalTest.java

@@ -8,7 +8,8 @@
 // No QuarkusTest annotation
 
 /**
- * It's likely we would never expect this to work; unit tests which don't have a @QuarkusTest
+ * It's likely we would never expect Quarkus bytecode changes to be visible in this kind of test; unit tests which don't have
+ * a @QuarkusTest
  * annotation would not be able to
  * benefit from bytecode manipulations from extensions.
  */