Refactored test suite to make sure the tests are running isolated

.github/workflows/tests.yml

@@ -24,7 +24,7 @@ jobs:
       run: docker run -d -p 7700:7700 getmeili/meilisearch:latest ./meilisearch --master-key=masterKey --no-analytics=true
 
     - name: Run test suite
-      run: vendor/bin/phpunit --color tests/
+      run: sh scripts/tests.sh
 
     - name: Run linter
       run: vendor/bin/php-cs-fixer fix -v --config=.php_cs.dist --using-cache=no --dry-run

README.md

@@ -293,7 +293,7 @@ Each PR should pass the tests and the linter to be accepted.
 ```bash
 # Tests
 $ docker run -d -p 7700:7700 getmeili/meilisearch:latest ./meilisearch --master-key=masterKey --no-analytics=true
-$ vendor/bin/phpunit --color tests/
+$ sh scripts/tests.sh
 # Linter (with auto-fix)
 $ vendor/bin/php-cs-fixer fix --verbose --config=.php_cs.dist
 # Linter (without auto-fix)

composer.json

@@ -26,10 +26,7 @@
     "autoload-dev": {
         "psr-4": {
             "Tests\\": "tests/"
-        },
-        "files": [
-            "tests/utils.php"
-        ]
+        }
     },
     "require-dev": {
         "phpunit/phpunit": "^8.5",

scripts/tests.sh

@@ -0,0 +1,7 @@
+#!/bin/sh
+
+echo 'Setting FDs limit to 1000'
+ulimit -Sn 1000
+
+echo "Launching tests..."
+vendor/bin/phpunit --color tests/

src/Client.php

@@ -21,6 +21,13 @@ public function deleteIndex($uid)
         return $this->indexInstance($uid)->delete();
     }
 
+    public function deleteAllIndexes()
+    {
+        foreach ($this->getAllIndexes() as $index) {
+            $this->deleteIndex($index['uid']);
+        }
+    }
+
     public function getIndex($uid)
     {
         return $this->indexInstance($uid);

src/Index.php

@@ -109,13 +109,14 @@ public function waitForPendingUpdate($update_id, $timeout_in_ms = 5000, $interva
 
     public function search($query, $options = null)
     {
-        if (isset($options)) {
-            $paramters = array_merge(['q' => $query], $options);
+        if ($options) {
+            $options = $this->flattenOptions($options);
+            $parameters = array_merge(['q' => $query], $options);
         } else {
-            $paramters = ['q' => $query];
+            $parameters = ['q' => $query];
         }
 
-        return $this->httpGet('/indexes/'.$this->uid.'/search', $paramters);
+        return $this->httpGet('/indexes/'.$this->uid.'/search', $parameters);
     }
 
     // Stats
@@ -255,4 +256,15 @@ public function updateAcceptNewFields($accept_new_fields)
     {
         return $this->httpPost('/indexes/'.$this->uid.'/settings/accept-new-fields', $accept_new_fields);
     }
+
+    private function flattenOptions(array $options)
+    {
+        return array_map(function ($entry) {
+            if (is_array($entry)) {
+                return implode(',', $entry);
+            }
+
+            return $entry;
+        }, $options);
+    }
 }