[Translation][Mailer] Convert $this calls to static ones in data providers

Author: alexandre-daubois

CHANGELOG.md

@@ -4,9 +4,9 @@ CHANGELOG
 5.4.21
 ------
 
-* [BC BREAK] The following data providers for `TransportFactoryTestCase` are now static:
+ * [BC BREAK] The following data providers for `TransportFactoryTestCase` are now static:
   `supportsProvider()`, `createProvider()`, `unsupportedSchemeProvider()`and `incompleteDsnProvider()`
-* [BC BREAK] The following data providers for `TransportTestCase` are now static:
+ * [BC BREAK] The following data providers for `TransportTestCase` are now static:
   `toStringProvider()`, `supportedMessagesProvider()` and `unsupportedMessagesProvider()`
 
 5.4

Test/TransportFactoryTestCase.php

@@ -13,6 +13,8 @@
 
 use PHPUnit\Framework\TestCase;
 use Psr\Log\LoggerInterface;
+use Psr\Log\NullLogger;
+use Symfony\Component\HttpClient\MockHttpClient;
 use Symfony\Component\Mailer\Exception\IncompleteDsnException;
 use Symfony\Component\Mailer\Exception\UnsupportedSchemeException;
 use Symfony\Component\Mailer\Transport\Dsn;
@@ -31,11 +33,11 @@ abstract class TransportFactoryTestCase extends TestCase
     protected const USER = 'u$er';
     protected const PASSWORD = 'pa$s';
 
-    protected $dispatcher;
-    protected $client;
-    protected $logger;
+    protected static $dispatcher;
+    protected static $client;
+    protected static $logger;
 
-    abstract public function getFactory(): TransportFactoryInterface;
+    abstract public static function getFactory(): TransportFactoryInterface;
 
     abstract public static function supportsProvider(): iterable;
 
@@ -100,18 +102,22 @@ public function testIncompleteDsnException(Dsn $dsn)
         $factory->create($dsn);
     }
 
-    protected function getDispatcher(): EventDispatcherInterface
+    protected static function getDispatcher(): EventDispatcherInterface
     {
-        return $this->dispatcher ?? $this->dispatcher = $this->createMock(EventDispatcherInterface::class);
+        return self::$dispatcher ?? self::$dispatcher = new class() implements EventDispatcherInterface {
+            public function dispatch($event, string $eventName = null): object
+            {
+            }
+        };
     }
 
-    protected function getClient(): HttpClientInterface
+    protected static function getClient(): HttpClientInterface
     {
-        return $this->client ?? $this->client = $this->createMock(HttpClientInterface::class);
+        return self::$client ?? self::$client = new MockHttpClient();
     }
 
-    protected function getLogger(): LoggerInterface
+    protected static function getLogger(): LoggerInterface
     {
-        return $this->logger ?? $this->logger = $this->createMock(LoggerInterface::class);
+        return self::$logger ?? self::$logger = new NullLogger();
     }
 }

Tests/Transport/NullTransportFactoryTest.php

@@ -19,9 +19,9 @@
 
 class NullTransportFactoryTest extends TransportFactoryTestCase
 {
-    public function getFactory(): TransportFactoryInterface
+    public static function getFactory(): TransportFactoryInterface
     {
-        return new NullTransportFactory($this->getDispatcher(), $this->getClient(), $this->getLogger());
+        return new NullTransportFactory(self::getDispatcher(), self::getClient(), self::getLogger());
     }
 
     public static function supportsProvider(): iterable
@@ -36,7 +36,7 @@ public static function createProvider(): iterable
     {
         yield [
             new Dsn('null', 'null'),
-            new NullTransport($this->getDispatcher(), $this->getLogger()),
+            new NullTransport(self::getDispatcher(), self::getLogger()),
         ];
     }
 }

Tests/Transport/SendmailTransportFactoryTest.php

@@ -19,9 +19,9 @@
 
 class SendmailTransportFactoryTest extends TransportFactoryTestCase
 {
-    public function getFactory(): TransportFactoryInterface
+    public static function getFactory(): TransportFactoryInterface
     {
-        return new SendmailTransportFactory($this->getDispatcher(), $this->getClient(), $this->getLogger());
+        return new SendmailTransportFactory(self::getDispatcher(), self::getClient(), self::getLogger());
     }
 
     public static function supportsProvider(): iterable
@@ -36,12 +36,12 @@ public static function createProvider(): iterable
     {
         yield [
             new Dsn('sendmail+smtp', 'default'),
-            new SendmailTransport(null, $this->getDispatcher(), $this->getLogger()),
+            new SendmailTransport(null, self::getDispatcher(), self::getLogger()),
         ];
 
         yield [
             new Dsn('sendmail+smtp', 'default', null, null, null, ['command' => '/usr/sbin/sendmail -oi -t']),
-            new SendmailTransport('/usr/sbin/sendmail -oi -t', $this->getDispatcher(), $this->getLogger()),
+            new SendmailTransport('/usr/sbin/sendmail -oi -t', self::getDispatcher(), self::getLogger()),
         ];
     }
 

Tests/Transport/Smtp/EsmtpTransportFactoryTest.php

@@ -20,9 +20,9 @@
 
 class EsmtpTransportFactoryTest extends TransportFactoryTestCase
 {
-    public function getFactory(): TransportFactoryInterface
+    public static function getFactory(): TransportFactoryInterface
     {
-        return new EsmtpTransportFactory($this->getDispatcher(), $this->getClient(), $this->getLogger());
+        return new EsmtpTransportFactory(self::getDispatcher(), self::getClient(), self::getLogger());
     }
 
     public static function supportsProvider(): iterable
@@ -45,8 +45,8 @@ public static function supportsProvider(): iterable
 
     public static function createProvider(): iterable
     {
-        $eventDispatcher = $this->getDispatcher();
-        $logger = $this->getLogger();
+        $eventDispatcher = self::getDispatcher();
+        $logger = self::getLogger();
 
         $transport = new EsmtpTransport('localhost', 25, false, $eventDispatcher, $logger);
 

composer.json

@@ -27,7 +27,7 @@
         "symfony/service-contracts": "^1.1|^2|^3"
     },
     "require-dev": {
-        "symfony/http-client-contracts": "^1.1|^2|^3",
+        "symfony/http-client": "^4.4|^5.0|^6.0",
         "symfony/messenger": "^4.4|^5.0|^6.0"
     },
     "conflict": {