Merge branch '5.2' into 5.x

* 5.2:
  [Notifier] Use abstract test cases in 5.2
  Rename parameter of @method configureContainer
  [DependencyInjection] Support PHP 8 builtin types in CheckTypeDeclarationsPass
  [DependencyInjection] Fix InvalidParameterTypeException for function parameters
  [Notifier] Cleanup changelog (5.2)
  [Notifier] Cleanup changelog (5.1)
  [VarDumper] fix mutating $GLOBALS while cloning it

Author: nicolas-grekas

Tests/LinkedInTransportFactoryTest.php

@@ -2,55 +2,41 @@
 
 namespace Symfony\Component\Notifier\Bridge\LinkedIn\Tests;
 
-use PHPUnit\Framework\TestCase;
 use Symfony\Component\Notifier\Bridge\LinkedIn\LinkedInTransportFactory;
-use Symfony\Component\Notifier\Exception\IncompleteDsnException;
-use Symfony\Component\Notifier\Exception\UnsupportedSchemeException;
-use Symfony\Component\Notifier\Transport\Dsn;
+use Symfony\Component\Notifier\Tests\TransportFactoryTestCase;
+use Symfony\Component\Notifier\Transport\TransportFactoryInterface;
 
-final class LinkedInTransportFactoryTest extends TestCase
+final class LinkedInTransportFactoryTest extends TransportFactoryTestCase
 {
-    public function testCreateWithDsn()
+    /**
+     * @return LinkedInTransportFactory
+     */
+    public function createFactory(): TransportFactoryInterface
     {
-        $factory = $this->createFactory();
-
-        $transport = $factory->create(Dsn::fromString('linkedin://accessToken:UserId@host.test'));
-
-        $this->assertSame('linkedin://host.test', (string) $transport);
-    }
-
-    public function testCreateWithOnlyAccessTokenOrUserIdThrowsIncompleteDsnException()
-    {
-        $factory = $this->createFactory();
-
-        $this->expectException(IncompleteDsnException::class);
-        $factory->create(Dsn::fromString('linkedin://AccessTokenOrUserId@default'));
+        return new LinkedInTransportFactory();
     }
 
-    public function testSupportsReturnsTrueWithSupportedScheme()
+    public function createProvider(): iterable
     {
-        $factory = $this->createFactory();
-
-        $this->assertTrue($factory->supports(Dsn::fromString('linkedin://host/path')));
+        yield [
+            'linkedin://host.test',
+            'linkedin://accessToken:UserId@host.test',
+        ];
     }
 
-    public function testSupportsReturnsFalseWithUnsupportedScheme()
+    public function supportsProvider(): iterable
     {
-        $factory = $this->createFactory();
-
-        $this->assertFalse($factory->supports(Dsn::fromString('somethingElse://host/path')));
+        yield [true, 'linkedin://host'];
+        yield [false, 'somethingElse://host'];
     }
 
-    public function testUnsupportedSchemeThrowsUnsupportedSchemeException()
+    public function incompleteDsnProvider(): iterable
     {
-        $factory = $this->createFactory();
-
-        $this->expectException(UnsupportedSchemeException::class);
-        $factory->create(Dsn::fromString('somethingElse://accessToken:UserId@default'));
+        yield 'missing account or user_id' => ['linkedin://AccessTokenOrUserId@default'];
     }
 
-    private function createFactory(): LinkedInTransportFactory
+    public function unsupportedSchemeProvider(): iterable
     {
-        return new LinkedInTransportFactory();
+        yield ['somethingElse://accessToken:UserId@default'];
     }
 }