serde trait check

Author: rudib

packed_struct/src/packing.rs

@@ -41,7 +41,7 @@ pub trait PackedStructSlice where Self: Sized {
     }
 }
 
-#[cfg_attr(feature = "use_serde", derive(Serialize))]
+#[cfg_attr(feature = "use_serde", derive(Serialize, Deserialize))]
 #[derive(Debug, Copy, Clone, PartialEq)]
 /// Packing errors that might occur during packing or unpacking
 pub enum PackingError {

packed_struct/src/types_num.rs

@@ -30,17 +30,18 @@ impl<T, B> Display for Integer<T, B> where T: Display {
 
 #[cfg(feature = "use_serde")]
 mod serialize {
-    
+    use serde::ser::{Serialize, Serializer};
+    use serde::de::{Deserialize, Deserializer};
 
-    impl<T, B> Serialize for Integer<T, B> where T: Serialize {
+    impl<T, B> Serialize for super::Integer<T, B> where T: Serialize {
         fn serialize<S>(&self, serializer: S) -> Result<S::Ok, S::Error>
             where S: Serializer
         {
             self.num.serialize(serializer)
         }
     }
 
-    impl<'de, T, B> Deserialize<'de> for Integer<T, B> where T: Deserialize<'de>, T: Into<Integer<T, B>> {
+    impl<'de, T, B> Deserialize<'de> for super::Integer<T, B> where T: Deserialize<'de>, T: Into<super::Integer<T, B>> {
         fn deserialize<D>(deserializer: D) -> Result<Self, D::Error>
             where D: Deserializer<'de>
         {

packed_struct/src/types_reserved.rs

@@ -15,6 +15,7 @@ pub trait ReservedBitValue {
 }
 
 #[cfg_attr(feature = "use_serde", derive(Serialize, Deserialize))]
+#[derive(Default, Copy, Clone, PartialEq)]
 pub struct BitOne;
 impl ReservedBitValue for BitOne {
     fn get_reserved_bit_value_byte() -> u8 {
@@ -23,6 +24,7 @@ impl ReservedBitValue for BitOne {
 }
 
 #[cfg_attr(feature = "use_serde", derive(Serialize, Deserialize))]
+#[derive(Default, Copy, Clone, PartialEq)]
 pub struct BitZero;
 impl ReservedBitValue for BitZero {
     fn get_reserved_bit_value_byte() -> u8 {
@@ -32,6 +34,7 @@ impl ReservedBitValue for BitZero {
 
 /// Always packs into the associated bit value. Ignores the input when unpacking.
 #[cfg_attr(feature = "use_serde", derive(Serialize, Deserialize))]
+#[derive(Default, Copy, Clone, PartialEq)]
 pub struct ReservedBits<V, B> {
     value: V,
     bits: PhantomData<B>

packed_struct_tests/Cargo.toml

@@ -5,6 +5,7 @@ authors = ["Rudi Benkovic <[email protected]>"]
 publish = false
 
 [dependencies]
-packed_struct = { version = "0.4", features = ["byte_types_64"] }
+packed_struct = { version = "0.4", features = ["byte_types_64", "use_serde"] }
 packed_struct_codegen = "0.4"
-error-chain = "0.12.0"
+error-chain = "0.12.0"
+serde = "1.0"

packed_struct_tests/tests/test_serde.rs

@@ -0,0 +1,22 @@
+extern crate packed_struct;
+extern crate serde;
+
+
+use packed_struct::prelude::*;
+
+#[test]
+fn test_serialization_traits() {
+
+    check_serde_support::<Integer<u8, packed_bits::Bits8>>();
+    check_serde_support::<PackingError>();
+    check_serde_support::<BitOne>();
+    check_serde_support::<BitZero>();
+    check_serde_support::<ReservedBits<u8, packed_bits::Bits8>>();
+}
+
+fn check_serde_support<'a, T>() 
+    where 
+        T: serde::Serialize,
+        T: serde::Deserialize<'a>
+{
+}