added missing query string auth, add to account api (#372)

Author: SecondeJK

src/Account/ClientFactory.php

@@ -14,6 +14,7 @@
 use Psr\Container\ContainerInterface;
 use Vonage\Client\APIResource;
 use Vonage\Client\Credentials\Handler\BasicHandler;
+use Vonage\Client\Credentials\Handler\BasicQueryHandler;
 
 class ClientFactory
 {
@@ -25,7 +26,7 @@ public function __invoke(ContainerInterface $container): Client
             ->setBaseUrl($accountApi->getClient()->getRestUrl())
             ->setIsHAL(false)
             ->setBaseUri('/account')
-            ->setAuthHandler(new BasicHandler())
+            ->setAuthHandler(new BasicQueryHandler())
         ;
 
         return new Client($accountApi);

src/Client/Credentials/Handler/BasicQueryHandler.php

@@ -0,0 +1,17 @@
+<?php
+
+namespace Vonage\Client\Credentials\Handler;
+
+use Psr\Http\Message\RequestInterface;
+use Vonage\Client\Credentials\Basic;
+use Vonage\Client\Credentials\CredentialsInterface;
+
+class BasicQueryHandler extends AbstractHandler
+{
+    public function __invoke(RequestInterface $request, CredentialsInterface $credentials): RequestInterface
+    {
+        $credentials = $this->extract(Basic::class, $credentials);
+
+        return $request->withUri($request->getUri()->withQuery(http_build_query($credentials->asArray())));
+    }
+}

test/Account/ClientTest.php

@@ -56,6 +56,7 @@ public function setUp(): void
         $this->api = new APIResource();
         $this->api->setBaseUrl('https://rest.nexmo.com')
             ->setIsHAL(false)
+            ->setAuthHandler(new Client\Credentials\Handler\BasicQueryHandler())
             ->setBaseUri('/account');
 
         $this->api->setClient($this->vonageClient->reveal());
@@ -163,6 +164,10 @@ public function testGetBalance(): void
             $this->assertEquals('rest.nexmo.com', $request->getUri()->getHost());
             $this->assertEquals('GET', $request->getMethod());
 
+            $uri = $request->getUri();
+            $uriString = $uri->__toString();
+            $this->assertEquals('https://rest.nexmo.com/account/get-balance?api_key=abc&api_secret=def', $uriString);
+
             return true;
         }))->shouldBeCalledTimes(1)->willReturn($this->getResponse('get-balance'));
 

test/Client/Credentials/Handler/BasicQueryHandlerTest.php

@@ -0,0 +1,23 @@
+<?php
+
+namespace VonageTest\Client\Credentials\Handler;
+
+use Laminas\Diactoros\Request;
+use Vonage\Client\Credentials\Basic;
+use Vonage\Client\Credentials\Handler\BasicQueryHandler;
+use PHPUnit\Framework\TestCase;
+
+class BasicQueryHandlerTest extends TestCase
+{
+    public function testWillAddCredentialsToRequest(): void
+    {
+        $request = new Request('https://example.com');
+        $credentials = new Basic('abc', 'xyz');
+        $handler = new BasicQueryHandler();
+        $request = $handler($request, $credentials);
+
+        $uri = $request->getUri();
+        $uriString = $uri->__toString();
+        $this->assertEquals('https://example.com?api_key=abc&api_secret=xyz', $uriString);
+    }
+}