adjust for symfony 5

Author: dbu

.travis.yml

@@ -10,7 +10,7 @@ cache:
 env:
     global:
         - TEST_COMMAND="composer test"
-        - SYMFONY_PHPUNIT_VERSION="6.5"
+        - SYMFONY_PHPUNIT_VERSION="8"
         - SYMFONY_PHPUNIT_DIR="$HOME/symfony-bridge/.phpunit"
 
 branches:
@@ -25,7 +25,6 @@ matrix:
           env: COMPOSER_FLAGS="--prefer-stable --prefer-lowest" SYMFONY_DEPRECATIONS_HELPER="weak"
 
           # Test the latest stable release
-        - php: 7.1
         - php: 7.2
         - php: 7.3
           env: COVERAGE=true TEST_COMMAND="composer test-ci" DEPENDENCIES="php-http/vcr-plugin:^1.0@dev"

composer.json

@@ -25,7 +25,7 @@
         }
     ],
     "require": {
-        "php": "^7.1",
+        "php": "^7.2",
         "php-http/client-common": "^1.9 || ^2.0",
         "php-http/client-implementation": "^1.0",
         "php-http/discovery": "^1.0",
@@ -47,7 +47,7 @@
     },
     "require-dev": {
         "guzzlehttp/psr7": "^1.0",
-        "matthiasnoback/symfony-dependency-injection-test": "^3.0",
+        "matthiasnoback/symfony-dependency-injection-test": "^4.0",
         "nyholm/nsa": "^1.1",
         "php-http/cache-plugin": "^1.6",
         "php-http/guzzle6-adapter": "^1.1.1 || ^2.0.1",

src/Collector/Collector.php

@@ -2,10 +2,11 @@
 
 namespace Http\HttplugBundle\Collector;
 
-use Exception;
+use Symfony\Component\HttpKernel\Kernel;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
 use Symfony\Component\HttpKernel\DataCollector\DataCollector;
+use Throwable;
 
 /**
  * The Collector hold profiled Stacks pushed by StackPlugin. It also have a list of configured clients.
@@ -18,7 +19,8 @@
  *
  * @internal
  */
-class Collector extends DataCollector
+// Make this class non-abstract and move collect into it when we drop support for Symfony 4.
+abstract class BaseCollector extends DataCollector
 {
     /**
      * @var Stack|null
@@ -30,14 +32,6 @@ public function __construct()
         $this->reset();
     }
 
-    /**
-     * {@inheritdoc}
-     */
-    public function collect(Request $request, Response $response, Exception $exception = null)
-    {
-        // We do not need to collect any data from the Symfony Request and Response
-    }
-
     /**
      * {@inheritdoc}
      */
@@ -199,3 +193,27 @@ public function getTotalDuration()
         }, 0);
     }
 }
+
+if (Kernel::MAJOR_VERSION >= 5) {
+    class Collector extends BaseCollector
+    {
+        /**
+         * {@inheritdoc}
+         */
+        public function collect(Request $request, Response $response, Throwable $exception = null)
+        {
+            // We do not need to collect any data from the Symfony Request and Response
+        }
+    }
+} else {
+    class Collector extends BaseCollector
+    {
+        /**
+         * {@inheritdoc}
+         */
+        public function collect(Request $request, Response $response, \Exception $exception = null)
+        {
+            // We do not need to collect any data from the Symfony Request and Response
+        }
+    }
+}

src/Collector/PluginClientFactoryListener.php

@@ -3,8 +3,16 @@
 namespace Http\HttplugBundle\Collector;
 
 use Http\Client\Common\PluginClientFactory as DefaultPluginClientFactory;
-use Symfony\Component\EventDispatcher\Event;
+use Symfony\Component\EventDispatcher\Event as LegacyEvent;
 use Symfony\Component\EventDispatcher\EventSubscriberInterface;
+use Symfony\Component\HttpKernel\Kernel;
+use Symfony\Contracts\EventDispatcher\Event;
+
+if (Kernel::MAJOR_VERSION >= 5) {
+    \class_alias(Event::class, 'Http\HttplugBundle\Collector\PluginClientFactoryListenerEventClass');
+} else {
+    \class_alias(LegacyEvent::class, 'Http\HttplugBundle\Collector\PluginClientFactoryListenerEventClass');
+}
 
 /**
  * This subscriber ensures that every PluginClient created when using Http\Client\Common\PluginClientFactory without
@@ -32,10 +40,8 @@ public function __construct(PluginClientFactory $factory)
 
     /**
      * Make sure to profile clients created using PluginClientFactory.
-     *
-     * @param Event $e
      */
-    public function onEvent(Event $e)
+    public function onEvent(PluginClientFactoryListenerEventClass $e)
     {
         DefaultPluginClientFactory::setFactory([$this->factory, 'createClient']);
     }

src/Discovery/ConfiguredClientsStrategy.php

@@ -6,8 +6,16 @@
 use Http\Client\HttpAsyncClient;
 use Http\Discovery\HttpClientDiscovery;
 use Http\Discovery\Strategy\DiscoveryStrategy;
-use Symfony\Component\EventDispatcher\Event;
+use Symfony\Component\EventDispatcher\Event as LegacyEvent;
 use Symfony\Component\EventDispatcher\EventSubscriberInterface;
