HBX-3234: Improve the Maven hbm2orm Mojo

  - Add an example for a simple transformation
  - Verify the proper execution of the example by adding an integration test for it

Signed-off-by: Koen Aers <[email protected]>

Author: koentsje

maven/docs/examples/hbm2orm/.gitignore

@@ -0,0 +1 @@
+*.mapping.xml

maven/docs/examples/hbm2orm/simple-default/README.md

@@ -0,0 +1,19 @@
+<!--
+  ~ Copyright 2004 - 2025 Red Hat, Inc.
+  ~
+  ~ Licensed under the Apache License, Version 2.0 (the "License");
+  ~ you may not use this file except in compliance with the License.
+  ~ You may obtain a copy of the License at
+  ~
+  ~     http://www.apache.org/licenses/LICENSE-2.0
+  ~
+  ~ Unless required by applicable law or agreed to in writing, software
+  ~ distributed under the License is distributed on an "AS IS" basis,
+  ~ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+  ~ See the License for the specific language governing permissions and
+  ~ limitations under the License.
+  -->
+To run this example:
+- Have [Apache Maven](https://maven.apache.org) installed
+- Issue the following commands from a command-line window opened in this folder:
+  `mvn org.hibernate.tool:hibernate-tools-maven:${hibernate.version}:hbm2orm`

maven/docs/examples/hbm2orm/simple-default/pom.xml

@@ -0,0 +1,26 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!--
+  ~ Copyright 2018 - 2025 Red Hat, Inc.
+  ~
+  ~ Licensed under the Apache License, Version 2.0 (the "License");
+  ~ you may not use this file except in compliance with the License.
+  ~ You may obtain a copy of the License at
+  ~
+  ~     http://www.apache.org/licenses/LICENSE-2.0
+  ~
+  ~ Unless required by applicable law or agreed to in writing, software
+  ~ distributed under the License is distributed on an "AS IS" basis,
+  ~ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+  ~ See the License for the specific language governing permissions and
+  ~ limitations under the License.
+  -->
+<project xmlns="http://maven.apache.org/POM/4.0.0"
+         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
+    <modelVersion>4.0.0</modelVersion>
+
+    <groupId>org.hibernate.tool.maven.test</groupId>
+    <artifactId>hbm2orm-simple-default</artifactId>
+    <version>0.0.1-SNAPSHOT</version>
+
+ </project>

maven/docs/examples/hbm2orm/simple-default/src/main/resources/simple.hbm.xml

@@ -0,0 +1,29 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!--
+  ~ Copyright 2004 - 2025 Red Hat, Inc.
+  ~
+  ~ Licensed under the Apache License, Version 2.0 (the "License");
+  ~ you may not use this file except in compliance with the License.
+  ~ You may obtain a copy of the License at
+  ~
+  ~     http://www.apache.org/licenses/LICENSE-2.0
+  ~
+  ~ Unless required by applicable law or agreed to in writing, software
+  ~ distributed under the License is distributed on an "AS IS" basis,
+  ~ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+  ~ See the License for the specific language governing permissions and
+  ~ limitations under the License.
+  -->
+<!DOCTYPE hibernate-mapping PUBLIC
+	"-//Hibernate/Hibernate Mapping DTD 3.0//EN"
+	"https://www.hibernate.org/dtd/hibernate-mapping-3.0.dtd">
+<hibernate-mapping>
+
+	<class name="org.bar.Foo">
+		<id name="id" type="long">
+			<generator class="assigned"/>
+		</id>
+		<property name="name" type="string"/>
+	</class>
+
+</hibernate-mapping>

maven/src/functionalTest/java/org/hibernate/tool/maven/ExamplesTestIT.java

@@ -4,6 +4,7 @@
 import static org.junit.jupiter.api.Assertions.assertFalse;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
+import org.hibernate.tool.api.version.Version;
 import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.io.TempDir;
@@ -12,10 +13,12 @@
 import org.apache.maven.cli.MavenCli;
 
 import java.io.File;
+import java.net.URL;
 import java.nio.file.Files;
 import java.sql.Connection;
 import java.sql.DriverManager;
 import java.sql.Statement;
+import java.util.Objects;
 
 public class ExamplesTestIT {
 
@@ -24,7 +27,6 @@ public class ExamplesTestIT {
     private static File localRepo;
 
     private File projectFolder;
-    private MavenCli mavenCli;
 
     @TempDir
     private File tempFolder;
@@ -94,7 +96,7 @@ public void testOutputDirectory() throws Exception {
         assertFalse(outputDirectory.exists());
         assertFalse(personFile.exists());
         runGenerateSources();
-        assertEquals(1, outputDirectory.list().length); // 1 file is generated in 'generated-classes'
+        assertEquals(1, Objects.requireNonNull(outputDirectory.list()).length); // 1 file is generated in 'generated-classes'
         assertTrue(personFile.exists()); // The Person.java file should have been generated
     }
 
@@ -122,6 +124,17 @@ public void testUseGenerics() throws Exception {
         assertGeneratedContains("Person.java", "Set<Item>");
     }
 
+    @Test
+    public void testHbm2OrmSimpleDefault() throws Exception {
+        projectFolder = new File(baseFolder, "hbm2orm/simple-default");
+        File ormXmlFile = new File(projectFolder, "src/main/resources/simple.mapping.xml");
+        assertFalse(ormXmlFile.exists());
+        runMavenCommand("org.hibernate.tool:hibernate-tools-maven:" + Version.versionString() + ":hbm2orm");
+        assertTrue(ormXmlFile.exists());
+        String ormXmlContents = Files.readString( ormXmlFile.toPath() );
+        assertTrue(ormXmlContents.contains("entity-mappings"));
+    }
+
     private void prepareProject(String projectName) throws Exception {
         projectFolder = new File(baseFolder, projectName);
         assertTrue(projectFolder.exists());
@@ -132,7 +145,7 @@ private void prepareProject(String projectName) throws Exception {
 
     private void createHibernatePropertiesFile(File projectFolder) throws Exception {
         File projectResourcesFolder = new File(projectFolder, "src/main/resources");
-        projectResourcesFolder.mkdirs();
+        assertTrue(projectResourcesFolder.mkdirs());
         File hibernatePropertiesFile = new File(projectResourcesFolder, "hibernate.properties");
         assertFalse(hibernatePropertiesFile.exists());
         String hibernatePropertiesFileContents =
@@ -147,8 +160,14 @@ private void createHibernatePropertiesFile(File projectFolder) throws Exception
     }
 
     private void runGenerateSources() {
+        runMavenCommand("generate-sources");
+    }
+
+    private void runMavenCommand(String command) {
         new MavenCli().doMain(
-                new String[]{"-Dmaven.repo.local=" + localRepo.getAbsolutePath(), "generate-sources"},
+                new String[]{
+                        "-Dmaven.repo.local=" + localRepo.getAbsolutePath(),
+                        command},
                 projectFolder.getAbsolutePath(),
                 null,
                 null);
@@ -166,8 +185,11 @@ private void assertGeneratedDoesNotContain(String fileName, String contents) thr
         assertFalse(readGeneratedContents(fileName).contains(contents));
     }
 
-    private void assertNumberOfGeneratedFiles(int amount) throws Exception {
-        assertEquals(amount, new File(projectFolder, "target/generated-sources").list().length);
+    private void assertNumberOfGeneratedFiles(int amount) {
+        assertEquals(
+                amount,
+                Objects.requireNonNull(
+                    new File(projectFolder, "target/generated-sources").list()).length);
     }
 
     private String readGeneratedContents(String fileName) throws Exception {
@@ -177,8 +199,14 @@ private String readGeneratedContents(String fileName) throws Exception {
     }
 
     private static File determineBaseFolder() throws Exception {
-        return new File(ExamplesTestIT.class.getClassLoader().getResource("5-minute-tutorial/pom.xml").toURI())
-                .getParentFile().getParentFile();
+        Class<?> thisClass = ExamplesTestIT.class;
+        URL classUrl = thisClass.getResource("/" + thisClass.getName().replace(".", "/") + ".class");
+        assert classUrl != null;
+        File result = new File(classUrl.toURI());
+        for (int i = 0; i < thisClass.getName().chars().filter(ch -> ch == '.').count() + 1; i++) {
+            result = result.getParentFile();
+        }
+        return result;
     }
 
     private void createDatabase() throws Exception {

maven/src/functionalTest/java/org/hibernate/tool/maven/TransformHbmTestIT.java

@@ -62,10 +62,10 @@ private void runTransformHbmToOrm() throws Exception {
     	File ormXmlFile = new File(destinationDir, "simple.mapping.xml");
     	assertFalse(ormXmlFile.exists());
         new MavenCli().doMain(
-                new String[]{
+                new String[] {
                 		"-Dmaven.repo.local=" + localRepo.getAbsolutePath(),
-                		"org.hibernate.tool:hibernate-tools-maven:" + Version.versionString() + ":hbm2orm", 
-                		"generate-sources"},
+                		"org.hibernate.tool:hibernate-tools-maven:" + Version.versionString() + ":hbm2orm"
+                },
                 projectPath.toAbsolutePath().toString(),
                 null,
                 null);