Add negative tests for build compatible extensions with invalid metho…

impl/src/main/java/org/jboss/cdi/tck/tests/build/compatible/extensions/invalid/AbstractInvalidExtensionParamTest.java

@@ -0,0 +1,28 @@
+package org.jboss.cdi.tck.tests.build.compatible.extensions.invalid;
+
+import org.jboss.cdi.tck.AbstractTest;
+import org.jboss.cdi.tck.shrinkwrap.WebArchiveBuilder;
+import org.jboss.shrinkwrap.impl.BeansXml;
+import org.testng.annotations.Test;
+
+public abstract class AbstractInvalidExtensionParamTest extends AbstractTest {
+
+    /**
+     * Prepared archive builder minus the extension being tested
+     */
+    static WebArchiveBuilder prepareArchiveBuilder() {
+        return new WebArchiveBuilder()
+                // add some class that will be type-discovered
+                .withClasses(SomeBean.class)
+                .withTestClass(AbstractInvalidExtensionParamTest.class)
+                // annotated discovery mode
+                .withBeansXml(new BeansXml());
+    }
+
+
+    @Test()
+    //@SpecAssertion(section = TODO, id = "TODO")
+    public void test() {
+        // deployment should fail
+    }
+}

impl/src/main/java/org/jboss/cdi/tck/tests/build/compatible/extensions/invalid/EnhancementMultipleParams2Test.java

@@ -0,0 +1,17 @@
+package org.jboss.cdi.tck.tests.build.compatible.extensions.invalid;
+
+import jakarta.enterprise.inject.spi.DefinitionException;
+import org.jboss.arquillian.container.test.api.Deployment;
+import org.jboss.arquillian.container.test.api.ShouldThrowException;
+import org.jboss.shrinkwrap.api.spec.WebArchive;
+import org.jboss.test.audit.annotations.SpecVersion;
+
+@SpecVersion(spec = "cdi", version = "4.0")
+public class EnhancementMultipleParams2Test extends AbstractInvalidExtensionParamTest {
+
+    @ShouldThrowException(DefinitionException.class)
+    @Deployment
+    public static WebArchive createTestArchive() {
+        return prepareArchiveBuilder().withBuildCompatibleExtension(EnhancementMultipleParamsExtension2.class).build();
+    }
+}

impl/src/main/java/org/jboss/cdi/tck/tests/build/compatible/extensions/invalid/EnhancementMultipleParamsExtension.java

@@ -0,0 +1,15 @@
+package org.jboss.cdi.tck.tests.build.compatible.extensions.invalid;
+
+import jakarta.enterprise.inject.build.compatible.spi.BuildCompatibleExtension;
+import jakarta.enterprise.inject.build.compatible.spi.ClassConfig;
+import jakarta.enterprise.inject.build.compatible.spi.Enhancement;
+import jakarta.enterprise.inject.build.compatible.spi.Messages;
+import jakarta.enterprise.lang.model.declarations.ClassInfo;
+
+public class EnhancementMultipleParamsExtension implements BuildCompatibleExtension {
+
+    @Enhancement(types = SomeBean.class)
+    public void enhance(ClassConfig cc, ClassInfo ci, Messages msg) {
+        // no-op deployment should fail
+    }
+}

impl/src/main/java/org/jboss/cdi/tck/tests/build/compatible/extensions/invalid/EnhancementMultipleParamsExtension2.java

@@ -0,0 +1,15 @@
+package org.jboss.cdi.tck.tests.build.compatible.extensions.invalid;
+
+import jakarta.enterprise.inject.build.compatible.spi.BuildCompatibleExtension;
+import jakarta.enterprise.inject.build.compatible.spi.ClassConfig;
+import jakarta.enterprise.inject.build.compatible.spi.Enhancement;
+import jakarta.enterprise.inject.build.compatible.spi.MethodConfig;
+import jakarta.enterprise.lang.model.declarations.FieldInfo;
+
+public class EnhancementMultipleParamsExtension2 implements BuildCompatibleExtension {
+
+    @Enhancement(types = SomeBean.class)
+    public void enhance(ClassConfig cc, MethodConfig mc, FieldInfo fi) {
+        // no-op deployment should fail
+    }
+}

