Renamed test modules and packages.

Signed-off-by: Tomáš Kraus <[email protected]>

Author: Tomáš Kraus

data/codegen/parser/etc/spotbugs/exclude.xml

@@ -21,9 +21,8 @@
         xsi:schemaLocation="https://github.com/spotbugs/filter/3.0.0 https://raw.githubusercontent.com/spotbugs/spotbugs/3.1.0/spotbugs/etc/findbugsfilter.xsd">
 
     <Match>
-        <!-- Doesn't construct SQL string. It converts string to statement -->
-        <Class name="io.helidon.data.codegen.parser.MethodNameParserImpl"/>
-        <Method name="dataQuery"/>
-        <Bug pattern="SF_SWITCH_FALLTHROUGH"/>
+        <!-- ANTLR generated code, can't fix. -->
+        <Class name="io.helidon.data.codegen.parser.MethodName"/>
+        <Bug pattern="SF_SWITCH_NO_DEFAULT"/>
     </Match>
 </FindBugsFilter>

data/codegen/parser/src/main/java/io/helidon/data/codegen/parser/MethodNameParserImpl.java

@@ -178,18 +178,18 @@ public <E extends Throwable> void run(EntityManager em, Functions.CheckedConsume
         // PersistenceException mapping
         private static void handlePersistenceException(PersistenceException pe) throws DataException {
             switch (pe) {
-            case jakarta.persistence.EntityExistsException ignore:
-                throw new EntityExistsException("Entity already exists.", pe);
-            case jakarta.persistence.EntityNotFoundException ignore:
-                throw new EntityNotFoundException("Entity was not found.", pe);
-            case jakarta.persistence.NoResultException ignore:
-                throw new NoResultException("Query returned no result.", pe);
-            case jakarta.persistence.NonUniqueResultException ignore:
-                throw new NonUniqueResultException("Query returned multiple result.", pe);
-            case jakarta.persistence.OptimisticLockException ignore:
-                throw new OptimisticLockException("Optimistic locking conflict.", pe);
-            default:
-                throw new DataException("Persistence session task failed.", pe);
+                case jakarta.persistence.EntityExistsException ex:
+                    throw new EntityExistsException("Entity already exists.", ex);
+                case jakarta.persistence.EntityNotFoundException ex:
+                    throw new EntityNotFoundException("Entity was not found.", ex);
+                case jakarta.persistence.NoResultException ex:
+                    throw new NoResultException("Query returned no result.", ex);
+                case jakarta.persistence.NonUniqueResultException ex:
+                    throw new NonUniqueResultException("Query returned multiple result.", ex);
+                case jakarta.persistence.OptimisticLockException ex:
+                    throw new OptimisticLockException("Optimistic locking conflict.", ex);
+                default:
+                    throw new DataException("Persistence session task failed.", pe);
             }
         }
     }

data/jakarta-persistence/jakarta-persistence/src/main/java/io/helidon/data/jakarta/persistence/JpaRepositoryExecutorImpl.java

@@ -211,16 +211,15 @@
 
     <profiles>
         <profile>
-            <id>testcontainers</id>
+            <id>tests</id>
             <build>
                 <plugins>
                     <plugin>
                         <groupId>org.apache.maven.plugins</groupId>
                         <artifactId>maven-surefire-plugin</artifactId>
                         <configuration>
                             <systemPropertyVariables>
-                                <junit.jupiter.extensions.autodetection.enabled>true
-                                </junit.jupiter.extensions.autodetection.enabled>
+                                <junit.jupiter.extensions.autodetection.enabled>true</junit.jupiter.extensions.autodetection.enabled>
                             </systemPropertyVariables>
                             <skip>false</skip>
                         </configuration>

data/sql/datasource/hikari/pom.xml

@@ -217,16 +217,15 @@
 
     <profiles>
         <profile>
-            <id>testcontainers</id>
+            <id>tests</id>
             <build>
                 <plugins>
                     <plugin>
                         <groupId>org.apache.maven.plugins</groupId>
                         <artifactId>maven-surefire-plugin</artifactId>
                         <configuration>
                             <systemPropertyVariables>
-                                <junit.jupiter.extensions.autodetection.enabled>true
-                                </junit.jupiter.extensions.autodetection.enabled>
+                                <junit.jupiter.extensions.autodetection.enabled>true</junit.jupiter.extensions.autodetection.enabled>
                             </systemPropertyVariables>
                             <skip>false</skip>
                         </configuration>

data/sql/datasource/ucp/pom.xml

@@ -51,7 +51,7 @@ private Config config() {
         return config;
     }
 
-    private void config(Config config) {
+    private static void config(Config config) {
         TestConfigFactory.config = config;
     }
 

data/sql/testing/src/main/java/io/helidon/data/sql/testing/TestConfigFactory.java

@@ -21,12 +21,12 @@
     <modelVersion>4.0.0</modelVersion>
 
     <parent>
-        <groupId>io.helidon.data.tests.codegen</groupId>
-        <artifactId>helidon-data-tests-codegen-project</artifactId>
+        <groupId>io.helidon.data.tests</groupId>
+        <artifactId>helidon-data-tests-project</artifactId>
         <version>4.2.0-SNAPSHOT</version>
     </parent>
 
-    <artifactId>helidon-data-tests-codegen-application</artifactId>
+    <artifactId>helidon-data-tests-application</artifactId>
     <name>Helidon Data Codegen Tests Application</name>
 
     <dependencies>

data/tests/codegen/application/pom.xml renamed to data/tests/application/pom.xml

@@ -13,7 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-package io.helidon.data.tests.codegen.application;
+package io.helidon.data.tests.application;
 
 import io.helidon.data.DataConfig;
 import io.helidon.data.DataRegistry;

data/tests/codegen/application/src/main/java/io/helidon/data/tests/codegen/application/ApplicationData.java renamed to data/tests/application/src/main/java/io/helidon/data/tests/application/ApplicationData.java

@@ -14,13 +14,13 @@
  * limitations under the License.
  */
 /**
- * Helidon Data Codegen Tests Application.
+ * Helidon Data Tests Application.
  */
-module io.helidon.data.tests.codegen.application {
+module io.helidon.data.tests.application {
 
     requires io.helidon.data;
     requires io.helidon.common.config;
 
-    exports io.helidon.data.tests.codegen.application;
+    exports io.helidon.data.tests.application;
 
 }