test: add conductor api serialization guard

.github/workflows/build.yaml

@@ -9,7 +9,7 @@ jobs:
   build:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
       - uses: docker/login-action@v2
         with:
           username: ${{ secrets.DOCKERHUB_USERNAME }}

.github/workflows/test.yaml

@@ -3,11 +3,16 @@ name: test
 on: [push]
 
 jobs:
-  cargo-test:
+  lint-and-test:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
-      - uses: cachix/install-nix-action@v18
-        with:
-          nix_path: nixpkgs=channel:nixos-unstable
-      - run: nix-shell --run "cargo test"
+      - uses: actions/checkout@v4
+      - uses: cachix/install-nix-action@v23
+      - name: Build Nix packages for dev shell
+        run: nix develop -c $SHELL -c "rustc --version --verbose"
+      - name: Run fmt
+        run: nix develop -c $SHELL -c "script-holochain-tests-static-fmt"
+      - name: Run clippy
+        run: nix develop -c $SHELL -c "script-holochain-tests-static-clippy"
+      - name: Run tests
+        run: nix develop -c $SHELL -c "cargo test"

.gitignore

@@ -2,6 +2,7 @@ Cargo.lock
 target
 .cargo
 **/*.rs.bk
+.pre-commit-config.yaml
 
 # Environment Cruft
 .idea

crates/holochain_serialized_bytes/Cargo.toml

@@ -12,7 +12,7 @@ repository = "https://github.com/holochain/holochain-serialization"
 edition = "2018"
 
 [dependencies]
-serde = { version = "1.0.123", features = ["serde_derive"] }
+serde = { version = "1.0.181", features = ["serde_derive"] }
 serde_json = { version = "1.0.51", features = ["preserve_order"] }
 holochain_serialized_bytes_derive = { version = "=0.0.53", path = "../holochain_serialized_bytes_derive" }
 rmp-serde = "0.15"

crates/holochain_serialized_bytes/src/lib.rs

@@ -310,9 +310,40 @@ impl<'a> TryFrom<&'a SerializedBytes> for SerializedBytes {
 #[cfg(test)]
 pub mod tests {
 
+    use serde_json::Value;
+
     use super::prelude::*;
     use std::convert::TryInto;
 
+    #[test]
+    fn conductor_api() {
+        use rmp_serde::Deserializer;
+
+        #[derive(Serialize, Deserialize, Debug)]
+        #[serde(rename_all = "snake_case", tag = "type", content = "data")]
+        enum ConductorApi {
+            Request { param: i32 },
+        }
+
+        let request = ConductorApi::Request { param: 100 };
+        let request_encoded = encode(&request).unwrap();
+        assert_eq!(
+            request_encoded,
+            [
+                130, 164, 116, 121, 112, 101, 129, 167, 114, 101, 113, 117, 101, 115, 116, 192,
+                164, 100, 97, 116, 97, 129, 165, 112, 97, 114, 97, 109, 100
+            ]
+        );
+
+        let mut deserializer = Deserializer::new(&*request_encoded);
+        let value: Value = Deserialize::deserialize(&mut deserializer).unwrap();
+        let json_string = serde_json::to_string(&value).unwrap();
+        assert_eq!(
+            r#"{"type":{"request":null},"data":{"param":100}}"#,
+            json_string
+        );
+    }
+
     /// struct with a utf8 string in it
     #[derive(Serialize, Deserialize, PartialEq, Debug, Clone, SerializedBytes)]
     struct Foo {