Add support for disabled api key

journy-io · Jun 20, 2021 · a7d3448 · a7d3448
1 parent eacd605
commit a7d3448
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 4 deletions.
diff --git a/src/Client.php b/src/Client.php
@@ -86,8 +86,17 @@ private function check(ResponseInterface $response)
                 false,
                 $this->getRemainingRequests($response),
                 $this->getMaxRequests($response),
-                [$json["message"]],
-                null
+                [$json["message"]]
+            );
+        }
+
+        if ($response->getStatusCode() === 403) {
+            return new CallResult(
+                false,
+                false,
+                $this->getRemainingRequests($response),
+                $this->getMaxRequests($response),
+                ["the api key is disabled"]
             );
         }
 
@@ -107,8 +116,7 @@ private function check(ResponseInterface $response)
                 true,
                 $this->getRemainingRequests($response),
                 $this->getMaxRequests($response),
-                ["rate limited"],
-                null
+                ["rate limited"]
             );
         }
     }

diff --git a/tests/ClientTest.php b/tests/ClientTest.php
@@ -668,6 +668,18 @@ public function test_it_deals_with_unexpected_error()
         );
     }
 
+    public function test_it_deals_with_forbidden_error()
+    {
+        $factory = new Psr17Factory();
+        $http = new HttpClientFixed(new Response(403, [], null));
+        $client = new Client($http, $factory, $factory, ["apiKey" => "key"]);
+
+        $this->assertEquals(
+            new CallResult(false, false, 0, 0, ["the api key is disabled"], null),
+            $client->addEvent(Event::forUser("login", UserIdentified::byUserId("1")))
+        );
+    }
+
     public function test_it_deals_with_unauthorized_error()
     {
         $factory = new Psr17Factory();