micronaut tests shouldn't run during the build

Author: romain-grecourt

config/config-mp/pom.xml

@@ -83,6 +83,11 @@
                             <artifactId>helidon-codegen-apt</artifactId>
                             <version>${helidon.version}</version>
                         </path>
+                        <path>
+                            <groupId>io.helidon.service</groupId>
+                            <artifactId>helidon-service-codegen</artifactId>
+                            <version>${helidon.version}</version>
+                        </path>
                         <path>
                             <groupId>io.helidon.config.metadata</groupId>
                             <artifactId>helidon-config-metadata-codegen</artifactId>
@@ -96,6 +101,11 @@
                         <artifactId>helidon-codegen-apt</artifactId>
                         <version>${helidon.version}</version>
                     </dependency>
+                    <dependency>
+                        <groupId>io.helidon.service</groupId>
+                        <artifactId>helidon-service-codegen</artifactId>
+                        <version>${helidon.version}</version>
+                    </dependency>
                     <dependency>
                         <groupId>io.helidon.config.metadata</groupId>
                         <artifactId>helidon-config-metadata-codegen</artifactId>

config/config-mp/src/main/java/io/helidon/config/mp/MpConfig.java

@@ -52,7 +52,7 @@ public static io.helidon.config.Config toHelidonConfig(Config mpConfig) {
 
         if (mpConfig instanceof MpConfigImpl) {
 
-            // If the mpConfig is based on an SE config (such as when we use meta configuration)pom.xml
+            // If the mpConfig is based on an SE config (such as when we use meta configuration)
             // we must reuse that se config instance
             Iterator<ConfigSource> configSources = mpConfig.getConfigSources().iterator();
             ConfigSource first = configSources.hasNext() ? configSources.next() : null;

integrations/micronaut/data/pom.xml

@@ -222,17 +222,9 @@
             <activation>
                 <jdk>!21</jdk>
             </activation>
-            <build>
-                <plugins>
-                    <plugin>
-                        <groupId>org.apache.maven.plugins</groupId>
-                        <artifactId>maven-surefire-plugin</artifactId>
-                        <configuration>
-                            <skipTests>true</skipTests>
-                        </configuration>
-                    </plugin>
-                </plugins>
-            </build>
+            <properties>
+                <skipTests>true</skipTests>
+            </properties>
         </profile>
     </profiles>
 </project>

integrations/micronaut/data/src/test/java/io/helidon/integrations/micronaut/cdi/data/MicronautDataCdiExtensionTest.java

@@ -23,8 +23,8 @@
 import io.helidon.integrations.micronaut.cdi.data.app.DbPetRepository;
 import io.helidon.integrations.micronaut.cdi.data.app.Owner;
 import io.helidon.integrations.micronaut.cdi.data.app.Pet;
-import io.helidon.microprofile.testing.junit5.AddBean;
-import io.helidon.microprofile.testing.junit5.Configuration;
+import io.helidon.microprofile.testing.AddBean;
+import io.helidon.microprofile.testing.Configuration;
 import io.helidon.microprofile.testing.junit5.HelidonTest;
 
 import jakarta.enterprise.context.ApplicationScoped;
@@ -58,7 +58,7 @@ void testPet() {
         Optional<Pet> dinoOptional = petRepository.findByName("Dino");
         assertThat(dinoOptional, is(optionalPresent()));
 
-        Pet pet = dinoOptional.get();
+        Pet pet = dinoOptional.orElseThrow();
         assertThat(pet.getName(), is("Dino"));
         Owner owner = pet.getOwner();
         assertThat(owner.getName(), is("Fred"));
@@ -70,7 +70,7 @@ void testOwner() {
         Optional<Owner> maybeBarney = ownerRepository.findByName("Barney");
         assertThat(maybeBarney, is(optionalPresent()));
 
-        Owner barney = maybeBarney.get();
+        Owner barney = maybeBarney.orElseThrow();
         assertThat(barney.getName(), is("Barney"));
         assertThat(barney.getAge(), is(40));
     }