Do not retry on 401 and its test

Author: AdamVyborny

src/Google/RestApi.php

@@ -319,7 +319,7 @@ protected function decideRetry(int $retries, int $maxRetries, ?ResponseInterface
             if ($statusCode >= 200 && $statusCode < 300) {
                 return false;
             }
-            if ($statusCode === 400) {
+            if ($statusCode === 400 || $statusCode === 401) {
                 return false;
             }
             if ($statusCode === 403) {

tests/RestApiTest.php

@@ -150,6 +150,29 @@ public function testRetries(): void
         }
     }
 
+    public function testDoNotRetryOnWrongCredentials(): void
+    {
+        $testHandler = new TestHandler();
+        $logger = new Logger('Google Rest API tests');
+        $logger->pushHandler($testHandler);
+        $api = new RestApi(
+            $this->getEnv('CLIENT_ID'),
+            $this->getEnv('CLIENT_SECRET') . 'invalid',
+            '',
+            $this->getEnv('REFRESH_TOKEN'),
+            $logger
+        );
+
+        try {
+            $api->refreshToken();
+        } catch (ClientException $e) {
+            $this->assertStringContainsString('401 Unauthorized', $e->getMessage());
+        }
+
+        $this->assertEmpty($testHandler->getRecords());
+
+    }
+
     protected function getEnv(string $name): string
     {
         $value = getenv($name);