Convert dataProvider annotations to attributes

bobvandevijver · bobvandevijver · commit d69147fb961a · 2025-08-05T19:52:58.000+02:00
diff --git a/src/Symfony/Component/Mailer/Bridge/MicrosoftGraph/Tests/Transport/MicrosoftGraphApiTransportTest.php b/src/Symfony/Component/Mailer/Bridge/MicrosoftGraph/Tests/Transport/MicrosoftGraphApiTransportTest.php
@@ -11,6 +11,7 @@
 
 namespace Symfony\Component\Mailer\Bridge\MicrosoftGraph\Tests\Transport;
 
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 use Symfony\Component\HttpClient\MockHttpClient;
 use Symfony\Component\HttpClient\Response\MockResponse;
@@ -23,9 +24,7 @@
 
 class MicrosoftGraphApiTransportTest extends TestCase
 {
-    /**
-     * @dataProvider getTransportData
-     */
+    #[DataProvider('getTransportData')]
     public function testToString(MicrosoftGraphApiTransport $transport, string $expected)
     {
         $this->assertSame($expected, (string) $transport);
@@ -230,7 +229,7 @@ public function testCustomHeader()
         $transport->send($mail);
     }
 
-    /** @dataProvider importanceProvider */
+    #[DataProvider('importanceProvider')]
     public function testImportance(string $expected, int $priority)
     {
         $client = new MockHttpClient(function (string $method, string $url, array $options) use ($expected): ResponseInterface {
diff --git a/src/Symfony/Component/Mailer/Bridge/MicrosoftGraph/Tests/Transport/MicrosoftGraphTransportFactoryTest.php b/src/Symfony/Component/Mailer/Bridge/MicrosoftGraph/Tests/Transport/MicrosoftGraphTransportFactoryTest.php
@@ -11,6 +11,7 @@
 
 namespace Symfony\Component\Mailer\Bridge\MicrosoftGraph\Tests\Transport;
 
+use PHPUnit\Framework\Attributes\DataProvider;
 use Psr\Log\NullLogger;
 use Symfony\Component\HttpClient\MockHttpClient;
 use Symfony\Component\Mailer\Bridge\MicrosoftGraph\TokenManager;
@@ -76,7 +77,7 @@ public static function incompleteDsnProvider(): iterable
         yield [new Dsn('microsoft+graphapi', 'non-default', self::USER, self::PASSWORD, null, ['tenantId' => self::TENANT])];
     }
 
-    /** @dataProvider invalidHttpDsnProvider */
+    #[DataProvider('invalidHttpDsnProvider')]
     public function testValidatesHttpNotProvided(string $graph, string $auth, string $failingType)
     {
         $factory = $this->getFactory();

Original file line number	Diff line number	Diff line change
`@@ -11,6 +11,7 @@`
`11`	`11`
`12`	`12`	`namespace Symfony\Component\Mailer\Bridge\MicrosoftGraph\Tests\Transport;`
`13`	`13`
	`14`	`+use PHPUnit\Framework\Attributes\DataProvider;`
`14`	`15`	`use PHPUnit\Framework\TestCase;`
`15`	`16`	`use Symfony\Component\HttpClient\MockHttpClient;`
`16`	`17`	`use Symfony\Component\HttpClient\Response\MockResponse;`
`@@ -23,9 +24,7 @@`
`23`	`24`
`24`	`25`	`class MicrosoftGraphApiTransportTest extends TestCase`
`25`	`26`	`{`
`26`		`- /**`
`27`		`- * @dataProvider getTransportData`
`28`		`- */`
	`27`	`+ #[DataProvider('getTransportData')]`
`29`	`28`	`public function testToString(MicrosoftGraphApiTransport $transport, string $expected)`
`30`	`29`	`{`
`31`	`30`	`$this->assertSame($expected, (string) $transport);`
`@@ -230,7 +229,7 @@ public function testCustomHeader()`
`230`	`229`	`$transport->send($mail);`
`231`	`230`	`}`
`232`	`231`
`233`		`- /** @dataProvider importanceProvider */`
	`232`	`+ #[DataProvider('importanceProvider')]`
`234`	`233`	`public function testImportance(string $expected, int $priority)`
`235`	`234`	`{`
`236`	`235`	`$client = new MockHttpClient(function (string $method, string $url, array $options) use ($expected): ResponseInterface {`