Default log levels to emergency and critical

subjective-php · Aug 12, 2018 · dddd843 · dddd843
1 parent 43862f9
commit dddd843
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 10 deletions.
diff --git a/src/NewRelicLogger.php b/src/NewRelicLogger.php
@@ -38,9 +38,7 @@ final class NewRelicLogger extends AbstractLogger implements LoggerInterface
      */
     const DEFAULT_OBSERVED_LEVELS = [
         LogLevel::EMERGENCY,
-        LogLevel::ALERT,
         LogLevel::CRITICAL,
-        LogLevel::ERROR,
     ];
 
     /**

diff --git a/tests/NewRelicLoggerTest.php b/tests/NewRelicLoggerTest.php
@@ -20,9 +20,9 @@ final class NewRelicLoggerTest extends \PHPUnit\Framework\TestCase
      */
     public function logWithOnlyMessage()
     {
-        $newRelicAgentMock = $this->getNewRelicAgentMock(LogLevel::ERROR, 'an error message', ['foo' => 'bar']);
+        $newRelicAgentMock = $this->getNewRelicAgentMock(LogLevel::CRITICAL, 'an error message', ['foo' => 'bar']);
         $logger = new NewRelicLogger($newRelicAgentMock);
-        $logger->log(LogLevel::ERROR, 'an error message', ['foo' => 'bar']);
+        $logger->log(LogLevel::CRITICAL, 'an error message', ['foo' => 'bar']);
     }
 
     /**
@@ -34,9 +34,9 @@ public function logWithOnlyMessage()
     public function logWithException()
     {
         $exception = new \RuntimeException();
-        $newRelicAgentMock = $this->getNewRelicAgentMock(LogLevel::ALERT, 'an alert message', [], $exception);
+        $newRelicAgentMock = $this->getNewRelicAgentMock(LogLevel::EMERGENCY, 'an alert message', [], $exception);
         $logger = new NewRelicLogger($newRelicAgentMock);
-        $logger->log(LogLevel::ALERT, 'an alert message', ['exception' => $exception]);
+        $logger->log(LogLevel::EMERGENCY, 'an alert message', ['exception' => $exception]);
     }
 
     /**
@@ -63,15 +63,15 @@ public function logIgnoredLevel()
     public function logWithNonScalarContext()
     {
         $newRelicAgentMock = $this->getNewRelicAgentMock(
-            LogLevel::ERROR,
+            LogLevel::CRITICAL,
             'an error message',
             [
                 'foo' => 'bar',
                 'extra' => var_export(new \StdClass(), true),
             ]
         );
         $logger = new NewRelicLogger($newRelicAgentMock);
-        $logger->log(LogLevel::ERROR, 'an error message', ['foo' => 'bar', 'extra' => new \StdClass()]);
+        $logger->log(LogLevel::CRITICAL, 'an error message', ['foo' => 'bar', 'extra' => new \StdClass()]);
     }
 
     /**
@@ -90,9 +90,9 @@ public function logError()
             $error->getFile(),
             $error->getLine()
         );
-        $newRelicAgentMock = $this->getNewRelicAgentMock(LogLevel::ALERT, 'an error message', [], $exception);
+        $newRelicAgentMock = $this->getNewRelicAgentMock(LogLevel::EMERGENCY, 'an error message', [], $exception);
         $logger = new NewRelicLogger($newRelicAgentMock);
-        $logger->log(LogLevel::ALERT, 'an error message', ['exception' => $error]);
+        $logger->log(LogLevel::EMERGENCY, 'an error message', ['exception' => $error]);
     }
 
     private function getNewRelicAgentMock(