Move tests around to satisfy crates.io dependency requirements

Author: robertwayne

.github/workflows/ci.yml

@@ -26,7 +26,7 @@ jobs:
             - name: rustfmt
               run: cargo fmt --all --check
             - name: Run tests
-              run: cargo test -p bufferfish-core -p bufferfish -p bufferfish-derive --lib
+              run: cargo test -p bufferfish-core -p bufferfish -p bufferfish-derive --lib --all-features --all-targets
 
     check-ts:
         needs: check-rust

rust/bufferfish-derive/Cargo.toml

@@ -20,7 +20,3 @@ syn = "2"
 quote = "1"
 proc-macro2 = "1"
 proc-macro-error = "1"
-
-[dev-dependencies]
-trybuild = "1"
-bufferfish = { version = "0.1", path = "../bufferfish", features = ["derive"] }

rust/bufferfish-derive/tests/basic_header.rs

@@ -17,6 +17,9 @@ derive = ["bufferfish-derive", "syn"]
 
 
 [dependencies]
-bufferfish-core = { path = "../bufferfish-core" }
-bufferfish-derive = { path = "../bufferfish-derive", optional = true }
+bufferfish-core = { version = "0.1", path = "../bufferfish-core" }
+bufferfish-derive = { version = "0.1", path = "../bufferfish-derive", optional = true }
 syn = { version = "2", features = ["full"], optional = true }
+
+[dev-dependencies]
+trybuild = "1"

rust/bufferfish-derive/tests/deserialize_struct.rs

@@ -7,6 +7,7 @@ pub use bufferfish_derive::{Decode, Encode};
 #[cfg(feature = "derive")]
 pub use compiler::generate;
 
+#[cfg(feature = "derive")]
 #[cfg(test)]
 mod tests {
     use bufferfish_core::Bufferfish;

rust/bufferfish-derive/tests/enum_header.rs

@@ -0,0 +1,19 @@
+#[cfg(feature = "derive")]
+mod tests {
+    use bufferfish::{Bufferfish, Encodable};
+    use bufferfish_derive::Encode;
+
+    #[test]
+    fn encode_struct() {
+        #[derive(Encode, Debug)]
+        #[bufferfish(0_u16)]
+        struct Foo {
+            bar: u8,
+        }
+
+        let foo = Foo { bar: 42 };
+        let bf: Bufferfish = foo.to_bufferfish().unwrap();
+
+        assert_eq!(bf.len(), 3);
+    }
+}

rust/bufferfish-derive/tests/serialize_struct.rs

@@ -0,0 +1,43 @@
+#[cfg(feature = "derive")]
+mod tests {
+    use bufferfish_core::{Bufferfish, Decodable};
+    use bufferfish_derive::Decode;
+
+    #[test]
+    fn decode_struct() {
+        #[derive(Decode, Debug, PartialEq)]
+        #[bufferfish(0_u16)]
+        struct Foo {
+            bar: u8,
+        }
+
+        let mut bf = Bufferfish::from(vec![0, 0, 42]);
+        let foo = Foo::decode(&mut bf).unwrap();
+
+        assert_eq!(foo, Foo { bar: 42 });
+    }
+
+    #[test]
+    fn decode_unit_struct() {
+        #[derive(Decode, Debug, PartialEq)]
+        #[bufferfish(0_u16)]
+        struct Foo;
+
+        let mut bf = Bufferfish::from(vec![0, 0]);
+        let foo = Foo::decode(&mut bf).unwrap();
+
+        assert_eq!(foo, Foo);
+    }
+
+    #[test]
+    fn decode_tuple_struct() {
+        #[derive(Decode, Debug, PartialEq)]
+        #[bufferfish(0_u16)]
+        struct Foo(u8);
+
+        let mut bf = Bufferfish::from(vec![0, 0, 42]);
+        let foo = Foo::decode(&mut bf).unwrap();
+
+        assert_eq!(foo, Foo(42));
+    }
+}

rust/bufferfish/Cargo.toml

@@ -0,0 +1,32 @@
+#[cfg(feature = "derive")]
+mod tests {
+    use bufferfish_core::{Bufferfish, Encodable};
+    use bufferfish_derive::Encode;
+
+    #[allow(dead_code)]
+    enum PacketId {
+        Ping,
+    }
+
+    impl From<PacketId> for u16 {
+        fn from(packet_id: PacketId) -> u16 {
+            match packet_id {
+                PacketId::Ping => 0,
+            }
+        }
+    }
+
+    #[derive(Encode, Debug)]
+    #[bufferfish(PacketId::Ping)]
+    struct Foo {
+        bar: u8,
+    }
+
+    #[test]
+    fn encode_struct() {
+        let foo = Foo { bar: 42 };
+        let bf: Bufferfish = foo.to_bufferfish().unwrap();
+
+        assert_eq!(bf.len(), 3);
+    }
+}

rust/bufferfish/src/lib.rs

@@ -0,0 +1,43 @@
+#[cfg(feature = "derive")]
+mod tests {
+    use bufferfish_core::{Bufferfish, Encodable};
+    use bufferfish_derive::Encode;
+
+    #[test]
+    fn encode_struct() {
+        #[derive(Encode, Debug)]
+        #[bufferfish(0_u16)]
+        struct Foo {
+            bar: u8,
+        }
+
+        let foo = Foo { bar: 42 };
+        let bf: Bufferfish = foo.to_bufferfish().unwrap();
+
+        assert_eq!(bf.len(), 3);
+    }
+
+    #[test]
+    fn encode_unit_struct() {
+        #[derive(Encode, Debug)]
+        #[bufferfish(0_u16)]
+        struct Foo;
+
+        let foo = Foo;
+        let bf: Bufferfish = foo.to_bufferfish().unwrap();
+
+        assert_eq!(bf.len(), 2);
+    }
+
+    #[test]
+    fn encode_tuple_struct() {
+        #[derive(Encode, Debug)]
+        #[bufferfish(0_u16)]
+        struct Foo(u8);
+
+        let foo = Foo(42);
+        let bf: Bufferfish = foo.to_bufferfish().unwrap();
+
+        assert_eq!(bf.len(), 3);
+    }
+}