+use Symfony\Component\HttpKernel\Kernel;
+use Symfony\Contracts\EventDispatcher\Event;
+
+if (Kernel::MAJOR_VERSION >= 5) {
+    \class_alias(Event::class, 'Http\HttplugBundle\Discovery\ConfiguredClientsStrategyEventClass');
+} else {
+    \class_alias(LegacyEvent::class, 'Http\HttplugBundle\Discovery\ConfiguredClientsStrategyEventClass');
+}
 
 /**
  * A strategy that provide clients configured with HTTPlug bundle. With help from this strategy
@@ -60,10 +68,8 @@ public static function getCandidates($type)
 
     /**
      * Make sure to use our custom strategy.
-     *
-     * @param Event $e
      */
-    public function onEvent(Event $e)
+    public function onEvent(ConfiguredClientsStrategyEventClass $e)
     {
         HttpClientDiscovery::prependStrategy(self::class);
     }

tests/Functional/DiscoveredClientsTest.php

@@ -11,7 +11,9 @@
 use Http\HttplugBundle\Discovery\ConfiguredClientsStrategy;
 use Nyholm\NSA;
 use Symfony\Bundle\FrameworkBundle\Test\WebTestCase;
-use Symfony\Component\EventDispatcher\Event;
+use Symfony\Component\HttpKernel\Kernel;
+use Symfony\Contracts\EventDispatcher\Event;
+use Symfony\Component\EventDispatcher\Event as LegacyEvent;
 
 class DiscoveredClientsTest extends WebTestCase
 {
@@ -124,8 +126,9 @@ protected function setUp(): void
         parent::setUp();
 
         // Reset values
-        $strategy = new ConfiguredClientsStrategy(null, null, null);
+        $strategy = new ConfiguredClientsStrategy(null, null);
         HttpClientDiscovery::setStrategies([CommonClassesStrategy::class]);
-        $strategy->onEvent(new Event());
+        $class = (Kernel::MAJOR_VERSION >= 5) ? Event::class : LegacyEvent::class;
+        $strategy->onEvent(new $class());
     }
 }

tests/Functional/DiscoveryTest.php

@@ -30,7 +30,7 @@ protected function setUp(): void
     /**
      * {@inheritdoc}
      */
-    protected function getContainerExtensions()
+    protected function getContainerExtensions(): array
     {
         return [
             new HttplugExtension(),

tests/Functional/ServiceInstantiationTest.php

@@ -17,6 +17,7 @@
 use Symfony\Component\EventDispatcher\EventDispatcherInterface;
 use Symfony\Component\HttpFoundation\Request as SymfonyRequest;
 use Symfony\Component\HttpKernel\Event\GetResponseEvent;
+use Symfony\Component\HttpKernel\Event\RequestEvent;
 use Symfony\Component\HttpKernel\HttpKernelInterface;
 use Symfony\Component\HttpKernel\Kernel;
 use Symfony\Component\HttpKernel\KernelEvents;
@@ -116,7 +117,8 @@ protected static function bootKernel(array $options = [])
         /** @var EventDispatcherInterface $dispatcher */
         $dispatcher = static::$kernel->getContainer()->get('event_dispatcher');
 
-        $event = new GetResponseEvent(static::$kernel, new SymfonyRequest(), HttpKernelInterface::MASTER_REQUEST);
+        $class = (Kernel::MAJOR_VERSION >= 5) ? RequestEvent::class : GetResponseEvent::class;
+        $event = new $class(static::$kernel, SymfonyRequest::create('/'), HttpKernelInterface::MASTER_REQUEST);
 
         if (version_compare(Kernel::VERSION, '4.3.0', '>=')) {
             $dispatcher->dispatch($event, KernelEvents::REQUEST);

tests/Unit/Collector/PluginClientFactoryListenerTest.php

@@ -9,8 +9,10 @@
 use Http\HttplugBundle\Collector\PluginClientFactoryListener;
 use Nyholm\NSA;
 use PHPUnit\Framework\TestCase;
-use Symfony\Component\EventDispatcher\Event;
+use Symfony\Component\EventDispatcher\Event as LegacyEvent;
+use Symfony\Component\HttpKernel\Kernel;
 use Symfony\Component\Stopwatch\Stopwatch;
+use Symfony\Contracts\EventDispatcher\Event;
 
 final class PluginClientFactoryListenerTest extends TestCase
 {
@@ -24,7 +26,8 @@ public function testRegisterPluginClientFactory(): void
 
         $listener = new PluginClientFactoryListener($factory);
 
-        $listener->onEvent(new Event());
+        $class = (Kernel::MAJOR_VERSION >= 5) ? Event::class : LegacyEvent::class;
+        $listener->onEvent(new $class());
 
         $this->assertTrue(is_callable(NSA::getProperty(DefaultPluginClientFactory::class, 'factory')));
     }

tests/Unit/Collector/ProfileClientTest.php

@@ -141,10 +141,6 @@ public function testSendRequest(): void
         $this->assertEquals('https', $this->activeStack->getRequestScheme());
     }
 
-    /**
-     * @expectedException \Error
-     * @expectedException "You set string to int prop"
-     */
     public function testSendRequestTypeError()
     {
         $this->client
@@ -158,6 +154,8 @@ public function testSendRequestTypeError()
             ->method('formatException')
             ->with($this->isInstanceOf(\Error::class));
 
+        $this->expectException(\Error::class);
+        $this->expectExceptionMessage('You set string to int prop');
         $this->subject->sendRequest($this->request);
     }