Add support for PHPUnit10 and remove support for PHPUnit8

.gitattributes

@@ -13,3 +13,4 @@
 /phpunit.xml.dist export-ignore
 /spec/            export-ignore
 /tests/           export-ignore
+/.phpunit.cache/  export-ignore

.github/workflows/guzzle.yml

@@ -11,7 +11,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php: [ '7.2', '7.3', '7.4', '8.0', '8.1', '8.2' ]
+        php: [ '7.3', '7.4', '8.0', '8.1', '8.2' ]
     uses: ./.github/workflows/integration.yml
     with:
       php: ${{ matrix.php }}

.github/workflows/laminas-legacy.yml

@@ -11,7 +11,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php: [ '7.2', '7.3', '7.4' ]
+        php: [ '7.3', '7.4' ]
     uses: ./.github/workflows/integration.yml
     with:
       php: ${{ matrix.php }}

.github/workflows/nyholm.yml

@@ -11,7 +11,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php: [ '7.2', '7.3', '7.4', '8.0', '8.1', '8.2' ]
+        php: [ '7.3', '7.4', '8.0', '8.1', '8.2' ]
     uses: ./.github/workflows/integration.yml
     with:
       php: ${{ matrix.php }}

.github/workflows/ringcentral.yml

@@ -11,7 +11,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php: [ '7.2', '7.3', '7.4', '8.0', '8.1', '8.2' ]
+        php: [ '7.3', '7.4', '8.0', '8.1', '8.2' ]
     uses: ./.github/workflows/integration.yml
     with:
       php: ${{ matrix.php }}

.gitignore

@@ -1,5 +1,6 @@
 /.tmp
 /.phpunit.result.cache
+/.phpunit.cache
 /behat.yml
 /build/
 /composer.lock

composer.json

