fix: Address CodeRabbit review feedback on dynamic search rules

- index.py: Fix upsert_dynamic_search_rule path to use top-level route
  /dynamic-search-rules/{uid} instead of index-scoped path
- .code-samples.meilisearch.yaml: Fix patch sample to use camelCase keys
  (documentIds, matchCondition) and correct conditions array structure
  with scope/operator/value fields per v1.41 API spec
- tests: Replace tautological assertion in test_list with meaningful
  checks (isinstance dict, results/meta key, results is list)
- tests: Replace try/except with pytest.raises(MeilisearchApiError)
  in test_delete verification; add MeilisearchApiError import

Author: diana

.code-samples.meilisearch.yaml

@@ -631,16 +631,20 @@ get_dynamic_search_rule_1: |-
 
 patch_dynamic_search_rule_1: |-
   client.index('movies').upsert_dynamic_search_rule('promote-new', {
-    'condition': "query = 'new'",
-    'match_condition': 'all',
+    'conditions': [
+      {
+        'scope': ['title'],
+        'operator': 'contains',
+        'value': 'new'
+      }
+    ],
     'actions': [
       {
         'action': 'promote',
-        'document_ids': ['1', '2'],
-        'position': 1
+        'documentIds': ['1', '2'],
+        'matchCondition': 'all'
       }
     ]
   })
-
 delete_dynamic_search_rule_1: |-
   client.index('movies').delete_dynamic_search_rule('promote-new')

meilisearch/index.py

@@ -1384,7 +1384,7 @@ def upsert_dynamic_search_rule(self, uid: str, body: Dict[str, Any]) -> TaskInfo
             An error containing details about why Meilisearch can't process your request. Meilisearch error codes are described here: https://www.meilisearch.com/docs/reference/errors/error_codes#meilisearch-errors
         """
         task = self.http.patch(
-            f"{self.config.paths.index}/{self.uid}/{self.config.paths.dynamic_search_rules}/{uid}",
+            f"{self.config.paths.dynamic_search_rules}/{uid}",
             body,
         )
 

tests/index/test_index_dynamic_search_rules_meilisearch.py

@@ -1,4 +1,5 @@
 import pytest
+from meilisearch.errors import MeilisearchApiError
 
 
 @pytest.fixture
@@ -12,7 +13,9 @@ def test_list_dynamic_search_rules(test_index):
     """Test listing dynamic search rules"""
     response = test_index.list_dynamic_search_rules()
     assert isinstance(response, dict)
-    assert "results" in response or "results" not in response  # May be empty
+    assert "results" in response or "meta" in response
+    if "results" in response:
+        assert isinstance(response["results"], list)
 
 
 def test_get_dynamic_search_rule(test_index):
@@ -91,9 +94,5 @@ def test_delete_dynamic_search_rule(test_index):
     test_index.wait_for_task(delete_response.task_uid)
 
     # Verify it's deleted - should raise error or return empty
-    try:
+    with pytest.raises(MeilisearchApiError):
         test_index.get_dynamic_search_rule(rule_uid)
-        assert False, "Rule should have been deleted"
-    except Exception:
-        # Expected - rule doesn't exist
-        pass