Merge pull request #8 from keboola/adamvyborny-COM-1354-log-retry-as-info

Change retry log level to info

Author: AdamVyborny

src/Google/RestApi.php

@@ -302,7 +302,7 @@ protected function logRetryRequest(
                 ];
             }
 
-            $this->logger->warning(sprintf('Retrying request (%sx)', $retries), $context);
+            $this->logger->info(sprintf('Retrying request (%sx)', $retries), $context);
         }
     }
 

tests/RestApiTest.php

@@ -4,7 +4,9 @@
 
 namespace Keboola\Google\ClientBundle\Tests;
 
+use GuzzleHttp\Exception\ClientException;
 use Keboola\Google\ClientBundle\Google\RestApi;
+use Monolog\Handler\TestHandler;
 use Monolog\Logger;
 use PHPUnit\Framework\TestCase;
 use SebastianBergmann\Timer\Timer;
@@ -23,12 +25,17 @@ class RestApiTest extends TestCase
     /** @var Logger */
     private $logger;
 
+    /** @var TestHandler */
+    private $testHandler;
+
     protected function initApi(): RestApi
     {
         $this->clientId = $this->getEnv('CLIENT_ID');
         $this->clientSecret = $this->getEnv('CLIENT_SECRET');
         $this->refreshToken = $this->getEnv('REFRESH_TOKEN');
+        $this->testHandler = new TestHandler();
         $this->logger = new Logger('Google Rest API tests');
+        $this->logger->pushHandler($this->testHandler);
 
         return new RestApi(
             $this->clientId,
@@ -99,6 +106,22 @@ public function testDelayFn(): void
         $this->assertGreaterThan(5, $time);
     }
 
+    public function testRetries(): void
+    {
+        $restApi = $this->initApi();
+        try {
+            $restApi->request('/auth/invalid-scope');
+        } catch (ClientException $e) {
+        }
+
+        $this->assertNotCount(0, $this->testHandler->getRecords());
+
+        foreach ($this->testHandler->getRecords() as $key => $value) {
+            $this->assertEquals(Logger::INFO, $value['level']);
+            $this->assertEquals(sprintf('Retrying request (%dx)', $key), $value['message']);
+        }
+    }
+
     protected function getEnv(string $name): string
     {
         $value = getenv($name);