Change app_hash field type in Genesis to AppHash

informalsystems · Nov 16, 2022 · 8c5565b · 8c5565b
1 parent 0cafd74
commit 8c5565b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/rpc/tests/kvstore_fixtures.rs b/rpc/tests/kvstore_fixtures.rs
@@ -656,7 +656,7 @@ fn incoming_fixtures() {
                 let result =
                     endpoint::genesis::Response::<Option<serde_json::Value>>::from_string(content)
                         .unwrap();
-                assert!(result.genesis.app_hash.is_empty());
+                assert!(result.genesis.app_hash.as_bytes().is_empty());
                 assert_eq!(result.genesis.chain_id.as_str(), CHAIN_ID);
                 assert_eq!(result.genesis.consensus_params.block.max_bytes, 22020096);
                 assert_eq!(result.genesis.consensus_params.block.max_gas, -1);

diff --git a/tendermint/src/genesis.rs b/tendermint/src/genesis.rs
@@ -2,7 +2,7 @@
 
 use serde::{Deserialize, Serialize};
 
-use crate::{chain, consensus, prelude::*, serializers, validator, Time};
+use crate::{chain, consensus, prelude::*, serializers, validator, AppHash, Time};
 
 /// Genesis data
 #[derive(Clone, Debug, Serialize, Deserialize)]
@@ -25,8 +25,8 @@ pub struct Genesis<AppState = serde_json::Value> {
     pub validators: Vec<validator::Info>,
 
     /// App hash
-    #[serde(with = "serializers::bytes::hexstring")]
-    pub app_hash: Vec<u8>,
+    #[serde(with = "serializers::apphash")]
+    pub app_hash: AppHash,
 
     /// App state
     pub app_state: AppState,