Fix panic tests (#160)

* Fix update tests

* Fix desrialization

Author: curquiza

src/errors.rs

@@ -20,7 +20,7 @@ pub enum Error {
     UnreachableServer,
     /// The MeiliSearch server returned invalid JSON for a request.
     ParseError(serde_json::Error),
-    /// This Meilisearch sdk generated an invalid request (which was not sent).
+    /// This MeiliSearch SDK generated an invalid request (which was not sent).
     /// It probably comes from an invalid API key resulting in an invalid HTTP header.
     InvalidRequest,
 
@@ -245,7 +245,7 @@ impl std::fmt::Display for Error {
             Error::UnreachableServer => write!(fmt, "The MeiliSearch server can't be reached."),
             Error::InvalidRequest => write!(fmt, "Unable to generate a valid HTTP request. It probably comes from an invalid API key."),
             Error::ParseError(e) => write!(fmt, "Error parsing response JSON: {}", e),
-            Error::HttpError(e) => write!(fmt, "HTTP request failed: {}", e)
+            Error::HttpError(e) => write!(fmt, "HTTP request failed: {}", e),
         }
     }
 }

src/indexes.rs

@@ -707,6 +707,7 @@ impl Index {
     /// let status = progress.get_status().await.unwrap();
     /// let from_progress = match status {
     ///    UpdateStatus::Enqueued{content} => content.update_id,
+    ///    UpdateStatus::Processing{content} => content.update_id,
     ///    UpdateStatus::Failed{content} => content.update_id,
     ///    UpdateStatus::Processed{content} => content.update_id,
     /// };
@@ -717,6 +718,7 @@ impl Index {
     ///
     /// let from_index = match status {
     ///    UpdateStatus::Enqueued{content} => content.update_id,
+    ///    UpdateStatus::Processing{content} => content.update_id,
     ///    UpdateStatus::Failed{content} => content.update_id,
     ///    UpdateStatus::Processed{content} => content.update_id,
     /// };

src/progress.rs

@@ -174,32 +174,22 @@ pub(crate) async fn async_sleep(interval: Duration) {
 }
 
 #[derive(Debug, Clone, Deserialize)]
-pub enum RankingRule {
-    Words,
-    Typo,
-    Proximity,
-    Attribute,
-    Exactness,
-    Asc(String),
-    Desc(String),
-}
-
-#[derive(Debug, Clone, Deserialize)]
-pub enum UpdateState<T> {
-    Update(T),
-    Clear,
-    Nothing,
-}
-
-#[derive(Debug, Clone, Deserialize)]
+#[serde(rename_all = "camelCase")]
 pub struct SettingsUpdate {
-    pub ranking_rules: UpdateState<Vec<RankingRule>>,
-    pub distinct_attribute: UpdateState<String>,
-    pub searchable_attributes: UpdateState<Vec<String>>,
-    pub displayed_attributes: UpdateState<BTreeSet<String>>,
-    pub stop_words: UpdateState<BTreeSet<String>>,
-    pub synonyms: UpdateState<BTreeMap<String, Vec<String>>>,
-    pub filterable_attributes: UpdateState<Vec<String>>,
+    #[serde(skip_serializing_if = "Option::is_none")]
+    pub ranking_rules: Option<Vec<String>>,
+    #[serde(skip_serializing_if = "Option::is_none")]
+    pub distinct_attribute: Option<String>,
+    #[serde(skip_serializing_if = "Vec::is_none")]
+    pub searchable_attributes: Option<Vec<String>>,
+    #[serde(skip_serializing_if = "BTreeSet::is_not_set")]
+    pub displayed_attributes: Option<BTreeSet<String>>,
+    #[serde(skip_serializing_if = "Option::is_none")]
+    pub stop_words: Option<BTreeSet<String>>,
+    #[serde(skip_serializing_if = "Option::is_none")]
+    pub synonyms: Option<BTreeMap<String, Vec<String>>>,
+    #[serde(skip_serializing_if = "Option::is_none")]
+    pub filterable_attributes: Option<Vec<String>>,
 }
 
 #[allow(clippy::large_enum_variant)]

src/settings.rs

@@ -468,11 +468,10 @@ impl Index {
     /// let mut movie_index = client.get_or_create("movies").await.unwrap();
     ///
     /// let ranking_rules = [
-    ///     "typo",
     ///     "words",
+    ///     "typo",
     ///     "proximity",
     ///     "attribute",
-    ///     "wordsPosition",
     ///     "exactness",
     ///     "asc(release_date)",
     ///     "desc(rank)",