Make logger arg a hard dependency, remove dead code and add tests

Author: yceruto

Debug/LoggingMiddleware.php

@@ -21,7 +21,7 @@ class LoggingMiddleware implements MiddlewareInterface
 {
     private $logger;
 
-    public function __construct(LoggerInterface $logger = null)
+    public function __construct(LoggerInterface $logger)
     {
         $this->logger = $logger;
     }
@@ -31,10 +31,6 @@ public function __construct(LoggerInterface $logger = null)
      */
     public function handle($message, callable $next)
     {
-        if (null === $this->logger) {
-            return $next($message);
-        }
-
         $this->logger->debug('Starting handling message {class}', array(
             'message' => $message,
             'class' => \get_class($message),

DependencyInjection/MessengerPass.php

@@ -49,7 +49,7 @@ public function process(ContainerBuilder $container)
             return;
         }
 
-        if (!$container->getParameter('kernel.debug') || !$container->has('logger')) {
+        if (!$container->getParameter('kernel.debug') || !$container->hasAlias('logger')) {
             $container->removeDefinition('messenger.middleware.debug.logging');
         }
 

Tests/Debug/LoggingMiddlewareTest.php

@@ -0,0 +1,69 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\Component\Messenger\Tests\Debug;
+
+use PHPUnit\Framework\TestCase;
+use Psr\Log\LoggerInterface;
+use Symfony\Component\Messenger\Debug\LoggingMiddleware;
+use Symfony\Component\Messenger\Tests\Fixtures\DummyMessage;
+
+class LoggingMiddlewareTest extends TestCase
+{
+    public function testDebugLogAndNextMiddleware()
+    {
+        $message = new DummyMessage('Hey');
+
+        $logger = $this->createMock(LoggerInterface::class);
+        $logger
+            ->expects($this->exactly(2))
+            ->method('debug')
+        ;
+        $next = $this->createPartialMock(\stdClass::class, array('__invoke'));
+        $next
+            ->expects($this->once())
+            ->method('__invoke')
+            ->with($message)
+            ->willReturn('Hello')
+        ;
+
+        $result = (new LoggingMiddleware($logger))->handle($message, $next);
+
+        $this->assertSame('Hello', $result);
+    }
+
+    /**
+     * @expectedException \Exception
+     */
+    public function testWarningLogOnException()
+    {
+        $message = new DummyMessage('Hey');
+
+        $logger = $this->createMock(LoggerInterface::class);
+        $logger
+            ->expects($this->once())
+            ->method('debug')
+        ;
+        $logger
+            ->expects($this->once())
+            ->method('warning')
+        ;
+        $next = $this->createPartialMock(\stdClass::class, array('__invoke'));
+        $next
+            ->expects($this->once())
+            ->method('__invoke')
+            ->with($message)
+            ->willThrowException(new \Exception())
+        ;
+
+        (new LoggingMiddleware($logger))->handle($message, $next);
+    }
+}

Tests/DependencyInjection/MessengerPassTest.php

@@ -44,6 +44,8 @@ public function testProcess()
 
         (new MessengerPass())->process($container);
 
+        $this->assertFalse($container->hasDefinition('messenger.middleware.debug.logging'));
+
         $handlerLocatorDefinition = $container->getDefinition($container->getDefinition('messenger.handler_resolver')->getArgument(0));
         $this->assertSame(ServiceLocator::class, $handlerLocatorDefinition->getClass());
         $this->assertEquals(

composer.json

@@ -19,14 +19,14 @@
         "php": "^7.1.3"
     },
     "require-dev": {
-        "symfony/serializer": "~3.4|~4.0",
+        "psr/log": "~1.0",
         "symfony/dependency-injection": "~3.4.6|~4.0",
         "symfony/http-kernel": "~3.4|~4.0",
+        "symfony/process": "~3.4|~4.0",
         "symfony/property-access": "~3.4|~4.0",
-        "symfony/var-dumper": "~3.4|~4.0",
-        "symfony/property-access": "~3.4|~4.0",
-        "symfony/process": "~4.0",
-        "symfony/validator": "~4.0"
+        "symfony/serializer": "~3.4|~4.0",
+        "symfony/validator": "~3.4|~4.0",
+        "symfony/var-dumper": "~3.4|~4.0"
     },
     "suggest": {
         "sroze/enqueue-bridge": "For using the php-enqueue library as an adapter."