impl/src/main/java/org/jboss/cdi/tck/tests/build/compatible/extensions/invalid/EnhancementMultipleParamsTest.java

@@ -0,0 +1,17 @@
+package org.jboss.cdi.tck.tests.build.compatible.extensions.invalid;
+
+import jakarta.enterprise.inject.spi.DefinitionException;
+import org.jboss.arquillian.container.test.api.Deployment;
+import org.jboss.arquillian.container.test.api.ShouldThrowException;
+import org.jboss.shrinkwrap.api.spec.WebArchive;
+import org.jboss.test.audit.annotations.SpecVersion;
+
+@SpecVersion(spec = "cdi", version = "4.0")
+public class EnhancementMultipleParamsTest extends AbstractInvalidExtensionParamTest {
+
+    @ShouldThrowException(DefinitionException.class)
+    @Deployment
+    public static WebArchive createTestArchive() {
+        return prepareArchiveBuilder().withBuildCompatibleExtension(EnhancementMultipleParamsExtension.class).build();
+    }
+}

impl/src/main/java/org/jboss/cdi/tck/tests/build/compatible/extensions/invalid/EnhancementNoParamExtension.java

@@ -0,0 +1,14 @@
+package org.jboss.cdi.tck.tests.build.compatible.extensions.invalid;
+
+import jakarta.enterprise.inject.build.compatible.spi.BuildCompatibleExtension;
+import jakarta.enterprise.inject.build.compatible.spi.Enhancement;
+import org.jboss.test.audit.annotations.SpecVersion;
+
+@SpecVersion(spec = "cdi", version = "4.0")
+public class EnhancementNoParamExtension implements BuildCompatibleExtension {
+
+    @Enhancement(types = SomeBean.class)
+    public void enhance() {
+        // no-op deployment should fail
+    }
+}

impl/src/main/java/org/jboss/cdi/tck/tests/build/compatible/extensions/invalid/EnhancementNoParamTest.java

@@ -0,0 +1,17 @@
+package org.jboss.cdi.tck.tests.build.compatible.extensions.invalid;
+
+import jakarta.enterprise.inject.spi.DefinitionException;
+import org.jboss.arquillian.container.test.api.Deployment;
+import org.jboss.arquillian.container.test.api.ShouldThrowException;
+import org.jboss.shrinkwrap.api.spec.WebArchive;
+import org.jboss.test.audit.annotations.SpecVersion;
+
+@SpecVersion(spec = "cdi", version = "4.0")
+public class EnhancementNoParamTest extends AbstractInvalidExtensionParamTest {
+
+    @ShouldThrowException(DefinitionException.class)
+    @Deployment
+    public static WebArchive createTestArchive() {
+        return prepareArchiveBuilder().withBuildCompatibleExtension(EnhancementNoParamExtension.class).build();
+    }
+}

impl/src/main/java/org/jboss/cdi/tck/tests/build/compatible/extensions/invalid/EnhancementOnlyMessagesExtension.java

@@ -0,0 +1,13 @@
+package org.jboss.cdi.tck.tests.build.compatible.extensions.invalid;
+
+import jakarta.enterprise.inject.build.compatible.spi.BuildCompatibleExtension;
+import jakarta.enterprise.inject.build.compatible.spi.Enhancement;
+import jakarta.enterprise.inject.build.compatible.spi.Messages;
+
+public class EnhancementOnlyMessagesExtension implements BuildCompatibleExtension {
+
+    @Enhancement(types = SomeBean.class)
+    public void enhance(Messages msg) {
+        // no-op deployment should fail
+    }
+}

impl/src/main/java/org/jboss/cdi/tck/tests/build/compatible/extensions/invalid/EnhancementOnlyMessagesTest.java

