NEW Add getLogger() to MonologErrorHandler and add test for exception without one

src/Logging/MonologErrorHandler.php

@@ -16,19 +16,31 @@ class MonologErrorHandler implements ErrorHandler
      * Set the PSR-3 logger to send errors & exceptions to
      *
      * @param LoggerInterface $logger
+     * @return $this
      */
     public function setLogger(LoggerInterface $logger)
     {
         $this->logger = $logger;
+        return $this;
+    }
+
+    /**
+     * Get the PSR-3 logger to send errors & exceptions to
+     *
+     * @return LoggerInterface
+     */
+    public function getLogger()
+    {
+        return $this->logger;
     }
 
     public function start()
     {
-        if (!$this->logger) {
+        if (!$this->getLogger()) {
             throw new \InvalidArgumentException("No Logger property passed to MonologErrorHandler."
                 . "Is your Injector config correct?");
         }
 
-        MonologHandler::register($this->logger);
+        MonologHandler::register($this->getLogger());
     }
 }

tests/php/Logging/MonologErrorHandlerTest.php

@@ -0,0 +1,19 @@
+<?php
+
+namespace SilverStripe\Logging\Tests;
+
+use SilverStripe\Dev\SapphireTest;
+use SilverStripe\Logging\MonologErrorHandler;
+
+class MonologErrorHandlerTest extends SapphireTest
+{
+    /**
+     * @expectedException \InvalidArgumentException
+     * @expectedExceptionMessageRegExp /No Logger property passed to MonologErrorHandler/
+     */
+    public function testStartThrowsExceptionWithoutLoggerDefined()
+    {
+        $handler = new MonologErrorHandler();
+        $handler->start();
+    }
+}