refactor: reduce map_err by adding some From<Error> implementations

Author: gazure

postgresql_archive/src/error.rs

@@ -224,4 +224,33 @@ mod test {
         let error = Error::from(parse_error);
         assert_eq!(error.to_string(), "empty host");
     }
+
+    #[cfg(feature = "maven")]
+    #[test]
+    fn test_from_quick_xml_error() {
+        let xml = "<invalid>";
+        let quick_xml_error = quick_xml::de::from_str::<String>(xml).expect_err("quick_xml error");
+        let error = Error::from(quick_xml_error);
+        assert!(matches!(error, Error::ParseError(_)));
+    }
+
+    #[cfg(feature = "zip")]
+    #[test]
+    fn test_from_zip_error() {
+        let zip_error = zip::result::ZipError::FileNotFound;
+        let error = Error::from(zip_error);
+        assert!(matches!(error, Error::Unexpected(_)));
+        assert!(
+            error
+                .to_string()
+                .contains("specified file not found in archive")
+        );
+    }
+
+    #[test]
+    fn test_from_poisoned_lock() {
+        let error = Error::from(std::sync::PoisonError::new(()));
+        assert!(matches!(error, Error::PoisonedLock(_)));
+        assert!(error.to_string().contains("poisoned lock"));
+    }
 }

postgresql_extensions/src/error.rs

@@ -38,3 +38,15 @@ impl<T> From<PoisonError<T>> for Error {
         Error::PoisonedLock(value.to_string())
     }
 }
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+
+    #[test]
+    fn test_from_poison_error() {
+        let error = Error::from(std::sync::PoisonError::new(()));
+        assert!(matches!(error, Error::PoisonedLock(_)));
+        assert!(error.to_string().contains("poisoned lock"));
+    }
+}