Skip to content

Commit

Permalink
HHH-17704 Add test for issue
Browse files Browse the repository at this point in the history
  • Loading branch information
dreab8 authored and mbladel committed Feb 8, 2024
1 parent 6057d99 commit d35cd46
Showing 1 changed file with 172 additions and 0 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1,172 @@
package org.hibernate.orm.test.proxy;

import org.hibernate.Hibernate;
import org.hibernate.LazyInitializationException;
import org.hibernate.query.Query;

import org.hibernate.testing.orm.junit.DomainModel;
import org.hibernate.testing.orm.junit.JiraKey;
import org.hibernate.testing.orm.junit.SessionFactory;
import org.hibernate.testing.orm.junit.SessionFactoryScope;
import org.junit.jupiter.api.BeforeAll;
import org.junit.jupiter.api.Test;

import jakarta.persistence.Entity;
import jakarta.persistence.FetchType;
import jakarta.persistence.Id;
import jakarta.persistence.Inheritance;
import jakarta.persistence.InheritanceType;
import jakarta.persistence.ManyToOne;

import static org.junit.jupiter.api.Assertions.assertFalse;
import static org.junit.jupiter.api.Assertions.assertThrows;

@DomainModel(
annotatedClasses = {
DetachedProxyAsQueryParameterTest.Department.class,
DetachedProxyAsQueryParameterTest.BasicDepartment.class,
DetachedProxyAsQueryParameterTest.SpecialDepartment.class,
DetachedProxyAsQueryParameterTest.Employee.class
}
)
@SessionFactory
@JiraKey("HHH-17704")
public class DetachedProxyAsQueryParameterTest {

private static final Integer EMPLOYEE_ID = 1;

@BeforeAll
public void setUp(SessionFactoryScope scope) {
scope.inTransaction(
session -> {
SpecialDepartment department = new SpecialDepartment( 2, "dep1" );
Employee employee = new Employee( EMPLOYEE_ID, "Fab", department );
session.persist( department );
session.persist( employee );
}
);
}

@Test
public void testQuery(SessionFactoryScope scope) {
Employee employee = scope.fromTransaction(
session ->
session.find( Employee.class, EMPLOYEE_ID )
);
assertFalse( Hibernate.isInitialized( employee.getDepartment() ) );
scope.inSession(
session -> {
Query<Employee> query = session.createQuery(
"select e from Employee e where e.department = :department",
Employee.class
);
query.setParameter( "department", employee.getDepartment() );
query.list();
}
);
scope.inSession(
session -> {
session.createQuery(
"select d from Department d where d = :department" )
.setParameter( "department", employee.getDepartment() ).list();
}
);


assertThrows(
LazyInitializationException.class, () -> {
scope.inSession(
session -> {
// In order to validate the parameter and check that it is an instance of SpecialDepartment
// the Department proxy have to be initialized but being a detached instance
// a LazyInitializationException is thrown
session.createQuery(
"select d from SpecialDepartment d where d = :department"
).setParameter( "department", employee.getDepartment() ).list();
}
);
}
);
}

@Entity(name = "Department")
@Inheritance(strategy = InheritanceType.SINGLE_TABLE)
public static class Department {
@Id
private Integer id;

public Department() {
}

public Department(Integer id) {
this.id = id;
}

public Integer getId() {
return id;
}
}

@Entity(name = "BasicDepartment")
public static class BasicDepartment extends Department {


private String name;

public BasicDepartment() {
}

public BasicDepartment(Integer id, String name) {
super( id );
this.name = name;
}


public String getName() {
return name;
}
}

@Entity(name = "SpecialDepartment")
public static class SpecialDepartment extends BasicDepartment {
public SpecialDepartment() {
}

public SpecialDepartment(Integer id, String name) {
super( id, name );
}
}

@Entity(name = "Employee")
public static class Employee {

@Id
private Integer id;

private String name;

@ManyToOne(fetch = FetchType.LAZY)
private Department department;

public Employee() {
}

public Employee(Integer id, String name, Department department) {
this.id = id;
this.name = name;
this.department = department;
}

public Integer getId() {
return id;
}

public String getName() {
return name;
}

public Department getDepartment() {
return department;
}
}
}

0 comments on commit d35cd46

Please sign in to comment.