Merge pull request #696

alcaeus · alcaeus · commit 9480f994de9f · 2019-11-15T09:21:00.000+01:00
diff --git a/src/Client.php b/src/Client.php
@@ -95,7 +95,7 @@ public function __construct($uri = 'mongodb://127.0.0.1/', array $uriOptions = [
     {
         $driverOptions += ['typeMap' => self::$defaultTypeMap];
 
-        if (isset($driverOptions['typeMap']) && ! is_array($driverOptions['typeMap'])) {
+        if (! is_array($driverOptions['typeMap'])) {
             throw InvalidArgumentException::invalidType('"typeMap" driver option', $driverOptions['typeMap'], 'array');
         }
 
diff --git a/src/GridFS/Bucket.php b/src/GridFS/Bucket.php
@@ -143,19 +143,19 @@ public function __construct(Manager $manager, $databaseName, array $options = []
             'disableMD5' => false,
         ];
 
-        if (isset($options['bucketName']) && ! is_string($options['bucketName'])) {
+        if (! is_string($options['bucketName'])) {
             throw InvalidArgumentException::invalidType('"bucketName" option', $options['bucketName'], 'string');
         }
 
-        if (isset($options['chunkSizeBytes']) && ! is_integer($options['chunkSizeBytes'])) {
+        if (! is_integer($options['chunkSizeBytes'])) {
             throw InvalidArgumentException::invalidType('"chunkSizeBytes" option', $options['chunkSizeBytes'], 'integer');
         }
 
-        if (isset($options['chunkSizeBytes']) && $options['chunkSizeBytes'] < 1) {
+        if ($options['chunkSizeBytes'] < 1) {
             throw new InvalidArgumentException(sprintf('Expected "chunkSizeBytes" option to be >= 1, %d given', $options['chunkSizeBytes']));
         }
 
-        if (isset($options['disableMD5']) && ! is_bool($options['disableMD5'])) {
+        if (! is_bool($options['disableMD5'])) {
             throw InvalidArgumentException::invalidType('"disableMD5" option', $options['disableMD5'], 'boolean');
         }
 
diff --git a/src/GridFS/WritableStream.php b/src/GridFS/WritableStream.php
@@ -114,15 +114,15 @@ public function __construct(CollectionWrapper $collectionWrapper, $filename, arr
             throw InvalidArgumentException::invalidType('"aliases" option', $options['aliases'], 'array of strings');
         }
 
-        if (isset($options['chunkSizeBytes']) && ! is_integer($options['chunkSizeBytes'])) {
+        if (! is_integer($options['chunkSizeBytes'])) {
             throw InvalidArgumentException::invalidType('"chunkSizeBytes" option', $options['chunkSizeBytes'], 'integer');
         }
 
-        if (isset($options['chunkSizeBytes']) && $options['chunkSizeBytes'] < 1) {
+        if ($options['chunkSizeBytes'] < 1) {
             throw new InvalidArgumentException(sprintf('Expected "chunkSizeBytes" option to be >= 1, %d given', $options['chunkSizeBytes']));
         }
 
-        if (isset($options['disableMD5']) && ! is_bool($options['disableMD5'])) {
+        if (! is_bool($options['disableMD5'])) {
             throw InvalidArgumentException::invalidType('"disableMD5" option', $options['disableMD5'], 'boolean');
         }
 
diff --git a/src/Operation/Watch.php b/src/Operation/Watch.php
@@ -178,10 +178,14 @@ public function __construct(Manager $manager, $databaseName, $collectionName, ar
             'readPreference' => new ReadPreference(ReadPreference::RP_PRIMARY),
         ];
 
-        if (isset($options['fullDocument']) && ! is_string($options['fullDocument'])) {
+        if (! is_string($options['fullDocument'])) {
             throw InvalidArgumentException::invalidType('"fullDocument" option', $options['fullDocument'], 'string');
         }
 
+        if (! $options['readPreference'] instanceof ReadPreference) {
+            throw InvalidArgumentException::invalidType('"readPreference" option', $options['readPreference'], ReadPreference::class);
+        }
+
         if (isset($options['resumeAfter']) && ! is_array($options['resumeAfter']) && ! is_object($options['resumeAfter'])) {
             throw InvalidArgumentException::invalidType('"resumeAfter" option', $options['resumeAfter'], 'array or object');
         }
diff --git a/tests/ClientTest.php b/tests/ClientTest.php
@@ -33,7 +33,7 @@ public function provideInvalidConstructorDriverOptions()
     {
         $options = [];
 
-        foreach ($this->getInvalidArrayValues() as $value) {
+        foreach ($this->getInvalidArrayValues(true) as $value) {
             $options[][] = ['typeMap' => $value];
         }
 
diff --git a/tests/GridFS/BucketFunctionalTest.php b/tests/GridFS/BucketFunctionalTest.php
@@ -62,15 +62,15 @@ public function provideInvalidConstructorOptions()
     {
         $options = [];
 
-        foreach ($this->getInvalidStringValues() as $value) {
+        foreach ($this->getInvalidStringValues(true) as $value) {
             $options[][] = ['bucketName' => $value];
         }
 
-        foreach ($this->getInvalidIntegerValues() as $value) {
+        foreach ($this->getInvalidIntegerValues(true) as $value) {
             $options[][] = ['chunkSizeBytes' => $value];
         }
 
-        foreach ($this->getInvalidBooleanValues() as $value) {
+        foreach ($this->getInvalidBooleanValues(true) as $value) {
             $options[][] = ['disableMD5' => $value];
         }
 
diff --git a/tests/GridFS/WritableStreamFunctionalTest.php b/tests/GridFS/WritableStreamFunctionalTest.php
@@ -50,11 +50,11 @@ public function provideInvalidConstructorOptions()
     {
         $options = [];
 
-        foreach ($this->getInvalidIntegerValues() as $value) {
+        foreach ($this->getInvalidIntegerValues(true) as $value) {
             $options[][] = ['chunkSizeBytes' => $value];
         }
 
-        foreach ($this->getInvalidBooleanValues() as $value) {
+        foreach ($this->getInvalidBooleanValues(true) as $value) {
             $options[][] = ['disableMD5' => $value];
         }
 
diff --git a/tests/Operation/AggregateTest.php b/tests/Operation/AggregateTest.php
@@ -27,7 +27,7 @@ public function provideInvalidConstructorOptions()
     {
         $options = [];
 
-        foreach ($this->getInvalidBooleanValues() as $value) {
+        foreach ($this->getInvalidBooleanValues(true) as $value) {
             $options[][] = ['allowDiskUse' => $value];
         }
 
@@ -79,7 +79,7 @@ public function provideInvalidConstructorOptions()
             $options[][] = ['typeMap' => $value];
         }
 
-        foreach ($this->getInvalidBooleanValues() as $value) {
+        foreach ($this->getInvalidBooleanValues(true) as $value) {
             $options[][] = ['useCursor' => $value];
         }
 
diff --git a/tests/Operation/BulkWriteTest.php b/tests/Operation/BulkWriteTest.php
@@ -412,7 +412,7 @@ public function provideInvalidConstructorOptions()
             $options[][] = ['bypassDocumentValidation' => $value];
         }
 
-        foreach ($this->getInvalidBooleanValues() as $value) {
+        foreach ($this->getInvalidBooleanValues(true) as $value) {
             $options[][] = ['ordered' => $value];
         }
 
diff --git a/tests/Operation/FindAndModifyTest.php b/tests/Operation/FindAndModifyTest.php
@@ -40,15 +40,15 @@ public function provideInvalidConstructorOptions()
             $options[][] = ['maxTimeMS' => $value];
         }
 
-        foreach ($this->getInvalidBooleanValues() as $value) {
+        foreach ($this->getInvalidBooleanValues(true) as $value) {
             $options[][] = ['new' => $value];
         }
 
         foreach ($this->getInvalidDocumentValues() as $value) {
             $options[][] = ['query' => $value];
         }
 
-        foreach ($this->getInvalidBooleanValues() as $value) {
+        foreach ($this->getInvalidBooleanValues(true) as $value) {
             $options[][] = ['remove' => $value];
         }
 
@@ -68,7 +68,7 @@ public function provideInvalidConstructorOptions()
             $options[][] = ['update' => $value];
         }
 
-        foreach ($this->getInvalidBooleanValues() as $value) {
+        foreach ($this->getInvalidBooleanValues(true) as $value) {
             $options[][] = ['upsert' => $value];
         }
 
diff --git a/tests/Operation/FindOneAndReplaceTest.php b/tests/Operation/FindOneAndReplaceTest.php
@@ -49,7 +49,7 @@ public function provideInvalidConstructorOptions()
             $options[][] = ['projection' => $value];
         }
 
-        foreach ($this->getInvalidIntegerValues() as $value) {
+        foreach ($this->getInvalidIntegerValues(true) as $value) {
             $options[][] = ['returnDocument' => $value];
         }
 
diff --git a/tests/Operation/FindOneAndUpdateTest.php b/tests/Operation/FindOneAndUpdateTest.php
@@ -49,7 +49,7 @@ public function provideInvalidConstructorOptions()
             $options[][] = ['projection' => $value];
         }
 
-        foreach ($this->getInvalidIntegerValues() as $value) {
+        foreach ($this->getInvalidIntegerValues(true) as $value) {
             $options[][] = ['returnDocument' => $value];
         }
 
diff --git a/tests/Operation/InsertManyTest.php b/tests/Operation/InsertManyTest.php
@@ -48,7 +48,7 @@ public function provideInvalidConstructorOptions()
             $options[][] = ['bypassDocumentValidation' => $value];
         }
 
-        foreach ($this->getInvalidBooleanValues() as $value) {
+        foreach ($this->getInvalidBooleanValues(true) as $value) {
             $options[][] = ['ordered' => $value];
         }
 
diff --git a/tests/Operation/UpdateTest.php b/tests/Operation/UpdateTest.php
@@ -52,15 +52,15 @@ public function provideInvalidConstructorOptions()
             $options[][] = ['collation' => $value];
         }
 
-        foreach ($this->getInvalidBooleanValues() as $value) {
+        foreach ($this->getInvalidBooleanValues(true) as $value) {
             $options[][] = ['multi' => $value];
         }
 
         foreach ($this->getInvalidSessionValues() as $value) {
             $options[][] = ['session' => $value];
         }
 
-        foreach ($this->getInvalidBooleanValues() as $value) {
+        foreach ($this->getInvalidBooleanValues(true) as $value) {
             $options[][] = ['upsert' => $value];
         }
 
diff --git a/tests/Operation/WatchTest.php b/tests/Operation/WatchTest.php
@@ -52,7 +52,7 @@ public function provideInvalidConstructorOptions()
             $options[][] = ['collation' => $value];
         }
 
-        foreach ($this->getInvalidStringValues() as $value) {
+        foreach ($this->getInvalidStringValues(true) as $value) {
             $options[][] = ['fullDocument' => $value];
         }
 
@@ -64,7 +64,7 @@ public function provideInvalidConstructorOptions()
             $options[][] = ['readConcern' => $value];
         }
 
-        foreach ($this->getInvalidReadPreferenceValues() as $value) {
+        foreach ($this->getInvalidReadPreferenceValues(true) as $value) {
             $options[][] = ['readPreference' => $value];
         }
 
diff --git a/tests/TestCase.php b/tests/TestCase.php
@@ -14,6 +14,7 @@
 use stdClass;
 use Traversable;
 use function array_map;
+use function array_merge;
 use function array_values;
 use function call_user_func;
 use function getenv;
@@ -165,91 +166,109 @@ protected function getDatabaseName()
     /**
      * Return a list of invalid array values.
      *
+     * @param boolean $includeNull
+     *
      * @return array
      */
-    protected function getInvalidArrayValues()
+    protected function getInvalidArrayValues($includeNull = false)
     {
-        return [123, 3.14, 'foo', true, new stdClass()];
+        return array_merge([123, 3.14, 'foo', true, new stdClass()], $includeNull ? [null] : []);
     }
 
     /**
      * Return a list of invalid boolean values.
      *
+     * @param boolean $includeNull
+     *
      * @return array
      */
-    protected function getInvalidBooleanValues()
+    protected function getInvalidBooleanValues($includeNull = false)
     {
-        return [123, 3.14, 'foo', [], new stdClass()];
+        return array_merge([123, 3.14, 'foo', [], new stdClass()], $includeNull ? [null] : []);
     }
 
     /**
      * Return a list of invalid document values.
      *
+     * @param boolean $includeNull
+     *
      * @return array
      */
-    protected function getInvalidDocumentValues()
+    protected function getInvalidDocumentValues($includeNull = false)
     {
-        return [123, 3.14, 'foo', true];
+        return array_merge([123, 3.14, 'foo', true], $includeNull ? [null] : []);
     }
 
     /**
      * Return a list of invalid integer values.
      *
+     * @param boolean $includeNull
+     *
      * @return array
      */
-    protected function getInvalidIntegerValues()
+    protected function getInvalidIntegerValues($includeNull = false)
     {
-        return [3.14, 'foo', true, [], new stdClass()];
+        return array_merge([3.14, 'foo', true, [], new stdClass()], $includeNull ? [null] : []);
     }
 
     /**
      * Return a list of invalid ReadPreference values.
      *
+     * @param boolean $includeNull
+     *
      * @return array
      */
-    protected function getInvalidReadConcernValues()
+    protected function getInvalidReadConcernValues($includeNull = false)
     {
-        return [123, 3.14, 'foo', true, [], new stdClass(), new ReadPreference(ReadPreference::RP_PRIMARY), new WriteConcern(1)];
+        return array_merge([123, 3.14, 'foo', true, [], new stdClass(), new ReadPreference(ReadPreference::RP_PRIMARY), new WriteConcern(1)], $includeNull ? [null] : []);
     }
 
     /**
      * Return a list of invalid ReadPreference values.
      *
+     * @param boolean $includeNull
+     *
      * @return array
      */
-    protected function getInvalidReadPreferenceValues()
+    protected function getInvalidReadPreferenceValues($includeNull = false)
     {
-        return [123, 3.14, 'foo', true, [], new stdClass(), new ReadConcern(), new WriteConcern(1)];
+        return array_merge([123, 3.14, 'foo', true, [], new stdClass(), new ReadConcern(), new WriteConcern(1)], $includeNull ? [null] : []);
     }
 
     /**
      * Return a list of invalid Session values.
      *
+     * @param boolean $includeNull
+     *
      * @return array
      */
-    protected function getInvalidSessionValues()
+    protected function getInvalidSessionValues($includeNull = false)
     {
-        return [123, 3.14, 'foo', true, [], new stdClass(), new ReadConcern(), new ReadPreference(ReadPreference::RP_PRIMARY), new WriteConcern(1)];
+        return array_merge([123, 3.14, 'foo', true, [], new stdClass(), new ReadConcern(), new ReadPreference(ReadPreference::RP_PRIMARY), new WriteConcern(1)], $includeNull ? [null] : []);
     }
 
     /**
      * Return a list of invalid string values.
      *
+     * @param boolean $includeNull
+     *
      * @return array
      */
-    protected function getInvalidStringValues()
+    protected function getInvalidStringValues($includeNull = false)
     {
-        return [123, 3.14, true, [], new stdClass()];
+        return array_merge([123, 3.14, true, [], new stdClass()], $includeNull ? [null] : []);
     }
 
     /**
      * Return a list of invalid WriteConcern values.
      *
+     * @param boolean $includeNull
+     *
      * @return array
      */
-    protected function getInvalidWriteConcernValues()
+    protected function getInvalidWriteConcernValues($includeNull = false)
     {
-        return [123, 3.14, 'foo', true, [], new stdClass(), new ReadConcern(), new ReadPreference(ReadPreference::RP_PRIMARY)];
+        return array_merge([123, 3.14, 'foo', true, [], new stdClass(), new ReadConcern(), new ReadPreference(ReadPreference::RP_PRIMARY)], $includeNull ? [null] : []);
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -95,7 +95,7 @@ public function __construct($uri = 'mongodb://127.0.0.1/', array $uriOptions = [`
`95`	`95`	`{`
`96`	`96`	`$driverOptions += ['typeMap' => self::$defaultTypeMap];`
`97`	`97`
`98`		`- if (isset($driverOptions['typeMap']) && ! is_array($driverOptions['typeMap'])) {`
	`98`	`+ if (! is_array($driverOptions['typeMap'])) {`
`99`	`99`	`throw InvalidArgumentException::invalidType('"typeMap" driver option', $driverOptions['typeMap'], 'array');`
`100`	`100`	`}`
`101`	`101`
Original file line number	Diff line number	Diff line change
`@@ -143,19 +143,19 @@ public function __construct(Manager $manager, $databaseName, array $options = []`
`143`	`143`	`'disableMD5' => false,`
`144`	`144`	`];`
`145`	`145`
`146`		`- if (isset($options['bucketName']) && ! is_string($options['bucketName'])) {`
	`146`	`+ if (! is_string($options['bucketName'])) {`
`147`	`147`	`throw InvalidArgumentException::invalidType('"bucketName" option', $options['bucketName'], 'string');`
`148`	`148`	`}`
`149`	`149`
`150`		`- if (isset($options['chunkSizeBytes']) && ! is_integer($options['chunkSizeBytes'])) {`
	`150`	`+ if (! is_integer($options['chunkSizeBytes'])) {`
`151`	`151`	`throw InvalidArgumentException::invalidType('"chunkSizeBytes" option', $options['chunkSizeBytes'], 'integer');`
`152`	`152`	`}`
`153`	`153`
`154`		`- if (isset($options['chunkSizeBytes']) && $options['chunkSizeBytes'] < 1) {`
	`154`	`+ if ($options['chunkSizeBytes'] < 1) {`
`155`	`155`	`throw new InvalidArgumentException(sprintf('Expected "chunkSizeBytes" option to be >= 1, %d given', $options['chunkSizeBytes']));`
`156`	`156`	`}`
`157`	`157`
`158`		`- if (isset($options['disableMD5']) && ! is_bool($options['disableMD5'])) {`
	`158`	`+ if (! is_bool($options['disableMD5'])) {`
`159`	`159`	`throw InvalidArgumentException::invalidType('"disableMD5" option', $options['disableMD5'], 'boolean');`
`160`	`160`	`}`
`161`	`161`
Original file line number	Diff line number	Diff line change
`@@ -114,15 +114,15 @@ public function __construct(CollectionWrapper $collectionWrapper, $filename, arr`
`114`	`114`	`throw InvalidArgumentException::invalidType('"aliases" option', $options['aliases'], 'array of strings');`
`115`	`115`	`}`
`116`	`116`
`117`		`- if (isset($options['chunkSizeBytes']) && ! is_integer($options['chunkSizeBytes'])) {`
	`117`	`+ if (! is_integer($options['chunkSizeBytes'])) {`
`118`	`118`	`throw InvalidArgumentException::invalidType('"chunkSizeBytes" option', $options['chunkSizeBytes'], 'integer');`
`119`	`119`	`}`
`120`	`120`
`121`		`- if (isset($options['chunkSizeBytes']) && $options['chunkSizeBytes'] < 1) {`
	`121`	`+ if ($options['chunkSizeBytes'] < 1) {`
`122`	`122`	`throw new InvalidArgumentException(sprintf('Expected "chunkSizeBytes" option to be >= 1, %d given', $options['chunkSizeBytes']));`
`123`	`123`	`}`
`124`	`124`
`125`		`- if (isset($options['disableMD5']) && ! is_bool($options['disableMD5'])) {`
	`125`	`+ if (! is_bool($options['disableMD5'])) {`
`126`	`126`	`throw InvalidArgumentException::invalidType('"disableMD5" option', $options['disableMD5'], 'boolean');`
`127`	`127`	`}`
`128`	`128`
Original file line number	Diff line number	Diff line change
`@@ -178,10 +178,14 @@ public function __construct(Manager $manager, $databaseName, $collectionName, ar`
`178`	`178`	`'readPreference' => new ReadPreference(ReadPreference::RP_PRIMARY),`
`179`	`179`	`];`
`180`	`180`
`181`		`- if (isset($options['fullDocument']) && ! is_string($options['fullDocument'])) {`
	`181`	`+ if (! is_string($options['fullDocument'])) {`
`182`	`182`	`throw InvalidArgumentException::invalidType('"fullDocument" option', $options['fullDocument'], 'string');`
`183`	`183`	`}`
`184`	`184`
	`185`	`+ if (! $options['readPreference'] instanceof ReadPreference) {`
	`186`	`+ throw InvalidArgumentException::invalidType('"readPreference" option', $options['readPreference'], ReadPreference::class);`
	`187`	`+ }`
	`188`	`+`
`185`	`189`	`if (isset($options['resumeAfter']) && ! is_array($options['resumeAfter']) && ! is_object($options['resumeAfter'])) {`
`186`	`190`	`throw InvalidArgumentException::invalidType('"resumeAfter" option', $options['resumeAfter'], 'array or object');`
`187`	`191`	`}`
Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ public function provideInvalidConstructorDriverOptions()`
`33`	`33`	`{`
`34`	`34`	`$options = [];`
`35`	`35`
`36`		`- foreach ($this->getInvalidArrayValues() as $value) {`
	`36`	`+ foreach ($this->getInvalidArrayValues(true) as $value) {`
`37`	`37`	`$options[][] = ['typeMap' => $value];`
`38`	`38`	`}`
`39`	`39`
Original file line number	Diff line number	Diff line change
`@@ -62,15 +62,15 @@ public function provideInvalidConstructorOptions()`
`62`	`62`	`{`
`63`	`63`	`$options = [];`
`64`	`64`
`65`		`- foreach ($this->getInvalidStringValues() as $value) {`
	`65`	`+ foreach ($this->getInvalidStringValues(true) as $value) {`
`66`	`66`	`$options[][] = ['bucketName' => $value];`
`67`	`67`	`}`
`68`	`68`
`69`		`- foreach ($this->getInvalidIntegerValues() as $value) {`
	`69`	`+ foreach ($this->getInvalidIntegerValues(true) as $value) {`
`70`	`70`	`$options[][] = ['chunkSizeBytes' => $value];`
`71`	`71`	`}`
`72`	`72`
`73`		`- foreach ($this->getInvalidBooleanValues() as $value) {`
	`73`	`+ foreach ($this->getInvalidBooleanValues(true) as $value) {`
`74`	`74`	`$options[][] = ['disableMD5' => $value];`
`75`	`75`	`}`
`76`	`76`
Original file line number	Diff line number	Diff line change
`@@ -50,11 +50,11 @@ public function provideInvalidConstructorOptions()`
`50`	`50`	`{`
`51`	`51`	`$options = [];`
`52`	`52`
`53`		`- foreach ($this->getInvalidIntegerValues() as $value) {`
	`53`	`+ foreach ($this->getInvalidIntegerValues(true) as $value) {`
`54`	`54`	`$options[][] = ['chunkSizeBytes' => $value];`
`55`	`55`	`}`
`56`	`56`
`57`		`- foreach ($this->getInvalidBooleanValues() as $value) {`
	`57`	`+ foreach ($this->getInvalidBooleanValues(true) as $value) {`
`58`	`58`	`$options[][] = ['disableMD5' => $value];`
`59`	`59`	`}`
`60`	`60`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ public function provideInvalidConstructorOptions()`
`27`	`27`	`{`
`28`	`28`	`$options = [];`
`29`	`29`
`30`		`- foreach ($this->getInvalidBooleanValues() as $value) {`
	`30`	`+ foreach ($this->getInvalidBooleanValues(true) as $value) {`
`31`	`31`	`$options[][] = ['allowDiskUse' => $value];`
`32`	`32`	`}`
`33`	`33`
`@@ -79,7 +79,7 @@ public function provideInvalidConstructorOptions()`
`79`	`79`	`$options[][] = ['typeMap' => $value];`
`80`	`80`	`}`
`81`	`81`
`82`		`- foreach ($this->getInvalidBooleanValues() as $value) {`
	`82`	`+ foreach ($this->getInvalidBooleanValues(true) as $value) {`
`83`	`83`	`$options[][] = ['useCursor' => $value];`
`84`	`84`	`}`
`85`	`85`
Original file line number	Diff line number	Diff line change
`@@ -412,7 +412,7 @@ public function provideInvalidConstructorOptions()`
`412`	`412`	`$options[][] = ['bypassDocumentValidation' => $value];`
`413`	`413`	`}`
`414`	`414`
`415`		`- foreach ($this->getInvalidBooleanValues() as $value) {`
	`415`	`+ foreach ($this->getInvalidBooleanValues(true) as $value) {`
`416`	`416`	`$options[][] = ['ordered' => $value];`
`417`	`417`	`}`
`418`	`418`
Original file line number	Diff line number	Diff line change
`@@ -40,15 +40,15 @@ public function provideInvalidConstructorOptions()`
`40`	`40`	`$options[][] = ['maxTimeMS' => $value];`
`41`	`41`	`}`
`42`	`42`
`43`		`- foreach ($this->getInvalidBooleanValues() as $value) {`
	`43`	`+ foreach ($this->getInvalidBooleanValues(true) as $value) {`
`44`	`44`	`$options[][] = ['new' => $value];`
`45`	`45`	`}`
`46`	`46`
`47`	`47`	`foreach ($this->getInvalidDocumentValues() as $value) {`
`48`	`48`	`$options[][] = ['query' => $value];`
`49`	`49`	`}`
`50`	`50`
`51`		`- foreach ($this->getInvalidBooleanValues() as $value) {`
	`51`	`+ foreach ($this->getInvalidBooleanValues(true) as $value) {`
`52`	`52`	`$options[][] = ['remove' => $value];`
`53`	`53`	`}`
`54`	`54`
`@@ -68,7 +68,7 @@ public function provideInvalidConstructorOptions()`
`68`	`68`	`$options[][] = ['update' => $value];`
`69`	`69`	`}`
`70`	`70`
`71`		`- foreach ($this->getInvalidBooleanValues() as $value) {`
	`71`	`+ foreach ($this->getInvalidBooleanValues(true) as $value) {`
`72`	`72`	`$options[][] = ['upsert' => $value];`
`73`	`73`	`}`
`74`	`74`