Merge pull request #8 from myLocalInfluence/bug/fix-wrong-logger-type#6

fix: Fix Wrong Logger Type

Author: kangoo13

src/DataDogFormatter.php

@@ -2,8 +2,8 @@
 
 namespace Myli\DatadogLogger;
 
+use Illuminate\Log\Logger;
 use Monolog\Formatter\JsonFormatter;
-use Monolog\Logger;
 
 /**
  * Class DataDogFormatter
@@ -25,6 +25,10 @@ class DataDogFormatter extends JsonFormatter
      */
     public function __invoke(Logger $logger)
     {
+        /**
+         * @var \Monolog\Logger $logger We can assume it's a monolog logger since
+         * Laravel's Logger has _call function calling the logger directly
+         */
         foreach ($logger->getHandlers() as $handler) {
             $handler->setFormatter($this);
         }

tests/DataDogFormatterTest.php

@@ -49,8 +49,9 @@ public function testInvoke()
         $loggerMock = $this->createMock(Logger::class);
         $streamHandler = new StreamHandler('foourl');
         $loggerMock->method('getHandlers')->willReturn([$streamHandler]);
+        $laravelLogger = new \Illuminate\Log\Logger($loggerMock);
         $dataDogFormatter = new \Myli\DatadogLogger\DataDogFormatter();
-        $dataDogFormatter->__invoke($loggerMock);
+        $dataDogFormatter->__invoke($laravelLogger);
         self::assertEquals(DataDogFormatter::class, get_class($streamHandler->getFormatter()));
     }