HHH-16378 Add test for issue

hibernate · Mar 30, 2023 · 9d64995 · 9d64995
1 parent 0c70c1e
commit 9d64995
Show file tree

Hide file tree

Showing 3 changed files with 491 additions and 0 deletions.
diff --git a/...ore/src/test/java/org/hibernate/orm/test/annotations/generics/GenericAssociationTest.java b/...ore/src/test/java/org/hibernate/orm/test/annotations/generics/GenericAssociationTest.java
@@ -0,0 +1,194 @@
+/*
+ * Hibernate, Relational Persistence for Idiomatic Java
+ *
+ * License: GNU Lesser General Public License (LGPL), version 2.1 or later
+ * See the lgpl.txt file in the root directory or http://www.gnu.org/licenses/lgpl-2.1.html
+ */
+package org.hibernate.orm.test.annotations.generics;
+
+import java.util.HashSet;
+import java.util.Set;
+
+import org.hibernate.query.criteria.JpaPath;
+
+import org.hibernate.testing.orm.junit.DomainModel;
+import org.hibernate.testing.orm.junit.Jira;
+import org.hibernate.testing.orm.junit.SessionFactory;
+import org.hibernate.testing.orm.junit.SessionFactoryScope;
+import org.junit.jupiter.api.AfterAll;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.Test;
+
+import jakarta.persistence.Entity;
+import jakarta.persistence.Id;
+import jakarta.persistence.ManyToOne;
+import jakarta.persistence.MappedSuperclass;
+import jakarta.persistence.OneToMany;
+import jakarta.persistence.criteria.CriteriaBuilder;
+import jakarta.persistence.criteria.CriteriaQuery;
+import jakarta.persistence.criteria.Join;
+import jakarta.persistence.criteria.Path;
+import jakarta.persistence.criteria.Root;
+
+import static org.assertj.core.api.Assertions.assertThat;
+
+/**
+ * @author Yoann Rodière
+ * @author Marco Belladelli
+ */
+@SessionFactory
+@DomainModel( annotatedClasses = {
+		GenericAssociationTest.AbstractParent.class,
+		GenericAssociationTest.Parent.class,
+		GenericAssociationTest.AbstractChild.class,
+		GenericAssociationTest.Child.class
+} )
+@Jira( "https://hibernate.atlassian.net/browse/HHH-16378" )
+public class GenericAssociationTest {
+	@BeforeAll
+	public void setUp(SessionFactoryScope scope) {
+		scope.inTransaction( session -> {
+			final Parent parent = new Parent( 1L );
+			final Child child = new Child( 2L );
+			child.setParent( parent );
+			parent.getChildren().add( child );
+			session.persist( parent );
+			session.persist( child );
+		} );
+	}
+
+	@AfterAll
+	public void tearDown(SessionFactoryScope scope) {
+		scope.inTransaction( session -> session.createQuery( "from Parent", Parent.class )
+				.getResultList()
+				.forEach( p -> p.getChildren().clear() ) );
+		scope.inTransaction( session -> {
+			session.createMutationQuery( "delete from Child" ).executeUpdate();
+			session.createMutationQuery( "delete from Parent" ).executeUpdate();
+		} );
+	}
+
+	@Test
+	public void testParentQuery(SessionFactoryScope scope) {
+		scope.inTransaction( session -> assertThat( session.createQuery(
+				"select parent.id from Child",
+				Long.class
+		).getSingleResult() ).isEqualTo( 1L ) );
+	}
+
+	@Test
+	public void testParentCriteriaQuery(SessionFactoryScope scope) {
+		scope.inTransaction( session -> {
+			final CriteriaBuilder cb = session.getCriteriaBuilder();
+			final CriteriaQuery<Long> query = cb.createQuery( Long.class );
+			final Root<Child> root = query.from( Child.class );
+			final Path<Parent> parent = root.get( "parent" );
+			// generic attributes are always reported as Object java type
+			assertThat( parent.getJavaType() ).isEqualTo( Object.class );
+			assertThat( parent.getModel() ).isSameAs( root.getModel().getAttribute( "parent" ) );
+			assertThat( ( (JpaPath<?>) parent ).getResolvedModel().getBindableJavaType() ).isEqualTo( Parent.class );
+			final Long result = session.createQuery( query.select( parent.get( "id" ) ) ).getSingleResult();
+			assertThat( result ).isEqualTo( 1L );
+		} );
+	}
+
+	@Test
+	public void testChildQuery(SessionFactoryScope scope) {
+		scope.inTransaction( session -> assertThat( session.createQuery(
+				"select c.id from Parent p join p.children c",
+				Long.class
+		).getSingleResult() ).isEqualTo( 2L ) );
+	}
+
+	@Test
+	public void testChildCriteriaQuery(SessionFactoryScope scope) {
+		scope.inTransaction( session -> {
+			final CriteriaBuilder cb = session.getCriteriaBuilder();
+			final CriteriaQuery<Long> query = cb.createQuery( Long.class );
+			final Root<Parent> root = query.from( Parent.class );
+			final Join<Parent, Child> join = root.join( "children" );
+			// generic attributes are always reported as Object java type
+			assertThat( join.getJavaType() ).isEqualTo( Object.class );
+			assertThat( join.getModel() ).isSameAs( root.getModel().getAttribute( "children" ) );
+			assertThat( ( (JpaPath<?>) join ).getResolvedModel().getBindableJavaType() ).isEqualTo( Child.class );
+			final Long result = session.createQuery( query.select( join.get( "id" ) ) ).getSingleResult();
+			assertThat( result ).isEqualTo( 2L );
+		} );
+	}
+
+	@Test
+	public void testElementQuery(SessionFactoryScope scope) {
+		scope.inTransaction( session -> assertThat( session.createQuery(
+				"select element(c).id from Parent p join p.children c",
+				Long.class
+		).getSingleResult() ).isEqualTo( 2L ) );
+	}
+
+	@MappedSuperclass
+	public abstract static class AbstractParent<E, T> {
+		@OneToMany
+		private Set<E> children;
+
+		public AbstractParent() {
+			this.children = new HashSet<>();
+		}
+
+		public Set<E> getChildren() {
+			return children;
+		}
+	}
+
+	@Entity( name = "Parent" )
+	public static class Parent extends AbstractParent<Child, String> {
+		@Id
+		private Long id;
+
+		public Parent() {
+		}
+
+		public Parent(Long id) {
+			this.id = id;
+		}
+
+		public Long getId() {
+			return this.id;
+		}
+	}
+
+	@MappedSuperclass
+	public abstract static class AbstractChild<T> {
+		@ManyToOne
+		private T parent;
+
+		public AbstractChild() {
+		}
+
+		public abstract Long getId();
+
+		public T getParent() {
+			return this.parent;
+		}
+
+		public void setParent(T parent) {
+			this.parent = parent;
+		}
+	}
+
+	@Entity( name = "Child" )
+	public static class Child extends AbstractChild<Parent> {
+		@Id
+		protected Long id;
+
+		public Child() {
+		}
+
+		public Child(Long id) {
+			this.id = id;
+		}
+
+		@Override
+		public Long getId() {
+			return this.id;
+		}
+	}
+}
diff --git a/...src/test/java/org/hibernate/orm/test/annotations/generics/GenericBasicValuedPathTest.java b/...src/test/java/org/hibernate/orm/test/annotations/generics/GenericBasicValuedPathTest.java
@@ -0,0 +1,106 @@
+/*
+ * Hibernate, Relational Persistence for Idiomatic Java
+ *
+ * License: GNU Lesser General Public License (LGPL), version 2.1 or later
+ * See the lgpl.txt file in the root directory or http://www.gnu.org/licenses/lgpl-2.1.html
+ */
+package org.hibernate.orm.test.annotations.generics;
+
+import org.hibernate.query.criteria.JpaPath;
+
+import org.hibernate.testing.orm.junit.DomainModel;
+import org.hibernate.testing.orm.junit.Jira;
+import org.hibernate.testing.orm.junit.SessionFactory;
+import org.hibernate.testing.orm.junit.SessionFactoryScope;
+
+import org.junit.jupiter.api.AfterAll;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.Test;
+
+import jakarta.persistence.Entity;
+import jakarta.persistence.Id;
+import jakarta.persistence.MappedSuperclass;
+import jakarta.persistence.criteria.CriteriaBuilder;
+import jakarta.persistence.criteria.CriteriaQuery;
+import jakarta.persistence.criteria.Path;
+import jakarta.persistence.criteria.Root;
+
+import static org.assertj.core.api.Assertions.assertThat;
+
+/**
+ * @author Marco Belladelli
+ */
+@SessionFactory
+@DomainModel( annotatedClasses = {
+		GenericBasicValuedPathTest.MySuper.class,
+		GenericBasicValuedPathTest.MyEntity.class
+} )
+@Jira( "https://hibernate.atlassian.net/browse/HHH-16378" )
+public class GenericBasicValuedPathTest {
+	@BeforeAll
+	public void setUp(SessionFactoryScope scope) {
+		scope.inTransaction( session -> session.persist( new MyEntity( 1, "my_entity" ) ) );
+	}
+
+	@AfterAll
+	public void tearDown(SessionFactoryScope scope) {
+		scope.inTransaction( session -> session.createMutationQuery( "delete from MyEntity" ).executeUpdate() );
+	}
+
+	@Test
+	public void testId(SessionFactoryScope scope) {
+		scope.inTransaction( session -> {
+			final CriteriaBuilder cb = session.getCriteriaBuilder();
+			final CriteriaQuery<Object> query = cb.createQuery();
+			final Root<MyEntity> root = query.from( MyEntity.class );
+			final Path<String> idPath = root.get( "id" );
+			// generic attributes are always reported as Object java type
+			assertThat( idPath.getJavaType() ).isEqualTo( Object.class );
+			assertThat( idPath.getModel() ).isSameAs( root.getModel().getAttribute( "id" ) );
+			assertThat( ( (JpaPath<?>) idPath ).getResolvedModel().getBindableJavaType() ).isEqualTo( Integer.class );
+			final Object result = session.createQuery( query.select( idPath ) ).getSingleResult();
+			assertThat( result ).isEqualTo( 1 );
+		} );
+	}
+
+	@Test
+	public void testProperty(SessionFactoryScope scope) {
+		scope.inTransaction( session -> {
+			final CriteriaBuilder cb = session.getCriteriaBuilder();
+			final CriteriaQuery<Object> query = cb.createQuery();
+			final Root<MyEntity> root = query.from( MyEntity.class );
+			final Path<String> dataPath = root.get( "data" );
+			// generic attributes are always reported as Object java type
+			assertThat( dataPath.getJavaType() ).isEqualTo( Object.class );
+			assertThat( dataPath.getModel() ).isSameAs( root.getModel().getAttribute( "data" ) );
+			assertThat( ( (JpaPath<?>) dataPath ).getResolvedModel().getBindableJavaType() ).isEqualTo( String.class );
+			final Object result = session.createQuery( query.select( dataPath ) ).getSingleResult();
+			assertThat( result ).isEqualTo( "my_entity" );
+		} );
+	}
+
+	@MappedSuperclass
+	public abstract static class MySuper<T, S> {
+		@Id
+		private T id;
+		private S data;
+
+		public MySuper() {
+		}
+
+		public MySuper(T id, S data) {
+			this.id = id;
+			this.data = data;
+		}
+	}
+
+	@Entity( name = "MyEntity" )
+	public static class MyEntity extends MySuper<Integer, String> {
+		public MyEntity() {
+		}
+
+		public MyEntity(Integer id, String data) {
+			super( id, data );
+		}
+	}
+}