Migrate to static data providers using rector/rector

Author: OskarStark

Tests/EventListener/MessageListenerTest.php

@@ -36,7 +36,7 @@ public function testHeaders(Headers $initialHeaders, Headers $defaultHeaders, He
         $this->assertEquals($expectedHeaders, $event->getMessage()->getHeaders());
     }
 
-    public function provideHeaders(): iterable
+    public static function provideHeaders(): iterable
     {
         $initialHeaders = new Headers();
         $defaultHeaders = (new Headers())

Tests/Exception/UnsupportedSchemeExceptionTest.php

@@ -59,7 +59,7 @@ public function testMessageWhereSchemeIsPartOfSchemeToPackageMap(string $scheme,
         );
     }
 
-    public function messageWhereSchemeIsPartOfSchemeToPackageMapProvider(): \Generator
+    public static function messageWhereSchemeIsPartOfSchemeToPackageMapProvider(): \Generator
     {
         yield ['gmail', 'symfony/google-mailer'];
         yield ['mailgun', 'symfony/mailgun-mailer'];
@@ -83,7 +83,7 @@ public function testMessageWhereSchemeIsNotPartOfSchemeToPackageMap(string $expe
         );
     }
 
-    public function messageWhereSchemeIsNotPartOfSchemeToPackageMapProvider(): \Generator
+    public static function messageWhereSchemeIsNotPartOfSchemeToPackageMapProvider(): \Generator
     {
         yield [
             'The "somethingElse" scheme is not supported.',

Tests/Transport/DsnTest.php

@@ -45,7 +45,7 @@ public function testInvalidDsn(string $dsn, string $exceptionMessage)
         Dsn::fromString($dsn);
     }
 
-    public function fromStringProvider(): iterable
+    public static function fromStringProvider(): iterable
     {
         yield 'simple smtp without user and pass' => [
             'smtp://example.com',
@@ -88,7 +88,7 @@ public function fromStringProvider(): iterable
         ];
     }
 
-    public function invalidDsnProvider(): iterable
+    public static function invalidDsnProvider(): iterable
     {
         yield [
             'some://',

Tests/Transport/NativeTransportFactoryTest.php

@@ -66,7 +66,7 @@ public function testCreateSendmailWithNoSendmailPath()
         $sut->create(Dsn::fromString('native://default'));
     }
 
-    public function provideCreateSendmailWithNoHostOrNoPort(): \Generator
+    public static function provideCreateSendmailWithNoHostOrNoPort(): \Generator
     {
         yield ['native://default', '', '', ''];
         yield ['native://default', '', 'localhost', ''];
@@ -95,7 +95,7 @@ public function testCreateSendmailWithNoHostOrNoPort(string $dsn, string $sendma
         $sut->create(Dsn::fromString($dsn));
     }
 
-    public function provideCreate(): \Generator
+    public static function provideCreate(): \Generator
     {
         yield ['native://default', '/usr/sbin/sendmail -t -i', '', '', new SendmailTransport('/usr/sbin/sendmail -t -i')];
 

Tests/Transport/Smtp/Stream/AbstractStreamTest.php

@@ -29,7 +29,7 @@ public function testReplace(string $expected, string $from, string $to, array $c
         $this->assertSame($expected, $result);
     }
 
-    public function provideReplace()
+    public static function provideReplace()
     {
         yield ['ca', 'ab', 'c', ['a', 'b', 'a']];
         yield ['ac', 'ab', 'c', ['a', 'ab']];

Tests/TransportTest.php

@@ -34,7 +34,7 @@ public function testFromString(string $dsn, TransportInterface $transport)
         $this->assertEquals($transport, $transportFactory->fromString($dsn));
     }
 
-    public function fromStringProvider(): iterable
+    public static function fromStringProvider(): iterable
     {
         $transportA = new DummyTransport('a');
         $transportB = new DummyTransport('b');
@@ -68,7 +68,7 @@ public function testFromDsn(string $dsn, TransportInterface $transport)
         $this->assertEquals($transport, Transport::fromDsn($dsn));
     }
 
-    public function fromDsnProvider(): iterable
+    public static function fromDsnProvider(): iterable
     {
         yield 'multiple transports' => [
             'failover(smtp://a smtp://b)',
@@ -88,7 +88,7 @@ public function testFromWrongString(string $dsn, string $error)
         $transportFactory->fromString($dsn);
     }
 
-    public function fromWrongStringProvider(): iterable
+    public static function fromWrongStringProvider(): iterable
     {
         yield 'garbage at the end' => ['dummy://a some garbage here', 'The DSN has some garbage at the end: " some garbage here".'];