feat: [Logger] handle logging via channel

Author: koshuang

src/Logger.php

@@ -48,15 +48,31 @@ public function __construct(LoggerInterface $logger, Dispatcher $dispatcher = nu
      */
     protected function writeLog($level, $message, $context)
     {
+        $info = $this->generateExtraContextInfo();
+        $context = array_merge($context, $info);
+
+        parent::writeLog($level, $message, $context);
+    }
+
+    protected function generateExtraContextInfo()
+    {
+        $info = [];
+
         // attach class_path
         // NOTE: it's hardcoded, should find a better way to get caller class
-        $caller = debug_backtrace();
-        $caller = $caller[4];
-        $context['class_path'] = $caller['class'];
+        $stack = debug_backtrace();
+        $caller = $stack[2];
+
+        if ($caller['class'] === 'Illuminate\Log\LogManager') {
+            // It means log from channel
+            $caller = $stack[4];
+        }
+
+        $info['class_path'] = $caller['class'];
 
         // attach tracking_id
-        $context['tracking_id'] = $this->debugId;
+        $info['tracking_id'] = $this->debugId;
 
-        parent::writeLog($level, $message, $context);
+        return $info;
     }
 }

tests/Unit/LoggerTest.php

@@ -2,13 +2,10 @@
 
 namespace Onramplab\LaravelLogEnhancement\Tests\Unit\Concerns;
 
-use Illuminate\Events\Dispatcher;
-use Illuminate\Foundation\Application;
-use Illuminate\Support\Facades\Log;
 use Mockery;
 use Onramplab\LaravelLogEnhancement\Tests\TestCase;
 use Onramplab\LaravelLogEnhancement\Logger;
-use Psr\Log\LoggerInterface;
+use Monolog\Logger as Monolog;
 
 class LoggerTest extends TestCase
 {
@@ -19,20 +16,15 @@ class LoggerTest extends TestCase
      */
     public function log_should_include_class_path_and_uuid_in_context()
     {
-      $this->app->instance(Logger::class, Mockery::mock(Logger::class));
+        $monolog = Mockery::mock(Monolog::class);
+        $logger = new Logger($monolog);
 
-      $logger = app()
-        ->make(Logger::class)
-        ->makePartial();
+        $monolog->shouldReceive('info')->withArgs(function ($message, $context) {
+            return $message === '123'
+                && isSet($context['class_path'])
+                && isSet($context['tracking_id']);
+        })->once();
 
-      $logger->shouldReceive('log', function ($logLevel, $message, $context) {
-          return $logLevel === 'info'
-            && $message === '123'
-            && isSet($context['class_path'])
-            && isSet($context['tracking_id']);
-        })
-        ->once();
-
-      $logger->info('123');
+        $logger->info('123');
     }
 }