From 7293187a0187539eea7fce40de46f5bf585740bf Mon Sep 17 00:00:00 2001 From: Gail Badner Date: Mon, 19 Jul 2010 22:09:56 +0000 Subject: [PATCH] HHH-3334 : Added FailureExpected test for Cascade-save breaks if parent ID is assigned (delays insert) and child has identity ID (early insert) git-svn-id: https://svn.jboss.org/repos/hibernate/core/trunk@19967 1b8cb986-b30d-0410-93ca-fae66ebed9b2 --- .../CascadeTestWithAssignedParentIdTest.java | 119 ++++++++++++++++++ .../ChildForParentWithAssignedId.hbm.xml | 11 ++ .../test/cascade/ParentWithAssignedId.hbm.xml | 14 +++ 3 files changed, 144 insertions(+) create mode 100644 testsuite/src/test/java/org/hibernate/test/cascade/CascadeTestWithAssignedParentIdTest.java create mode 100644 testsuite/src/test/java/org/hibernate/test/cascade/ChildForParentWithAssignedId.hbm.xml create mode 100644 testsuite/src/test/java/org/hibernate/test/cascade/ParentWithAssignedId.hbm.xml diff --git a/testsuite/src/test/java/org/hibernate/test/cascade/CascadeTestWithAssignedParentIdTest.java b/testsuite/src/test/java/org/hibernate/test/cascade/CascadeTestWithAssignedParentIdTest.java new file mode 100644 index 0000000000..e9869e9665 --- /dev/null +++ b/testsuite/src/test/java/org/hibernate/test/cascade/CascadeTestWithAssignedParentIdTest.java @@ -0,0 +1,119 @@ +/* + * Hibernate, Relational Persistence for Idiomatic Java + * + * Copyright (c) 2010, Red Hat Middleware LLC or third-party contributors as + * indicated by the @author tags or express copyright attribution + * statements applied by the authors. All third-party contributions are + * distributed under license by Red Hat Middleware LLC. + * + * This copyrighted material is made available to anyone wishing to use, modify, + * copy, or redistribute it subject to the terms and conditions of the GNU + * Lesser General Public License, as published by the Free Software Foundation. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY + * or FITNESS FOR A PARTICULAR PURPOSE. See the GNU Lesser General Public License + * for more details. + * + * You should have received a copy of the GNU Lesser General Public License + * along with this distribution; if not, write to: + * Free Software Foundation, Inc. + * 51 Franklin Street, Fifth Floor + * Boston, MA 02110-1301 USA + */ +package org.hibernate.test.cascade; + +import java.util.Collections; + +import junit.framework.Test; + +import org.hibernate.Session; +import org.hibernate.Transaction; +import org.hibernate.id.IdentityGenerator; +import org.hibernate.testing.junit.functional.FunctionalTestCase; +import org.hibernate.testing.junit.functional.FunctionalTestClassTestSuite; + +/** + * @author Wallace Wadge (based on code by Gail Badner) + */ + +/** + * Test case to illustrate that when a child table attempts to cascade to a parent and the parent's Id + * is set to assigned, an exception thrown (not-null property references a null or transient value). + * This error only occurs if the parent link in marked as not nullable. + */ +public class CascadeTestWithAssignedParentIdTest extends FunctionalTestCase { + + public CascadeTestWithAssignedParentIdTest(String name) { + super( name ); + } + + public String[] getMappings() { + return new String[] { + "cascade/ChildForParentWithAssignedId.hbm.xml", + "cascade/ParentWithAssignedId.hbm.xml" + }; + } + + public static Test suite() { + return new FunctionalTestClassTestSuite( CascadeTestWithAssignedParentIdTest.class ); + } + + + /** + * Saves the child object with the parent when both the one-to-many and + * many-to-one associations use cascade="all" + */ + public void testSaveChildWithParentFailureExpected() { + if ( ! IdentityGenerator.class.isAssignableFrom( getDialect().getNativeIdentifierGeneratorClass() ) ) { + reportSkip( "FailureExpected test passes when native id generator is not IdentityGenerator", + "parent insert is assigned (delayed) when child is identity (early insert)" ); + fail( "test is expected to fail" ); + return; + } + Session session = openSession(); + Transaction txn = session.beginTransaction(); + Parent parent = new Parent(); + Child child = new Child(); + child.setParent( parent ); + parent.setChildren( Collections.singleton( child ) ); + parent.setId(new Long(123L)); + // this should figure out that the parent needs saving first since id is assigned. + session.save( child ); + txn.commit(); + session.close(); + + session = openSession(); + txn = session.beginTransaction(); + parent = ( Parent ) session.get( Parent.class, parent.getId() ); + assertEquals( 1, parent.getChildren().size() ); + txn.commit(); + session.close(); + } + + public void testSaveChildWithParent() { + if ( IdentityGenerator.class.isAssignableFrom( getDialect().getNativeIdentifierGeneratorClass() ) ) { + reportSkip( "test is known to fail when native id generator is IdentityGenerator", + "parent insert is assigned (delayed) when child has identity (early insert)" ); + return; + } + Session session = openSession(); + Transaction txn = session.beginTransaction(); + Parent parent = new Parent(); + Child child = new Child(); + child.setParent( parent ); + parent.setChildren( Collections.singleton( child ) ); + parent.setId(new Long(123L)); + // this should figure out that the parent needs saving first since id is assigned. + session.save( child ); + txn.commit(); + session.close(); + + session = openSession(); + txn = session.beginTransaction(); + parent = ( Parent ) session.get( Parent.class, parent.getId() ); + assertEquals( 1, parent.getChildren().size() ); + txn.commit(); + session.close(); + } +} diff --git a/testsuite/src/test/java/org/hibernate/test/cascade/ChildForParentWithAssignedId.hbm.xml b/testsuite/src/test/java/org/hibernate/test/cascade/ChildForParentWithAssignedId.hbm.xml new file mode 100644 index 0000000000..1d377cafd1 --- /dev/null +++ b/testsuite/src/test/java/org/hibernate/test/cascade/ChildForParentWithAssignedId.hbm.xml @@ -0,0 +1,11 @@ + + + + + + + + + + + diff --git a/testsuite/src/test/java/org/hibernate/test/cascade/ParentWithAssignedId.hbm.xml b/testsuite/src/test/java/org/hibernate/test/cascade/ParentWithAssignedId.hbm.xml new file mode 100644 index 0000000000..f99523cbf4 --- /dev/null +++ b/testsuite/src/test/java/org/hibernate/test/cascade/ParentWithAssignedId.hbm.xml @@ -0,0 +1,14 @@ + + + + + + + + + + + + + +