@@ -0,0 +1,17 @@
+package org.jboss.cdi.tck.tests.build.compatible.extensions.invalid;
+
+import jakarta.enterprise.inject.spi.DefinitionException;
+import org.jboss.arquillian.container.test.api.Deployment;
+import org.jboss.arquillian.container.test.api.ShouldThrowException;
+import org.jboss.shrinkwrap.api.spec.WebArchive;
+import org.jboss.test.audit.annotations.SpecVersion;
+
+@SpecVersion(spec = "cdi", version = "4.0")
+public class EnhancementOnlyMessagesTest extends AbstractInvalidExtensionParamTest {
+
+    @ShouldThrowException(DefinitionException.class)
+    @Deployment
+    public static WebArchive createTestArchive() {
+        return prepareArchiveBuilder().withBuildCompatibleExtension(EnhancementOnlyMessagesExtension.class).build();
+    }
+}

impl/src/main/java/org/jboss/cdi/tck/tests/build/compatible/extensions/invalid/EnhancementOnlyTypesExtension.java

@@ -0,0 +1,13 @@
+package org.jboss.cdi.tck.tests.build.compatible.extensions.invalid;
+
+import jakarta.enterprise.inject.build.compatible.spi.BuildCompatibleExtension;
+import jakarta.enterprise.inject.build.compatible.spi.Enhancement;
+import jakarta.enterprise.inject.build.compatible.spi.Types;
+
+public class EnhancementOnlyTypesExtension implements BuildCompatibleExtension {
+
+    @Enhancement(types = SomeBean.class)
+    public void enhance(Types types) {
+        // no-op deployment should fail
+    }
+}

impl/src/main/java/org/jboss/cdi/tck/tests/build/compatible/extensions/invalid/EnhancementOnlyTypesTest.java

@@ -0,0 +1,17 @@
+package org.jboss.cdi.tck.tests.build.compatible.extensions.invalid;
+
+import jakarta.enterprise.inject.spi.DefinitionException;
+import org.jboss.arquillian.container.test.api.Deployment;
+import org.jboss.arquillian.container.test.api.ShouldThrowException;
+import org.jboss.shrinkwrap.api.spec.WebArchive;
+import org.jboss.test.audit.annotations.SpecVersion;
+
+@SpecVersion(spec = "cdi", version = "4.0")
+public class EnhancementOnlyTypesTest extends AbstractInvalidExtensionParamTest {
+
+    @ShouldThrowException(DefinitionException.class)
+    @Deployment
+    public static WebArchive createTestArchive() {
+        return prepareArchiveBuilder().withBuildCompatibleExtension(EnhancementOnlyTypesExtension.class).build();
+    }
+}

impl/src/main/java/org/jboss/cdi/tck/tests/build/compatible/extensions/invalid/RegistrationMultipleParams2Test.java

@@ -0,0 +1,17 @@
+package org.jboss.cdi.tck.tests.build.compatible.extensions.invalid;
+
+import jakarta.enterprise.inject.spi.DefinitionException;
+import org.jboss.arquillian.container.test.api.Deployment;
+import org.jboss.arquillian.container.test.api.ShouldThrowException;
+import org.jboss.shrinkwrap.api.spec.WebArchive;
+import org.jboss.test.audit.annotations.SpecVersion;
+
+@SpecVersion(spec = "cdi", version = "4.0")
+public class RegistrationMultipleParams2Test extends AbstractInvalidExtensionParamTest {
+
+    @ShouldThrowException(DefinitionException.class)
+    @Deployment
+    public static WebArchive createTestArchive() {
+        return prepareArchiveBuilder().withBuildCompatibleExtension(RegistrationMultipleParamsExtension2.class).build();
+    }
+}

impl/src/main/java/org/jboss/cdi/tck/tests/build/compatible/extensions/invalid/RegistrationMultipleParamsExtension.java

