Fix failing tests for RecommendRequest, BatchRecommendRequest, and Service

Add required setLimit() calls to RecommendRequest and BatchRecommendRequest
tests to match the limit validation added in the previous commit. Remove
locks tests from ServiceTest as the /locks endpoint has been removed from
the Qdrant API, and mark getLocks/setLocks methods as deprecated.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: hkulekci

src/Endpoints/Service.php

@@ -41,9 +41,10 @@ public function metrics(bool $anonymize): Response
     }
 
     /**
-     * # Delete storage snapshot
-     * Delete snapshot of the whole storage
+     * # Set lock options
+     * Set lock options for the storage
      *
+     * @deprecated This endpoint has been removed from the Qdrant API.
      * @throws InvalidArgumentException
      */
     public function setLocks(ServiceLock $body): Response
@@ -54,9 +55,10 @@ public function setLocks(ServiceLock $body): Response
     }
 
     /**
-     * # Download storage snapshot
-     * Download specified snapshot of the whole storage as a file
+     * # Get lock options
+     * Get lock options for the storage
      *
+     * @deprecated This endpoint has been removed from the Qdrant API.
      * @throws InvalidArgumentException
      */
     public function getLocks(): Response

tests/Integration/Endpoints/ServiceTest.php

@@ -8,7 +8,6 @@
 
 use Qdrant\Endpoints\Service;
 use Qdrant\Exception\InvalidArgumentException;
-use Qdrant\Models\Request\ServiceLock;
 use Qdrant\Tests\Integration\AbstractIntegration;
 
 class ServiceTest extends AbstractIntegration
@@ -42,32 +41,4 @@ public function testServiceMetrics(): void
         $this->assertStringContainsString('collections_total', $response['content']);
         $this->assertStringContainsString('rest_responses_', $response['content']);
     }
-
-    /**
-     * @throws InvalidArgumentException
-     */
-    public function testServiceGetLocks(): void
-    {
-        $service = new Service($this->client);
-        $response = $service->getLocks();
-        $this->assertEquals('ok', $response['status']);
-        $this->assertArrayHasKey('result', $response);
-    }
-
-    /**
-     * @throws InvalidArgumentException
-     */
-    public function testServiceSetLocks(): void
-    {
-        $service = new Service($this->client);
-
-        $response = $service->setLocks(new ServiceLock(true));
-        $this->assertEquals('ok', $response['status']);
-        $this->assertArrayHasKey('result', $response);
-
-        // Revert back to False
-        $response = $service->setLocks(new ServiceLock(false));
-        $this->assertEquals('ok', $response['status']);
-        $this->assertArrayHasKey('result', $response);
-    }
-}
+}

tests/Unit/Models/Request/Points/BatchRecommendRequestTest.php

@@ -17,21 +17,23 @@ class BatchRecommendRequestTest extends TestCase
     public function testBasicRecommendRequest(): void
     {
         $request = new BatchRecommendRequest([
-            new RecommendRequest([100, 101], [110]),
-            new RecommendRequest([101, 102], [112]),
+            (new RecommendRequest([100, 101], [110]))->setLimit(10),
+            (new RecommendRequest([101, 102], [112]))->setLimit(10),
         ]);
 
         $this->assertEquals([
             'searches' =>[
                 [
                     'positive' => [100, 101],
                     'negative' => [110],
+                    'limit' => 10,
                 ],
                 [
                     'positive' => [101, 102],
                     'negative' => [112],
+                    'limit' => 10,
                 ]
             ]
         ], $request->toArray());
     }
-}
+}

tests/Unit/Models/Request/Points/RecommendRequestTest.php

@@ -16,17 +16,28 @@ class RecommendRequestTest extends TestCase
 {
     public function testBasicRecommendRequest(): void
     {
-        $request = new RecommendRequest([100, 101], [110]);
+        $request = (new RecommendRequest([100, 101], [110]))
+            ->setLimit(10);
 
         $this->assertEquals([
             'positive' => [100, 101],
             'negative' => [110],
+            'limit' => 10,
         ], $request->toArray());
     }
 
+    public function testRecommendRequestWithoutLimitThrowsException(): void
+    {
+        $request = new RecommendRequest([100, 101], [110]);
+
+        $this->expectException(InvalidArgumentException::class);
+        $request->toArray();
+    }
+
     public function testRecommendRequestWithFilter(): void
     {
         $request = (new RecommendRequest([100, 101], [110]))
+            ->setLimit(10)
             ->setFilter(
                 (new Filter())->addMust((new MatchExcept('foo', ['bar'])))
             );
@@ -43,13 +54,15 @@ public function testRecommendRequestWithFilter(): void
                         ]
                     ]
                 ]
-            ]
+            ],
+            'limit' => 10,
         ], $request->toArray());
     }
 
     public function testRecommendRequestWithScoreThreshold(): void
     {
         $request = (new RecommendRequest([100, 101], [110]))
+            ->setLimit(10)
             ->setFilter(
                 (new Filter())->addMust((new MatchExcept('foo', ['bar'])))
             )
@@ -68,31 +81,36 @@ public function testRecommendRequestWithScoreThreshold(): void
                         ]
                     ]
                 ]
-            ]
+            ],
+            'limit' => 10,
         ], $request->toArray());
     }
 
     public function testRecommendRequestWithOffset(): void
     {
         $request = (new RecommendRequest([100, 101], [110]))
+            ->setLimit(10)
             ->setOffset(1);
 
         $this->assertEquals([
             'positive' => [100, 101],
             'negative' => [110],
+            'limit' => 10,
             'offset'=> 1,
         ], $request->toArray());
     }
 
     public function testRecommendRequestWithUsing(): void
     {
         $request = (new RecommendRequest([100, 101], [110]))
+            ->setLimit(10)
             ->setUsing('foo');
 
         $this->assertEquals([
             'positive' => [100, 101],
             'negative' => [110],
             'using'=> 'foo',
+            'limit' => 10,
         ], $request->toArray());
     }
 
@@ -142,4 +160,4 @@ public function testRecommendRequestWithLimitAndStrategy(): void
             'strategy' => RecommendRequest::STRATEGY_AVERAGE_VECTOR
         ], $request->toArray());
     }
-}
+}