test: add build url tests

Author: andrepimpao

README.md

@@ -69,7 +69,6 @@ class YourApi extends Api
 - [HTTP client (PSR-18) and HTTP factories (PSR-17)](#http-client-psr-18-and-http-factories-psr-17)
 - [Cache (PSR-6)](#cache-psr-6)
 - [Logger (PSR-3)](#logger-psr-3)
-- [Configure options](#configure-options)
 
 ### Base URL
 

tests/Integration/ApiTest.php

@@ -5,10 +5,12 @@
 use Http\Message\Authentication;
 use Http\Mock\Client;
 use Nyholm\Psr7\Response;
+use PHPUnit\Framework\Attributes\DataProvider;
 use ProgrammatorDev\Api\Api;
 use ProgrammatorDev\Api\Builder\CacheBuilder;
 use ProgrammatorDev\Api\Builder\ClientBuilder;
 use ProgrammatorDev\Api\Builder\LoggerBuilder;
+use ProgrammatorDev\Api\Event\PreRequestEvent;
 use ProgrammatorDev\Api\Event\ResponseContentsEvent;
 use ProgrammatorDev\Api\Test\AbstractTestCase;
 use ProgrammatorDev\Api\Test\MockResponse;
@@ -192,6 +194,30 @@ public function testResponseContentsListener()
         $this->assertIsArray($response);
     }
 
+    #[DataProvider('provideBuildUrlData')]
+    public function testBuildUrl(?string $baseUrl, string $path, array $query, string $expectedUrl)
+    {
+        $this->api->addPreRequestListener(function(PreRequestEvent $event) use ($expectedUrl) {
+            $url = (string) $event->getRequest()->getUri();
+
+            $this->assertSame($expectedUrl, $url);
+        });
+
+        $this->api->setBaseUrl($baseUrl);
+        $this->api->request(method: 'GET', path: $path, query: $query);
+    }
+
+    public static function provideBuildUrlData(): \Generator
+    {
+        yield 'no base url' => [null, '/path', [], '/path'];
+        yield 'base url' => [self::BASE_URL, '/path', [], 'https://base.com/url/path'];
+        yield 'path full url' => [self::BASE_URL, 'https://fullurl.com/path', [], 'https://fullurl.com/path'];
+        yield 'duplicated slashes' => [self::BASE_URL, '////path', [], 'https://base.com/url/path'];
+        yield 'query' => [self::BASE_URL, '/path', ['foo' => 'bar'], 'https://base.com/url/path?foo=bar'];
+        yield 'path query' => [self::BASE_URL, '/path?test=true', ['foo' => 'bar'], 'https://base.com/url/path?test=true&foo=bar'];
+        yield 'query replace' => [self::BASE_URL, '/path?test=true', ['test' => 'false'], 'https://base.com/url/path?test=false'];
+    }
+
     public function testBuildPath()
     {
         $path = $this->api->buildPath('/path/{parameter1}/multiple/{parameter2}', [