@@ -0,0 +1,14 @@
+package org.jboss.cdi.tck.tests.build.compatible.extensions.invalid;
+
+import jakarta.enterprise.inject.build.compatible.spi.BeanInfo;
+import jakarta.enterprise.inject.build.compatible.spi.BuildCompatibleExtension;
+import jakarta.enterprise.inject.build.compatible.spi.InterceptorInfo;
+import jakarta.enterprise.inject.build.compatible.spi.Registration;
+
+public class RegistrationMultipleParamsExtension implements BuildCompatibleExtension {
+
+    @Registration(types = {SomeBean.class})
+    public void register(BeanInfo bi, InterceptorInfo ii) {
+        // no-op, deployment should fail
+    }
+}

impl/src/main/java/org/jboss/cdi/tck/tests/build/compatible/extensions/invalid/RegistrationMultipleParamsExtension2.java

@@ -0,0 +1,15 @@
+package org.jboss.cdi.tck.tests.build.compatible.extensions.invalid;
+
+import jakarta.enterprise.inject.build.compatible.spi.BeanInfo;
+import jakarta.enterprise.inject.build.compatible.spi.BuildCompatibleExtension;
+import jakarta.enterprise.inject.build.compatible.spi.ObserverInfo;
+import jakarta.enterprise.inject.build.compatible.spi.Registration;
+import jakarta.enterprise.inject.build.compatible.spi.Types;
+
+public class RegistrationMultipleParamsExtension2 implements BuildCompatibleExtension {
+
+    @Registration(types = {SomeBean.class})
+    public void register(ObserverInfo oi, Types t, BeanInfo bi) {
+        // no-op, deployment should fail
+    }
+}

impl/src/main/java/org/jboss/cdi/tck/tests/build/compatible/extensions/invalid/RegistrationMultipleParamsTest.java

@@ -0,0 +1,17 @@
+package org.jboss.cdi.tck.tests.build.compatible.extensions.invalid;
+
+import jakarta.enterprise.inject.spi.DefinitionException;
+import org.jboss.arquillian.container.test.api.Deployment;
+import org.jboss.arquillian.container.test.api.ShouldThrowException;
+import org.jboss.shrinkwrap.api.spec.WebArchive;
+import org.jboss.test.audit.annotations.SpecVersion;
+
+@SpecVersion(spec = "cdi", version = "4.0")
+public class RegistrationMultipleParamsTest extends AbstractInvalidExtensionParamTest {
+
+    @ShouldThrowException(DefinitionException.class)
+    @Deployment
+    public static WebArchive createTestArchive() {
+        return prepareArchiveBuilder().withBuildCompatibleExtension(RegistrationMultipleParamsExtension.class).build();
+    }
+}

impl/src/main/java/org/jboss/cdi/tck/tests/build/compatible/extensions/invalid/RegistrationNoParamExtension.java

@@ -0,0 +1,12 @@
+package org.jboss.cdi.tck.tests.build.compatible.extensions.invalid;
+
+import jakarta.enterprise.inject.build.compatible.spi.BuildCompatibleExtension;
+import jakarta.enterprise.inject.build.compatible.spi.Registration;
+
+public class RegistrationNoParamExtension implements BuildCompatibleExtension {
+
+    @Registration(types = {SomeBean.class})
+    public void register() {
+        // no-op, deployment should fail
+    }
+}

impl/src/main/java/org/jboss/cdi/tck/tests/build/compatible/extensions/invalid/RegistrationNoParamTest.java

@@ -0,0 +1,17 @@
+package org.jboss.cdi.tck.tests.build.compatible.extensions.invalid;
+
+import jakarta.enterprise.inject.spi.DefinitionException;
+import org.jboss.arquillian.container.test.api.Deployment;
+import org.jboss.arquillian.container.test.api.ShouldThrowException;
+import org.jboss.shrinkwrap.api.spec.WebArchive;
+import org.jboss.test.audit.annotations.SpecVersion;
+
+@SpecVersion(spec = "cdi", version = "4.0")
+public class RegistrationNoParamTest extends AbstractInvalidExtensionParamTest {
+
+    @ShouldThrowException(DefinitionException.class)
+    @Deployment
+    public static WebArchive createTestArchive() {
+        return prepareArchiveBuilder().withBuildCompatibleExtension(RegistrationNoParamExtension.class).build();
+    }
+}

