PHPC-976: Add more test cases for default read and write concerns

Author: jmikola

tests/readConcern/readconcern-isdefault-001.phpt

@@ -10,8 +10,10 @@ $tests = [
     new MongoDB\Driver\ReadConcern(MongoDB\Driver\ReadConcern::MAJORITY),
     new MongoDB\Driver\ReadConcern('not-yet-supported'),
     (new MongoDB\Driver\Manager('mongodb://127.0.0.1/?readconcernlevel=local'))->getReadConcern(),
+    (new MongoDB\Driver\Manager('mongodb://127.0.0.1/?readconcernlevel='))->getReadConcern(),
     (new MongoDB\Driver\Manager(null, ['readconcernlevel' => 'local']))->getReadConcern(),
     (new MongoDB\Driver\Manager(null, ['readconcernlevel' => '']))->getReadConcern(),
+    // Cannot test ['readconcernlevel' => null] since a string type is expected (PHPC-887)
     (new MongoDB\Driver\Manager)->getReadConcern(),
 ];
 
@@ -31,5 +33,6 @@ bool(false)
 bool(false)
 bool(false)
 bool(false)
+bool(false)
 bool(true)
 ===DONE===

tests/writeConcern/writeconcern-isdefault-001.phpt

@@ -4,9 +4,32 @@ MongoDB\Driver\WriteConcern::isDefault()
 <?php
 
 $tests = [
+    new MongoDB\Driver\WriteConcern(-3), // MONGOC_WRITE_CONCERN_W_MAJORITY
+    new MongoDB\Driver\WriteConcern(-2), // MONGOC_WRITE_CONCERN_W_DEFAULT
+    new MongoDB\Driver\WriteConcern(-1), // MONGOC_WRITE_CONCERN_W_ERRORS_IGNORED
+    new MongoDB\Driver\WriteConcern(0),  // MONGOC_WRITE_CONCERN_W_UNACKNOWLEDGED
     new MongoDB\Driver\WriteConcern(1),
+    new MongoDB\Driver\WriteConcern(2),
+    new MongoDB\Driver\WriteConcern('tag'),
+    new MongoDB\Driver\WriteConcern(MongoDB\Driver\WriteConcern::MAJORITY),
+    // mongoc_uri_parse_option() ignores empty string for w
+    (new MongoDB\Driver\Manager('mongodb://127.0.0.1/?w='))->getWriteConcern(),
+    // Cannot test "w=-3" since libmongoc URI parsing expects integers >= -2
+    (new MongoDB\Driver\Manager('mongodb://127.0.0.1/?w=-2'))->getWriteConcern(),
+    (new MongoDB\Driver\Manager('mongodb://127.0.0.1/?w=-1'))->getWriteConcern(),
+    (new MongoDB\Driver\Manager('mongodb://127.0.0.1/?w=0'))->getWriteConcern(),
     (new MongoDB\Driver\Manager('mongodb://127.0.0.1/?w=1'))->getWriteConcern(),
+    (new MongoDB\Driver\Manager('mongodb://127.0.0.1/?w=2'))->getWriteConcern(),
+    (new MongoDB\Driver\Manager('mongodb://127.0.0.1/?w=tag'))->getWriteConcern(),
+    (new MongoDB\Driver\Manager('mongodb://127.0.0.1/?w=majority'))->getWriteConcern(),
+    // Cannot test ['w' => null] since an integer or string type is expected (PHPC-887)
+    // Cannot test ['w' => -3] or ['w' => -2] since php_phongo_apply_wc_options_to_uri() expects integers >= -1
+    (new MongoDB\Driver\Manager(null, ['w' => -1]))->getWriteConcern(),
+    (new MongoDB\Driver\Manager(null, ['w' => 0]))->getWriteConcern(),
     (new MongoDB\Driver\Manager(null, ['w' => 1]))->getWriteConcern(),
+    (new MongoDB\Driver\Manager(null, ['w' => 2]))->getWriteConcern(),
+    (new MongoDB\Driver\Manager(null, ['w' => 'tag']))->getWriteConcern(),
+    (new MongoDB\Driver\Manager(null, ['w' => 'majority']))->getWriteConcern(),
     (new MongoDB\Driver\Manager)->getWriteConcern(),
 ];
 
@@ -19,6 +42,25 @@ foreach ($tests as $wc) {
 <?php exit(0); ?>
 --EXPECT--
 bool(false)
+bool(true)
+bool(false)
+bool(false)
+bool(false)
+bool(false)
+bool(false)
+bool(false)
+bool(true)
+bool(true)
+bool(false)
+bool(false)
+bool(false)
+bool(false)
+bool(false)
+bool(false)
+bool(false)
+bool(false)
+bool(false)
+bool(false)
 bool(false)
 bool(false)
 bool(true)