From 61eb7d7d374b0b5db210c21bb21d43cfd97fb5b2 Mon Sep 17 00:00:00 2001
From: Ahmed TAILOULOUTE <ahmed.tailouloute@gmail.com>
Date: Mon, 8 Jun 2020 21:05:34 +0200
Subject: [PATCH] [Mailer] Fixed generator bug when creating multiple
 transports using Transport::fromDsn

---
 .../Component/Mailer/Tests/TransportTest.php     | 16 ++++++++++++++++
 src/Symfony/Component/Mailer/Transport.php       |  2 +-
 2 files changed, 17 insertions(+), 1 deletion(-)

diff --git a/src/Symfony/Component/Mailer/Tests/TransportTest.php b/src/Symfony/Component/Mailer/Tests/TransportTest.php
index 95eb5b6ebf03a..5a284bba96a9b 100644
--- a/src/Symfony/Component/Mailer/Tests/TransportTest.php
+++ b/src/Symfony/Component/Mailer/Tests/TransportTest.php
@@ -60,6 +60,22 @@ public function fromStringProvider(): iterable
         ];
     }
 
+    /**
+     * @dataProvider fromDnsProvider
+     */
+    public function testFromDsn(string $dsn, TransportInterface $transport): void
+    {
+        $this->assertEquals($transport, Transport::fromDsn($dsn));
+    }
+
+    public function fromDnsProvider(): iterable
+    {
+        yield 'multiple transport' => [
+            'failover(smtp://a smtp://b)',
+            new FailoverTransport([new Transport\Smtp\EsmtpTransport('a'), new Transport\Smtp\EsmtpTransport('b')])
+        ];
+    }
+
     /**
      * @dataProvider fromWrongStringProvider
      */
diff --git a/src/Symfony/Component/Mailer/Transport.php b/src/Symfony/Component/Mailer/Transport.php
index a8f159dc7c491..c0cd0c5fb9dad 100644
--- a/src/Symfony/Component/Mailer/Transport.php
+++ b/src/Symfony/Component/Mailer/Transport.php
@@ -51,7 +51,7 @@ class Transport
 
     public static function fromDsn(string $dsn, EventDispatcherInterface $dispatcher = null, HttpClientInterface $client = null, LoggerInterface $logger = null): TransportInterface
     {
-        $factory = new self(self::getDefaultFactories($dispatcher, $client, $logger));
+        $factory = new self(iterator_to_array(self::getDefaultFactories($dispatcher, $client, $logger)));
 
         return $factory->fromString($dsn);
     }