impl/src/main/java/org/jboss/cdi/tck/tests/build/compatible/extensions/invalid/RegistrationOnlyMessagesExtension.java

@@ -0,0 +1,13 @@
+package org.jboss.cdi.tck.tests.build.compatible.extensions.invalid;
+
+import jakarta.enterprise.inject.build.compatible.spi.BuildCompatibleExtension;
+import jakarta.enterprise.inject.build.compatible.spi.Messages;
+import jakarta.enterprise.inject.build.compatible.spi.Registration;
+
+public class RegistrationOnlyMessagesExtension implements BuildCompatibleExtension {
+
+    @Registration(types = {SomeBean.class})
+    public void register(Messages m) {
+        // no-op, deployment should fail
+    }
+}

impl/src/main/java/org/jboss/cdi/tck/tests/build/compatible/extensions/invalid/RegistrationOnlyMessagesTest.java

@@ -0,0 +1,17 @@
+package org.jboss.cdi.tck.tests.build.compatible.extensions.invalid;
+
+import jakarta.enterprise.inject.spi.DefinitionException;
+import org.jboss.arquillian.container.test.api.Deployment;
+import org.jboss.arquillian.container.test.api.ShouldThrowException;
+import org.jboss.shrinkwrap.api.spec.WebArchive;
+import org.jboss.test.audit.annotations.SpecVersion;
+
+@SpecVersion(spec = "cdi", version = "4.0")
+public class RegistrationOnlyMessagesTest extends AbstractInvalidExtensionParamTest {
+
+    @ShouldThrowException(DefinitionException.class)
+    @Deployment
+    public static WebArchive createTestArchive() {
+        return prepareArchiveBuilder().withBuildCompatibleExtension(RegistrationOnlyMessagesExtension.class).build();
+    }
+}

impl/src/main/java/org/jboss/cdi/tck/tests/build/compatible/extensions/invalid/RegistrationOnlyTypesExtension.java

@@ -0,0 +1,13 @@
+package org.jboss.cdi.tck.tests.build.compatible.extensions.invalid;
+
+import jakarta.enterprise.inject.build.compatible.spi.BuildCompatibleExtension;
+import jakarta.enterprise.inject.build.compatible.spi.Registration;
+import jakarta.enterprise.inject.build.compatible.spi.Types;
+
+public class RegistrationOnlyTypesExtension implements BuildCompatibleExtension {
+
+    @Registration(types = {SomeBean.class})
+    public void register(Types t) {
+        // no-op, deployment should fail
+    }
+}

impl/src/main/java/org/jboss/cdi/tck/tests/build/compatible/extensions/invalid/RegistrationOnlyTypesTest.java

@@ -0,0 +1,17 @@
+package org.jboss.cdi.tck.tests.build.compatible.extensions.invalid;
+
+import jakarta.enterprise.inject.spi.DefinitionException;
+import org.jboss.arquillian.container.test.api.Deployment;
+import org.jboss.arquillian.container.test.api.ShouldThrowException;
+import org.jboss.shrinkwrap.api.spec.WebArchive;
+import org.jboss.test.audit.annotations.SpecVersion;
+
+@SpecVersion(spec = "cdi", version = "4.0")
+public class RegistrationOnlyTypesTest extends AbstractInvalidExtensionParamTest {
+
+    @ShouldThrowException(DefinitionException.class)
+    @Deployment
+    public static WebArchive createTestArchive() {
+        return prepareArchiveBuilder().withBuildCompatibleExtension(RegistrationOnlyTypesExtension.class).build();
+    }
+}

impl/src/main/java/org/jboss/cdi/tck/tests/build/compatible/extensions/invalid/SomeBean.java

@@ -0,0 +1,8 @@
+package org.jboss.cdi.tck.tests.build.compatible.extensions.invalid;
+
+import jakarta.enterprise.context.Dependent;
+
+// Dummy bean class just to have some in the bean archive
+@Dependent
+public class SomeBean {
+}