Guice @Inject doesn't create default constructor for MVC routes

- fix #3567

Author: jknack

modules/jooby-apt/pom.xml

@@ -32,6 +32,12 @@
       <scope>test</scope>
     </dependency>
 
+    <dependency>
+      <groupId>com.google.inject</groupId>
+      <artifactId>guice</artifactId>
+      <scope>test</scope>
+    </dependency>
+
     <!-- Test dependencies -->
     <dependency>
       <groupId>com.google.testing.compile</groupId>

modules/jooby-apt/src/main/java/io/jooby/internal/apt/MvcRouter.java

@@ -298,7 +298,8 @@ private StringBuilder constructors(String generatedName, boolean kt) {
   }
 
   private static Predicate<Element> hasInjectAnnotation() {
-    var injectAnnotations = Set.of("javax.inject.Inject", "jakarta.inject.Inject");
+    var injectAnnotations =
+        Set.of("javax.inject.Inject", "jakarta.inject.Inject", "com.google.inject.Inject");
     return it ->
         injectAnnotations.stream()
             .anyMatch(annotation -> findAnnotationByName(it, annotation) != null);

modules/jooby-apt/src/test/java/tests/i3567/C3567.java

@@ -0,0 +1,26 @@
+/*
+ * Jooby https://jooby.io
+ * Apache License Version 2.0 https://jooby.io/LICENSE.txt
+ * Copyright 2014 Edgar Espina
+ */
+package tests.i3567;
+
+import com.google.inject.Inject;
+import com.typesafe.config.Config;
+import io.jooby.annotation.GET;
+import io.jooby.annotation.QueryParam;
+
+public class C3567 {
+
+  private final Config config;
+
+  @Inject
+  public C3567(Config config) {
+    this.config = config;
+  }
+
+  @GET("/3567")
+  public String webMethod(@QueryParam String property) {
+    return config.getString(property);
+  }
+}

modules/jooby-apt/src/test/java/tests/i3567/Issue3567.java

@@ -0,0 +1,36 @@
+/*
+ * Jooby https://jooby.io
+ * Apache License Version 2.0 https://jooby.io/LICENSE.txt
+ * Copyright 2014 Edgar Espina
+ */
+package tests.i3567;
+
+import static org.junit.jupiter.api.Assertions.*;
+
+import org.junit.jupiter.api.Test;
+
+import com.typesafe.config.ConfigFactory;
+import com.typesafe.config.ConfigValueFactory;
+import io.jooby.apt.ProcessorRunner;
+
+public class Issue3567 {
+
+  @Test
+  public void shouldSupportGoogleInjectAnnotation() throws Exception {
+    var value = "bar";
+    var conf = ConfigFactory.empty().withValue("foo", ConfigValueFactory.fromAnyRef(value));
+    new ProcessorRunner(new C3567(conf))
+        .withSourceCode(
+            source -> {
+              assertTrue(
+                  source.contains(
+                      "public C3567_() {\n"
+                          + "      this(C3567.class);\n"
+                          + "    }\n"
+                          + "\n"
+                          + "    public C3567_(Class<C3567> type) {\n"
+                          + "      this(ctx -> ctx.require(type));\n"
+                          + "    }"));
+            });
+  }
+}