Allow using the {,raw}{bson,doc} with types implementing Into<{,Raw}Bson> (#450)

Author: tyilo

src/macros.rs

@@ -181,7 +181,7 @@ macro_rules! bson {
     // Any Into<Bson> type.
     // Must be below every other rule.
     ($other:expr) => {
-        $crate::Bson::from($other)
+        <_ as ::std::convert::Into<$crate::Bson>>::into($other)
     };
 }
 
@@ -392,7 +392,7 @@ macro_rules! rawbson {
     // Any Into<RawBson> type.
     // Must be below every other rule.
     ($other:expr) => {
-        $crate::RawBson::from($other)
+        <_ as ::std::convert::Into<$crate::RawBson>>::into($other)
     };
 }
 

src/tests/modules/macros.rs

@@ -242,3 +242,34 @@ fn recursive_macro() {
 
     assert_eq!(rawdoc.into_bytes(), crate::to_vec(&doc).unwrap());
 }
+
+#[test]
+#[allow(clippy::from_over_into)]
+fn can_use_macro_with_into_bson() {
+    struct Custom;
+
+    impl Into<Bson> for Custom {
+        fn into(self) -> Bson {
+            "foo".into()
+        }
+    }
+
+    impl Into<RawBson> for Custom {
+        fn into(self) -> RawBson {
+            "foo".into()
+        }
+    }
+
+    _ = bson!({
+        "a": Custom,
+    });
+    _ = doc! {
+        "a": Custom,
+    };
+    _ = rawbson!({
+        "a": Custom,
+    });
+    _ = rawdoc! {
+        "a": Custom,
+    };
+}