fix: add missing reflection hints for type used with ReflectTools (#23059) (#23063)

WebComponentExporter, I18NProvider, and MenuAccessControl might be instantiated
via ReflectTools.createInstance and were missing reflection hints in
VaadinBeanFactoryInitializationAotProcessor, which could cause issues at runtime
in native executables.

Fixes #23060

Co-authored-by: Marco Collovati <marco@vaadin.com>

Author: vaadin-bot

vaadin-spring/src/main/java/com/vaadin/flow/spring/springnative/VaadinBeanFactoryInitializationAotProcessor.java

@@ -43,14 +43,17 @@
 import com.vaadin.flow.component.Component;
 import com.vaadin.flow.component.ComponentEvent;
 import com.vaadin.flow.component.UI;
+import com.vaadin.flow.component.WebComponentExporter;
 import com.vaadin.flow.component.page.AppShellConfigurator;
+import com.vaadin.flow.i18n.I18NProvider;
 import com.vaadin.flow.router.HasErrorParameter;
 import com.vaadin.flow.router.HasUrlParameter;
 import com.vaadin.flow.router.Layout;
 import com.vaadin.flow.router.Route;
 import com.vaadin.flow.router.RouteAlias;
 import com.vaadin.flow.router.RouterLayout;
 import com.vaadin.flow.server.PWA;
+import com.vaadin.flow.server.auth.MenuAccessControl;
 
 /**
  * Bean factory initialization AOT processor for Vaadin applications.
@@ -122,6 +125,9 @@ public BeanFactoryInitializationAotContribution processAheadOfTime(
                 registerSubTypes(hints, pkg, HasUrlParameter.class);
                 registerSubTypes(hints, pkg,
                         "com.vaadin.flow.data.converter.Converter");
+                registerSubTypes(hints, pkg, WebComponentExporter.class);
+                registerSubTypes(hints, pkg, I18NProvider.class);
+                registerSubTypes(hints, pkg, MenuAccessControl.class);
             }
         };
     }

vaadin-spring/src/test/java/com/vaadin/flow/spring/springnative/VaadinBeanFactoryInitializationAotProcessorTest.java

@@ -39,7 +39,10 @@
 import com.vaadin.flow.component.ComponentEvent;
 import com.vaadin.flow.component.Tag;
 import com.vaadin.flow.component.UI;
+import com.vaadin.flow.component.WebComponentExporter;
 import com.vaadin.flow.component.page.AppShellConfigurator;
+import com.vaadin.flow.component.webcomponent.WebComponent;
+import com.vaadin.flow.i18n.I18NProvider;
 import com.vaadin.flow.router.BeforeEnterEvent;
 import com.vaadin.flow.router.BeforeEvent;
 import com.vaadin.flow.router.ErrorParameter;
@@ -50,6 +53,7 @@
 import com.vaadin.flow.router.RouteAlias;
 import com.vaadin.flow.router.RouterLayout;
 import com.vaadin.flow.server.PWA;
+import com.vaadin.flow.server.auth.MenuAccessControl;
 
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.mockito.ArgumentMatchers.any;
@@ -550,6 +554,73 @@ void getSubtypesOf_excludesParentType() {
                 .isNotEmpty().doesNotContain(RouterLayout.class);
     }
 
+    @Test
+    void getSubtypesOf_findsWebComponentExporterSubclasses() {
+        VaadinBeanFactoryInitializationAotProcessor processor = new VaadinBeanFactoryInitializationAotProcessor();
+
+        Collection<Class<?>> exporters = processor.getSubtypesOf(
+                getClass().getPackageName(), WebComponentExporter.class);
+
+        assertThat(exporters).as("Should find WebComponentExporter subclasses")
+                .contains(TestWebComponentExporter.class);
+    }
+
+    @Test
+    void getSubtypesOf_findsI18NProviderImplementations() {
+        VaadinBeanFactoryInitializationAotProcessor processor = new VaadinBeanFactoryInitializationAotProcessor();
+
+        Collection<Class<?>> providers = processor
+                .getSubtypesOf(getClass().getPackageName(), I18NProvider.class);
+
+        assertThat(providers).as("Should find I18NProvider implementations")
+                .contains(TestI18NProvider.class);
+    }
+
+    @Test
+    void getSubtypesOf_findsMenuAccessControlImplementations() {
+        VaadinBeanFactoryInitializationAotProcessor processor = new VaadinBeanFactoryInitializationAotProcessor();
+
+        Collection<Class<?>> accessControls = processor.getSubtypesOf(
+                getClass().getPackageName(), MenuAccessControl.class);
+
+        assertThat(accessControls)
+                .as("Should find MenuAccessControl implementations")
+                .contains(TestMenuAccessControl.class);
+    }
+
+    @Test
+    void processAheadOfTime_webComponentExporterSubtype_reflectionHintRegistered() {
+        RuntimeHints hints = processAotForHintsWithSubtypes(
+                TestWebComponentExporter.class, WebComponentExporter.class);
+
+        assertThat(RuntimeHintsPredicates.reflection()
+                .onType(TestWebComponentExporter.class))
+                .as("WebComponentExporter subtype should be registered for reflection")
+                .accepts(hints);
+    }
+
+    @Test
+    void processAheadOfTime_i18nProviderSubtype_reflectionHintRegistered() {
+        RuntimeHints hints = processAotForHintsWithSubtypes(
+                TestI18NProvider.class, I18NProvider.class);
+
+        assertThat(RuntimeHintsPredicates.reflection()
+                .onType(TestI18NProvider.class))
+                .as("I18NProvider subtype should be registered for reflection")
+                .accepts(hints);
+    }
+
+    @Test
+    void processAheadOfTime_menuAccessControlSubtype_reflectionHintRegistered() {
+        RuntimeHints hints = processAotForHintsWithSubtypes(
+                TestMenuAccessControl.class, MenuAccessControl.class);
+
+        assertThat(RuntimeHintsPredicates.reflection()
+                .onType(TestMenuAccessControl.class))
+                .as("MenuAccessControl subtype should be registered for reflection")
+                .accepts(hints);
+    }
+
     @Test
     void getAnnotatedClasses_findsClassesWithSpecificAnnotation() {
         VaadinBeanFactoryInitializationAotProcessor processor = new VaadinBeanFactoryInitializationAotProcessor();
@@ -809,4 +880,41 @@ public static class RouteWithUILayout extends Component {
     @Tag("div")
     public static class RouteWithDefaultLayout extends Component {
     }
+
+    public static class TestWebComponentExporter
+            extends WebComponentExporter<Component> {
+        public TestWebComponentExporter() {
+            super("test-exporter");
+        }
+
+        @Override
+        protected void configureInstance(WebComponent<Component> webComponent,
+                Component component) {
+        }
+    }
+
+    public static class TestI18NProvider implements I18NProvider {
+        @Override
+        public java.util.List<java.util.Locale> getProvidedLocales() {
+            return java.util.List.of(java.util.Locale.ENGLISH);
+        }
+
+        @Override
+        public String getTranslation(String key, java.util.Locale locale,
+                Object... params) {
+            return key;
+        }
+    }
+
+    public static class TestMenuAccessControl implements MenuAccessControl {
+        @Override
+        public void setPopulateClientSideMenu(
+                PopulateClientMenu populateClientMenu) {
+        }
+
+        @Override
+        public PopulateClientMenu getPopulateClientSideMenu() {
+            return PopulateClientMenu.AUTOMATIC;
+        }
+    }
 }