feature #31288 [Messenger] RoutableMessageBus route to default bus (dirk39)

This PR was squashed before being merged into the 4.3-dev branch (closes #31288).

Discussion
----------

[Messenger] RoutableMessageBus route to default bus

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #31200
| License       | MIT
| Doc PR        | Not needed

Hi! I modified the behaviour of RoutableMessageBus to route message without BusNameStamp to default bus instead of raise an exception.

Made with ❤️ by [![Open Source Saturday](https://img.shields.io/badge/%E2%9D%A4%EF%B8%8F-open%20source%20saturday-F64060.svg)](https://www.meetup.com/it-IT/Open-Source-Saturday-Milano/)

Commits
-------

c4e5e83b56 [Messenger] RoutableMessageBus route to default bus

Author: fabpot

RoutableMessageBus.php

@@ -45,14 +45,12 @@ public function dispatch($envelope, array $stamps = []): Envelope
 
         /** @var BusNameStamp $busNameStamp */
         $busNameStamp = $envelope->last(BusNameStamp::class);
-        if (null === $busNameStamp) {
-            throw new InvalidArgumentException('Envelope does not contain a BusNameStamp.');
-        }
+        $busName = null !== $busNameStamp ? $busNameStamp->getBusName() : MessageBusInterface::class;
 
-        if (!$this->busLocator->has($busNameStamp->getBusName())) {
-            throw new InvalidArgumentException(sprintf('Invalid bus name "%s" on BusNameStamp.', $busNameStamp->getBusName()));
+        if (!$this->busLocator->has($busName)) {
+            throw new InvalidArgumentException(sprintf('Bus name "%s" does not exists.', $busName));
         }
 
-        return $this->busLocator->get($busNameStamp->getBusName())->dispatch($envelope, $stamps);
+        return $this->busLocator->get($busName)->dispatch($envelope, $stamps);
     }
 }

Tests/RoutableMessageBusTest.php

@@ -41,15 +41,35 @@ public function testItRoutesToTheCorrectBus()
         $this->assertSame($envelope, $routableBus->dispatch($envelope, [$stamp]));
     }
 
-    public function testItExceptionOnMissingStamp()
+    public function testItRoutesToDefaultBus()
+    {
+        $envelope = new Envelope(new \stdClass());
+        $stamp = new DelayStamp(5);
+        $defaultBus = $this->createMock(MessageBusInterface::class);
+        $defaultBus->expects($this->once())->method('dispatch')->with($envelope, [$stamp])
+            ->willReturn($envelope);
+
+        $container = $this->createMock(ContainerInterface::class);
+        $container->expects($this->once())->method('has')->with(MessageBusInterface::class)
+            ->willReturn(true);
+        $container->expects($this->once())->method('get')->with(MessageBusInterface::class)
+            ->willReturn($defaultBus);
+
+        $routableBus = new RoutableMessageBus($container);
+
+        $this->assertSame($envelope, $routableBus->dispatch($envelope, [$stamp]));
+    }
+
+    public function testItExceptionOnDefaultBusNotFound()
     {
         $this->expectException(InvalidArgumentException::class);
-        $this->expectExceptionMessage('does not contain a BusNameStamp');
+        $this->expectExceptionMessage(sprintf('Bus name "%s" does not exists.', MessageBusInterface::class));
 
         $envelope = new Envelope(new \stdClass());
 
         $container = $this->createMock(ContainerInterface::class);
-        $container->expects($this->never())->method('has');
+        $container->expects($this->once())->method('has')->with(MessageBusInterface::class)
+            ->willReturn(false);
 
         $routableBus = new RoutableMessageBus($container);
         $routableBus->dispatch($envelope);
@@ -58,7 +78,7 @@ public function testItExceptionOnMissingStamp()
     public function testItExceptionOnBusNotFound()
     {
         $this->expectException(InvalidArgumentException::class);
-        $this->expectExceptionMessage('Invalid bus name');
+        $this->expectExceptionMessage(sprintf('Bus name "%s" does not exists.', 'foo_bus'));
 
         $envelope = new Envelope(new \stdClass(), [new BusNameStamp('foo_bus')]);