improved tests for transport and httpFactoryTrait

Author: hkulekci

composer.json

@@ -19,7 +19,8 @@
         "mockery/mockery": "^1.5",
         "phpunit/php-code-coverage": "^10.1@dev",
         "symfony/http-client": "^6.4.3|^7.1.0",
-        "nyholm/psr7": "^1.8@dev"
+        "nyholm/psr7": "^1.8@dev",
+        "php-http/mock-client": "1.x-dev"
     },
     "autoload": {
         "psr-4": {

tests/Unit/Http/TransportTest.php

@@ -0,0 +1,32 @@
+<?php
+/**
+ * @since     Mar 2023
+ * @author    Haydar KULEKCI <haydarkulekci@gmail.com>
+ */
+
+namespace Qdrant\Tests\Unit\Http;
+
+use Http\Discovery\Psr17FactoryDiscovery;
+use Http\Mock\Client;
+use PHPUnit\Framework\TestCase;
+use Psr\Http\Message\ResponseInterface;
+use Qdrant\Config;
+use Qdrant\Http\Transport;
+
+class TransportTest extends TestCase
+{
+    public function testTransportApiKeyHeader(): void
+    {
+        $client = new Client();
+        $config = new Config('127.0.0.1', 3232);
+        $config->setApiKey('foo-bar');
+        $transport = new Transport($client, $config);
+
+        $httpFactory = Psr17FactoryDiscovery::findRequestFactory();
+        $request = $httpFactory->createRequest('GET', '/');
+
+        $this->assertInstanceOf(ResponseInterface::class, $transport->sendRequest($request));
+        $lastRequest = $client->getLastRequest();
+        $this->assertEquals(['foo-bar'], $lastRequest->getHeader('api-key'));
+    }
+}

tests/Unit/Models/Traits/HttpFactoryTraitTest.php

@@ -0,0 +1,46 @@
+<?php
+
+namespace Qdrant\Tests\Unit\Models\Traits;
+
+use InvalidArgumentException;
+use PHPUnit\Framework\TestCase;
+use Psr\Http\Message\RequestFactoryInterface;
+use Psr\Http\Message\RequestInterface;
+use Qdrant\Endpoints\HttpFactoryTrait;
+use Qdrant\Models\Traits\ProtectedPropertyAccessor;
+
+class HttpFactoryTraitTest extends TestCase
+{
+    public function testCustomHttpRequestFactory()
+    {
+        $requestFactory = new class implements RequestFactoryInterface {
+            public function createRequest(string $method, $uri): RequestInterface
+            {
+            }
+        };
+
+        $mock = new class {
+            use HttpFactoryTrait;
+        };
+        $mock->setHttpFactory($requestFactory);
+        $this->assertInstanceOf(get_class($requestFactory), $mock->getHttpFactory());
+    }
+
+    public function testHeaders()
+    {
+        $mock = new class {
+            use HttpFactoryTrait;
+
+            public function test(string $method, string $uri, array $body = [], $queryString = []): RequestInterface
+            {
+                return $this->createRequest($method, $uri . $this->queryBuild($queryString), $body);
+            }
+        };
+
+        $request = $mock->test('GET', '/api', ['foo' => 'bar'], ['query' => 'string']);
+
+        $this->assertEquals('{"foo":"bar"}', $request->getBody()->getContents());
+        $this->assertEquals('query=string', $request->getUri()->getQuery());
+        $this->assertEquals('/api', $request->getUri()->getPath());
+    }
+}