feature symfony#58351 [Mailer] deprecate the TransportFactoryTestCase (xabbuh)

This PR was merged into the 7.2 branch.

Discussion
----------

[Mailer] deprecate the TransportFactoryTestCase

| Q             | A
| ------------- | ---
| Branch?       | 7.2
| Bug fix?      | no
| New feature?  | no
| Deprecations? | yes
| Issues        |
| License       | MIT

Commits
-------

e878066 deprecate the TransportFactoryTestCase

Author: nicolas-grekas

UPGRADE-7.2.md

@@ -47,6 +47,14 @@ Ldap
 
  * Add methods for `saslBind()` and `whoami()` to `ConnectionInterface` and `LdapInterface`
 
+Mailer
+------
+
+* Deprecate `TransportFactoryTestCase`, extend `AbstractTransportFactoryTestCase` instead
+
+  The `testIncompleteDsnException()` test is no longer provided by default. If you make use of it by implementing the `incompleteDsnProvider()` data providers,
+  you now need to use the `IncompleteDsnTestTrait`.
+
 Messenger
 ---------
 

src/Symfony/Component/Mailer/Bridge/Amazon/Tests/Transport/SesTransportFactoryTest.php

@@ -19,12 +19,15 @@
 use Symfony\Component\Mailer\Bridge\Amazon\Transport\SesHttpAsyncAwsTransport;
 use Symfony\Component\Mailer\Bridge\Amazon\Transport\SesSmtpTransport;
 use Symfony\Component\Mailer\Bridge\Amazon\Transport\SesTransportFactory;
-use Symfony\Component\Mailer\Test\TransportFactoryTestCase;
+use Symfony\Component\Mailer\Test\AbstractTransportFactoryTestCase;
+use Symfony\Component\Mailer\Test\IncompleteDsnTestTrait;
 use Symfony\Component\Mailer\Transport\Dsn;
 use Symfony\Component\Mailer\Transport\TransportFactoryInterface;
 
