Fixes from_bytes throwing split_to out of bounds error

pumpkin-nbt/src/deserializer.rs

@@ -126,7 +126,8 @@ impl<'de, T: Buf> de::Deserializer<'de> for &mut Deserializer<'de, T> {
 
             if self.is_named {
                 // Consume struct name
-                NbtTag::deserialize(self.input)?;
+                let len = self.input.get_u16() as usize;
+                self.input.advance(len);
             }
         }
 

pumpkin-nbt/src/lib.rs

@@ -205,6 +205,8 @@ impl_array!(BytesArray, "byte");
 mod test {
     use serde::{Deserialize, Serialize};
 
+    use crate::deserializer::from_bytes;
+    use crate::serializer::to_bytes;
     use crate::BytesArray;
     use crate::IntArray;
     use crate::LongArray;
@@ -236,6 +238,23 @@ mod test {
         assert_eq!(test, recreated_struct);
     }
 
+    #[test]
+    fn test_simple_ser_de_named() {
+        let name = String::from("Test");
+        let test = Test {
+            byte: 123,
+            short: 1342,
+            int: 4313,
+            long: 34,
+            float: 1.00,
+            string: "Hello test".to_string(),
+        };
+        let mut bytes = to_bytes(&test, name).unwrap();
+        let recreated_struct: Test = from_bytes(&mut bytes).unwrap();
+
+        assert_eq!(test, recreated_struct);
+    }
+
     #[derive(Serialize, Deserialize, PartialEq, Debug)]
     struct TestArray {
         #[serde(with = "BytesArray")]
@@ -258,4 +277,18 @@ mod test {
 
         assert_eq!(test, recreated_struct);
     }
+
+    #[test]
+    fn test_simple_ser_de_array_named() {
+        let name = String::from("Test");
+        let test = TestArray {
+            byte_array: vec![0, 3, 2],
+            int_array: vec![13, 1321, 2],
+            long_array: vec![1, 0, 200301, 1],
+        };
+        let mut bytes = to_bytes(&test, name).unwrap();
+        let recreated_struct: TestArray = from_bytes(&mut bytes).unwrap();
+
+        assert_eq!(test, recreated_struct);
+    }
 }