Date: Tuesday, December 6, 2005 @ 17:54:01
  Author: gilles
    Path: /cvsroot/carob/carob/test

Modified: TestBeginCommitRollback.cpp (1.9 -> 1.10)
          TestExecReadRequest.cpp (1.14 -> 1.15) TestStatement.cpp (1.12
          -> 1.13)

Added resultSet deletes in order to check memory leaking on these tests


-----------------------------+
 TestBeginCommitRollback.cpp |    6 ++++--
 TestExecReadRequest.cpp     |    2 ++
 TestStatement.cpp           |    1 +
 3 files changed, 7 insertions(+), 2 deletions(-)


Index: carob/test/TestBeginCommitRollback.cpp
diff -u carob/test/TestBeginCommitRollback.cpp:1.9 
carob/test/TestBeginCommitRollback.cpp:1.10
--- carob/test/TestBeginCommitRollback.cpp:1.9  Fri Dec  2 15:53:07 2005
+++ carob/test/TestBeginCommitRollback.cpp      Tue Dec  6 17:54:01 2005
@@ -34,12 +34,13 @@
   connectionPtr->setAutoCommit(false);
   RequestWithResultSetParameters readReq(L"select * from address");
   readReq.setEscapeProcessing(false).setTimeoutInSeconds(2);
-  connectionPtr->statementExecuteQuery(readReq);
+  DriverResultSet* drsPtr = connectionPtr->statementExecuteQuery(readReq);
   connectionPtr->commit();
   if (isDebugEnabled())
   {
     logDebug(fctName, L"Read and commit succeeded");
   }
+  delete drsPtr;
 }
 
 void TestBeginCommitRollback::testReadRequestAndRollback()
@@ -48,12 +49,13 @@
   connectionPtr->setAutoCommit(false);
   RequestWithResultSetParameters readReq(L"select * from address");
   readReq.setEscapeProcessing(false).setTimeoutInSeconds(2);
-  connectionPtr->statementExecuteQuery(readReq);
+  DriverResultSet* drsPtr = connectionPtr->statementExecuteQuery(readReq);
   connectionPtr->rollback();
   if (isDebugEnabled())
   {
     logDebug(fctName, L"Read and rollback succeeded");
   }
+  delete drsPtr;
 }
 
 void TestBeginCommitRollback::testWriteRequestAndCommit()
Index: carob/test/TestExecReadRequest.cpp
diff -u carob/test/TestExecReadRequest.cpp:1.14 
carob/test/TestExecReadRequest.cpp:1.15
--- carob/test/TestExecReadRequest.cpp:1.14     Fri Dec  2 15:53:07 2005
+++ carob/test/TestExecReadRequest.cpp  Tue Dec  6 17:54:01 2005
@@ -109,6 +109,8 @@
               <<L"\t"<<drsPtr->getString(2)
               <<L"\t\t"<<drsPtr->getString(3)<<endl;
   }
+  //We have to free the allocated result...
+  delete drsPtr;
 }
 
 CppUnit::Test* TestExecReadRequest::suite()
Index: carob/test/TestStatement.cpp
diff -u carob/test/TestStatement.cpp:1.12 carob/test/TestStatement.cpp:1.13
--- carob/test/TestStatement.cpp:1.12   Fri Dec  2 15:53:07 2005
+++ carob/test/TestStatement.cpp        Tue Dec  6 17:54:01 2005
@@ -218,6 +218,7 @@
     
   //This next() should return false because we asked just 1 row
   CPPUNIT_ASSERT(drsPtr->next() == false);
+  delete drsPtr;
   delete statementPtr;
 }
 

_______________________________________________
Carob-commits mailing list
[email protected]
https://forge.continuent.org/mailman/listinfo/carob-commits

Reply via email to