[Messenger] Rename some classes

fabpot · fabpot · commit f526af00ad7b · 2022-08-05T08:42:18.000+02:00
diff --git a/src/Symfony/Component/Messenger/CHANGELOG.md b/src/Symfony/Component/Messenger/CHANGELOG.md
@@ -5,7 +5,7 @@ CHANGELOG
 ---
 
  * Add new `messenger:stats` command that return a list of transports with their "to be processed" message count
- * Add `ViaSenderStamp` to change the transport while dispatching a message
+ * Add `TransportNamesStamp` to change the transport while dispatching a message
 
 6.1
 ---
diff --git a/src/Symfony/Component/Messenger/Stamp/OverrideSendersStamp.php b/src/Symfony/Component/Messenger/Stamp/OverrideSendersStamp.php
diff --git a/src/Symfony/Component/Messenger/Stamp/TransportNamesStamp.php b/src/Symfony/Component/Messenger/Stamp/TransportNamesStamp.php
@@ -0,0 +1,30 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <fabien@symfony.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\Component\Messenger\Stamp;
+
+/**
+ * Stamp used to override the transport names specified in the Messenger routing configuration file.
+ */
+final class TransportNamesStamp implements StampInterface
+{
+    /**
+     * @param string[] $transports Transport names to be used for the message
+     */
+    public function __construct(private array $transports)
+    {
+    }
+
+    public function getTransportNames(): array
+    {
+        return $this->transports;
+    }
+}
diff --git a/src/Symfony/Component/Messenger/Tests/Stamp/TransportNamesStampTest.php b/src/Symfony/Component/Messenger/Tests/Stamp/TransportNamesStampTest.php
@@ -12,15 +12,15 @@
 namespace Symfony\Component\Messenger\Tests\Stamp;
 
 use PHPUnit\Framework\TestCase;
-use Symfony\Component\Messenger\Stamp\ViaSenderStamp;
+use Symfony\Component\Messenger\Stamp\TransportNamesStamp;
 
-class ViaSenderStampTest extends TestCase
+class TransportNamesStampTest extends TestCase
 {
     public function testGetSenders()
     {
         $configuredSenders = ['first_transport', 'second_transport', 'other_transport'];
-        $stamp = new ViaSenderStamp($configuredSenders);
-        $stampSenders = $stamp->getSenders();
+        $stamp = new TransportNamesStamp($configuredSenders);
+        $stampSenders = $stamp->getTransportNames();
         $this->assertEquals(\count($configuredSenders), \count($stampSenders));
 
         foreach ($configuredSenders as $key => $sender) {
diff --git a/src/Symfony/Component/Messenger/Tests/Transport/Sender/SendersLocatorTest.php b/src/Symfony/Component/Messenger/Tests/Transport/Sender/SendersLocatorTest.php
@@ -14,7 +14,7 @@
 use PHPUnit\Framework\TestCase;
 use Psr\Container\ContainerInterface;
 use Symfony\Component\Messenger\Envelope;
-use Symfony\Component\Messenger\Stamp\ViaSenderStamp;
+use Symfony\Component\Messenger\Stamp\TransportNamesStamp;
 use Symfony\Component\Messenger\Tests\Fixtures\DummyMessage;
 use Symfony\Component\Messenger\Tests\Fixtures\SecondMessage;
 use Symfony\Component\Messenger\Transport\Sender\SenderInterface;
@@ -36,7 +36,7 @@ public function testItReturnsTheSenderBasedOnTheMessageClass()
         $this->assertSame([], iterator_to_array($locator->getSenders(new Envelope(new SecondMessage()))));
     }
 
-    public function testItReturnsTheSenderBasedOnViaSenderStamp()
+    public function testItReturnsTheSenderBasedOnTransportNamesStamp()
     {
         $mySender = $this->createMock(SenderInterface::class);
         $otherSender = $this->createMock(SenderInterface::class);
@@ -48,7 +48,7 @@ public function testItReturnsTheSenderBasedOnViaSenderStamp()
             DummyMessage::class => ['my_sender'],
         ], $sendersLocator);
 
-        $this->assertSame(['other_sender' => $otherSender], iterator_to_array($locator->getSenders(new Envelope(new DummyMessage('a'), [new ViaSenderStamp(['other_sender'])]))));
+        $this->assertSame(['other_sender' => $otherSender], iterator_to_array($locator->getSenders(new Envelope(new DummyMessage('a'), [new TransportNamesStamp(['other_sender'])]))));
         $this->assertSame([], iterator_to_array($locator->getSenders(new Envelope(new SecondMessage()))));
     }
 
diff --git a/src/Symfony/Component/Messenger/Transport/Sender/SendersLocator.php b/src/Symfony/Component/Messenger/Transport/Sender/SendersLocator.php
@@ -15,7 +15,7 @@
 use Symfony\Component\Messenger\Envelope;
 use Symfony\Component\Messenger\Exception\RuntimeException;
 use Symfony\Component\Messenger\Handler\HandlersLocator;
-use Symfony\Component\Messenger\Stamp\ViaSenderStamp;
+use Symfony\Component\Messenger\Stamp\TransportNamesStamp;
 
 /**
  * Maps a message to a list of senders.
@@ -42,10 +42,8 @@ public function __construct(array $sendersMap, ContainerInterface $sendersLocato
      */
     public function getSenders(Envelope $envelope): iterable
     {
-        if (\count($envelope->all(ViaSenderStamp::class)) > 0) {
-            $viaSenderStamp = $envelope->last(ViaSenderStamp::class);
-
-            foreach ($viaSenderStamp->getSenders() as $senderAlias) {
+        if ($envelope->all(TransportNamesStamp::class)) {
+            foreach ($envelope->last(TransportNamesStamp::class)->getTransportNames() as $senderAlias) {
                 yield from $this->getSenderFromAlias($senderAlias);
             }
 
