Skip to content

Commit 374ee55

Browse files
committed
Use CommandBus interface instead of concrete implementation
1 parent 3daa279 commit 374ee55

File tree

3 files changed

+8
-7
lines changed

3 files changed

+8
-7
lines changed

src/Client.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
namespace ApiClients\Foundation;
44

5-
use ApiClients\Tools\CommandBus\CommandBus;
5+
use ApiClients\Tools\CommandBus\CommandBusInterface;
66
use Interop\Container\ContainerInterface;
77
use React\Promise\CancellablePromiseInterface;
88

@@ -14,7 +14,7 @@ final class Client
1414
private $container;
1515

1616
/**
17-
* @var CommandBus
17+
* @var CommandBusInterface
1818
*/
1919
private $commandBus;
2020

@@ -24,7 +24,7 @@ final class Client
2424
public function __construct(ContainerInterface $container)
2525
{
2626
$this->container = $container;
27-
$this->commandBus = $this->container->get(CommandBus::class);
27+
$this->commandBus = $this->container->get(CommandBusInterface::class);
2828
}
2929

3030
/**

src/Factory.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
use ApiClients\Foundation\Hydrator\Hydrator;
77
use ApiClients\Foundation\Transport\Client as TransportClient;
88
use ApiClients\Foundation\Transport\Factory as TransportFactory;
9-
use ApiClients\Tools\CommandBus\CommandBus;
9+
use ApiClients\Tools\CommandBus\CommandBusInterface;
1010
use ApiClients\Tools\CommandBus\Factory as CommandBusFactory;
1111
use DI\ContainerBuilder;
1212
use Interop\Container\ContainerInterface;
@@ -39,7 +39,7 @@ private static function createContainer(LoopInterface $loop, array $options): Co
3939
Hydrator::class => function (ContainerInterface $container) use ($options) {
4040
return self::createHydrator($container, $options);
4141
},
42-
CommandBus::class => function (ContainerInterface $container) {
42+
CommandBusInterface::class => function (ContainerInterface $container) {
4343
return CommandBusFactory::create($container);
4444
},
4545
]);

tests/ClientTest.php

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44

55
use ApiClients\Foundation\Client;
66
use ApiClients\Tools\CommandBus\CommandBus;
7+
use ApiClients\Tools\CommandBus\CommandBusInterface;
78
use ApiClients\Tools\TestUtilities\TestCase;
89
use DI\ContainerBuilder;
910
use League\Tactician\Handler\CommandHandlerMiddleware;
@@ -41,11 +42,11 @@ public function handle($command)
4142
$commandBus = new CommandBus($loop, $handlerMiddleware);
4243

4344
$container = ContainerBuilder::buildDevContainer();
44-
$container->set(CommandBus::class, $commandBus);
45+
$container->set(CommandBusInterface::class, $commandBus);
4546
$client = new Client($container);
4647

4748
$this->assertSame($container, $client->getContainer());
48-
$this->assertSame($commandBus, $client->getFromContainer(CommandBus::class));
49+
$this->assertSame($commandBus, $client->getFromContainer(CommandBusInterface::class));
4950
$this->assertSame($command, await($client->handle($command), $loop));
5051
}
5152

0 commit comments

Comments
 (0)