close cursor before skipping the tests

to avoid SQL Server to throw an error "Connection is busy with results for another command"
cakephp · Jan 29, 2015 · 9dbbea0 · 9dbbea0
1 parent 86025b3
commit 9dbbea0
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/tests/TestCase/Database/QueryTest.php b/tests/TestCase/Database/QueryTest.php
@@ -3144,10 +3144,10 @@ public function testUnbufferedQuery()
             ->bufferResults(false)
             ->execute();
 
-        $this->skipIf(
-            !method_exists($result, 'bufferResults'),
-            'This driver does not support unbuffered queries'
-        );
+        if (!method_exists($result, 'bufferResults')) {
+            $result->closeCursor();
+            $this->skipIf(true, 'This driver does not support unbuffered queries');
+        }
 
         $this->assertCount(0, $result);
         $list = $result->fetchAll('assoc');