From bd90e1249e6396562902a3dae8fe22725e7b049c Mon Sep 17 00:00:00 2001 From: PoulavBhowmick03 Date: Thu, 17 Jul 2025 09:39:36 +0530 Subject: [PATCH 1/3] change OutputV0 to camelCase --- crates/supervisor/types/src/types.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/crates/supervisor/types/src/types.rs b/crates/supervisor/types/src/types.rs index cb5ddccbf3..e8ca4f6e47 100644 --- a/crates/supervisor/types/src/types.rs +++ b/crates/supervisor/types/src/types.rs @@ -31,7 +31,7 @@ impl BlockSeal { } /// Output data for version 0 of the protocol. #[derive(Debug, Clone, PartialEq, Eq, Serialize, Deserialize, Default)] -#[serde(rename_all = "PascalCase")] +#[serde(rename_all = "camelCase")] pub struct OutputV0 { /// The state root hash pub state_root: B256, From 53355c1aee6fe42c0bf649dcbd7165b62d363e1b Mon Sep 17 00:00:00 2001 From: Arun Dhyani Date: Tue, 26 Aug 2025 17:05:32 +0530 Subject: [PATCH 2/3] test cases added --- crates/supervisor/types/src/types.rs | 40 ++++++++++++++++++++++++++++ tests/devnets/simple-supervisor.yaml | 6 ++--- 2 files changed, 43 insertions(+), 3 deletions(-) diff --git a/crates/supervisor/types/src/types.rs b/crates/supervisor/types/src/types.rs index 02a8afeb9e..37a89ba781 100644 --- a/crates/supervisor/types/src/types.rs +++ b/crates/supervisor/types/src/types.rs @@ -58,3 +58,43 @@ pub struct SubscriptionEvent { /// Represents the event data sent by the node pub data: Option, } + +#[cfg(test)] +mod tests { + use super::*; + use alloy_primitives::B256; + use serde_json::{json, Value}; + + #[test] + fn test_output_v0_serialize_camel_case() { + let output = OutputV0 { + state_root: B256::from([1u8; 32]), + message_passer_storage_root: B256::from([2u8; 32]), + block_hash: B256::from([3u8; 32]), + }; + + let json_str = serde_json::to_string(&output).unwrap(); + let v: Value = serde_json::from_str(&json_str).unwrap(); + + // Check that keys are camelCase + assert!(v.get("stateRoot").is_some()); + assert!(v.get("messagePasserStorageRoot").is_some()); + assert!(v.get("blockHash").is_some()); + } + + #[test] + fn test_output_v0_deserialize_camel_case() { + let json_obj = json!({ + "stateRoot": "0x0101010101010101010101010101010101010101010101010101010101010101", + "messagePasserStorageRoot": "0x0202020202020202020202020202020202020202020202020202020202020202", + "blockHash": "0x0303030303030303030303030303030303030303030303030303030303030303" + }); + + let json_str = serde_json::to_string(&json_obj).unwrap(); + let output: OutputV0 = serde_json::from_str(&json_str).unwrap(); + + assert_eq!(output.state_root, B256::from([1u8; 32])); + assert_eq!(output.message_passer_storage_root, B256::from([2u8; 32])); + assert_eq!(output.block_hash, B256::from([3u8; 32])); + } +} \ No newline at end of file diff --git a/tests/devnets/simple-supervisor.yaml b/tests/devnets/simple-supervisor.yaml index 225b10373a..e7659b49a8 100644 --- a/tests/devnets/simple-supervisor.yaml +++ b/tests/devnets/simple-supervisor.yaml @@ -10,7 +10,7 @@ optimism_package: image: us-docker.pkg.dev/oplabs-tools-artifacts/images/op-geth:v1.101511.0 cl: type: op-node - image: us-docker.pkg.dev/oplabs-tools-artifacts/images/op-node:v1.13.5 + image: us-docker.pkg.dev/oplabs-tools-artifacts/images/op-node:develop log_level: debug extra_params: [--experimental.sequencer-api=true] network_params: @@ -35,7 +35,7 @@ optimism_package: image: us-docker.pkg.dev/oplabs-tools-artifacts/images/op-geth:v1.101511.0 cl: type: op-node - image: us-docker.pkg.dev/oplabs-tools-artifacts/images/op-node:v1.13.5 + image: us-docker.pkg.dev/oplabs-tools-artifacts/images/op-node:develop log_level: debug extra_params: [--experimental.sequencer-api=true] network_params: @@ -62,7 +62,7 @@ optimism_package: image: kona-supervisor:local superchain: superchain extra_env_vars: - RUST_LOG: "supervisor=info" + RUST_LOG: "supervisor=info,supervisor::chain_processor=trace" test-sequencers: sequencer: enabled: true From 926827d914c6a712a3c0e6c13b12725a02de5b3e Mon Sep 17 00:00:00 2001 From: Arun Dhyani Date: Tue, 26 Aug 2025 17:08:13 +0530 Subject: [PATCH 3/3] lintfix --- crates/supervisor/types/src/types.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/crates/supervisor/types/src/types.rs b/crates/supervisor/types/src/types.rs index 37a89ba781..5bd183911e 100644 --- a/crates/supervisor/types/src/types.rs +++ b/crates/supervisor/types/src/types.rs @@ -63,7 +63,7 @@ pub struct SubscriptionEvent { mod tests { use super::*; use alloy_primitives::B256; - use serde_json::{json, Value}; + use serde_json::{Value, json}; #[test] fn test_output_v0_serialize_camel_case() { @@ -97,4 +97,4 @@ mod tests { assert_eq!(output.message_passer_storage_root, B256::from([2u8; 32])); assert_eq!(output.block_hash, B256::from([3u8; 32])); } -} \ No newline at end of file +}