Merge pull request #4 from flotzilla/tests-fix

Minor fixes

src/Channel/Channel.php

@@ -51,7 +51,7 @@ public function __construct(
         string $minLogLevel = null
     )
     {
-        if (!$channelName){
+        if (!$channelName) {
             throw new InvalidChannelNameException();
         }
 
@@ -82,7 +82,7 @@ public function __construct(
     private function setHandlers(array $handlers): void
     {
         foreach ($handlers as $handler) {
-            if (!$handler instanceof HandlerInterface){
+            if (!$handler instanceof HandlerInterface) {
                 throw new InvalidConfigurationException('Array arguments should be instance of HandlerInterface');
             }
 

src/Handler/ConsoleHandler.php

@@ -33,7 +33,7 @@ public function handle(
     {
         $formattedMessage = $this->formatter->format($message, $context, $level, $date);
 
-        echo $formattedMessage;
+        echo $formattedMessage . PHP_EOL;
 
         return true;
     }

src/Handler/FileHandler.php

@@ -82,7 +82,7 @@ public function handle(
     ): bool
     {
         $formattedMessage = $this->formatter->format($message, $context, $level, $date);
-        if (!$isSuccess = $this->appendLog($formattedMessage . PHP_EOL)){
+        if (!$isSuccess = $this->appendLog($formattedMessage . PHP_EOL)) {
             throw new HandlerException('Error during witting log message to file', $message, $context, $level, $date);
         }
 
@@ -127,12 +127,11 @@ private function appendLog(string $log): bool
             return $result;
         }
 
-        if ($file = fopen($this->fileName, 'a')) {
+        if ($file = fopen($this->fileName, 'w')) {
             $result = fwrite($file, $log) !== false;
+            fclose($file);
         }
 
-        fclose($file);
-
         return $result;
     }
 

src/Helper/Helper.php

@@ -14,7 +14,7 @@ class Helper
      */
     public static function isTimeFormatValid(string $format): bool
     {
-        return \DateTime::createFromFormat($format , date($format)) != false;
+        return \DateTime::createFromFormat($format, date($format)) != false;
     }
 
     /**

src/Logger.php

@@ -105,13 +105,12 @@ public function getChannels(): array
      */
     public function setChannels(array $channels): void
     {
-        foreach ($channels as $channel)
-        {
-            if (!$channel instanceof ChannelInterface){
+        foreach ($channels as $channel) {
+            if (!$channel instanceof ChannelInterface) {
                 throw new InvalidConfigurationException('Array arguments should be instance of ChannelInterface');
             }
 
-            $this->addChannel($channel);;
+            $this->addChannel($channel);
         }
     }
 
@@ -121,11 +120,9 @@ public function setChannels(array $channels): void
      */
     public function addChannel(ChannelInterface $channel): void
     {
-        if (array_key_exists($channel->getChannelName(), $this->channels))
-        {
+        if (array_key_exists($channel->getChannelName(), $this->channels)) {
             throw new InvalidConfigurationException(
-                "Channel with name {$channel->getChannelName()} already exist in runtime")
-            ;
+                "Channel with name {$channel->getChannelName()} already exist in runtime");
         }
 
         $this->channels[$channel->getChannelName()] = $channel;
@@ -137,7 +134,7 @@ public function addChannel(ChannelInterface $channel): void
      */
     public function getChannel(string $name): ?ChannelInterface
     {
-        return array_key_exists($name, $this->channels)? $this->channels[$name]: new NullChannel;
+        return array_key_exists($name, $this->channels) ? $this->channels[$name] : new NullChannel;
     }
 
     /**