fixed tests

Author: mmelograno

composer.json

@@ -31,7 +31,8 @@
   "require-dev": {
     "phpunit/phpunit": "^9.0.0",
     "squizlabs/php_codesniffer": "3.*",
-    "rogervila/php-sonarqube-scanner": "1.1.0"
+    "rogervila/php-sonarqube-scanner": "1.1.0",
+    "monolog/monolog": "^2.8|^3.0"
   },
 
   "autoload": {

src/SplitIO/Component/Initialization/LoggerFactory.php

@@ -8,12 +8,16 @@
 
 class LoggerFactory
 {
+    private static function setDefaultLogger(array $options) {
+        $adapter = (isset($options['adapter'])) ? $options['adapter'] : null;
+        $level = (isset($options['level'])) ? $options['level'] : null;
+
+        LoggerTrait::addLogger($adapter, $level);
+    }
+
     public static function setupLogger(array $options) {
         if (!isset($options['psr3-instance'])) {
-            $adapter = (isset($options['adapter'])) ? $options['adapter'] : null;
-            $level = (isset($options['level'])) ? $options['level'] : null;
-
-            LoggerTrait::addLogger($adapter, $level);
+            self::setDefaultLogger($options);
             return;
         }
 
@@ -33,10 +37,7 @@ public static function setupLogger(array $options) {
                 LoggerTrait::addLogger(null, null, new Logger(new LoggerAdapterPSR3v1($options['psr3-instance'])));
                 break;
             default:
-                $adapter = (isset($options['adapter'])) ? $options['adapter'] : null;
-                $level = (isset($options['level'])) ? $options['level'] : null;
-
-                LoggerTrait::addLogger($adapter, $level);
+                self::setDefaultLogger($options);
         }
     }
 }

src/SplitIO/Component/Log/LoggerAdapterPSR3v2.php

@@ -36,7 +36,7 @@ public function write($logLevel, $message)
             if (!is_string($message) || !$message instanceof Stringable) {
                 $message = json_encode($message);
             }
-            $this->logger->write($logLevel, $message);
+            $this->logger->log($logLevel, $message);
         } catch (\Exception $e) {
         }
     }

src/SplitIO/Component/Log/LoggerAdapterPSR3v3.php

@@ -36,7 +36,7 @@ public function write($logLevel, $message)
             if (!is_string($message) || !$message instanceof Stringable) {
                 $message = json_encode($message);
             }
-            $this->logger->write($logLevel, $message);
+            $this->logger->log($logLevel, $message);
         } catch (\Exception $e) {
         }
     }

tests/Suite/Sdk/SdkClientTest.php

@@ -2,6 +2,8 @@
 namespace SplitIO\Test\Suite\Sdk;
 
 use \stdClass;
+use Monolog\Logger;
+use Monolog\Handler\ErrorLogHandler;
 use SplitIO\Component\Common\Di;
 use SplitIO\Test\Suite\Redis\ReflectiveTools;
 use SplitIO\Component\Cache\ImpressionCache;
@@ -423,16 +425,16 @@ public function testClient()
      */
     public function testCustomLog()
     {
-        /*
         Di::set(Di::KEY_FACTORY_TRACKER, false);
         // create a log channel
         $log = new Logger('SplitIO');
+        $log->pushHandler(new ErrorLogHandler(ErrorLogHandler::OPERATING_SYSTEM, Logger::INFO));
 
         $parameters = array('scheme' => 'redis', 'host' => REDIS_HOST, 'port' => REDIS_PORT, 'timeout' => 881);
         $options = array('prefix' => TEST_PREFIX);
 
         $sdkConfig = array(
-            'log' => array('psr3-instance' => $log),
+            'log' =>  array('psr3-instance' => $log, 'standard' => 'psr3-v3'),
             'cache' => array('adapter' => 'predis', 'parameters' => $parameters, 'options' => $options)
         );
 
@@ -451,7 +453,6 @@ public function testCustomLog()
 
         $this->assertTrue($splitSdk->isTreatment('user1', 'sample_feature', 'on'));
         $this->assertFalse($splitSdk->isTreatment('user1', 'sample_feature', 'invalid_treatment'));
-        */
     }
 
     public function testInvalidCacheAdapter()