From 80c156df3fae586b80ca222ea9a90cbee2bc4c35 Mon Sep 17 00:00:00 2001
From: walter <walter.beller.morales@gmail.com>
Date: Sun, 5 Jun 2022 00:49:36 -0400
Subject: [PATCH] Add custom TaskTypeError for TaskType

---
 meilisearch-http/src/task.rs | 37 ++++++++++++++++++++++++++----------
 1 file changed, 27 insertions(+), 10 deletions(-)

diff --git a/meilisearch-http/src/task.rs b/meilisearch-http/src/task.rs
index d9360039d..d098d6710 100644
--- a/meilisearch-http/src/task.rs
+++ b/meilisearch-http/src/task.rs
@@ -1,4 +1,5 @@
-use std::fmt::Write;
+use std::error::Error;
+use std::fmt::{self, Write};
 use std::str::FromStr;
 use std::write;
 
@@ -39,10 +40,29 @@ impl From<TaskContent> for TaskType {
     }
 }
 
-impl FromStr for TaskType {
-    type Err = String;
+#[derive(Debug)]
+pub struct TaskTypeError {
+    invalid_type: String,
+}
 
-    fn from_str(status: &str) -> Result<Self, String> {
+impl fmt::Display for TaskTypeError {
+    fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
+        write!(
+            f,
+            "invalid task type `{}`, expecting one of: \
+            indexCreation, indexUpdate, indexDeletion, documentAdditionOrUpdate, \
+            documentDeletion, settingsUpdate, dumpCreation",
+            self.invalid_type
+        )
+    }
+}
+
+impl Error for TaskTypeError {}
+
+impl FromStr for TaskType {
+    type Err = TaskTypeError;
+
+    fn from_str(status: &str) -> Result<Self, TaskTypeError> {
         if status.eq_ignore_ascii_case("indexCreation") {
             Ok(TaskType::IndexCreation)
         } else if status.eq_ignore_ascii_case("indexUpdate") {
@@ -58,12 +78,9 @@ impl FromStr for TaskType {
         } else if status.eq_ignore_ascii_case("dumpCreation") {
             Ok(TaskType::DumpCreation)
         } else {
-            Err(format!(
-                "invalid task type `{}`, expecting one of: \
-                indexCreation, indexUpdate, indexDeletion, documentAdditionOrUpdate, \
-                documentDeletion, settingsUpdate, dumpCreation",
-                status
-            ))
+            Err(TaskTypeError {
+                invalid_type: status.to_string(),
+            })
         }
     }
 }