Hibernate
/
hibernate-orm
mirror of https://github.com/hibernate/hibernate-orm
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

HHH-9933 - Uninitialized PersistentSet with lazy="extra" doesn't respect added elements 

(cherry picked from commit 63e4702629)

HHH-9933 : Fix test case to work in 5.1 branch
This commit is contained in:
lrcuhuh 2015-07-16 14:15:23 +02:00 committed by Gail Badner
parent 45eee165f2
commit dab6c4d0bf
2 changed files with 169 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
hibernate-core/src/main/java/org/hibernate/collection/internal/PersistentSet.java
View File

@ -326,8 +326,8 @@ public class PersistentSet extends AbstractPersistentCollection implements java.
public boolean endRead() {
set.addAll( tempList );
tempList = null;
setInitialized();
return true;
// ensure that operationQueue is considered
return super.endRead();
}
@Override

167
hibernate-core/src/test/java/org/hibernate/test/extralazy/ExtraLazyCollectionConsistencyTest.java Normal file
View File

@ -0,0 +1,167 @@
/*
* 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.extralazy;
import org.hibernate.Session;
import org.hibernate.testing.TestForIssue;
import org.hibernate.testing.junit4.BaseCoreFunctionalTestCase;
import org.junit.Test;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertTrue;
/**
* @author Vlad Mihalcea
*/
public class ExtraLazyCollectionConsistencyTest extends BaseCoreFunctionalTestCase {
private User user;
@Override
public String[] getMappings() {
return new String[] { "extralazy/UserGroup.hbm.xml","extralazy/Parent.hbm.xml","extralazy/Child.hbm.xml" };
}
@Override
protected void prepareTest() {
Session session = openSession();
session.beginTransaction();
{
user = new User("victor", "hugo");
session.persist(user);
}
session.getTransaction().commit();
session.close();
}
@Override
protected boolean isCleanupTestDataRequired() {
return true;
}
@Test
@TestForIssue(jiraKey = "HHH-9933")
public void testSetSize() {
Session session = openSession();
session.beginTransaction();
{
User _user = session.get(User.class, user.getName());
Document document = new Document("Les Miserables", "sad", _user);
assertEquals(1, _user.getDocuments().size());
}
session.getTransaction().commit();
session.close();
}
@Test
@TestForIssue(jiraKey = "HHH-9933")
public void testSetIterator() {
Session session = openSession();
session.beginTransaction();
{
User _user = session.get(User.class, user.getName());
Document document = new Document("Les Miserables", "sad", _user);
assertTrue(_user.getDocuments().iterator().hasNext());
}
session.getTransaction().commit();
session.close();
}
@Test
@TestForIssue(jiraKey = "HHH-9933")
public void testSetIsEmpty() {
Session session = openSession();
session.beginTransaction();
{
User _user = session.get(User.class, user.getName());
Document document = new Document("Les Miserables", "sad", _user);
assertFalse(_user.getDocuments().isEmpty());
}
session.getTransaction().commit();
session.close();
}
@Test
@TestForIssue(jiraKey = "HHH-9933")
public void testSetContains() {
Session session = openSession();
session.beginTransaction();
{
User _user = session.get(User.class, user.getName());
Document document = new Document("Les Miserables", "sad", _user);
assertTrue(_user.getDocuments().contains(document));
}
session.getTransaction().commit();
session.close();
}
@Test
@TestForIssue(jiraKey = "HHH-9933")
public void testSetAdd() {
Session session = openSession();
session.beginTransaction();
{
User _user = session.get(User.class, user.getName());
Document document = new Document();
document.setTitle("Les Miserables");
document.setContent("sad");
document.setOwner(_user);
assertTrue("not added", _user.getDocuments().add(document));
assertFalse("added", _user.getDocuments().add(document));
}
session.getTransaction().commit();
session.close();
}
@Test
@TestForIssue(jiraKey = "HHH-9933")
public void testSetRemove() {
Session session = openSession();
session.beginTransaction();
{
User _user = session.get(User.class, user.getName());
Document document = new Document("Les Miserables", "sad", _user);
assertTrue("not removed", _user.getDocuments().remove(document));
}
session.getTransaction().commit();
session.close();
}
@Test
@TestForIssue(jiraKey = "HHH-9933")
public void testSetToArray() {
Session session = openSession();
session.beginTransaction();
{
User _user = session.get(User.class, user.getName());
Document document = new Document("Les Miserables", "sad", _user);
assertEquals(1, _user.getDocuments().toArray().length);
}
session.getTransaction().commit();
session.close();
}
@Test
@TestForIssue(jiraKey = "HHH-9933")
public void testSetToArrayTyped() {
Session session = openSession();
session.beginTransaction();
{
User _user = session.get(User.class, user.getName());
Document document = new Document("Les Miserables", "sad", _user);
assertEquals(1, _user.getDocuments().toArray(new Document[0]).length);
}
session.getTransaction().commit();
session.close();
}
}