Fix bug: Search params are not applied when they are a list of strings

Author: eskombro

meilisearch/index.py

@@ -246,6 +246,9 @@ def search(self, query, opt_params=None):
         if opt_params is None:
             opt_params = {}
         search_param = {'q': query}
+        for key in opt_params:
+            if isinstance(opt_params[key], list):
+                opt_params[key] = ",".join(opt_params[key])
         params = {**search_param, **opt_params}
         return self.http.get(
             '{}/{}/{}?{}'.format(

meilisearch/tests/index/test_index_search_meilisearch.py

@@ -86,5 +86,20 @@ def test_basic_search_params_with_simple_string(self):
         assert "title" in response['hits'][0]['_formatted']
         assert not "release_date" in response['hits'][0]['_formatted']
 
-    # Add def test_basic_search_params_with_string_list(self):
-    # when bug (issue #85) is fixed
+    def test_basic_search_params_with_string_list(self):
+        """Tests search with string list in query params"""
+        response = self.index.search(
+            'a',
+            {
+                'limit': 5,
+                'attributesToRetrieve': ['title', 'overview'],
+                "attributesToHighlight": ["title"],
+            }
+        )
+        assert isinstance(response, object)
+        assert len(response['hits']) == 5
+        assert "title" in response['hits'][0]
+        assert "overview" in response['hits'][0]
+        assert not "release_date" in response['hits'][0]
+        assert "title" in response['hits'][0]['_formatted']
+        assert not "overview" in response['hits'][0]['_formatted']