Using the context level logger in a message handler

php-service-bus · Jan 15, 2020 · e0bf54b · e0bf54b
1 parent 4a6db29
commit e0bf54b
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 18 deletions.
diff --git a/tests/Application/Bootstrap/services/TestBootstrapService.php b/tests/Application/Bootstrap/services/TestBootstrapService.php
@@ -31,7 +31,7 @@ public function do(TestBootstrapCommand $command, ServiceBusContext $context): v
  /**
  * @EventListener()
  */
- public function when(TestBootstrapEvent $event, ServiceBusContext $context): void
+ public function when(TestBootstrapEven $event, ServiceBusContext $context): void
  {
  }
 }
diff --git a/tests/EntryPoint/EntryPointTest.php b/tests/EntryPoint/EntryPointTest.php
@@ -31,7 +31,6 @@
 use ServiceBus\Services\Configuration\ServiceMessageHandler;
 use ServiceBus\Transport\Common\Queue;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
-use function Amp\delay;
 use function ServiceBus\Tests\filterLogMessages;
 
 /**
@@ -77,8 +76,7 @@ protected function setUp(): void
  new MessageArgumentResolver(),
  new ContextArgumentResolver(),
  new ContainerArgumentResolver($containerBuilder)
- ],
- $this->logger
+ ]
  );
 
  $messageRouter = new Router();

diff --git a/tests/MessageExecutor/DefaultMessageExecutorTest.php b/tests/MessageExecutor/DefaultMessageExecutorTest.php
@@ -70,15 +70,16 @@ static function () use (&$variable): void
  $closure,
  new \SplObjectStorage(),
  DefaultHandlerOptions::createForCommandHandler('some description'),
- $this->argumentResolvers,
- $this->logger
+ $this->argumentResolvers
  );
 
- wait($processor(new EmptyMessage(), new TestContext()));
+ $context = new TestContext();
+
+ wait($processor(new EmptyMessage(), $context));
 
  static::assertSame('handled', $variable);
 
- $messages = filterLogMessages($this->logHandler);
+ $messages = filterLogMessages($context->testLogHandler);
 
  static::assertContains('some description', $messages);
  }
@@ -102,8 +103,7 @@ static function (): void
  $closure,
  new \SplObjectStorage(),
  $options,
- $this->argumentResolvers,
- $this->logger
+ $this->argumentResolvers
  );
 
  wait($processor(new EmptyMessage(), $context));
@@ -133,8 +133,7 @@ static function (): void
  $closure,
  new \SplObjectStorage(),
  DefaultHandlerOptions::createForCommandHandler('some description'),
- $this->argumentResolvers,
- $this->logger
+ $this->argumentResolvers
  );
 
  wait($processor(new EmptyMessage(), new TestContext()));

diff --git a/tests/MessageExecutor/MessageValidationExecutorTest.php b/tests/MessageExecutor/MessageValidationExecutorTest.php
@@ -94,8 +94,7 @@ static function (): void
  $closure,
  new \SplObjectStorage(),
  $options,
- $this->argumentResolvers,
- $this->logger
+ $this->argumentResolvers
  ),
  $options,
  $this->validator
@@ -127,8 +126,7 @@ static function (): void
  $closure,
  new \SplObjectStorage(),
  $options,
- $this->argumentResolvers,
- $this->logger
+ $this->argumentResolvers
  ),
  $options,
  $this->validator
@@ -165,8 +163,7 @@ static function (): void
  $closure,
  new \SplObjectStorage(),
  $options,
- $this->argumentResolvers,
- $this->logger
+ $this->argumentResolvers
  ),
  $options,
  $this->validator