Update HTTP methods for v0.28.0 (#312)

* Update task api v0.28.0

* Update tests to be relevant with the structures where they are ran

* Update types in code-samples

* Remove clearAll task type

* Add index_uid as an optional in the Tasks

* Add comments on best builder implementation

* Add execute function on get_tasks

* Update code samples

* Remove out of context comment

* Fix tests

* Fix pagination tests

* Update HTTP methods for v0.28.0

* Fix clippy

* Remove comment of task tests since the tests are now sucesful

* Fix doc tests

* Fix get tasks inside index structure

* Fix none doc tests with new get_tasks api

* Add from and limit in tasks params

* Add warning on failing test

* Update task API

* Remove useless comment

* Rename type in Task structure

* Removed useless newlines

* Fix typo in comment

* Add missing semi-column

* Fix clippy errors

Author: bidoubiwa

src/dumps.rs

@@ -43,7 +43,7 @@ use serde::Deserialize;
 
 /// The status of a dump.\
 /// Contained in [`DumpInfo`].
-#[derive(Debug, Deserialize, Clone, PartialEq)]
+#[derive(Debug, Deserialize, Clone, Eq, PartialEq)]
 #[serde(rename_all = "snake_case")]
 pub enum DumpStatus {
     /// Dump creation is in progress.

src/indexes.rs

@@ -97,7 +97,7 @@ impl Index {
         request::<serde_json::Value, serde_json::Value>(
             &format!("{}/indexes/{}", self.client.host, self.uid),
             &self.client.api_key,
-            Method::Put(json!({ "primaryKey": primary_key.as_ref() })),
+            Method::Patch(json!({ "primaryKey": primary_key.as_ref() })),
             200,
         )
         .await?;

src/search.rs

@@ -3,7 +3,7 @@ use serde::{de::DeserializeOwned, Deserialize, Serialize, Serializer};
 use serde_json::{Map, Value};
 use std::collections::HashMap;
 
-#[derive(Deserialize, Debug, PartialEq)]
+#[derive(Deserialize, Debug, Eq, PartialEq)]
 pub struct MatchRange {
     pub start: usize,
     pub length: usize,

src/settings.rs

@@ -482,7 +482,7 @@ impl Index {
         request::<&Settings, TaskInfo>(
             &format!("{}/indexes/{}/settings", self.client.host, self.uid),
             &self.client.api_key,
-            Method::Post(settings),
+            Method::Patch(settings),
             202,
         )
         .await
@@ -522,7 +522,7 @@ impl Index {
                 self.client.host, self.uid
             ),
             &self.client.api_key,
-            Method::Post(synonyms),
+            Method::Put(synonyms),
             202,
         )
         .await
@@ -558,7 +558,7 @@ impl Index {
                 self.client.host, self.uid
             ),
             &self.client.api_key,
-            Method::Post(
+            Method::Put(
                 stop_words
                     .into_iter()
                     .map(|v| v.as_ref().to_string())
@@ -608,7 +608,7 @@ impl Index {
                 self.client.host, self.uid
             ),
             &self.client.api_key,
-            Method::Post(
+            Method::Put(
                 ranking_rules
                     .into_iter()
                     .map(|v| v.as_ref().to_string())
@@ -649,7 +649,7 @@ impl Index {
                 self.client.host, self.uid
             ),
             &self.client.api_key,
-            Method::Post(
+            Method::Put(
                 filterable_attributes
                     .into_iter()
                     .map(|v| v.as_ref().to_string())
@@ -690,7 +690,7 @@ impl Index {
                 self.client.host, self.uid
             ),
             &self.client.api_key,
-            Method::Post(
+            Method::Put(
                 sortable_attributes
                     .into_iter()
                     .map(|v| v.as_ref().to_string())
@@ -730,7 +730,7 @@ impl Index {
                 self.client.host, self.uid
             ),
             &self.client.api_key,
-            Method::Post(distinct_attribute.as_ref().to_string()),
+            Method::Put(distinct_attribute.as_ref().to_string()),
             202,
         )
         .await
@@ -765,7 +765,7 @@ impl Index {
                 self.client.host, self.uid
             ),
             &self.client.api_key,
-            Method::Post(
+            Method::Put(
                 searchable_attributes
                     .into_iter()
                     .map(|v| v.as_ref().to_string())
@@ -805,7 +805,7 @@ impl Index {
                 self.client.host, self.uid
             ),
             &self.client.api_key,
-            Method::Post(
+            Method::Put(
                 displayed_attributes
                     .into_iter()
                     .map(|v| v.as_ref().to_string())

src/task_info.rs

@@ -164,7 +164,6 @@ mod test {
     }
 
     #[meilisearch_test]
-    // TODO: failing because settings routes now uses PUT instead of POST as http method
     async fn test_failing_task(client: Client, movies: Index) -> Result<(), Error> {
         let task_info = movies.set_ranking_rules(["wrong_ranking_rule"]).await?;
         let task = client.wait_for_task(task_info, None, None).await?;

src/tasks.rs

@@ -694,7 +694,6 @@ mod test {
     }
 
     #[meilisearch_test]
-    // TODO: failing because settings routes now uses PUT instead of POST as http method
     async fn test_failing_task(client: Client, movies: Index) -> Result<(), Error> {
         let task_info = movies.set_ranking_rules(["wrong_ranking_rule"]).await?;