Migrate to static data providers using rector/rector

Author: OskarStark

Tests/Channel/ChannelPolicyTest.php

@@ -39,7 +39,7 @@ public function testCanRetrieveChannels(array $policy, string $importance, array
         $this->assertSame($expectedChannels, $channels);
     }
 
-    public function provideValidPolicies(): \Generator
+    public static function provideValidPolicies(): \Generator
     {
         yield [['urgent' => ['chat']], 'urgent', ['chat']];
         yield [['urgent' => ['chat', 'sms']], 'urgent', ['chat', 'sms']];

Tests/Event/FailedMessageEventTest.php

@@ -95,7 +95,7 @@ public function __toString(): string
         }
     }
 
-    public function messagesProvider(): iterable
+    public static function messagesProvider(): iterable
     {
         yield [$message = new ChatMessage('subject'), $error = new \RuntimeException(), new FailedMessageEvent($message, $error)];
         yield [$message = new SmsMessage('+3312345678', 'subject'), $error = new \Exception(), new FailedMessageEvent($message, $error)];

Tests/Event/SentMessageEventTest.php

@@ -35,7 +35,7 @@ public function testGetMessage(SentMessage $message, SentMessageEvent $event)
         $this->assertSame($message, $event->getMessage());
     }
 
-    public function messagesProvider(): iterable
+    public static function messagesProvider(): iterable
     {
         yield [$message = new SentMessage(new ChatMessage('subject'), 'null_transport'), new SentMessageEvent($message)];
         yield [$message = new SentMessage(new SmsMessage('+3312345678', 'subject'), 'null_transport'), new SentMessageEvent($message)];

Tests/Exception/UnsupportedSchemeExceptionTest.php

@@ -127,7 +127,7 @@ public function testMessageWhereSchemeIsPartOfSchemeToPackageMap(string $scheme,
         );
     }
 
-    public function messageWhereSchemeIsPartOfSchemeToPackageMapProvider(): \Generator
+    public static function messageWhereSchemeIsPartOfSchemeToPackageMapProvider(): \Generator
     {
         yield ['allmysms', 'symfony/all-my-sms-notifier'];
         yield ['sns', 'symfony/amazon-sns-notifier'];
@@ -183,7 +183,7 @@ public function testMessageWhereSchemeIsNotPartOfSchemeToPackageMap(string $expe
         );
     }
 
-    public function messageWhereSchemeIsNotPartOfSchemeToPackageMapProvider(): \Generator
+    public static function messageWhereSchemeIsNotPartOfSchemeToPackageMapProvider(): \Generator
     {
         yield [
             'The "somethingElse" scheme is not supported.',

Tests/Message/NullMessageTest.php

@@ -36,7 +36,7 @@ public function testCanBeConstructed(MessageInterface $message)
             : $this->assertSame($message->getTransport(), $nullMessage->getTransport());
     }
 
-    public function messageDataProvider(): \Generator
+    public static function messageDataProvider(): \Generator
     {
         yield [new DummyMessageWithoutTransport()];
         yield [new DummyMessageWithTransport()];

Tests/Recipient/RecipientTest.php

@@ -38,7 +38,7 @@ public function testCanBeConstructed(string $email, string $phone)
         $this->assertSame($phone, $recipient->getPhone());
     }
 
-    public function provideValidEmailAndPhone()
+    public static function provideValidEmailAndPhone()
     {
         yield ['[email protected]', '+0815'];
         yield ['[email protected]', ''];

Tests/Transport/DsnTest.php

@@ -35,7 +35,7 @@ public function testConstruct(string $dsnString, string $scheme, string $host, s
         $this->assertSame($options, $dsn->getOptions());
     }
 
-    public function constructProvider(): iterable
+    public static function constructProvider(): iterable
     {
         yield 'simple dsn' => [
             'scheme://localhost',
@@ -151,7 +151,7 @@ public function testInvalidDsn(string $dsnString, string $exceptionMessage)
         new Dsn($dsnString);
     }
 
-    public function invalidDsnProvider(): iterable
+    public static function invalidDsnProvider(): iterable
     {
         yield [
             'some://',
@@ -179,7 +179,7 @@ public function testGetOption($expected, string $dsnString, string $option, stri
         $this->assertSame($expected, $dsn->getOption($option, $default));
     }
 
-    public function getOptionProvider(): iterable
+    public static function getOptionProvider(): iterable
     {
         yield [
             'foo',
@@ -217,7 +217,7 @@ public function testGetRequiredOption(string $expectedValue, string $options, st
         $this->assertSame($expectedValue, $dsn->getRequiredOption($option));
     }
 
-    public function getRequiredOptionProvider(): iterable
+    public static function getRequiredOptionProvider(): iterable
     {
         yield [
             'value',
@@ -245,7 +245,7 @@ public function testGetRequiredOptionThrowsMissingRequiredOptionException(string
         $dsn->getRequiredOption($option);
     }
 
-    public function getRequiredOptionThrowsMissingRequiredOptionExceptionProvider(): iterable
+    public static function getRequiredOptionThrowsMissingRequiredOptionExceptionProvider(): iterable
     {
         yield [
             'The option "foo_bar" is required but missing.',