handle pkg private

Author: SentryMan

inject-generator/src/main/java/io/avaje/inject/generator/MethodReader.java

@@ -75,7 +75,7 @@ final class MethodReader {
       var lazyPrism = Util.isLazy(element);
       lazy = lazyPrism != null;
       conditions.readAll(element);
-      this.lazyProxyType = lazy ? Util.lazyProxy(element) : null;
+      this.lazyProxyType = lazy ? Util.lazyProxy(element, beanType) : null;
       this.proxyLazy = lazy && lazyProxyType != null;
       if (lazy && !proxyLazy) {
         if (lazyPrism.enforceProxy()) {

inject-generator/src/main/java/io/avaje/inject/generator/Util.java

@@ -395,15 +395,19 @@ static void validateBeanTypes(Element origin, List<TypeMirror> beanType) {
     });
   }
 
-  static TypeElement lazyProxy(Element element) {
+  static TypeElement lazyProxy(TypeElement element) {
+    return lazyProxy(element, element);
+  }
+
+  static TypeElement lazyProxy(Element element, TypeElement source) {
     TypeElement type =
       element instanceof TypeElement
         ? (TypeElement) element
         : APContext.asTypeElement(((ExecutableElement) element).getReturnType());
 
     if (type.getModifiers().contains(Modifier.FINAL)
         || !type.getKind().isInterface() && !Util.hasNoArgConstructor(type)
-        || Util.hasFinalMethods(type)) {
+        || Util.cantOverride(type, source)) {
 
       return BeanTypesPrism.getOptionalOn(element)
           .map(BeanTypesPrism::value)
@@ -418,11 +422,23 @@ static TypeElement lazyProxy(Element element) {
     return type;
   }
 
-  private static boolean hasFinalMethods(TypeElement type) {
+  private static boolean cantOverride(TypeElement type, TypeElement source) {
+
+    var pkgPrivate =
+        source
+            .getEnclosingElement()
+            .getSimpleName()
+            .contentEquals(type.getEnclosingElement().getSimpleName());
+
     return ElementFilter.methodsIn(type.getEnclosedElements()).stream()
         .filter(x -> !x.getModifiers().contains(Modifier.STATIC))
         .filter(x -> !x.getModifiers().contains(Modifier.PRIVATE))
-        .anyMatch(m -> m.getModifiers().contains(Modifier.FINAL));
+        .anyMatch(
+            m ->
+                m.getModifiers().contains(Modifier.FINAL)
+                    || !m.getModifiers().contains(Modifier.PUBLIC)
+                        && !m.getModifiers().contains(Modifier.PROTECTED)
+                        && !pkgPrivate);
   }
 
   static boolean hasNoArgConstructor(TypeElement beanType) {

inject-generator/src/test/java/io/avaje/inject/generator/models/valid/lazy/LazyBeanPkgPrivate.java

@@ -0,0 +1,7 @@
+package io.avaje.inject.generator.models.valid.lazy;
+
+public class LazyBeanPkgPrivate {
+
+  void something() {}
+
+}

…enerator/models/valid/RandomFactory.java …tor/models/valid/lazy/RandomFactory.javainject-generator/src/test/java/io/avaje/inject/generator/models/valid/RandomFactory.java renamed to inject-generator/src/test/java/io/avaje/inject/generator/models/valid/lazy/RandomFactory.java inject-generator/src/test/java/io/avaje/inject/generator/models/valid/RandomFactory.java renamed to inject-generator/src/test/java/io/avaje/inject/generator/models/valid/lazy/RandomFactory.java

@@ -1,4 +1,4 @@
-package io.avaje.inject.generator.models.valid;
+package io.avaje.inject.generator.models.valid.lazy;
 
 import java.security.SecureRandom;
 

inject-generator/src/test/java/io/avaje/inject/generator/models/valid/lazy/generic/LazyGenericFactory.java

@@ -3,6 +3,7 @@
 import io.avaje.inject.Bean;
 import io.avaje.inject.Factory;
 import io.avaje.inject.Lazy;
+import io.avaje.inject.generator.models.valid.lazy.LazyBeanPkgPrivate;
 
 @Factory
 public class LazyGenericFactory {
@@ -12,4 +13,10 @@ public class LazyGenericFactory {
   LazyGenericInterface<String> lazyInterface() {
     return null;
   }
+
+  @Bean
+  @Lazy
+  LazyBeanPkgPrivate pkgPrivateMethods() {
+    return null;
+  }
 }