fix: basic search query

oramasearch · Jul 2, 2024 · 3cc2a1a · 3cc2a1a
1 parent aad5274
commit 3cc2a1a
Show file tree

Hide file tree

Showing 4 changed files with 26 additions and 12 deletions.
diff --git a/src/Client.php b/src/Client.php
@@ -1,4 +1,5 @@
 <?php
+
 namespace OramaCloud;
 
 use GuzzleHttp\Client as HttpClient;
@@ -21,13 +22,12 @@ public function __construct($endpoint, $apiKey, $answersApiBaseURL = null) {
 
     public function search(Query $query) {
         $endpoint = "{$this->endpoint}/search?api-key={$this->apiKey}";
-        $options = [
+
+        $response = $this->http->request('POST', $endpoint, [
             'form_params' => [
                 'q' => $query->toJson()
             ]
-        ];
-
-        $response = $this->http->request('POST', $endpoint, $options);
+        ]);
 
         return json_decode($response->getBody(), true);
     }

diff --git a/src/Query.php b/src/Query.php
@@ -1,4 +1,5 @@
 <?php
+
 namespace OramaCloud;
 
 class Query {
@@ -29,11 +30,6 @@ public function setMode($mode) {
         return $this;
     }
 
-    public static function fromArray($array) {
-        extract($array);
-        return new Query($term, $mode);
-    }
-
     public function toArray() {
         return [
             'term' => $this->term,
@@ -44,4 +40,13 @@ public function toArray() {
     public function toJson() {
         return json_encode($this->toArray());
     }
+
+    public static function fromArray($array) {
+        $query = new Query();
+        $query
+            ->setTerm(isset($array['term']) ? $array['term'] : '')
+            ->setMode(isset($array['mode']) ? $array['mode'] : 'fulltext');
+
+        return $query;
+    }
 }
diff --git a/tests/ClientTest.php b/tests/ClientTest.php
@@ -4,8 +4,8 @@
 use OramaCloud\Client;
 use OramaCloud\Query;
 
-const ENDPOINT = 'https://cloud.orama.run/v1/indexes/askorama-ai-development-uc6oxa';
-const API_KEY = 'P9buEfpy8rWvT265McikCG1tP4pT6cBg';
+const API_ENDPOINT = 'https://cloud.orama.run/v1/indexes/askorama-ai-development-uc6oxa';
+const PUBLIC_API_KEY = 'P9buEfpy8rWvT265McikCG1tP4pT6cBg';
 
 class ClientTest extends TestCase {
 
@@ -14,7 +14,7 @@ class ClientTest extends TestCase {
     protected function setUp(): void {
         parent::setUp();
 
-        $this->client = new Client(ENDPOINT, API_KEY);
+        $this->client = new Client(API_ENDPOINT, PUBLIC_API_KEY);
     }
 
     public function testBasicFullTextSearch() {

diff --git a/tests/QueryTest.php b/tests/QueryTest.php
@@ -17,6 +17,15 @@ public function testQueryParams() {
         $this->assertEquals($mode, $query->getMode());
     }
 
+    public function testDefaultQueryParamsFromArray() {
+        $query = Query::fromArray([]);
+
+        $this->assertEquals($query->toArray(), [
+            'term' => '',
+            'mode' => 'fulltext'
+        ]);
+    }
+
     public function testQueryParamsFromArray() {
         $params = [
             'term' => 'mock-term',