-class SesTransportFactoryTest extends TransportFactoryTestCase
+class SesTransportFactoryTest extends AbstractTransportFactoryTestCase
 {
+    use IncompleteDsnTestTrait;
+
     public function getFactory(): TransportFactoryInterface
     {
         return new SesTransportFactory(null, new MockHttpClient(), new NullLogger());

src/Symfony/Component/Mailer/Bridge/Amazon/composer.json

@@ -18,7 +18,7 @@
     "require": {
         "php": ">=8.2",
         "async-aws/ses": "^1.3",
-        "symfony/mailer": "^6.4|^7.0"
+        "symfony/mailer": "^7.2"
     },
     "require-dev": {
         "symfony/http-client": "^6.4|^7.0"

src/Symfony/Component/Mailer/Bridge/Azure/Tests/Transport/AzureTransportFactoryTest.php

@@ -15,12 +15,15 @@
 use Symfony\Component\HttpClient\MockHttpClient;
 use Symfony\Component\Mailer\Bridge\Azure\Transport\AzureApiTransport;
 use Symfony\Component\Mailer\Bridge\Azure\Transport\AzureTransportFactory;
-use Symfony\Component\Mailer\Test\TransportFactoryTestCase;
+use Symfony\Component\Mailer\Test\AbstractTransportFactoryTestCase;
+use Symfony\Component\Mailer\Test\IncompleteDsnTestTrait;
 use Symfony\Component\Mailer\Transport\Dsn;
 use Symfony\Component\Mailer\Transport\TransportFactoryInterface;
 
-class AzureTransportFactoryTest extends TransportFactoryTestCase
+class AzureTransportFactoryTest extends AbstractTransportFactoryTestCase
 {
+    use IncompleteDsnTestTrait;
+
     public function getFactory(): TransportFactoryInterface
     {
         return new AzureTransportFactory(null, new MockHttpClient(), new NullLogger());

src/Symfony/Component/Mailer/Bridge/Azure/composer.json

@@ -17,7 +17,7 @@
     ],
     "require": {
         "php": ">=8.2",
-        "symfony/mailer": "^6.4|^7.0"
+        "symfony/mailer": "^7.2"
     },
     "require-dev": {
         "symfony/http-client": "^6.4|^7.0"

src/Symfony/Component/Mailer/Bridge/Brevo/Tests/Transport/BrevoTransportFactoryTest.php

@@ -16,12 +16,15 @@
 use Symfony\Component\Mailer\Bridge\Brevo\Transport\BrevoApiTransport;
 use Symfony\Component\Mailer\Bridge\Brevo\Transport\BrevoSmtpTransport;
 use Symfony\Component\Mailer\Bridge\Brevo\Transport\BrevoTransportFactory;
-use Symfony\Component\Mailer\Test\TransportFactoryTestCase;
+use Symfony\Component\Mailer\Test\AbstractTransportFactoryTestCase;
+use Symfony\Component\Mailer\Test\IncompleteDsnTestTrait;
 use Symfony\Component\Mailer\Transport\Dsn;
 use Symfony\Component\Mailer\Transport\TransportFactoryInterface;
 
-class BrevoTransportFactoryTest extends TransportFactoryTestCase
+class BrevoTransportFactoryTest extends AbstractTransportFactoryTestCase
 {
+    use IncompleteDsnTestTrait;
+
     public function getFactory(): TransportFactoryInterface
     {
         return new BrevoTransportFactory(null, new MockHttpClient(), new NullLogger());

src/Symfony/Component/Mailer/Bridge/Brevo/composer.json

@@ -17,7 +17,7 @@
     ],
     "require": {
         "php": ">=8.1",
-        "symfony/mailer": "^5.4.21|^6.2.7|^7.0"
+        "symfony/mailer": "^7.2"
     },
     "require-dev": {
         "symfony/http-client": "^6.3|^7.0",

src/Symfony/Component/Mailer/Bridge/Google/Tests/Transport/GmailTransportFactoryTest.php

@@ -15,12 +15,15 @@
 use Symfony\Component\HttpClient\MockHttpClient;
 use Symfony\Component\Mailer\Bridge\Google\Transport\GmailSmtpTransport;
 use Symfony\Component\Mailer\Bridge\Google\Transport\GmailTransportFactory;
-use Symfony\Component\Mailer\Test\TransportFactoryTestCase;
+use Symfony\Component\Mailer\Test\AbstractTransportFactoryTestCase;
+use Symfony\Component\Mailer\Test\IncompleteDsnTestTrait;
 use Symfony\Component\Mailer\Transport\Dsn;
 use Symfony\Component\Mailer\Transport\TransportFactoryInterface;
 
-class GmailTransportFactoryTest extends TransportFactoryTestCase
+class GmailTransportFactoryTest extends AbstractTransportFactoryTestCase
 {
+    use IncompleteDsnTestTrait;
+
     public function getFactory(): TransportFactoryInterface
     {
         return new GmailTransportFactory(null, new MockHttpClient(), new NullLogger());

src/Symfony/Component/Mailer/Bridge/Google/composer.json

@@ -17,7 +17,7 @@
     ],
     "require": {
         "php": ">=8.2",
-        "symfony/mailer": "^6.4|^7.0"
+        "symfony/mailer": "^7.2"
     },
     "require-dev": {
         "symfony/http-client": "^6.4|^7.0"

src/Symfony/Component/Mailer/Bridge/Infobip/Tests/Transport/InfobipApiTransportFactoryTest.php

@@ -16,12 +16,15 @@
 use Symfony\Component\Mailer\Bridge\Infobip\Transport\InfobipApiTransport;
 use Symfony\Component\Mailer\Bridge\Infobip\Transport\InfobipSmtpTransport;
 use Symfony\Component\Mailer\Bridge\Infobip\Transport\InfobipTransportFactory;
-use Symfony\Component\Mailer\Test\TransportFactoryTestCase;
+use Symfony\Component\Mailer\Test\AbstractTransportFactoryTestCase;
+use Symfony\Component\Mailer\Test\IncompleteDsnTestTrait;
 use Symfony\Component\Mailer\Transport\Dsn;
 use Symfony\Component\Mailer\Transport\TransportFactoryInterface;
 
-class InfobipApiTransportFactoryTest extends TransportFactoryTestCase
+class InfobipApiTransportFactoryTest extends AbstractTransportFactoryTestCase
 {
+    use IncompleteDsnTestTrait;
+
     public function getFactory(): TransportFactoryInterface
     {
         return new InfobipTransportFactory(null, new MockHttpClient(), new NullLogger());