From 56b99ffff11fa8c2da057c6132c38b0be9fb9e3c Mon Sep 17 00:00:00 2001 From: Avior Date: Wed, 17 Apr 2019 01:29:57 +0200 Subject: [PATCH] Finished Logger tests --- src/AdminPanel/Logger.php | 2 +- tests/LoggerTest.php | 34 +++++++++++++++++++++++++++++++++- 2 files changed, 34 insertions(+), 2 deletions(-) diff --git a/src/AdminPanel/Logger.php b/src/AdminPanel/Logger.php index 59192b6..b228b50 100644 --- a/src/AdminPanel/Logger.php +++ b/src/AdminPanel/Logger.php @@ -59,7 +59,7 @@ class Logger extends AbstractLogger file_put_contents($this->file, $this->fillMessage($this->start . $this->exception, $context), FILE_APPEND); } if ($message === null && !isset($context["exception"])) { - throw new InvalidArgumentException("no exception nor message was found"); + throw new InvalidArgumentException("Message and Exception not set"); } } } diff --git a/tests/LoggerTest.php b/tests/LoggerTest.php index 87c0234..3bc967f 100644 --- a/tests/LoggerTest.php +++ b/tests/LoggerTest.php @@ -3,6 +3,7 @@ use PHPUnit\Framework\TestCase; use AdminPanel\Cache\FileCache; use AdminPanel\Logger; +use Psr\Log\InvalidArgumentException; final class LoggerTest extends TestCase { @@ -29,7 +30,6 @@ final class LoggerTest extends TestCase unlink($file); } $logger = new Logger($file); - $this->assertFileExists($file); $exception = new Exception("hello phpunit"); $logger->alert("test", array( @@ -40,4 +40,36 @@ final class LoggerTest extends TestCase "[Alert]: " . (new \DateTime())->format("Y-m-d H:i:s") . " test\n[Alert]: " . (new \DateTime())->format("Y-m-d H:i:s") . " 0 " . $exception->getFile() . "[Exception] 0 " . $exception->getMessage() . "\n" ); } + + public function testLevelException() + { + $file = "tests/logs.log"; + if (file_exists($file)) { + unlink($file); + } + $logger = new Logger($file); + try { + $logger->log('incorrect_level', "level incorrect"); + } catch (InvalidArgumentException $e) { + $this->assertEquals($e->getMessage(), "Level not supported"); + return; + } + $this->fail(); + } + + public function testNothingException() + { + $file = "tests/logs.log"; + if (file_exists($file)) { + unlink($file); + } + $logger = new Logger($file); + try { + $logger->log("info"); + } catch (InvalidArgumentException $e) { + $this->assertEquals($e->getMessage(), "Message and Exception not set"); + return; + } + $this->fail(); + } }