Fix api endpoint URL

JoppeDC · Apr 9, 2023 · 44bcec2 · 44bcec2
1 parent 60cd90f
commit 44bcec2
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/src/Client.php b/src/Client.php
@@ -16,6 +16,8 @@
 
 class Client
 {
+    public const API_URL = 'https://api.logsnag.com/v1';
+
     use HandlesLog;
     use HandlesInsight;
 
@@ -24,14 +26,13 @@ class Client
     private Insight $insight;
 
     public function __construct(
-        string $url,
         string $apiKey = null,
         ClientInterface $httpClient = null,
         RequestFactoryInterface $requestFactory = null,
         array $clientAgents = [],
         StreamFactoryInterface $streamFactory = null
     ) {
-        $this->http = new LogSnagClient($url, $apiKey, $httpClient, $requestFactory, $clientAgents, $streamFactory);
+        $this->http = new LogSnagClient(self::API_URL, $apiKey, $httpClient, $requestFactory, $clientAgents, $streamFactory);
 
         $this->log = new Log($this->http);
         $this->insight = new Insight($this->http);

diff --git a/tests/ClientTest.php b/tests/ClientTest.php
@@ -6,16 +6,17 @@
 
 use JoppeDc\LogsnagPhpSdk\Client;
 use JoppeDc\LogsnagPhpSdk\Contracts\LogPayload;
+use JoppeDc\LogsnagPhpSdk\Exceptions\ApiException;
 use JoppeDc\LogsnagPhpSdk\Exceptions\CommunicationException;
 use PHPUnit\Framework\TestCase;
 
 class ClientTest extends TestCase
 {
     public function testThrowCommunicationException(): void
     {
-        $client = new Client('https://doesntexist:1234');
+        $client = new Client('testkey');
 
-        $this->expectException(CommunicationException::class);
+        $this->expectException(ApiException::class);
         $client->createLog(new LogPayload('', '', ''));
     }
 }