Fix deserialize_skip_error function (#24)

https://github.com/serde-rs/serde/issues/1726#issuecomment-577425541

src/deser/helpers.rs

@@ -58,9 +58,26 @@ where
     T: Deserialize<'de> + Default,
     D: Deserializer<'de>,
 {
-    let result = Deserialize::deserialize(deserializer);
+    let value = serde_json::Value::deserialize(deserializer)?;
-    Ok(match result {
+    let inner = T::deserialize(value).unwrap_or_default();
-        Ok(o) => o,
+    Ok(inner)
-        Err(_) => Default::default(),
+}
-    })
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+
+    #[test]
+    fn test_deserialize_skip_error() {
+        #[derive(Debug, Deserialize)]
+        pub struct MyData {
+            #[serde(deserialize_with = "deserialize_skip_error")]
+            pub data: Option<String>,
+        }
+        // data has type object
+        let _: MyData = serde_json::from_str(r#"{ "data": {} }"#).unwrap();
+
+        // data has type array
+        let _: MyData = serde_json::from_str(r#"{"data": []}"#).unwrap();
+    }
 }