Use consistent db and collection name in example tests to ensure cleanup is done in tearDown

Author: GromNaN

tests/DocumentationExamplesTest.php

@@ -1194,11 +1194,10 @@ public function testRunCommand_example_1(): void
     public function testRunCommand_example_2(): void
     {
         $db = new Database($this->manager, $this->getDatabaseName());
-        $db->dropCollection('restaurants');
-        $db->createCollection('restaurants');
+        $db->createCollection('inventory');
 
         // Start runCommand Example 2
-        $cursor = $db->command(['collStats' => 'restaurants']);
+        $cursor = $db->command(['collStats' => 'inventory']);
         $result = $cursor->toArray()[0];
         // End runCommand Example 2
 
@@ -1210,7 +1209,7 @@ public function testIndex_example_1(): void
         $db = new Database($this->manager, $this->getDatabaseName());
 
         // Start Index Example 1
-        $indexName = $db->records->createIndex(['score' => 1]);
+        $indexName = $db->inventory->createIndex(['score' => 1]);
         // End Index Example 1
 
         $this->assertEquals('score_1', $indexName);
@@ -1221,7 +1220,7 @@ public function testIndex_example_2(): void
         $db = new Database($this->manager, $this->getDatabaseName());
 
         // Start Index Example 2
-        $indexName = $db->restaurants->createIndex(
+        $indexName = $db->inventory->createIndex(
             ['cuisine' => 1, 'name' => 1],
             ['partialFilterExpression' => ['rating' => ['$gt' => 5]]]
         );
@@ -1496,13 +1495,14 @@ public function testCausalConsistency(): void
 
         // Prep
         $client = static::createTestClient();
-        $items = $client->selectDatabase(
-            'test',
+        $databaseName = $this->getDatabaseName();
+        $inventory = $client->selectDatabase(
+            $databaseName,
             ['writeConcern' => new WriteConcern(WriteConcern::MAJORITY)]
-        )->items;
+        )->inventory;
 
-        $items->drop();
-        $items->insertOne(
+        $inventory->drop();
+        $inventory->insertOne(
             ['sku' => '111', 'name' => 'Peanuts', 'start' => new UTCDateTime()]
         );
 
@@ -1511,33 +1511,33 @@ public function testCausalConsistency(): void
              * mode, so using $manager->selectServer does not work here. To work
              * around this, we run a query on a secondary and rely on an
              * exception to let us know that no secondary is available. */
-            $items->countDocuments([], ['readPreference' => new ReadPreference(ReadPreference::RP_SECONDARY)]);
+            $inventory->countDocuments([], ['readPreference' => new ReadPreference(ReadPreference::RP_SECONDARY)]);
         } catch (Exception $e) {
             $this->markTestSkipped('Secondary is not available');
         }
 
         // phpcs:disable SlevomatCodingStandard.Namespaces.ReferenceUsedNamesOnly
         // Start Causal Consistency Example 1
-        $items = $client->selectDatabase(
-            'test',
+        $inventory = $client->selectDatabase(
+            $databaseName,
             [
                 'readConcern' => new \MongoDB\Driver\ReadConcern(\MongoDB\Driver\ReadConcern::MAJORITY),
                 'writeConcern' => new \MongoDB\Driver\WriteConcern(\MongoDB\Driver\WriteConcern::MAJORITY, 1000),
             ]
-        )->items;
+        )->inventory;
 
         $s1 = $client->startSession(
             ['causalConsistency' => true]
         );
 
         $currentDate = new \MongoDB\BSON\UTCDateTime();
 
-        $items->updateOne(
+        $inventory->updateOne(
             ['sku' => '111', 'end' => ['$exists' => false]],
             ['$set' => ['end' => $currentDate]],
             ['session' => $s1]
         );
-        $items->insertOne(
+        $inventory->insertOne(
             ['sku' => '111-nuts', 'name' => 'Pecans', 'start' => $currentDate],
             ['session' => $s1]
         );
@@ -1554,16 +1554,16 @@ public function testCausalConsistency(): void
         $s2->advanceClusterTime($s1->getClusterTime());
         $s2->advanceOperationTime($s1->getOperationTime());
 
-        $items = $client->selectDatabase(
-            'test',
+        $inventory = $client->selectDatabase(
+            $databaseName,
             [
                 'readPreference' => new \MongoDB\Driver\ReadPreference(\MongoDB\Driver\ReadPreference::RP_SECONDARY),
                 'readConcern' => new \MongoDB\Driver\ReadConcern(\MongoDB\Driver\ReadConcern::MAJORITY),
                 'writeConcern' => new \MongoDB\Driver\WriteConcern(\MongoDB\Driver\WriteConcern::MAJORITY, 1000),
             ]
-        )->items;
+        )->inventory;
 
-        $result = $items->find(
+        $result = $inventory->find(
             ['end' => ['$exists' => false]],
             ['session' => $s2]
         );

tests/FunctionalTestCase.php

@@ -273,9 +273,9 @@ protected function createCollection(array $options = []): void
      * server, a majority write concern will be used. This is helpful for tests
      * using transactions or secondary reads.
      */
-    protected function dropCollection(array $options = []): void
+    protected function dropCollection(): void
     {
-        $options += ['writeConcern' => new WriteConcern(WriteConcern::MAJORITY)];
+        $options = ['writeConcern' => new WriteConcern(WriteConcern::MAJORITY)];
 
         $operation = new DropCollection($this->getDatabaseName(), $this->getCollectionName(), $options);
         $operation->execute($this->getPrimaryServer());