[hibernate-commits] Hibernate SVN: r15723 - in core/branches/Branch_3_2_4_SP1_CP/test/org/hibernate/test: tm and 1 other directory.

[hibernate-commits at lists.jboss.org]

Author: gbadner
Date: 2008-12-22 18:27:26 -0500 (Mon, 22 Dec 2008)
New Revision: 15723

Modified:
   core/branches/Branch_3_2_4_SP1_CP/test/org/hibernate/test/connections/AggressiveReleaseTest.java
   core/branches/Branch_3_2_4_SP1_CP/test/org/hibernate/test/tm/CMTTest.java
Log:
JBPAPP-1527 HHH-3675 - Limitiations on Sybase ResultSet implementation causes unit test failures


Modified: core/branches/Branch_3_2_4_SP1_CP/test/org/hibernate/test/connections/AggressiveReleaseTest.java
===================================================================
--- core/branches/Branch_3_2_4_SP1_CP/test/org/hibernate/test/connections/AggressiveReleaseTest.java	2008-12-22 19:16:28 UTC (rev 15722)
+++ core/branches/Branch_3_2_4_SP1_CP/test/org/hibernate/test/connections/AggressiveReleaseTest.java	2008-12-22 23:27:26 UTC (rev 15723)
@@ -107,6 +107,10 @@
 			// expected behavior
 		}
 
+		// getting the first row only because Sybase throws NullPointerException
+		// if data is not read before closing the ResultSet
+		sr.next();
+
 		// Closing the ScrollableResults does currently force the batcher to
 		// aggressively release the connection
 		sr.close();

Modified: core/branches/Branch_3_2_4_SP1_CP/test/org/hibernate/test/tm/CMTTest.java
===================================================================
--- core/branches/Branch_3_2_4_SP1_CP/test/org/hibernate/test/tm/CMTTest.java	2008-12-22 19:16:28 UTC (rev 15722)
+++ core/branches/Branch_3_2_4_SP1_CP/test/org/hibernate/test/tm/CMTTest.java	2008-12-22 23:27:26 UTC (rev 15723)
@@ -443,17 +443,18 @@
 		DummyTransactionManager.INSTANCE.begin();
 		s = getSessions().getCurrentSession();
 		results = s.createQuery( "from Item" ).scroll();
-		while ( !results.isLast() ) {
-			results.next();
+		while ( results.next() ) {
+			// do nothing
 		}
+
 		DummyTransactionManager.INSTANCE.getTransaction().commit();
 
 		// Next, scroll the entire result (closing)
 		DummyTransactionManager.INSTANCE.begin();
 		s = getSessions().getCurrentSession();
 		results = s.createQuery( "from Item" ).scroll();
-		while ( !results.isLast() ) {
-			results.next();
+		while ( results.next() ) {
+			// do nothing
 		}
 		results.close();
 		DummyTransactionManager.INSTANCE.getTransaction().commit();