HHH-19629 Adjust how Hibernate Processor constructs the type name as String

Author: marko-bekhta

tooling/metamodel-generator/src/jakartaData/java/org/hibernate/processor/test/data/constraint/DataTest.java

@@ -0,0 +1,24 @@
+/*
+ * SPDX-License-Identifier: Apache-2.0
+ * Copyright Red Hat Inc. and Hibernate Authors
+ */
+package org.hibernate.processor.test.data.constraint;
+
+import org.hibernate.processor.test.util.CompilationTest;
+import org.hibernate.processor.test.util.WithClasses;
+import org.junit.jupiter.api.Test;
+
+import static org.hibernate.processor.test.util.TestUtil.assertMetamodelClassGeneratedFor;
+import static org.hibernate.processor.test.util.TestUtil.getMetaModelSourceAsString;
+
+@CompilationTest
+class DataTest {
+@Test
+@WithClasses({MyEntity.class, MyConstrainedRepository.class})
+void test() {
+System.out.println( getMetaModelSourceAsString( MyEntity.class ) );
+System.out.println( getMetaModelSourceAsString( MyConstrainedRepository.class ) );
+assertMetamodelClassGeneratedFor( MyEntity.class );
+//	assertMetamodelClassGeneratedFor( MyConstrainedRepository.class );
+}
+}

tooling/metamodel-generator/src/jakartaData/java/org/hibernate/processor/test/data/constraint/MyConstrainedRepository.java

@@ -0,0 +1,18 @@
+/*
+ * SPDX-License-Identifier: Apache-2.0
+ * Copyright Red Hat Inc. and Hibernate Authors
+ */
+package org.hibernate.processor.test.data.constraint;
+
+import jakarta.data.repository.CrudRepository;
+import jakarta.data.repository.Find;
+import jakarta.data.repository.Repository;
+import jakarta.validation.constraints.NotNull;
+import jakarta.validation.constraints.Size;
+
+@Repository
+public interface MyConstrainedRepository extends CrudRepository<MyEntity, Long> {
+
+@Find
+MyEntity findByName(@NotNull @Size(min = 5) String name);
+}

tooling/metamodel-generator/src/jakartaData/java/org/hibernate/processor/test/data/constraint/MyEntity.java

@@ -0,0 +1,16 @@
+/*
+ * SPDX-License-Identifier: Apache-2.0
+ * Copyright Red Hat Inc. and Hibernate Authors
+ */
+package org.hibernate.processor.test.data.constraint;
+
+import jakarta.persistence.Entity;
+import jakarta.persistence.Id;
+
+@Entity
+public class MyEntity {
+
+@Id
+private Long id;
+private String name;
+}

tooling/metamodel-generator/src/main/java/org/hibernate/processor/annotation/AnnotationMetaEntity.java

@@ -62,6 +62,7 @@
 import java.util.Set;
 import java.util.StringTokenizer;
 import java.util.regex.Pattern;
+import java.util.stream.Collectors;
 import java.util.stream.Stream;
 
 import jakarta.persistence.AccessType;
@@ -3425,15 +3426,21 @@ private List<String> parameterTypes(ExecutableElement method) {
  * Workaround for a bug in Java 20/21. Should not be necessary!
  */
 private String typeAsString(TypeMirror type) {
-String result = type.toString();
-for ( AnnotationMirror annotation : type.getAnnotationMirrors() ) {
-final String annotationString = annotation.toString();
-result = result
-// if it has a space after it, we need to remove that too
-.replace(annotationString + ' ', "")
-// just in case it did not have a space after it
-.replace(annotationString, "");
+String result;
+if ( type instanceof DeclaredType dt && dt.asElement() instanceof TypeElement te ) {
+// get the "fqcn" without any type arguments
+result = te.getQualifiedName().toString();
+// add the < ? ,? ....> as necessary:
+if ( !dt.getTypeArguments().isEmpty() ) {
+result += dt.getTypeArguments().stream()
+.map( this::typeAsString )
+.collect( Collectors.joining( ",", "<", ">" ) );
+}
 }
+else {
+result = type.toString();
+}
+
 for ( AnnotationMirror annotation : type.getAnnotationMirrors() ) {
 result = annotation.toString() + ' ' + result;
 }