PHPLIB-597: Remove explicit dependency on PHPUnit in favour of phpunit-bridge

Author: alcaeus

.evergreen/run-tests.sh

@@ -26,10 +26,10 @@ PATH=/opt/php/${PHP_VERSION}-64bit/bin:$OLD_PATH
 case "$TESTS" in
    atlas-data-lake*)
       MONGODB_URI="mongodb://mhuser:pencil@127.0.0.1:27017"
-      php vendor/bin/phpunit --configuration phpunit.evergreen.xml --testsuite "Atlas Data Lake Test Suite"
+      php vendor/bin/simple-phpunit --configuration phpunit.evergreen.xml --testsuite "Atlas Data Lake Test Suite"
       ;;
 
    *)
-      php vendor/bin/phpunit --configuration phpunit.evergreen.xml
+      php vendor/bin/simple-phpunit --configuration phpunit.evergreen.xml
       ;;
 esac

.travis.yml

@@ -165,7 +165,7 @@ before_script:
 script:
   - export MONGODB_URI=`cat /tmp/uri.txt`
   - echo $MONGODB_URI
-  - vendor/bin/phpunit -v
+  - vendor/bin/simple-phpunit -v
 
 before_cache:
   - rm -f ${HOME}/.cache/pip/log/debug.log

CONTRIBUTING.md

@@ -20,13 +20,14 @@ Installation directions for Composer may be found in its
 
 ## Testing
 
-The library's test suite uses [PHPUnit](https://phpunit.de/), which should be
-installed as a development dependency by Composer.
+The library's test suite uses [PHPUnit](https://phpunit.de/), which is installed
+through the [PHPUnit Bridge](https://symfony.com/phpunit-bridge) dependency by
+Composer.
 
 The test suite may be executed with:
 
 ```
-$ vendor/bin/phpunit
+$ vendor/bin/simple-phpunit
 ```
 
 The `phpunit.xml.dist` file is used as the default configuration file for the

composer.json

@@ -16,10 +16,8 @@
         "jean85/pretty-package-versions": "^1.2"
     },
     "require-dev": {
-        "phpunit/phpunit": "^6.5",
-        "sebastian/comparator": "^2.0 || ^3.0",
         "squizlabs/php_codesniffer": "^3.5, <3.5.5",
-        "symfony/phpunit-bridge": "^4.4@dev"
+        "symfony/phpunit-bridge": "^5.2@beta"
     },
     "autoload": {
         "psr-4": { "MongoDB\\": "src/" },
@@ -29,11 +27,19 @@
         "psr-4": { "MongoDB\\Tests\\": "tests/" },
         "// Manually include assertion functions for PHPUnit 8.x and earlier ":"",
         "// See: https://github.com/sebastianbergmann/phpunit/issues/3746 ":"",
-        "files": [ "vendor/phpunit/phpunit/src/Framework/Assert/Functions.php" ]
+        "files": [
+            "vendor/bin/.phpunit/phpunit/src/Framework/Assert/Functions.php",
+            "tests/PHPUnit/FunctionsPolyfill.php"
+        ]
     },
     "extra": {
         "branch-alias": {
             "dev-master": "1.8.x-dev"
         }
+    },
+    "scripts": {
+        "// Ensure PHPUnit is installed through the PHPUnit bridge before ":"",
+        "// dumping the autoloader to avoid errors due to missing files.":"",
+        "pre-autoload-dump": "if test -f vendor/bin/simple-phpunit; then vendor/bin/simple-phpunit --version > /dev/null; fi"
     }
 }

phpcs.xml.dist

@@ -113,6 +113,6 @@
     </rule>
 
     <rule ref="PSR1.Classes.ClassDeclaration.MultipleClasses">
-        <exclude-pattern>/tests/Compat/PolyfillAssertTrait.php</exclude-pattern>
+        <exclude-pattern>/tests/PHPUnit/ConstraintTrait.php</exclude-pattern>
     </rule>
 </ruleset>

tests/Compat/PolyfillAssertTrait.php

@@ -721,7 +721,7 @@ public function testDownloadToStreamFails()
         $this->bucket->uploadFromStream('filename', $this->createStream('foo'), ['_id' => ['foo' => 'bar']]);
 
         $this->expectException(StreamException::class);
-        $this->expectExceptionMessageRegExp('#^Downloading file from "gridfs://.*/.*/.*" to "php://temp" failed. GridFS identifier: "{ "_id" : { "foo" : "bar" } }"$#');
+        $this->expectExceptionMessageMatches('#^Downloading file from "gridfs://.*/.*/.*" to "php://temp" failed. GridFS identifier: "{ "_id" : { "foo" : "bar" } }"$#');
         $this->bucket->downloadToStream(['foo' => 'bar'], fopen('php://temp', 'r'));
     }
 
