feat: result err to boxed for pass clippy

Author: ahabhgk

src/api.rs

@@ -468,14 +468,14 @@ pub struct CompileResult {
 }
 
 impl TryFrom<CompileResponse> for CompileResult {
-  type Error = Exception;
+  type Error = Box<Exception>;
 
   fn try_from(response: CompileResponse) -> Result<Self> {
     let res = response.result.unwrap();
     match res {
       compile_response::Result::Success(success) => Ok(success.into()),
       compile_response::Result::Failure(failure) => {
-        Err(Exception::from(failure))
+        Err(Exception::from(failure).into())
       }
     }
   }

src/embedded.rs

@@ -83,7 +83,9 @@ impl Embedded {
 
     let host = Host::new(importer_registry, logger_registry);
     let conn = self.channel.connect(host)?;
-    let response = conn.compile_request(request)?;
+    let response = conn
+      .compile_request(request)
+      .map_err(|e| Box::new(e.into()))?;
     CompileResult::try_from(response)
   }
 
@@ -171,7 +173,9 @@ impl Embedded {
 
     let host = Host::new(importer_registry, logger_registry);
     let conn = self.channel.connect(host)?;
-    let response = conn.compile_request(request)?;
+    let response = conn
+      .compile_request(request)
+      .map_err(|e| Box::new(e.into()))?;
     CompileResult::try_from(response)
   }
 
@@ -181,7 +185,7 @@ impl Embedded {
     let importer_registry = ImporterRegistry::default();
     let host = Host::new(importer_registry, logger_registry);
     let conn = self.channel.connect(host)?;
-    let response = conn.version_request()?;
+    let response = conn.version_request().map_err(|e| Box::new(e.into()))?;
     Ok(format!(
       "sass-embedded\t#{}",
       response.implementation_version

src/error.rs

@@ -8,7 +8,7 @@ use crate::{
 };
 
 /// An alias for [std::result::Result<T, Exception>].
-pub type Result<T> = std::result::Result<T, Exception>;
+pub type Result<T> = std::result::Result<T, Box<Exception>>;
 
 /// An exception for this crate, thrown because a Sass compilation failed or `io::Error`.
 ///

tests/importer_spec.rs

@@ -241,7 +241,7 @@ fn wraps_an_error_in_canonicalize() {
       _: &str,
       _: &ImporterOptions,
     ) -> Result<Option<Url>> {
-      Err(Exception::new("this import is bad actually"))
+      Err(Exception::new("this import is bad actually").into())
     }
 
     fn load(&self, _: &Url) -> Result<Option<ImporterResult>> {
@@ -275,7 +275,7 @@ fn wraps_an_error_in_load() {
     }
 
     fn load(&self, _: &Url) -> Result<Option<ImporterResult>> {
-      Err(Exception::new("this import is bad actually"))
+      Err(Exception::new("this import is bad actually").into())
     }
   }
 
@@ -305,7 +305,7 @@ fn avoids_importer_when_canonicalize_returns_nil() {
     }
 
     fn load(&self, _: &Url) -> Result<Option<ImporterResult>> {
-      Err(Exception::new("this import is bad actually"))
+      Err(Exception::new("this import is bad actually").into())
     }
   }
 
@@ -1006,7 +1006,7 @@ mod file_importer {
         _: &str,
         _: &ImporterOptions,
       ) -> Result<Option<Url>> {
-        Err(Exception::new("this import is bad actually"))
+        Err(Exception::new("this import is bad actually").into())
       }
     }