Add tests for log level overrides

mtibben · mtibben · commit c3d15fad8354 · 2017-12-27T16:12:26.000+11:00
diff --git a/src/Symfony/Component/HttpKernel/Tests/EventListener/ExceptionListenerTest.php b/src/Symfony/Component/HttpKernel/Tests/EventListener/ExceptionListenerTest.php
@@ -19,6 +19,7 @@
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
 use Symfony\Component\HttpKernel\Tests\Logger;
+use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
 
 /**
  * ExceptionListenerTest.
@@ -126,6 +127,44 @@ public function testSubRequestFormat()
         $response = $event->getResponse();
         $this->assertEquals('xml', $response->getContent());
     }
+
+    public function testHttp4xxLogLevel()
+    {
+        $logger = new TestLogger();
+        $l = new ExceptionListener('foo', $logger);
+        $exception = new \Exception('foo');
+        $event = new GetResponseForExceptionEvent(new TestKernelThatThrowsHttp4xxException(), Request::create('/'), HttpKernelInterface::MASTER_REQUEST, $exception);
+        try {
+            $l->logKernelException($event);
+            $l->onKernelException($event);
+            $this->fail('NotFoundHttpException expected');
+        } catch (NotFoundHttpException $e) {
+            $this->assertSame('4xx', $e->getMessage());
+            $this->assertSame('foo', $e->getPrevious()->getMessage());
+        }
+
+        $this->assertEquals(1, $logger->countErrors());
+        $this->assertCount(1, $logger->getLogs('warning'));
+    }
+
+    public function testHttpLogLevelOverride()
+    {
+        $logger = new TestLogger();
+        $l = new ExceptionListener('foo', $logger, array(404 => 'notice'));
+        $exception = new \Exception('foo');
+        $event = new GetResponseForExceptionEvent(new TestKernelThatThrowsHttp4xxException(), Request::create('/'), HttpKernelInterface::MASTER_REQUEST, $exception);
+        try {
+            $l->logKernelException($event);
+            $l->onKernelException($event);
+            $this->fail('NotFoundHttpException expected');
+        } catch (NotFoundHttpException $e) {
+            $this->assertSame('4xx', $e->getMessage());
+            $this->assertSame('foo', $e->getPrevious()->getMessage());
+        }
+
+        $this->assertEquals(1, $logger->countErrors());
+        $this->assertCount(1, $logger->getLogs('notice'));
+    }
 }
 
 class TestLogger extends Logger implements DebugLoggerInterface
@@ -151,3 +190,11 @@ public function handle(Request $request, $type = self::MASTER_REQUEST, $catch =
         throw new \RuntimeException('bar');
     }
 }
+
+class TestKernelThatThrowsHttp4xxException implements HttpKernelInterface
+{
+    public function handle(Request $request, $type = self::MASTER_REQUEST, $catch = true)
+    {
+        throw new NotFoundHttpException('4xx');
+    }
+}