@@ -14,9 +14,9 @@
         }
     ],
     "require": {
-        "php": "^7.2 || ^8.0",
-        "phpunit/phpunit": "^8.0 || ^9.3",
-        "psr/http-message": "^1.0 || ^2.0"
+        "php": "^7.3 || ^8.0",
+        "phpunit/phpunit": "^9.3 || ^10.0",
+        "psr/http-message": "^1.0"
     },
     "require-dev": {
         "guzzlehttp/psr7": "^1.7 || ^2.0",

phpunit.xml.dist

@@ -1,44 +1,39 @@
 <?xml version="1.0" encoding="UTF-8"?>
-
-<phpunit backupGlobals="false"
-  backupStaticAttributes="true"
-  colors="true"
-  convertErrorsToExceptions="true"
-  convertNoticesToExceptions="true"
-  convertWarningsToExceptions="true"
-  processIsolation="false"
-  stopOnFailure="false"
-  bootstrap="vendor/autoload.php"
-  >
+<phpunit
+    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+    backupGlobals="false"
+    colors="true"
+    processIsolation="false"
+    stopOnFailure="false"
+    bootstrap="vendor/autoload.php"
+    xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.0/phpunit.xsd"
+    cacheDirectory=".phpunit.cache"
+    backupStaticProperties="true"
+>
+  <coverage>
+    <include>
+      <directory>./</directory>
+    </include>
+    <exclude>
+      <directory>./Tests</directory>
+      <directory>./vendor</directory>
+    </exclude>
+  </coverage>
   <testsuites>
     <testsuite name="Guzzle">
       <directory>./tests/Guzzle/</directory>
     </testsuite>
-
     <testsuite name="RingCentral">
       <directory>./tests/RingCentral/</directory>
     </testsuite>
-
     <testsuite name="Slim">
-        <directory>./tests/Slim/</directory>
+      <directory>./tests/Slim/</directory>
     </testsuite>
-
     <testsuite name="Laminas">
-        <directory>./tests/Laminas/</directory>
+      <directory>./tests/Laminas/</directory>
     </testsuite>
-
     <testsuite name="Nyholm">
-        <directory>./vendor/nyholm/psr7/tests/Integration/</directory>
+      <directory>./vendor/nyholm/psr7/tests/Integration/</directory>
     </testsuite>
   </testsuites>
-
-  <filter>
-    <whitelist>
-      <directory>./</directory>
-      <exclude>
-        <directory>./Tests</directory>
-        <directory>./vendor</directory>
-      </exclude>
-    </whitelist>
-  </filter>
 </phpunit>

src/BaseTest.php

@@ -5,12 +5,12 @@
 use GuzzleHttp\Psr7\Stream as GuzzleStream;
 use GuzzleHttp\Psr7\UploadedFile as GuzzleUploadedFile;
 use GuzzleHttp\Psr7\Uri as GuzzleUri;
+use GuzzleHttp\Psr7\Utils as GuzzleUtils;
 use Laminas\Diactoros\StreamFactory as LaminasStreamFactory;
 use Laminas\Diactoros\Uri as LaminasUri;
 use Laminas\Diactoros\UploadedFile as LaminasUploadedFile;
 use Nyholm\Psr7\Factory\Psr17Factory as NyholmFactory;
 use PHPUnit\Framework\TestCase;
-use Psr\Http\Message\UriInterface;
 use RingCentral\Psr7\Uri as RingCentralUri;
 use function RingCentral\Psr7\stream_for as ring_central_stream_for;
 use Slim\Psr7\Uri as SlimUri;
@@ -33,11 +33,11 @@ protected function buildUri($uri)
         if (defined('URI_FACTORY')) {
             $factoryClass = URI_FACTORY;
             $factory = new $factoryClass();
-            if ($factory instanceof \Http\Message\UriFactory) {
+            if ($factory instanceof PsrUriFactory) {
                 return $factory->createUri($uri);
             }
-            if ($factory instanceof \Psr\Http\Message\UriFactoryInterface) {
-                if ($uri instanceof UriInterface) {
+            if ($factory instanceof PsrUriFactoryInterface) {
+                if ($uri instanceof PsrUriInterface) {
                     return $uri;
                 }
 
@@ -79,10 +79,10 @@ protected function buildStream($data)
         if (defined('STREAM_FACTORY')) {
             $factoryClass = STREAM_FACTORY;
             $factory = new $factoryClass();
-            if ($factory instanceof \Http\Message\StreamFactory) {
+            if ($factory instanceof HttpPlugStreamFactory) {
                 return $factory->createStream($data);
             }
-            if ($factory instanceof \Psr\Http\Message\StreamFactoryInterface) {
+            if ($factory instanceof PsrStreamFactoryInterface) {
                 if (is_string($data)) {
                     return $factory->createStream($data);
                 }
@@ -94,7 +94,7 @@ protected function buildStream($data)
         }
 
         if (class_exists(GuzzleStream::class)) {
-            return \GuzzleHttp\Psr7\Utils::streamFor($data);
+            return GuzzleUtils::streamFor($data);
         }
 
         $factory = null;
@@ -127,7 +127,7 @@ protected function buildUploadableFile($data)
         if (defined('UPLOADED_FILE_FACTORY')) {
             $factoryClass = UPLOADED_FILE_FACTORY;
             $factory = new $factoryClass();
-            if (!$factory instanceof \Psr\Http\Message\UploadedFileFactoryInterface) {
+            if (!$factory instanceof PsrUploadedFileFactoryInterface) {
                 throw new \RuntimeException('Constant "UPLOADED_FILE_FACTORY" must be a reference to a Psr\Http\Message\UploadedFileFactoryInterface');
             }
 

src/MessageTrait.php

@@ -160,7 +160,7 @@ public function testWithHeaderInvalidArguments($name, $value)
         }
     }
 
-    public function getInvalidHeaderArguments()
+    public static function getInvalidHeaderArguments()
     {
         return [
             [[], 'foo'],
@@ -288,11 +288,6 @@ public function testBody()
 
     private function assertMatchesRegexp(string $pattern, string $string, string $message = ''): void
     {
-        // @TODO remove when package require phpunit 9.1
-        if (function_exists('PHPUnit\Framework\assertMatchesRegularExpression')) {
-            $this->assertMatchesRegularExpression($pattern, $string, $message);
-        } else {
-            $this->assertRegExp($pattern, $string, $message);
-        }
+        $this->assertMatchesRegularExpression($pattern, $string, $message);
     }
 }

src/RequestIntegrationTest.php

@@ -118,7 +118,7 @@ public function testMethodWithInvalidArguments($method)
         }
     }
 
-    public function getInvalidMethods()
+    public static function getInvalidMethods()
     {
         return [
             'null' => [null],

src/ResponseIntegrationTest.php

@@ -80,7 +80,7 @@ public function testStatusCodeInvalidArgument($statusCode)
         }
     }
 
-    public function getInvalidStatusCodeArguments()
+    public static function getInvalidStatusCodeArguments()
     {
         return [
             'true' => [true],

src/ServerRequestIntegrationTest.php

@@ -99,7 +99,7 @@ public function testGetParsedBody($value)
         $this->assertEquals($value, $new->getParsedBody());
     }
 
-    public function validParsedBodyParams()
+    public static function validParsedBodyParams()
     {
         return [
             [null],
@@ -125,7 +125,7 @@ public function testGetParsedBodyInvalid($value)
         }
     }
 
-    public function invalidParsedBodyParams()
+    public static function invalidParsedBodyParams()
     {
         return [
             [4711],

src/UploadedFileIntegrationTest.php

@@ -102,13 +102,8 @@ public function testGetSize()
 
         $file = $this->createSubject();
         $size = $file->getSize();
-        if (false !== $size) {
-            // @TODO remove when package require phpunit 9.1
-            if (function_exists('PHPUnit\Framework\assertMatchesRegularExpression')) {
-                $this->assertMatchesRegularExpression('|^[0-9]+$|', (string) $size);
-            } else {
-                $this->assertRegExp('|^[0-9]+$|', (string) $size);
-            }
+        if ($size) {
+            $this->assertMatchesRegularExpression('|^[0-9]+$|', (string) $size);
         } else {
             $this->assertNull($size);
         }

src/UriIntegrationTest.php

@@ -69,7 +69,7 @@ public function testWithSchemeInvalidArguments($schema)
         }
     }
 
-    public function getInvalidSchemaArguments()
+    public static function getInvalidSchemaArguments()
     {
         return [
             [true],
@@ -155,69 +155,69 @@ public function testPort()
     /**
      * @dataProvider getPaths
      */
-    public function testPath(UriInterface $uri, $expected)
+    public function testPath(string $uri, string $expected)
     {
         if (isset($this->skippedTests[__FUNCTION__])) {
             $this->markTestSkipped($this->skippedTests[__FUNCTION__]);
         }
 
-        $this->assertSame($expected, $uri->getPath());
+        $this->assertSame($expected, $this->createUri($uri)->getPath());
     }
 
-    public function getPaths()
+    public static function getPaths()
     {
         return [
-            [$this->createUri('http://www.foo.com/'), '/'],
-            [$this->createUri('http://www.foo.com'), ''],
-            [$this->createUri('foo/bar'), 'foo/bar'],
-            [$this->createUri('http://www.foo.com/foo bar'), '/foo%20bar'],
-            [$this->createUri('http://www.foo.com/foo%20bar'), '/foo%20bar'],
-            [$this->createUri('http://www.foo.com/foo%2fbar'), '/foo%2fbar'],
+            ['http://www.foo.com/', '/'],
+            ['http://www.foo.com', ''],
+            ['foo/bar', 'foo/bar'],
+            ['http://www.foo.com/foo bar', '/foo%20bar'],
+            ['http://www.foo.com/foo%20bar', '/foo%20bar'],
+            ['http://www.foo.com/foo%2fbar', '/foo%2fbar'],
         ];
     }
 
     /**
      * @dataProvider getQueries
      */
-    public function testQuery(UriInterface $uri, $expected)
+    public function testQuery(string $uri, string $expected)
     {
         if (isset($this->skippedTests[__FUNCTION__])) {
             $this->markTestSkipped($this->skippedTests[__FUNCTION__]);
         }
 
-        $this->assertSame($expected, $uri->getQuery());
+        $this->assertSame($expected, $this->createUri($uri)->getQuery());
     }
 
-    public function getQueries()
+    public static function getQueries()
     {
         return [
-            [$this->createUri('http://www.foo.com'), ''],
-            [$this->createUri('http://www.foo.com?'), ''],
-            [$this->createUri('http://www.foo.com?foo=bar'), 'foo=bar'],
-            [$this->createUri('http://www.foo.com?foo=bar%26baz'), 'foo=bar%26baz'],
-            [$this->createUri('http://www.foo.com?foo=bar&baz=biz'), 'foo=bar&baz=biz'],
+            ['http://www.foo.com', ''],
+            ['http://www.foo.com?', ''],
+            ['http://www.foo.com?foo=bar', 'foo=bar'],
+            ['http://www.foo.com?foo=bar%26baz', 'foo=bar%26baz'],
+            ['http://www.foo.com?foo=bar&baz=biz', 'foo=bar&baz=biz'],
         ];
     }
 
     /**
      * @dataProvider getFragments
      */
-    public function testFragment(UriInterface $uri, $expected)
+    public function testFragment(string $uri, string $expected)
     {
         if (isset($this->skippedTests[__FUNCTION__])) {
             $this->markTestSkipped($this->skippedTests[__FUNCTION__]);
         }
 
-        $this->assertEquals($expected, $uri->getFragment());
+        $this->assertEquals($expected, $this->createUri($uri)->getFragment());
     }
 
-    public function getFragments()
+    public static function getFragments()
     {
         return [
-            [$this->createUri('http://www.foo.com'), ''],
-            [$this->createUri('http://www.foo.com#'), ''],
-            [$this->createUri('http://www.foo.com#foo'), 'foo'],
-            [$this->createUri('http://www.foo.com#foo%20bar'), 'foo%20bar'],
+            ['http://www.foo.com', ''],
+            ['http://www.foo.com#', ''],
+            ['http://www.foo.com#foo', 'foo'],
+            ['http://www.foo.com#foo%20bar', 'foo%20bar'],
         ];
     }
 

tests/Guzzle/RequestTest.php

@@ -16,7 +16,7 @@ public function createSubject()
         return new Request('GET', '/');
     }
 
-    public function getInvalidHeaderArguments()
+    public static function getInvalidHeaderArguments()
     {
         $testCases = parent::getInvalidHeaderArguments();
 

tests/Guzzle/ResponseTest.php

@@ -12,7 +12,7 @@ public function createSubject()
         return new Response();
     }
 
-    public function getInvalidHeaderArguments()
+    public static function getInvalidHeaderArguments()
     {
         $testCases = parent::getInvalidHeaderArguments();