feat: drop support for php < 8.1, drop support for symfony < 5.4 (WIP)

.github/workflows/static.yml

@@ -15,9 +15,6 @@ jobs:
       - name: Checkout code
         uses: actions/checkout@v4
 
-      - name: Remove phpspec
-        run: composer remove --dev friends-of-phpspec/phpspec-code-coverage phpspec/phpspec
-
       - name: PHPStan
         uses: docker://oskarstark/phpstan-ga
         env:

.github/workflows/tests.yml

@@ -2,8 +2,8 @@ name: tests
 
 on:
   push:
-      branches:
-          - '*.x'
+    branches:
+      - '*.x'
   pull_request:
 
 jobs:
@@ -12,7 +12,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        php: ['7.1', '7.2', '7.3', '7.4', '8.0', '8.1', '8.2', '8.3']
+        php: ['8.1', '8.2', '8.3']
 
     steps:
       - name: Checkout code
@@ -36,7 +36,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        php: ['7.1', '7.4', '8.2', '8.3']
+        php: ['8.1']
 
     steps:
       - name: Checkout code
@@ -61,10 +61,8 @@ jobs:
     strategy:
       matrix:
         include:
-          - symfony: '4.4.*'
-            php-version: '7.1'
           - symfony: '5.4.*'
-            php-version: '7.4'
+            php-version: '8.1'
           - symfony: '6.4.*'
             php-version: '8.2'
           - symfony: '7.0.*'
@@ -103,7 +101,7 @@ jobs:
       - name: Setup PHP
         uses: shivammathur/setup-php@v2
         with:
-          php-version: 7.4
+          php-version: 8.1
           tools: composer:v2
           coverage: xdebug
 

.gitignore

@@ -9,3 +9,4 @@
 /vendor/
 
 .phpunit.result.cache
+.phpunit.cache

CHANGELOG.md

@@ -2,6 +2,8 @@
 
 ## unreleased
 
+- drop support for php < 8.1
+- drop support for symfony < 5.4
 - Cleaned up phpdoc.
 
 ## 2.7.1 - 2023-11-30

composer.json

@@ -11,22 +11,22 @@
         }
     ],
     "require": {
-        "php": "^7.1 || ^8.0",
-        "php-http/httplug": "^2.0",
+        "php": "^8.1",
+        "php-http/httplug": "^2.4",
         "php-http/message": "^1.6",
         "psr/http-client": "^1.0",
         "psr/http-factory": "^1.0",
-        "psr/http-message": "^1.0 || ^2.0",
-        "symfony/options-resolver": "~4.0.15 || ~4.1.9 || ^4.2.1 || ^5.0 || ^6.0 || ^7.0",
-        "symfony/polyfill-php80": "^1.17"
+        "psr/http-message": "^2.0",
+        "symfony/options-resolver": "^5.4 || ^6.0 || ^7.0"
     },
     "require-dev": {
-        "doctrine/instantiator": "^1.1",
-        "guzzlehttp/psr7": "^1.4",
-        "nyholm/psr7": "^1.2",
-        "phpspec/phpspec": "^5.1 || ^6.3 || ^7.1",
-        "phpspec/prophecy": "^1.10.2",
-        "phpunit/phpunit": "^7.5.20 || ^8.5.33 || ^9.6.7"
+        "doctrine/instantiator": "^2.0",
+        "guzzlehttp/psr7": "^2.6",
+        "nyholm/psr7": "^1.8",
+        "phpspec/phpspec": "^7.5",
+        "phpspec/prophecy": "^1.18",
+        "phpunit/phpunit": "^10.5.8",
+        "phpspec/prophecy-phpunit": "^2.1"
     },
     "suggest": {
         "ext-json": "To detect JSON responses with the ContentTypePlugin",

phpunit.xml.dist

@@ -1,14 +1,15 @@
 <?xml version="1.0" encoding="UTF-8"?>
-
-<phpunit bootstrap="./vendor/autoload.php"
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         bootstrap="./vendor/autoload.php"
          colors="true"
-         convertErrorsToExceptions="true"
-         convertNoticesToExceptions="true"
-         convertWarningsToExceptions="true">
+         xsi:noNamespaceSchemaLocation="vendor/phpunit/phpunit/phpunit.xsd"
+         cacheDirectory=".phpunit.cache"
+>
+
+  <testsuites>
+    <testsuite name="HTTPlug unit tests">
+      <directory>./tests</directory>
+    </testsuite>
+  </testsuites>
 
-    <testsuites>
-        <testsuite name="HTTPlug unit tests">
-            <directory suffix="Test.php">./tests</directory>
-        </testsuite>
-    </testsuites>
 </phpunit>

tests/Plugin/RedirectPluginTest.php

@@ -78,7 +78,7 @@ function (RequestInterface $request) {
         $this->assertSame('https://example.com/other', $response->getHeaderLine('uri'));
     }
 
-    public function provideRedirections(): array
+    public static function provideRedirections(): array
     {
         return [
             'no path on target' => ['https://example.com/path?query=value', 'https://example.com?query=value', 'https://example.com?query=value'],

tests/PluginChainTest.php

@@ -9,10 +9,13 @@
 use Http\Client\Common\PluginChain;
 use Http\Promise\Promise;
 use PHPUnit\Framework\TestCase;
+use Prophecy\PhpUnit\ProphecyTrait;
 use Psr\Http\Message\RequestInterface;
 
 class PluginChainTest extends TestCase
 {
+    use ProphecyTrait;
+
     private function createPlugin(callable $func): Plugin
     {
         return new class($func) implements Plugin {

tests/PluginClientBuilderTest.php

@@ -10,9 +10,12 @@
 use Http\Client\HttpAsyncClient;
 use Http\Client\HttpClient;
 use PHPUnit\Framework\TestCase;
+use Prophecy\PhpUnit\ProphecyTrait;
 
 class PluginClientBuilderTest extends TestCase
 {
+    use ProphecyTrait;
+
     /** @dataProvider clientProvider */
     public function testPriority(string $client): void
     {
@@ -71,7 +74,7 @@ function (): array {
         $this->assertSame(5, $options['max_restarts']);
     }
 
-    public function clientProvider(): iterable
+    public static function clientProvider(): iterable
     {
         yield 'sync\'d http client' => [HttpClient::class];
         yield 'async\'d http client' => [HttpAsyncClient::class];

tests/PluginClientTest.php

@@ -31,7 +31,7 @@ public function testRestartChain(PluginClient $client, string $method, string $r
         $this->assertInstanceOf($returnType, $result);
     }
 
-    public function clientAndMethodProvider()
+    public static function clientAndMethodProvider()
     {
         $syncClient = new class() implements ClientInterface {
             public function sendRequest(RequestInterface $request): ResponseInterface