RUST-1899 Fix UUID string deserialization (#468)

Author: isabelatkinson

src/de/raw.rs

@@ -208,15 +208,6 @@ impl<'de> Deserializer<'de> {
     where
         V: serde::de::Visitor<'de>,
     {
-        if let DeserializerHint::BinarySubtype(expected_st) = hint {
-            if self.current_type != ElementType::Binary {
-                return Err(Error::custom(format!(
-                    "expected Binary with subtype {:?}, instead got {:?}",
-                    expected_st, self.current_type
-                )));
-            }
-        }
-
         match self.current_type {
             ElementType::Int32 => visitor.visit_i32(read_i32(&mut self.bytes)?),
             ElementType::Int64 => visitor.visit_i64(read_i64(&mut self.bytes)?),

src/de/serde.rs

@@ -637,13 +637,12 @@ impl Deserializer {
 
         let is_rawbson = matches!(hint, DeserializerHint::RawBson);
 
-        if let DeserializerHint::BinarySubtype(expected_st) = hint {
-            match value {
-                Bson::Binary(ref b) if b.subtype == expected_st => {}
-                ref b => {
+        if let DeserializerHint::BinarySubtype(expected_subtype) = hint {
+            if let Bson::Binary(ref binary) = value {
+                if binary.subtype != expected_subtype {
                     return Err(Error::custom(format!(
-                        "expected Binary with subtype {:?}, instead got {:?}",
-                        expected_st, b
+                        "expected Binary with subtype {:?}, instead got subtype {:?}",
+                        expected_subtype, binary.subtype
                     )));
                 }
             }

src/uuid/test.rs

@@ -1,4 +1,6 @@
 use crate::{
+    from_document,
+    from_slice,
     spec::BinarySubtype,
     uuid::{Uuid, UuidRepresentation},
     Binary,
@@ -269,3 +271,22 @@ fn interop_1() {
     let d_uuid = doc! { "uuid": uuid_uuid };
     assert_eq!(d_bson, d_uuid);
 }
+
+#[test]
+fn deserialize_uuid_from_string() {
+    #[derive(Deserialize)]
+    struct UuidWrapper {
+        uuid: Uuid,
+    }
+
+    let uuid = Uuid::new();
+
+    let doc = doc! { "uuid": uuid.to_string() };
+    let wrapper: UuidWrapper = from_document(doc).expect("failed to deserialize document");
+    assert_eq!(wrapper.uuid, uuid);
+
+    let raw_doc = rawdoc! { "uuid": uuid.to_string() };
+    let wrapper: UuidWrapper =
+        from_slice(raw_doc.as_bytes()).expect("failed to deserialize raw document");
+    assert_eq!(wrapper.uuid, uuid);
+}