feat: Add comprehensive API feature tests, refactor API client creation, and enhance request payload generation.

Author: CodeWithKyrian

.github/workflows/test.yml

@@ -39,5 +39,8 @@ jobs:
       - name: Install Composer dependencies
         run: composer update --${{ matrix.dependency-version }} --no-interaction --prefer-dist
 
+      - name: Start Chroma Server
+        run: chroma run tests/chroma.yaml
+
       - name: Run tests
         run: composer test

composer.json

@@ -53,4 +53,4 @@
     "test": "vendor/bin/pest",
     "test:coverage": "XDEBUG_MODE=coverage ./vendor/bin/pest --coverage"
   }
-}
+}

src/Factory.php

@@ -104,21 +104,26 @@ public function withHeaders(array $headers): self
     }
 
     public function connect(): Client
+    {
+        $api = $this->createApi();
+
+        return new Client($api, $this->database, $this->tenant);
+    }
+
+    public function createApi(): Api
     {
         $baseUrl = $this->port ? "$this->host:$this->port" : $this->host;
 
         $httpClient = Psr18ClientDiscovery::find();
         $requestFactory = Psr17FactoryDiscovery::findRequestFactory();
         $streamFactory = Psr17FactoryDiscovery::findStreamFactory();
 
-        $api = new Api(
+        return new Api(
             $httpClient,
             $requestFactory,
             $streamFactory,
             $baseUrl,
             $this->headers
         );
-
-        return new Client($api, $this->database, $this->tenant);
     }
 }

src/Models/Collection.php

@@ -266,7 +266,7 @@ public function query(
         $include ??= ['embeddings', 'metadatas', 'distances'];
 
         if (
-            !(($queryEmbeddings != null xor $queryTexts != null xor $queryImages != null))
+            !(($queryEmbeddings != null xor $queryTexts  != null xor $queryImages != null))
         ) {
             throw new \InvalidArgumentException(
                 'You must provide only one of queryEmbeddings, queryTexts, queryImages, or queryUris'
@@ -350,7 +350,7 @@ function validate(
             || $images != null && count($images) != count($ids)
         ) {
             throw new \InvalidArgumentException(
-                'The number of ids, embeddings, metadatas, documents, and images  must be the same'
+                'The number of ids, embeddings, metadatas, documents, and images must be the same'
             );
         }
 

src/Requests/AddItemsRequest.php

@@ -39,11 +39,12 @@ public static function create(array $data): self
 
     public function toArray(): array
     {
-        return [
+        return array_filter([
             'embeddings' => $this->embeddings,
             'metadatas' => $this->metadatas,
             'ids' => $this->ids,
             'documents' => $this->documents,
-        ];
+            'images' => $this->images,
+        ], fn($value) => $value !== null);
     }
 }

src/Requests/DeleteItemsRequest.php

@@ -29,10 +29,10 @@ public static function create(array $data): self
 
     public function toArray(): array
     {
-        return [
+        return array_filter([
             'ids' => $this->ids,
             'where' => $this->where,
             'where_document' => $this->whereDocument,
-        ];
+        ], fn($value) => $value !== null);
     }
 }

src/Requests/GetEmbeddingRequest.php

@@ -44,14 +44,14 @@ public static function create(array $data): self
 
     public function toArray(): array
     {
-        return [
+        return array_filter([
             'ids' => $this->ids,
             'where' => $this->where,
             'whereDocument' => $this->whereDocument,
             'sort' => $this->sort,
             'limit' => $this->limit,
             'offset' => $this->offset,
             'include' => $this->include,
-        ];
+        ], fn($value) => $value !== null);
     }
 }