Hibernate/hibernate-orm
1
0
Fork 0
mirror of https://github.com/hibernate/hibernate-orm synced 2025-02-17 08:35:13 +00:00
Code Issues Packages Projects Releases Wiki Activity

HHH-9777 : Dereferenced collections are not processed properly

Browse Source 
HHH-9777 : Dereferenced collections are not processed properly (test case changes only)
This commit is contained in:
Gail Badner 2015-05-07 01:02:17 -07:00 committed by Andrea Boriero
parent e07eef3db1
commit 93f56ecf5d
5 changed files with 5 additions and 49 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
hibernate-core/src/main/java/org/hibernate/type/CollectionType.java
View File

@ -321,7 +321,7 @@ public boolean isDirty(Object old, Object current, SessionImplementor session)
// TODO: I don't really like this implementation; it would be better if
// this was handled by searchForDirtyCollections()
return isOwnerVersioned( session ) && super.isDirty( old, current, session );
return super.isDirty( old, current, session );
// return false;
}

5
hibernate-core/src/test/java/org/hibernate/test/collection/dereferenced/AbstractDereferencedCollectionTest.java
View File

@ -31,6 +31,7 @@
import org.hibernate.engine.spi.CollectionEntry;
import org.hibernate.engine.spi.EntityEntry;
import org.hibernate.engine.spi.SessionImplementor;
import org.hibernate.testing.TestForIssue;
import org.hibernate.testing.junit4.BaseCoreFunctionalTestCase;
import java.lang.InstantiationException;
@ -48,6 +49,7 @@
public abstract class AbstractDereferencedCollectionTest extends BaseCoreFunctionalTestCase {
@Test
@TestForIssue( jiraKey = "HHH-9777" )
public void testMergeNullCollection() {
Session s = openSession();
s.getTransaction().begin();
@ -121,6 +123,7 @@ public void testMergeNullCollection() {
}
@Test
@TestForIssue( jiraKey = "HHH-9777" )
public void testGetAndNullifyCollection() {
Session s = openSession();
s.getTransaction().begin();
@ -196,7 +199,7 @@ public void testGetAndNullifyCollection() {
}
@Test
//@FailureExpected( jiraKey = "HHH-9777")
@TestForIssue( jiraKey = "HHH-9777" )
public void testGetAndReplaceCollection() {
Session s = openSession();
s.getTransaction().begin();

25
hibernate-core/src/test/java/org/hibernate/test/collection/dereferenced/UnversionedCascadeDereferencedCollectionTest.java
View File

@ -23,10 +23,6 @@
*/
package org.hibernate.test.collection.dereferenced;
import org.junit.Test;
import org.hibernate.testing.FailureExpected;
/**
* @author Gail Badner
*/
@ -36,25 +32,4 @@ public class UnversionedCascadeDereferencedCollectionTest extends AbstractDerefe
protected Class<?> getCollectionOwnerClass() {
return UnversionedCascadeOne.class;
}
@Override
@Test
@FailureExpected(jiraKey = "HHH-9777")
public void testMergeNullCollection() {
super.testMergeNullCollection();
}
@Override
@Test
@FailureExpected(jiraKey = "HHH-9777")
public void testGetAndNullifyCollection() {
super.testGetAndNullifyCollection();
}
@Override
@Test
@FailureExpected(jiraKey = "HHH-9777")
public void testGetAndReplaceCollection() {
super.testGetAndReplaceCollection();
}
}

21
hibernate-core/src/test/java/org/hibernate/test/collection/dereferenced/UnversionedNoCascadeDereferencedCollectionTest.java
View File

@ -36,25 +36,4 @@ public class UnversionedNoCascadeDereferencedCollectionTest extends AbstractDere
protected Class<?> getCollectionOwnerClass() {
return UnversionedNoCascadeOne.class;
}
@Override
@Test
@FailureExpected(jiraKey = "HHH-9777")
public void testMergeNullCollection() {
super.testMergeNullCollection();
}
@Override
@Test
@FailureExpected(jiraKey = "HHH-9777")
public void testGetAndNullifyCollection() {
super.testGetAndNullifyCollection();
}
@Override
@Test
@FailureExpected(jiraKey = "HHH-9777")
public void testGetAndReplaceCollection() {
super.testGetAndReplaceCollection();
}
}

1
hibernate-core/src/test/java/org/hibernate/test/legacy/FooBarTest.java
View File

@ -118,7 +118,6 @@ public String[] getMappings() {
}
@Test
@FailureExpected(jiraKey = "HHH-9777")
public void testSaveOrUpdateCopyAny() throws Exception {
Session s = openSession();
s.beginTransaction();