PHPC-1878 and PHPC-2008: Fix tests using local database (#1277)

* PHPC-1878: Explicitly use w:1 for local database

* PHPC-2008: Skip test requiring role to drop local collection

The "restore" role is required to drop collections in the "local" database. Mongo Orchestration does not yet grant that role for its users.

Author: jmikola

tests/replicaset/manager-selectserver-001.phpt

@@ -10,7 +10,8 @@ MongoDB\Driver\Manager::selectServer() select a server from SDAM based on ReadPr
 require_once __DIR__ . "/../utils/basic.inc";
 
 // Disable retryWrites since the test writes to the unreplicated "local" database
-$manager = create_test_manager(URI, ['retryWrites' => false]);
+// Explicitly use w:1 to work around MongoDB 5.0 applying w:majority (SERVER-61790)
+$manager = create_test_manager(URI, ['retryWrites' => false, 'w' => 1]);
 
 $rp = new MongoDB\Driver\ReadPreference(MongoDB\Driver\ReadPreference::RP_PRIMARY);
 $server = $manager->selectServer($rp);

tests/replicaset/writeresult-getserver-002.phpt

@@ -2,6 +2,7 @@
 MongoDB\Driver\Server: Manager->getServer() returning correct server
 --SKIPIF--
 <?php require __DIR__ . "/../utils/basic-skipif.inc"; ?>
+<?php skip_if_auth(); /* TODO: PHPC-2008 */ ?>
 <?php skip_if_not_replica_set(); ?>
 <?php skip_if_no_secondary(); ?>
 <?php skip_if_not_clean(); ?>
@@ -11,8 +12,8 @@ MongoDB\Driver\Server: Manager->getServer() returning correct server
 require_once __DIR__ . "/../utils/basic.inc";
 
 // Disable retryWrites since the test writes to the unreplicated "local" database
-$manager = create_test_manager(URI, ['retryWrites' => false]);
-
+// Explicitly use w:1 to work around MongoDB 5.0 applying w:majority (SERVER-61790)
+$manager = create_test_manager(URI, ['retryWrites' => false, 'w' => 1]);
 
 $doc = array("example" => "document");
 $bulk = new \MongoDB\Driver\BulkWrite();
@@ -80,6 +81,8 @@ object(MongoDB\Driver\WriteResult)#%d (%d) {
   NULL
   ["writeConcern"]=>
   object(MongoDB\Driver\WriteConcern)#%d (%d) {
+    ["w"]=>
+    int(1)
   }
 }
 string(%d) "%s"

tests/server/server-executeBulkWrite-005.phpt

@@ -10,7 +10,8 @@ MongoDB\Driver\Server::executeBulkWrite() with write concern (replica set second
 require_once __DIR__ . "/../utils/basic.inc";
 
 // Disable retryWrites since the test writes to the unreplicated "local" database
-$manager = create_test_manager(URI, ['retryWrites' => false]);
+// Explicitly use w:1 to work around MongoDB 5.0 applying w:majority (SERVER-61790)
+$manager = create_test_manager(URI, ['retryWrites' => false, 'w' => 1]);
 $server = $manager->selectServer(new MongoDB\Driver\ReadPreference(MongoDB\Driver\ReadPreference::RP_SECONDARY));
 
 /* The server ignores write concerns with w>2 for writes to the local database,

tests/utils/tools.php

@@ -51,15 +51,18 @@ function drop_collection($uri, $databaseName, $collectionName)
     $command = new Command(['drop' => $collectionName]);
 
     try {
-        /* We need to use WriteConcern::MAJORITY here due to the issue
-         * explained in SERVER-35613: "drop" uses a two phase commit, and due
-         * to that, it is possible that a lock can't be acquired for a
-         * transaction that gets quickly started as the "drop" reaper hasn't
-         * completed yet. */
+        /* Unless we are dropping a collection within the "local" database,
+         * which does not support a write concern, we need to use w:majority due
+         * to the issue explained in SERVER-35613: "drop" uses a two phase
+         * commit, and due to that, it is possible that a lock can't be acquired
+         * for a transaction that gets quickly started as the "drop" reaper
+         * hasn't completed yet. */
+        $wc = $databaseName === 'local' ? new WriteConcern(1) : new WriteConcern(WriteConcern::MAJORITY);
+
         $server->executeCommand(
             $databaseName,
             $command,
-            ['writeConcern' => new WriteConcern(WriteConcern::MAJORITY)]
+            ['writeConcern' => $wc]
         );
     } catch (RuntimeException $e) {
         if ($e->getMessage() !== 'ns not found') {