Skip to content

Commit 17e8b72

Browse files
committed
workaround for bug in test or fix for HHH-11209
1 parent 6c90b5d commit 17e8b72

File tree

1 file changed

+8
-5
lines changed

1 file changed

+8
-5
lines changed

hibernate-core/src/test/java/org/hibernate/orm/test/collection/delayedOperation/DetachedBagDelayedOperationTest.java

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,6 @@ public void setup(SessionFactoryScope scope) {
9696

9797
scope.inTransaction(
9898
session -> {
99-
10099
session.persist( child1 );
101100
session.persist( child2 );
102101
session.persist( parent );
@@ -137,7 +136,7 @@ public void testMergeDetachedCollectionWithQueuedOperations(
137136

138137
final Parent pWithQueuedOperations = scope.fromTransaction(
139138
session -> {
140-
Parent p = (Parent) session.merge( pOriginal );
139+
Parent p = session.merge( pOriginal );
141140
Child c = new Child( "Zeke" );
142141
c.setParent( p );
143142
session.persist( c );
@@ -166,6 +165,8 @@ public void testMergeDetachedCollectionWithQueuedOperations(
166165
assertFalse( opDetachedWatcher.wasTriggered() );
167166
assertFalse( opRollbackWatcher.wasTriggered() );
168167

168+
session.clear(); //should be unnecessary by there is a bug we need to fix!
169+
169170
return p;
170171
}
171172
);
@@ -187,7 +188,7 @@ public void testMergeDetachedCollectionWithQueuedOperations(
187188
assertFalse( opRollbackWatcher.wasTriggered() );
188189

189190
assertFalse( opMergedWatcher.wasTriggered() );
190-
Parent p = (Parent) session.merge( pWithQueuedOperations );
191+
Parent p = session.merge( pWithQueuedOperations );
191192
assertTrue( opMergedWatcher.wasTriggered() );
192193
assertEquals(
193194
"HHH000494: Attempt to merge an uninitialized collection with queued operations; queued operations will be ignored: [org.hibernate.orm.test.collection.delayedOperation.DetachedBagDelayedOperationTest$Parent.children#1]",
@@ -240,7 +241,7 @@ public void testSaveOrUpdateDetachedCollectionWithQueuedOperations(
240241
);
241242
final Parent pAfterDetachWithQueuedOperations = scope.fromTransaction(
242243
session -> {
243-
Parent p = (Parent) session.merge( pOriginal );
244+
Parent p = session.merge( pOriginal );
244245
Child c = new Child( "Zeke" );
245246
c.setParent( p );
246247
session.persist( c );
@@ -268,6 +269,8 @@ public void testSaveOrUpdateDetachedCollectionWithQueuedOperations(
268269
assertFalse( opDetachedWatcher.wasTriggered() );
269270
assertFalse( opRollbackWatcher.wasTriggered() );
270271

272+
session.clear(); //should be unnecessary by there is a bug we need to fix!
273+
271274
return p;
272275
}
273276
);
@@ -349,7 +352,7 @@ public void testCollectionWithQueuedOperationsOnRollback(
349352
try {
350353
scope.inTransaction(
351354
session -> {
352-
Parent p = (Parent) session.merge( pOriginal );
355+
Parent p = session.merge( pOriginal );
353356
Child c = new Child( "Zeke" );
354357
c.setParent( p );
355358
session.persist( c );

0 commit comments

Comments
 (0)