@@ -730,7 +730,7 @@ public function testDownloadToStreamByNameFails()
         $this->bucket->uploadFromStream('filename', $this->createStream('foo'));
 
         $this->expectException(StreamException::class);
-        $this->expectExceptionMessageRegExp('#^Downloading file from "gridfs://.*/.*/.*" to "php://temp" failed. GridFS filename: "filename"$#');
+        $this->expectExceptionMessageMatches('#^Downloading file from "gridfs://.*/.*/.*" to "php://temp" failed. GridFS filename: "filename"$#');
         $this->bucket->downloadToStreamByName('filename', fopen('php://temp', 'r'));
     }
 
@@ -744,7 +744,7 @@ public function testUploadFromStreamFails()
         $source = fopen('unusable://temp', 'w');
 
         $this->expectException(StreamException::class);
-        $this->expectExceptionMessageRegExp('#^Uploading file from "unusable://temp" to "gridfs://.*/.*/.*" failed. GridFS filename: "filename"$#');
+        $this->expectExceptionMessageMatches('#^Uploading file from "unusable://temp" to "gridfs://.*/.*/.*" failed. GridFS filename: "filename"$#');
         $this->bucket->uploadFromStream('filename', $source);
     }
 

tests/GridFS/BucketFunctionalTest.php

@@ -171,7 +171,7 @@ public function testUnacknowledgedWriteConcern()
     public function testUnacknowledgedWriteConcernAccessesDeletedCount(BulkWriteResult $result)
     {
         $this->expectException(BadMethodCallException::class);
-        $this->expectExceptionMessageRegExp('/[\w:\\\\]+ should not be called for an unacknowledged write result/');
+        $this->expectExceptionMessageMatches('/[\w:\\\\]+ should not be called for an unacknowledged write result/');
         $result->getDeletedCount();
     }
 
@@ -181,7 +181,7 @@ public function testUnacknowledgedWriteConcernAccessesDeletedCount(BulkWriteResu
     public function testUnacknowledgedWriteConcernAccessesInsertCount(BulkWriteResult $result)
     {
         $this->expectException(BadMethodCallException::class);
-        $this->expectExceptionMessageRegExp('/[\w:\\\\]+ should not be called for an unacknowledged write result/');
+        $this->expectExceptionMessageMatches('/[\w:\\\\]+ should not be called for an unacknowledged write result/');
         $result->getInsertedCount();
     }
 
@@ -191,7 +191,7 @@ public function testUnacknowledgedWriteConcernAccessesInsertCount(BulkWriteResul
     public function testUnacknowledgedWriteConcernAccessesMatchedCount(BulkWriteResult $result)
     {
         $this->expectException(BadMethodCallException::class);
-        $this->expectExceptionMessageRegExp('/[\w:\\\\]+ should not be called for an unacknowledged write result/');
+        $this->expectExceptionMessageMatches('/[\w:\\\\]+ should not be called for an unacknowledged write result/');
         $result->getMatchedCount();
     }
 
@@ -201,7 +201,7 @@ public function testUnacknowledgedWriteConcernAccessesMatchedCount(BulkWriteResu
     public function testUnacknowledgedWriteConcernAccessesModifiedCount(BulkWriteResult $result)
     {
         $this->expectException(BadMethodCallException::class);
-        $this->expectExceptionMessageRegExp('/[\w:\\\\]+ should not be called for an unacknowledged write result/');
+        $this->expectExceptionMessageMatches('/[\w:\\\\]+ should not be called for an unacknowledged write result/');
         $result->getModifiedCount();
     }
 
@@ -211,7 +211,7 @@ public function testUnacknowledgedWriteConcernAccessesModifiedCount(BulkWriteRes
     public function testUnacknowledgedWriteConcernAccessesUpsertedCount(BulkWriteResult $result)
     {
         $this->expectException(BadMethodCallException::class);
-        $this->expectExceptionMessageRegExp('/[\w:\\\\]+ should not be called for an unacknowledged write result/');
+        $this->expectExceptionMessageMatches('/[\w:\\\\]+ should not be called for an unacknowledged write result/');
         $result->getUpsertedCount();
     }
 
@@ -221,7 +221,7 @@ public function testUnacknowledgedWriteConcernAccessesUpsertedCount(BulkWriteRes
     public function testUnacknowledgedWriteConcernAccessesUpsertedIds(BulkWriteResult $result)
     {
         $this->expectException(BadMethodCallException::class);
-        $this->expectExceptionMessageRegExp('/[\w:\\\\]+ should not be called for an unacknowledged write result/');
+        $this->expectExceptionMessageMatches('/[\w:\\\\]+ should not be called for an unacknowledged write result/');
         $result->getUpsertedIds();
     }