Use dropCollection() helper to ensure collections are cleaned up

Author: jmikola

tests/SpecTests/Crud/Prose3_BulkWriteSplitsOnMaxWriteBatchSizeTest.php

@@ -29,6 +29,7 @@ public function testSplitOnMaxWriteBatchSize(): void
         $maxWriteBatchSize = $this->getPrimaryServer()->getInfo()['maxWriteBatchSize'] ?? null;
         self::assertIsInt($maxWriteBatchSize);
 
+        $this->dropCollection($this->getDatabaseName(), $this->getCollectionName());
         $collection = $client->selectCollection($this->getDatabaseName(), $this->getCollectionName());
         $bulkWrite = ClientBulkWrite::createWithCollection($collection);
 

tests/SpecTests/Crud/Prose4_BulkWriteSplitsOnMaxMessageSizeBytesTest.php

@@ -35,6 +35,7 @@ public function testSplitOnMaxWriteBatchSize(): void
         $numModels = (int) ($maxMessageSizeBytes / $maxBsonObjectSize + 1);
         $document = ['a' => str_repeat('b', $maxBsonObjectSize - 500)];
 
+        $this->dropCollection($this->getDatabaseName(), $this->getCollectionName());
         $collection = $client->selectCollection($this->getDatabaseName(), $this->getCollectionName());
         $bulkWrite = ClientBulkWrite::createWithCollection($collection);
 

tests/SpecTests/Crud/Prose5_BulkWriteCollectsWriteConcernErrorsAcrossBatchesTest.php

@@ -42,6 +42,7 @@ public function testCollectWriteConcernErrors(): void
             ],
         ]);
 
+        $this->dropCollection($this->getDatabaseName(), $this->getCollectionName());
         $collection = $client->selectCollection($this->getDatabaseName(), $this->getCollectionName());
         $bulkWrite = ClientBulkWrite::createWithCollection($collection);
 

tests/SpecTests/Crud/Prose6_BulkWriteHandlesWriteErrorsAcrossBatchesTest.php

@@ -35,8 +35,8 @@ public function testOrdered(): void
         $maxWriteBatchSize = $this->getPrimaryServer()->getInfo()['maxWriteBatchSize'] ?? null;
         self::assertIsInt($maxWriteBatchSize);
 
+        $this->dropCollection($this->getDatabaseName(), $this->getCollectionName());
         $collection = $client->selectCollection($this->getDatabaseName(), $this->getCollectionName());
-        $collection->drop();
         $collection->insertOne(['_id' => 1]);
 
         $bulkWrite = ClientBulkWrite::createWithCollection($collection, ['ordered' => true]);
@@ -64,8 +64,8 @@ public function testUnordered(): void
         $maxWriteBatchSize = $this->getPrimaryServer()->getInfo()['maxWriteBatchSize'] ?? null;
         self::assertIsInt($maxWriteBatchSize);
 
+        $this->dropCollection($this->getDatabaseName(), $this->getCollectionName());
         $collection = $client->selectCollection($this->getDatabaseName(), $this->getCollectionName());
-        $collection->drop();
         $collection->insertOne(['_id' => 1]);
 
         $bulkWrite = ClientBulkWrite::createWithCollection($collection, ['ordered' => false]);

tests/SpecTests/Crud/Prose7_BulkWriteHandlesCursorRequiringGetMoreTest.php

@@ -31,8 +31,8 @@ public function testHandlesCursor(): void
         $maxBsonObjectSize = $this->getPrimaryServer()->getInfo()['maxBsonObjectSize'] ?? null;
         self::assertIsInt($maxBsonObjectSize);
 
+        $this->dropCollection($this->getDatabaseName(), $this->getCollectionName());
         $collection = $client->selectCollection($this->getDatabaseName(), $this->getCollectionName());
-        $collection->drop();
 
         $bulkWrite = ClientBulkWrite::createWithCollection($collection, ['verboseResults' => true]);
         $bulkWrite->updateOne(

tests/SpecTests/Crud/Prose8_BulkWriteHandlesCursorRequiringGetMoreWithinTransactionTest.php

@@ -32,8 +32,8 @@ public function testHandlesCursorWithinTransaction(): void
         $maxBsonObjectSize = $this->getPrimaryServer()->getInfo()['maxBsonObjectSize'] ?? null;
         self::assertIsInt($maxBsonObjectSize);
 
+        $this->dropCollection($this->getDatabaseName(), $this->getCollectionName());
         $collection = $client->selectCollection($this->getDatabaseName(), $this->getCollectionName());
-        $collection->drop();
 
         $bulkWrite = ClientBulkWrite::createWithCollection($collection, ['verboseResults' => true]);
         $bulkWrite->updateOne(

tests/SpecTests/Crud/Prose9_BulkWriteHandlesGetMoreErrorTest.php

@@ -43,8 +43,8 @@ public function testHandlesGetMoreError(): void
             ],
         ]);
 
+        $this->dropCollection($this->getDatabaseName(), $this->getCollectionName());
         $collection = $client->selectCollection($this->getDatabaseName(), $this->getCollectionName());
-        $collection->drop();
 
         $bulkWrite = ClientBulkWrite::createWithCollection($collection, ['verboseResults' => true]);
         $bulkWrite->updateOne(