Merge pull request #43761 from mkouba/issue-43751

ArC: detect incorrect PostConstruct and PreDestroy

extensions/arc/deployment/src/main/java/io/quarkus/arc/deployment/WrongAnnotationUsageProcessor.java

@@ -53,6 +53,9 @@ void detect(ArcConfig config, ApplicationIndexBuildItem applicationIndex, Custom
         unsupported.add(new UnsupportedAnnotation("org.gradle.internal.impldep.javax.inject.Inject",
                 correctInject));
 
+        unsupported.add(new UnsupportedAnnotation("javax.annotation.PostConstruct", "@jakarta.annotation.PostConstruct"));
+        unsupported.add(new UnsupportedAnnotation("javax.annotation.PreDestroy", "@jakarta.annotation.PreDestroy"));
+
         Map<AnnotationInstance, String> wrongUsages = new HashMap<>();
 
         for (UnsupportedAnnotation annotation : unsupported) {

extensions/arc/deployment/src/test/java/io/quarkus/arc/test/wrongannotations/WrongPostConstructPreDestroyTest.java

@@ -0,0 +1,48 @@
+package io.quarkus.arc.test.wrongannotations;
+
+import static org.junit.jupiter.api.Assertions.assertTrue;
+import static org.junit.jupiter.api.Assertions.fail;
+
+import javax.annotation.PostConstruct;
+import javax.annotation.PreDestroy;
+
+import jakarta.inject.Singleton;
+
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.RegisterExtension;
+
+import io.quarkus.runtime.util.ExceptionUtil;
+import io.quarkus.test.QuarkusUnitTest;
+
+public class WrongPostConstructPreDestroyTest {
+
+    @RegisterExtension
+    static final QuarkusUnitTest config = new QuarkusUnitTest()
+            .withApplicationRoot(root -> root
+                    .addClasses(MySingleton.class))
+            .assertException(t -> {
+                Throwable rootCause = ExceptionUtil.getRootCause(t);
+                assertTrue(rootCause.getMessage().contains("javax.annotation.PostConstruct"), t.toString());
+                assertTrue(rootCause.getMessage().contains("javax.annotation.PreDestroy"), t.toString());
+            });
+
+    @Test
+    public void testValidationFailed() {
+        // This method should not be invoked
+        fail();
+    }
+
+    @Singleton
+    static class MySingleton {
+
+        @PostConstruct
+        void init() {
+        }
+
+        @PreDestroy
+        void destroy() {
+        }
+
+    }
+
+}