Cover Content::Seq case in VariantRefDeserializer::struct_variant

Mingun · Mingun · commit 6588b0ad3777 · 2024-08-24T04:52:25.000+05:00
diff --git a/serde/src/private/de.rs b/serde/src/private/de.rs
@@ -2217,6 +2217,8 @@ mod content {
                 Some(Content::Map(v)) => {
                     de::Deserializer::deserialize_any(MapRefDeserializer::new(v), visitor)
                 }
+                // Covered by tests/test_enum_untagged.rs
+                //      newtype_enum::struct_from_seq
                 Some(Content::Seq(v)) => {
                     de::Deserializer::deserialize_any(SeqRefDeserializer::new(v), visitor)
                 }
diff --git a/test_suite/tests/test_enum_untagged.rs b/test_suite/tests/test_enum_untagged.rs
@@ -214,6 +214,25 @@ mod newtype_enum {
             ],
         );
     }
+
+    // Reaches crate::private::de::content::VariantRefDeserializer::struct_variant
+    // Content::Seq case
+    #[test]
+    fn struct_from_seq() {
+        assert_de_tokens(
+            &Outer::Inner(Inner::Struct { f: 1 }),
+            &[
+                Token::Map { len: Some(1) },
+                // tag
+                Token::Str("Struct"),
+                // content
+                Token::Seq { len: Some(1) },
+                Token::U8(1),
+                Token::SeqEnd,
+                Token::MapEnd,
+            ],
+        );
+    }
 }
 
 // Reaches crate::private::de::content::ContentRefDeserializer::deserialize_option

Original file line number	Diff line number	Diff line change
`@@ -2217,6 +2217,8 @@ mod content {`
`2217`	`2217`	`Some(Content::Map(v)) => {`
`2218`	`2218`	`de::Deserializer::deserialize_any(MapRefDeserializer::new(v), visitor)`
`2219`	`2219`	`}`
	`2220`	`+ // Covered by tests/test_enum_untagged.rs`
	`2221`	`+ // newtype_enum::struct_from_seq`
`2220`	`2222`	`Some(Content::Seq(v)) => {`
`2221`	`2223`	`de::Deserializer::deserialize_any(SeqRefDeserializer::new(v), visitor)`
`2222`	`2224`	`}`