fix(log): Fix empty logs

Author: julienloizelet

tests/GeolocationTest.php

@@ -163,7 +163,7 @@ public function testCanVerifyIpAndCountryWithMaxmindInStreamMode(array $maxmindC
         // Warm BlockList cache up
         $bouncer->refreshBlocklistCache();
 
-        $this->logger->debug('Refresh the cache just after the warm up. Nothing should append.');
+        $this->logger->debug('', ['message' =>'Refresh the cache just after the warm up. Nothing should append.']);
         $bouncer->refreshBlocklistCache();
 
         $this->assertEquals(

tests/IpVerificationTest.php

@@ -111,13 +111,13 @@ public function testCanVerifyIpInLiveModeWithCacheSystem(AbstractAdapter $cacheA
         unset($bouncerConfig['max_remediation_level']);
         $bouncer->configure($bouncerConfig);
 
-        $this->logger->info('set "Large IPV4 range banned" state');
+        $this->logger->info('', ['message' => 'set "Large IPV4 range banned" state']);
         $this->watcherClient->deleteAllDecisions();
         $this->watcherClient->addDecision(new \DateTime(), '24h', '+24 hours', TestHelpers::BAD_IP.'/'.TestHelpers::LARGE_IPV4_RANGE, 'ban');
         $cappedRemediation = $bouncer->getRemediationForIp(TestHelpers::BAD_IP);
         $this->assertEquals('ban', $cappedRemediation, 'The remediation for the banned IP with a too large range should now be "ban" as we are in live mode');
 
-        $this->logger->info('set "IPV6 range banned" state');
+        $this->logger->info('',['message' => 'set "IPV6 range banned" state']);
         $this->watcherClient->deleteAllDecisions();
         $this->watcherClient->addDecision(new \DateTime(), '24h', '+24 hours', TestHelpers::BAD_IPV6.'/'.TestHelpers::IPV6_RANGE, 'ban');
         $cappedRemediation = $bouncer->getRemediationForIp(TestHelpers::BAD_IPV6);
@@ -163,7 +163,7 @@ public function testCanVerifyIpInStreamModeWithCacheSystem(AbstractAdapter $cach
 
         $bouncer->refreshBlocklistCache();
 
-        $this->logger->debug('Refresh the cache just after the warm up. Nothing should append.');
+        $this->logger->debug('', ['message' => 'Refresh the cache just after the warm up. Nothing should append.']);
         $bouncer->refreshBlocklistCache();
 
         $this->assertEquals(
@@ -199,10 +199,10 @@ public function testCanVerifyIpInStreamModeWithCacheSystem(AbstractAdapter $cach
         // Pull updates
         $bouncer->refreshBlocklistCache();
 
-        $this->logger->debug('Refresh 2nd time the cache. Nothing should append.');
+        $this->logger->debug('', ['message' => 'Refresh 2nd time the cache. Nothing should append.']);
         $bouncer->refreshBlocklistCache();
 
-        $this->logger->debug('Refresh 3rd time the cache. Nothing should append.');
+        $this->logger->debug('', ['message' => 'Refresh 3rd time the cache. Nothing should append.']);
         $bouncer->refreshBlocklistCache();
 
         $this->assertEquals(
@@ -245,7 +245,7 @@ public function testCanVerifyIpInStreamModeWithCacheSystem(AbstractAdapter $cach
             'The cache warm up should be stored across each instanciation'
         );
 
-        $this->logger->info('set "Large IPV4 range banned" + "IPV6 range banned" state');
+        $this->logger->info('', ['message' => 'set "Large IPV4 range banned" + "IPV6 range banned" state']);
         $this->watcherClient->deleteAllDecisions();
         $this->watcherClient->addDecision(new \DateTime(), '24h', '+24 hours', TestHelpers::BAD_IP.'/'.TestHelpers::LARGE_IPV4_RANGE, 'ban');
         $this->watcherClient->addDecision(new \DateTime(), '24h', '+24 hours', TestHelpers::BAD_IPV6.'/'.TestHelpers::IPV6_RANGE, 'ban');

tests/WatcherClient.php

@@ -45,13 +45,13 @@ public function configure(): void
         ];
         $this->watcherClient = new RestClient($this->logger);
         $this->watcherClient->configure($apiUrl, $this->baseHeaders, 2);
-        $this->logger->info('Watcher client initialized');
+        $this->logger->info('', ['message' => 'Watcher client initialized']);
     }
 
     /** Set the initial watcher state */
     public function setInitialState(): void
     {
-        $this->logger->info('Set initial state');
+        $this->logger->info('', ['message' => 'Set initial state']);
         $this->deleteAllDecisions();
         $now = new DateTime();
         $this->addDecision($now, '12h', '+12 hours', TestHelpers::BAD_IP, 'captcha');
@@ -62,7 +62,7 @@ public function setInitialState(): void
     /** Set the second watcher state */
     public function setSecondState(): void
     {
-        $this->logger->info('Set "second" state');
+        $this->logger->info('', ['message' => 'Set "second" state']);
         $this->deleteAllDecisions();
         $now = new DateTime();
         $this->addDecision($now, '36h', '+36 hours', TestHelpers::NEWLY_BAD_IP, 'ban');
@@ -102,7 +102,7 @@ private function request(string $endpoint, array $queryParams = null, array $bod
     public function deleteAllDecisions(): void
     {
         // Delete all existing decisions.
-        $this->logger->info('Delete all decisions');
+        $this->logger->info('', ['message' => 'Delete all decisions']);
         $this->request('/v1/decisions', null, null, 'DELETE');
     }
 
@@ -146,6 +146,6 @@ public function addDecision(DateTime $now, string $durationString, string $dateT
             'stop_at' => $stopAt,
           ];
         $result = $this->request('/v1/alerts', null, [$body], 'POST');
-        $this->logger->info('Decision '.$result[0].' added: '.$body['decisions'][0]['scenario'].'');
+        $this->logger->info('', ['message' => 'Decision '.$result[0].' added: '.$body['decisions'][0]['scenario'].'']);
     }
 }