[Mailer] Fixed generator bug when creating multiple transports using …

…Transport::fromDsn
symfony · Jun 8, 2020 · 5d9bc15 · 5d9bc15
1 parent 3755efd
commit 5d9bc15
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git 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
@@ -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);
     }