HHH-13627 Add test for issue

Author: mbellade

hibernate-core/src/test/java/org/hibernate/test/cache/CacheModeGetUpdateTest.java

@@ -0,0 +1,222 @@
+/*
+ * 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.test.cache;
+
+import java.util.HashSet;
+import java.util.Objects;
+import java.util.Set;
+import javax.persistence.Cacheable;
+import javax.persistence.Column;
+import javax.persistence.Entity;
+import javax.persistence.FetchType;
+import javax.persistence.Id;
+import javax.persistence.JoinColumn;
+import javax.persistence.ManyToOne;
+import javax.persistence.OneToMany;
+
+import org.hibernate.CacheMode;
+import org.hibernate.annotations.Cache;
+import org.hibernate.annotations.CacheConcurrencyStrategy;
+
+import org.hibernate.testing.junit4.BaseCoreFunctionalTestCase;
+import org.junit.After;
+import org.junit.Before;
+import org.junit.Test;
+
+import static org.assertj.core.api.Assertions.assertThat;
+
+/**
+ * @author Marco Belladelli
+ */
+public class CacheModeGetUpdateTest extends BaseCoreFunctionalTestCase {
+	private static final long PHONE_ID = 1L;
+	private static final long PERSON_ID = 2L;
+
+	@Override
+	protected Class<?>[] getAnnotatedClasses() {
+		return new Class[] {
+				Phone.class,
+				Person.class
+		};
+	}
+
+	@Before
+	public void setUp() {
+		inTransaction( session -> {
+			session.persist( new Phone( PHONE_ID, "123" ) );
+			session.persist( new Person( PERSON_ID, "Marco" ) );
+		} );
+	}
+
+	@After
+	public void tearDown() {
+		inTransaction( session -> {
+			session.createQuery( "delete from Phone" ).executeUpdate();
+			session.createQuery( "delete from Person" ).executeUpdate();
+		} );
+	}
+
+	@Test
+	public void test() {
+		inTransaction( session -> {
+			session.setCacheMode( CacheMode.GET );
+			final Phone phone = session.get( Phone.class, PHONE_ID );
+			final Person person = session.get( Person.class, PERSON_ID );
+			phone.setPerson( person );
+			person.getPhones().add( phone );
+			session.persist( phone );
+		} );
+		// in a different transaction
+		inTransaction( session -> {
+			final Phone phone = session.get( Phone.class, PHONE_ID );
+			assertThat( phone.getPerson() ).isNotNull();
+		} );
+	}
+
+	@Entity( name = "Phone" )
+	@Cacheable
+	@Cache( usage = CacheConcurrencyStrategy.TRANSACTIONAL )
+	public static class Phone {
+		@Id
+		private Long id;
+
+		@Column( name = "phone_number" )
+		private String number;
+
+		@ManyToOne( fetch = FetchType.LAZY )
+		@JoinColumn
+		private Person person;
+
+		public Phone() {
+		}
+
+		public Phone(final long id, final String number) {
+			setId( id );
+			setNumber( number );
+		}
+
+		public Long getId() {
+			return id;
+		}
+
+		public void setId(Long id) {
+			this.id = id;
+		}
+
+		public String getNumber() {
+			return number;
+		}
+
+		public void setNumber(String number) {
+			this.number = number;
+		}
+
+		public Person getPerson() {
+			return person;
+		}
+
+		public void setPerson(Person person) {
+			this.person = person;
+		}
+
+		@Override
+		public String toString() {
+			return "Phone{" +
+					"id=" + id +
+					", number='" + number + '\'' +
+					", person=" + person +
+					'}';
+		}
+
+		@Override
+		public boolean equals(Object o) {
+			if ( this == o ) {
+				return true;
+			}
+			if ( o == null || getClass() != o.getClass() ) {
+				return false;
+			}
+			Phone phone = (Phone) o;
+			return Objects.equals( id, phone.id ) && Objects.equals( number, phone.number ) && Objects.equals(
+					person,
+					phone.person
+			);
+		}
+
+		@Override
+		public int hashCode() {
+			return Objects.hash( id, number, person );
+		}
+	}
+
+	@Entity( name = "Person" )
+	@Cacheable
+	@Cache( usage = CacheConcurrencyStrategy.TRANSACTIONAL )
+	public static class Person {
+
+		@Id
+		private Long id;
+
+		private String name;
+
+		@OneToMany( fetch = FetchType.LAZY, mappedBy = "person" )
+		private final Set<Phone> phones = new HashSet<>();
+
+		public Person() {
+		}
+
+		public Person(final long id, final String name) {
+			setId( id );
+			setName( name );
+		}
+
+		public Long getId() {
+			return id;
+		}
+
+		public void setId(final Long id) {
+			this.id = id;
+		}
+
+		public String getName() {
+			return name;
+		}
+
+		@Override
+		public String toString() {
+			return "Person{" +
+					"id=" + id +
+					", name='" + name +
+					'}';
+		}
+
+		@Override
+		public boolean equals(Object o) {
+			if ( this == o ) {
+				return true;
+			}
+			if ( o == null || getClass() != o.getClass() ) {
+				return false;
+			}
+			Person person = (Person) o;
+			return Objects.equals( id, person.id ) && Objects.equals( name, person.name );
+		}
+
+		@Override
+		public int hashCode() {
+			return Objects.hash( id, name );
+		}
+
+		public void setName(final String name) {
+			this.name = name;
+		}
+
+		public Set<Phone> getPhones() {
+			return phones;
+		}
+	}
+}