#3187 Add back testing for DialogProviderAnnotationProcessor

bundle/pom.xml

@@ -498,6 +498,11 @@
             <artifactId>hamcrest-all</artifactId>
             <scope>test</scope>
         </dependency>
+        <dependency>
+            <groupId>org.junit.jupiter</groupId>
+            <artifactId>junit-jupiter</artifactId>
+            <scope>test</scope>
+        </dependency>
         <dependency>
             <groupId>junit</groupId>
             <artifactId>junit</artifactId>

bundle/src/main/java/com/adobe/acs/commons/mcp/form/DialogProviderAnnotationProcessor.java

@@ -69,7 +69,7 @@ public SourceVersion getSupportedSourceVersion() {
     private void processDialogProviderAnnotation(Element element) throws IOException {
         TypeElement t = (TypeElement) element;
         String className = t.getQualifiedName().toString();
-        String serviceClassName = DialogResourceProvider.getServiceClassName(className);
+        String serviceClassName = getServiceClassName(className);
         if (providesResourceType(t)) {
             if (LOG.isLoggable(Level.INFO)) {
                 LOG.log(Level.INFO, String.format("Generated resource provider service for class %s => %s", className, serviceClassName));
@@ -128,4 +128,24 @@ private boolean elementProvidesResourceType(Element t) {
                 return false;
         }
     }
+
+    private static String getServiceClassName(String modelClass) {
+        String[] parts = StringUtils.split(modelClass, '.');
+        StringBuilder name = new StringBuilder();
+        String separator = ".";
+        for (String part : parts) {
+            char firstChar = part.charAt(0);
+            String newSeparator = separator;
+            if (firstChar >= 'A' && firstChar <= 'Z' && separator.equals(".")) {
+                newSeparator = "$";
+                name.append(".impl");
+            }
+            if (name.length() > 0) {
+                name.append(separator);
+            }
+            name.append(part);
+            separator = newSeparator;
+        }
+        return name + "_dialogResourceProvider";
+    }
 }

bundle/src/main/java/com/adobe/acs/commons/mcp/form/DialogResourceProvider.java

@@ -37,14 +37,14 @@
  */
 @SuppressWarnings("squid:S1214") // There are no constants declared here, not sure why this code smell is being detected
 public interface DialogResourceProvider {
-    Class getTargetClass();
+    Class<?> getTargetClass();
 
     default DialogProvider getDialogProvider() {
-        return (DialogProvider) getTargetClass().getAnnotation(DialogProvider.class);
+        return getTargetClass().getAnnotation(DialogProvider.class);
     }
 
     @SuppressWarnings("squid:S2386")
-    public static Map<Class, ServiceRegistration> registeredProviders = Collections.synchronizedMap(new HashMap<>());
+    public static Map<Class<?>, ServiceRegistration<?>> registeredProviders = Collections.synchronizedMap(new HashMap<>());
 
     @SuppressWarnings("squid:S1149") // Yes HashTable sucks but it's required here.
     default void doActivate(BundleContext bundleContext) throws InstantiationException, IllegalAccessException, IllegalArgumentException, InvocationTargetException, NoSuchMethodException, SecurityException   {
@@ -54,36 +54,15 @@ default void doActivate(BundleContext bundleContext) throws InstantiationExcepti
         props.put(ResourceProvider.PROPERTY_NAME, provider.getRoot());
         props.put(ResourceProvider.PROPERTY_ROOT, provider.getRoot());
         props.put(ResourceProvider.PROPERTY_USE_RESOURCE_ACCESS_SECURITY, Boolean.FALSE);
-        ServiceRegistration providerRegistration = bundleContext.registerService(ResourceProvider.class, provider, props);
+        ServiceRegistration<?> providerRegistration = bundleContext.registerService(ResourceProvider.class, provider, props);
         registeredProviders.put(getTargetClass(), providerRegistration);
     }
 
     default void doDeactivate() {
-        ServiceRegistration providerRegistration = registeredProviders.get(getTargetClass());
+        ServiceRegistration<?> providerRegistration = registeredProviders.get(getTargetClass());
         if (providerRegistration != null) {
             providerRegistration.unregister();
         }
         registeredProviders.remove(getTargetClass());
     }
-
-    public static String getServiceClassName(String modelClass) {
-        String[] parts = modelClass.split("\\.");
-        String name = "";
-        String separator = ".";
-        for (String part : parts) {
-            char firstChar = part.charAt(0);
-            String newSeparator = separator;
-            if (firstChar >= 'A' && firstChar <= 'Z' && separator.equals(".")) {
-                newSeparator = "$";
-                name += ".impl";
-            }
-            if (name.length() > 0) {
-                name += separator;
-            }
-            name += part;
-            separator = newSeparator;
-        }
-        return name + "_dialogResourceProvider";
-    }
-
 }

bundle/src/test/java/com/adobe/acs/commons/mcp/form/DialogProviderAnnotationProcessorTest.java

@@ -0,0 +1,154 @@
+package com.adobe.acs.commons.mcp.form;
+
+import com.google.testing.compile.Compilation;
+import com.google.testing.compile.CompilationSubject;
+import com.google.testing.compile.Compiler;
+import com.google.testing.compile.JavaFileObjects;
+import org.jetbrains.annotations.NotNull;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.params.ParameterizedTest;
+import org.junit.jupiter.params.provider.Arguments;
+import org.junit.jupiter.params.provider.MethodSource;
+
+import javax.annotation.processing.Processor;
+import javax.lang.model.SourceVersion;
+import java.util.stream.Stream;
+
+import static org.hamcrest.MatcherAssert.assertThat;
+import static org.hamcrest.Matchers.contains;
+import static org.hamcrest.Matchers.empty;
+import static org.hamcrest.Matchers.is;
+import static org.junit.jupiter.api.Assertions.assertAll;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.params.provider.Arguments.arguments;
+
+public class DialogProviderAnnotationProcessorTest {
+    private static final Processor UNDER_TEST = new DialogProviderAnnotationProcessor();
+    @Test
+    void testConfiguration() {
+        assertAll(
+                () -> assertThat(UNDER_TEST.getSupportedAnnotationTypes(), contains(
+                        DialogProvider.class.getCanonicalName()
+                )),
+                () -> assertEquals(SourceVersion.latestSupported(), UNDER_TEST.getSupportedSourceVersion())
+        );
+    }
+
+    @Test
+    void whenTheComputedFileAlreadyExistsAnExceptionIsThrownHoweverThatShouldBeIgnored() {
+        for (int i = 0; i < 2; i++) {
+            final Compilation compilation = compile("Example1", "package a; @com.adobe.acs.commons.mcp.form.DialogProvider public class Example1 {public String getResourceType(){return \"Something else\";}}");
+        }
+    }
+
+    @Test
+    void whenResourceTypeIsNotSuppliedTheAnnotationProcessorWillNotProduceAnAdditionalSourceFileForThatClass() {
+        final Compilation compilation = compile("ExampleFaulty", "package a; @com.adobe.acs.commons.mcp.form.DialogProvider public class ExampleFaulty {public String getResult(){return \"Something else\";}}");
+        assertThat(compilation.generatedSourceFiles(), is(empty()));
+    }
+
+    @ParameterizedTest
+    @MethodSource
+    void annotationProviderProducesAdditionalSourceFile(@NotNull final String name, @NotNull final String source, @NotNull final String expectedResult) {
+        final Compilation compilation = compile(name, source);
+        assertAll(
+                () -> CompilationSubject
+                        .assertThat(compilation).succeeded(),
+                () -> CompilationSubject
+                        .assertThat(compilation)
+                        .generatedSourceFile("a/impl/" + name + "_dialogResourceProvider")
+                        .contentsAsUtf8String()
+                        .isEqualTo(expectedResult)
+        );
+    }
+
+    @NotNull
+    private static Compilation compile(@NotNull String name, @NotNull String source) {
+        return Compiler.javac()
+                .withProcessors(new DialogProviderAnnotationProcessor())
+                .compile(JavaFileObjects.forSourceString("a." + name, source));
+    }
+
+    private static Stream<Arguments> annotationProviderProducesAdditionalSourceFile() {
+        return Stream.of(
+                arguments("Example1", "package a; @com.adobe.acs.commons.mcp.form.DialogProvider public class Example1 {public String getResourceType(){return \"my.type\";}}", "package a.impl;\n" +
+                        "\n" +
+                        "import javax.annotation.Generated;\n" +
+                        "import org.osgi.annotation.versioning.ConsumerType;\n" +
+                        "import org.osgi.framework.BundleContext;\n" +
+                        "import org.osgi.service.component.annotations.*;\n" +
+                        "\n" +
+                        "@Generated(\"Created by the ACS Commons DialogProviderAnnotationProcessor\")\n" +
+                        "@ConsumerType\n" +
+                        "@Component(service = com.adobe.acs.commons.mcp.form.DialogResourceProvider.class, immediate = true)\n" +
+                        "public class Example1_dialogResourceProvider implements com.adobe.acs.commons.mcp.form.DialogResourceProvider {\n" +
+                        "\n" +
+                        "    @Override\n" +
+                        "    public Class getTargetClass() {\n" +
+                        "        return a.Example1.class;\n" +
+                        "    }\n" +
+                        "    @Activate\n" +
+                        "    public void activate(BundleContext context) throws InstantiationException, IllegalAccessException, ReflectiveOperationException {\n" +
+                        "        this.doActivate(context);\n" +
+                        "    }\n" +
+                        "\n" +
+                        "    @Deactivate\n" +
+                        "    public void deactivate(BundleContext context) {\n" +
+                        "        this.doDeactivate();\n" +
+                        "    }\n" +
+                        "}\n"),
+                arguments("Example2", "package a; @com.adobe.acs.commons.mcp.form.DialogProvider public class Example2 {public String resourceType=\"my.type\";}", "package a.impl;\n" +
+                        "\n" +
+                        "import javax.annotation.Generated;\n" +
+                        "import org.osgi.annotation.versioning.ConsumerType;\n" +
+                        "import org.osgi.framework.BundleContext;\n" +
+                        "import org.osgi.service.component.annotations.*;\n" +
+                        "\n" +
+                        "@Generated(\"Created by the ACS Commons DialogProviderAnnotationProcessor\")\n" +
+                        "@ConsumerType\n" +
+                        "@Component(service = com.adobe.acs.commons.mcp.form.DialogResourceProvider.class, immediate = true)\n" +
+                        "public class Example2_dialogResourceProvider implements com.adobe.acs.commons.mcp.form.DialogResourceProvider {\n" +
+                        "\n" +
+                        "    @Override\n" +
+                        "    public Class getTargetClass() {\n" +
+                        "        return a.Example2.class;\n" +
+                        "    }\n" +
+                        "    @Activate\n" +
+                        "    public void activate(BundleContext context) throws InstantiationException, IllegalAccessException, ReflectiveOperationException {\n" +
+                        "        this.doActivate(context);\n" +
+                        "    }\n" +
+                        "\n" +
+                        "    @Deactivate\n" +
+                        "    public void deactivate(BundleContext context) {\n" +
+                        "        this.doDeactivate();\n" +
+                        "    }\n" +
+                        "}\n"),
+                arguments("Example3", "package a; @org.apache.sling.models.annotations.Model(adaptables=org.apache.sling.api.resource.Resource.class, resourceType=\"my.type\") @com.adobe.acs.commons.mcp.form.DialogProvider public class Example3 {}", "package a.impl;\n" +
+                        "\n" +
+                        "import javax.annotation.Generated;\n" +
+                        "import org.osgi.annotation.versioning.ConsumerType;\n" +
+                        "import org.osgi.framework.BundleContext;\n" +
+                        "import org.osgi.service.component.annotations.*;\n" +
+                        "\n" +
+                        "@Generated(\"Created by the ACS Commons DialogProviderAnnotationProcessor\")\n" +
+                        "@ConsumerType\n" +
+                        "@Component(service = com.adobe.acs.commons.mcp.form.DialogResourceProvider.class, immediate = true)\n" +
+                        "public class Example3_dialogResourceProvider implements com.adobe.acs.commons.mcp.form.DialogResourceProvider {\n" +
+                        "\n" +
+                        "    @Override\n" +
+                        "    public Class getTargetClass() {\n" +
+                        "        return a.Example3.class;\n" +
+                        "    }\n" +
+                        "    @Activate\n" +
+                        "    public void activate(BundleContext context) throws InstantiationException, IllegalAccessException, ReflectiveOperationException {\n" +
+                        "        this.doActivate(context);\n" +
+                        "    }\n" +
+                        "\n" +
+                        "    @Deactivate\n" +
+                        "    public void deactivate(BundleContext context) {\n" +
+                        "        this.doDeactivate();\n" +
+                        "    }\n" +
+                        "}\n")
+        );
+    }
+}

pom.xml

@@ -604,6 +604,12 @@ Service-Component: OSGI-INF/*.xml
                 <version>${sl4fj.version}</version>
                 <scope>test</scope>
             </dependency>
+            <dependency>
+                <groupId>org.junit.jupiter</groupId>
+                <artifactId>junit-jupiter</artifactId>
+                <version>5.10.0</version>
+                <scope>test</scope>
+            </dependency>
             <dependency>
                 <groupId>junit</groupId>
                 <artifactId>junit</artifactId>