Add logger() helper

commit 622f16d
Author: Adam Tomat <[email protected]>
Date:   Mon Jul 22 10:45:24 2019 +0100

    Fix bug with logger() helper test

commit 1320d23
Author: Adam Tomat <[email protected]>
Date:   Wed Jul 17 13:12:14 2019 +0100

    Add logger helper

Author: joelambert

src/Helpers.php

@@ -9,6 +9,7 @@
 use Rareloop\Lumberjack\Http\Responses\RedirectResponse;
 use Rareloop\Lumberjack\Http\Responses\TimberResponse;
 use Rareloop\Lumberjack\Facades\Session;
+use Rareloop\Lumberjack\Facades\Log;
 
 class Helpers
 {
@@ -80,4 +81,13 @@ public static function back()
     {
         return static::redirect(static::session()->previousUrl());
     }
+
+    public static function logger($message = null, $context = [])
+    {
+        if (is_null($message)) {
+            return static::app('logger');
+        }
+
+        return Log::debug($message, $context);
+    }
 }

src/functions.php

@@ -64,3 +64,10 @@ function request()
         return call_user_func_array([Helpers::class, 'request'], func_get_args());
     }
 }
+
+if (!function_exists('logger')) {
+    function logger()
+    {
+        return call_user_func_array([Helpers::class, 'logger'], func_get_args());
+    }
+}

tests/Unit/HelpersTest.php

@@ -17,6 +17,7 @@
 use Rareloop\Lumberjack\Exceptions\HandlerInterface;
 use Rareloop\Lumberjack\Http\Responses\TimberResponse;
 use Rareloop\Lumberjack\Http\Responses\RedirectResponse;
+use Monolog\Logger;
 
 /**
  * @runTestsInSeparateProcesses
@@ -140,8 +141,7 @@ public function can_get_a_url_for_a_named_route()
         $app = new Application;
         FacadeFactory::setContainer($app);
         $router = new Router;
-        $router->get('test/route', function () {
-        })->name('test.route');
+        $router->get('test/route', function () { })->name('test.route');
         $app->bind('router', $router);
 
         $url = Helpers::route('test.route');
@@ -155,8 +155,7 @@ public function can_get_a_url_for_a_named_route_with_params()
         $app = new Application;
         FacadeFactory::setContainer($app);
         $router = new Router;
-        $router->get('test/{name}', function ($name) {
-        })->name('test.route');
+        $router->get('test/{name}', function ($name) { })->name('test.route');
         $app->bind('router', $router);
 
         $url = Helpers::route('test.route', [
@@ -306,11 +305,58 @@ public function can_get_server_request()
 
         $this->assertSame($request, Helpers::request());
     }
+
+    /** @test */
+    public function can_get_logger()
+    {
+        $app = new Application;
+        FacadeFactory::setContainer($app);
+
+        $logger = new Logger('app');
+        $app->bind('logger', $logger);
+
+        $newLogger = Helpers::logger();
+
+        $this->assertInstanceOf(Logger::class, $newLogger);
+        $this->assertSame($logger, $newLogger);
+    }
+
+    /** @test */
+    public function can_write_debug_log()
+    {
+        $app = new Application;
+        FacadeFactory::setContainer($app);
+
+        $logger = \Mockery::mock(Logger::class)->makePartial();
+        $logger->shouldReceive('debug')->with('Example message', [])->once();
+
+        $app->bind('logger', $logger);
+
+        Helpers::logger('Example message');
+    }
+
+    /** @test */
+    public function can_write_debug_log_with_context()
+    {
+        $app = new Application;
+        FacadeFactory::setContainer($app);
+
+        $logger = \Mockery::mock(Logger::class)->makePartial();
+        $logger->shouldReceive('debug')->with('Example message', [
+            'test' => 123,
+        ])->once();
+
+        $app->bind('logger', $logger);
+
+        Helpers::logger('Example message', [
+            'test' => 123,
+        ]);
+    }
 }
 
 class TestExceptionHandler extends Handler
 {
-
+    // ...
 }
 
 class RequiresConstructorParams