chore(core, sequencer): add names to all snapshot tests (#1690)

## Summary Added names to all snapshot tests which did not have them. ## Background If a certain test had multiple snapshot assertions, they would be named based on their ordering. Hence any reordering, removal, or addition of the snapshots would break the test, which should not be the case. This change aims to fix this problem as well as establish a best practice of naming snapshots moving forward. ## Changes - Added names to all snapshot tests and regenerated all snapshots not within Conductor (due to code freeze) ## Testing Passing all tests. ## Related Issues closes #1656
astriaorg · Dec 6, 2024 · 7e00378 · 7e00378
1 parent 88adcb3
commit 7e00378
Show file tree

Hide file tree

Showing 57 changed files with 83 additions and 80 deletions.
diff --git a/crates/astria-core/src/primitive/v1/asset/denom.rs b/crates/astria-core/src/primitive/v1/asset/denom.rs
@@ -792,10 +792,10 @@ mod serde_impl {
         }
         #[test]
         fn snapshots() {
-            insta::assert_json_snapshot!(ibc_prefixed());
-            insta::assert_json_snapshot!(trace_prefixed());
-            insta::assert_json_snapshot!(Denom::from(ibc_prefixed()));
-            insta::assert_json_snapshot!(Denom::from(trace_prefixed()));
+            insta::assert_json_snapshot!("ibc_prefixed", ibc_prefixed());
+            insta::assert_json_snapshot!("trace_prefixed", trace_prefixed());
+            insta::assert_json_snapshot!("ibc_prefixed_denom", Denom::from(ibc_prefixed()));
+            insta::assert_json_snapshot!("trace_prefixed_denom", Denom::from(trace_prefixed()));
         }
     }
 }

diff --git a/..._denom__serde_impl__tests__snapshots.snap → ...nom__serde_impl__tests__ibc_prefixed.snap b/..._denom__serde_impl__tests__snapshots.snap → ...nom__serde_impl__tests__ibc_prefixed.snap
diff --git a/...enom__serde_impl__tests__snapshots-3.snap → ...erde_impl__tests__ibc_prefixed_denom.snap b/...enom__serde_impl__tests__snapshots-3.snap → ...erde_impl__tests__ibc_prefixed_denom.snap
diff --git a/...enom__serde_impl__tests__snapshots-2.snap → ...m__serde_impl__tests__trace_prefixed.snap b/...enom__serde_impl__tests__snapshots-2.snap → ...m__serde_impl__tests__trace_prefixed.snap
diff --git a/...enom__serde_impl__tests__snapshots-4.snap → ...de_impl__tests__trace_prefixed_denom.snap b/...enom__serde_impl__tests__snapshots-4.snap → ...de_impl__tests__trace_prefixed_denom.snap
diff --git a/crates/astria-core/src/primitive/v1/mod.rs b/crates/astria-core/src/primitive/v1/mod.rs
@@ -410,15 +410,15 @@ mod tests {
             .prefix(ASTRIA_ADDRESS_PREFIX)
             .try_build()
             .unwrap();
-        insta::assert_json_snapshot!(&main_address.to_raw());
+        insta::assert_json_snapshot!("main_bech32m_address", &main_address.to_raw());
 
         let compat_address = main_address
             .to_prefix(ASTRIA_COMPAT_ADDRESS_PREFIX)
             .unwrap()
             .to_format::<Bech32>();
         // We don't allow serializing non bech32m addresses due to
         // its impl via the protobuf type.
-        insta::assert_snapshot!(&compat_address);
+        insta::assert_snapshot!("compat_bech32_non_m_address", &compat_address);
     }
 
     #[test]

diff --git a/...e__primitive__v1__tests__snapshots-2.snap → ...__tests__compat_bech32_non_m_address.snap b/...e__primitive__v1__tests__snapshots-2.snap → ...__tests__compat_bech32_non_m_address.snap
@@ -1,5 +1,5 @@
 ---
 source: crates/astria-core/src/primitive/v1/mod.rs
-expression: "&bech32_address"
+expression: "&compat_address"
 ---
 astriacompat19g4z52329g4z52329g4z52329g4z52322jspvr
diff --git a/...ore__primitive__v1__tests__snapshots.snap → ...ive__v1__tests__main_bech32m_address.snap b/...ore__primitive__v1__tests__snapshots.snap → ...ive__v1__tests__main_bech32m_address.snap
@@ -1,6 +1,6 @@
 ---
 source: crates/astria-core/src/primitive/v1/mod.rs
-expression: address
+expression: "&main_address.to_raw()"
 ---
 {
   "bech32m": "astria19g4z52329g4z52329g4z52329g4z5232ayenag"

diff --git a/...1__tests__genesis_state_is_unchanged.snap → ...l__genesis__v1__tests__genesis_state.snap b/...1__tests__genesis_state_is_unchanged.snap → ...l__genesis__v1__tests__genesis_state.snap
@@ -1,5 +1,5 @@
 ---
-source: crates/astria-core/src/protocol/genesis/v1alpha1.rs
+source: crates/astria-core/src/protocol/genesis/v1.rs
 expression: genesis_state()
 ---
 {

diff --git a/crates/astria-core/src/protocol/genesis/v1.rs b/crates/astria-core/src/protocol/genesis/v1.rs
@@ -1046,6 +1046,6 @@ mod tests {
     #[cfg(feature = "serde")]
     #[test]
     fn genesis_state_is_unchanged() {
-        insta::assert_json_snapshot!(genesis_state());
+        insta::assert_json_snapshot!("genesis_state", genesis_state());
     }
 }
diff --git a/...ics20_transfer_deposit_memo_snapshot.snap → ...__tests__ics20_transfer_deposit_memo.snap b/...ics20_transfer_deposit_memo_snapshot.snap → ...__tests__ics20_transfer_deposit_memo.snap
@@ -1,6 +1,5 @@
 ---
-source: crates/astria-core/src/protocol/memos/v1alpha1.rs
-assertion_line: 28
+source: crates/astria-core/src/protocol/memos/v1.rs
 expression: memo
 ---
 {

diff --git a/...withdrawal_from_rollup_memo_snapshot.snap → ...s__ics20_withdrawal_from_rollup_memo.snap b/...withdrawal_from_rollup_memo_snapshot.snap → ...s__ics20_withdrawal_from_rollup_memo.snap
@@ -1,6 +1,5 @@
 ---
-source: crates/astria-core/src/protocol/memos/v1alpha1.rs
-assertion_line: 19
+source: crates/astria-core/src/protocol/memos/v1.rs
 expression: memo
 ---
 {

diff --git a/crates/astria-core/src/protocol/memos/v1.rs b/crates/astria-core/src/protocol/memos/v1.rs
@@ -16,7 +16,7 @@ mod tests {
             memo: "hello".to_string(),
         };
 
-        insta::assert_json_snapshot!(memo);
+        insta::assert_json_snapshot!("ics20_withdrawal_from_rollup_memo", memo);
     }
 
     #[test]
@@ -25,6 +25,6 @@ mod tests {
             rollup_deposit_address: "an-address-on-the-rollup".to_string(),
         };
 
-        insta::assert_json_snapshot!(memo);
+        insta::assert_json_snapshot!("ics20_transfer_deposit_memo", memo);
     }
 }
diff --git a/crates/astria-core/src/protocol/transaction/v1/mod.rs b/crates/astria-core/src/protocol/transaction/v1/mod.rs
@@ -621,7 +621,7 @@ mod tests {
             body_bytes: body.to_raw().encode_to_vec().into(),
         };
 
-        insta::assert_json_snapshot!(tx.id().to_raw());
+        insta::assert_json_snapshot!("transaction_id", tx.id().to_raw());
     }
 
     #[test]

diff --git a/crates/astria-sequencer/src/accounts/storage/keys.rs b/crates/astria-sequencer/src/accounts/storage/keys.rs
@@ -85,8 +85,8 @@ mod tests {
 
     #[test]
     fn keys_should_not_change() {
-        insta::assert_snapshot!(balance(&address(), &asset()));
-        insta::assert_snapshot!(nonce(&address()));
+        insta::assert_snapshot!("balance_key", balance(&address(), &asset()));
+        insta::assert_snapshot!("nonce_key", nonce(&address()));
     }
 
     #[test]

diff --git a/..._keys__tests__keys_should_not_change.snap → ...s__storage__keys__tests__balance_key.snap b/..._keys__tests__keys_should_not_change.snap → ...s__storage__keys__tests__balance_key.snap
diff --git a/...eys__tests__keys_should_not_change-2.snap → ...nts__storage__keys__tests__nonce_key.snap b/...eys__tests__keys_should_not_change-2.snap → ...nts__storage__keys__tests__nonce_key.snap
diff --git a/crates/astria-sequencer/src/address/storage/keys.rs b/crates/astria-sequencer/src/address/storage/keys.rs
@@ -9,8 +9,8 @@ mod tests {
 
     #[test]
     fn keys_should_not_change() {
-        insta::assert_snapshot!(BASE_PREFIX);
-        insta::assert_snapshot!(IBC_COMPAT_PREFIX);
+        insta::assert_snapshot!("base_prefix", BASE_PREFIX);
+        insta::assert_snapshot!("ibc_compat_prefix", IBC_COMPAT_PREFIX);
     }
 
     #[test]

diff --git a/..._keys__tests__keys_should_not_change.snap → ...s__storage__keys__tests__base_prefix.snap b/..._keys__tests__keys_should_not_change.snap → ...s__storage__keys__tests__base_prefix.snap
@@ -1,6 +1,5 @@
 ---
 source: crates/astria-sequencer/src/address/storage/keys.rs
-assertion_line: 10
-expression: BASE_PREFIX_KEY
+expression: BASE_PREFIX
 ---
 address/prefixes/base
diff --git a/...eys__tests__keys_should_not_change-2.snap → ...rage__keys__tests__ibc_compat_prefix.snap b/...eys__tests__keys_should_not_change-2.snap → ...rage__keys__tests__ibc_compat_prefix.snap
@@ -1,6 +1,5 @@
 ---
 source: crates/astria-sequencer/src/address/storage/keys.rs
-assertion_line: 11
-expression: IBC_COMPAT_PREFIX_KEY
+expression: IBC_COMPAT_PREFIX
 ---
 address/prefixes/ibc_compat
diff --git a/...eaking_changes__app_genesis_snapshot.snap → ...reaking_changes__app_hash_at_genesis.snap b/...eaking_changes__app_genesis_snapshot.snap → ...reaking_changes__app_hash_at_genesis.snap
diff --git a/...ansaction_with_every_action_snapshot.snap → ...anges__app_hash_execute_every_action.snap b/...ansaction_with_every_action_snapshot.snap → ...anges__app_hash_execute_every_action.snap
diff --git a/...changes__app_finalize_block_snapshot.snap → ...ing_changes__app_hash_finalize_block.snap b/...changes__app_finalize_block_snapshot.snap → ...ing_changes__app_hash_finalize_block.snap
diff --git a/crates/astria-sequencer/src/app/storage/keys.rs b/crates/astria-sequencer/src/app/storage/keys.rs
@@ -15,11 +15,11 @@ mod tests {
 
     #[test]
     fn keys_should_not_change() {
-        insta::assert_snapshot!(CHAIN_ID);
-        insta::assert_snapshot!(REVISION_NUMBER);
-        insta::assert_snapshot!(BLOCK_HEIGHT);
-        insta::assert_snapshot!(BLOCK_TIMESTAMP);
-        insta::assert_snapshot!(storage_version_by_height(42));
+        insta::assert_snapshot!("chain_id_key", CHAIN_ID);
+        insta::assert_snapshot!("revision_number_key", REVISION_NUMBER);
+        insta::assert_snapshot!("block_height_key", BLOCK_HEIGHT);
+        insta::assert_snapshot!("block_timestamp_key", BLOCK_TIMESTAMP);
+        insta::assert_snapshot!("storage_version_key", storage_version_by_height(42));
     }
 
     #[test]

diff --git a/...eys__tests__keys_should_not_change-3.snap → ...orage__keys__tests__block_height_key.snap b/...eys__tests__keys_should_not_change-3.snap → ...orage__keys__tests__block_height_key.snap
@@ -1,6 +1,5 @@
 ---
 source: crates/astria-sequencer/src/app/storage/keys.rs
-assertion_line: 22
-expression: BLOCK_HEIGHT_KEY
+expression: BLOCK_HEIGHT
 ---
 app/block_height
diff --git a/...eys__tests__keys_should_not_change-4.snap → ...ge__keys__tests__block_timestamp_key.snap b/...eys__tests__keys_should_not_change-4.snap → ...ge__keys__tests__block_timestamp_key.snap
@@ -1,6 +1,5 @@
 ---
 source: crates/astria-sequencer/src/app/storage/keys.rs
-assertion_line: 23
-expression: BLOCK_TIMESTAMP_KEY
+expression: BLOCK_TIMESTAMP
 ---
 app/block_timestamp
diff --git a/..._keys__tests__keys_should_not_change.snap → ...__storage__keys__tests__chain_id_key.snap b/..._keys__tests__keys_should_not_change.snap → ...__storage__keys__tests__chain_id_key.snap
@@ -1,6 +1,5 @@
 ---
 source: crates/astria-sequencer/src/app/storage/keys.rs
-assertion_line: 18
-expression: CHAIN_ID_KEY
+expression: CHAIN_ID
 ---
 app/chain_id
diff --git a/...eys__tests__keys_should_not_change-2.snap → ...ge__keys__tests__revision_number_key.snap b/...eys__tests__keys_should_not_change-2.snap → ...ge__keys__tests__revision_number_key.snap
@@ -1,6 +1,5 @@
 ---
 source: crates/astria-sequencer/src/app/storage/keys.rs
-assertion_line: 21
-expression: REVISION_NUMBER_KEY
+expression: REVISION_NUMBER
 ---
 app/revision_number
diff --git a/...eys__tests__keys_should_not_change-5.snap → ...ge__keys__tests__storage_version_key.snap b/...eys__tests__keys_should_not_change-5.snap → ...ge__keys__tests__storage_version_key.snap
@@ -1,6 +1,5 @@
 ---
 source: crates/astria-sequencer/src/app/storage/keys.rs
-assertion_line: 22
 expression: storage_version_by_height(42)
 ---
 app/storage_version/42
diff --git a/crates/astria-sequencer/src/app/tests_breaking_changes.rs b/crates/astria-sequencer/src/app/tests_breaking_changes.rs
@@ -81,7 +81,7 @@ use crate::{
 #[tokio::test]
 async fn app_genesis_snapshot() {
     let app = initialize_app(None, vec![]).await;
-    insta::assert_json_snapshot!(app.app_hash.as_bytes());
+    insta::assert_json_snapshot!("app_hash_at_genesis", app.app_hash.as_bytes());
 }
 
 #[tokio::test]
@@ -161,7 +161,7 @@ async fn app_finalize_block_snapshot() {
         .await
         .unwrap();
     app.commit(storage.clone()).await;
-    insta::assert_json_snapshot!(app.app_hash.as_bytes());
+    insta::assert_json_snapshot!("app_hash_finalize_block", app.app_hash.as_bytes());
 }
 
 // Note: this tests every action except for `Ics20Withdrawal` and `IbcRelay`.
@@ -368,5 +368,5 @@ async fn app_execute_transaction_with_every_action_snapshot() {
     app.prepare_commit(storage.clone()).await.unwrap();
     app.commit(storage.clone()).await;
 
-    insta::assert_json_snapshot!(app.app_hash.as_bytes());
+    insta::assert_json_snapshot!("app_hash_execute_every_action", app.app_hash.as_bytes());
 }
diff --git a/crates/astria-sequencer/src/assets/storage/keys.rs b/crates/astria-sequencer/src/assets/storage/keys.rs
@@ -29,8 +29,8 @@ mod tests {
 
     #[test]
     fn keys_should_not_change() {
-        insta::assert_snapshot!(NATIVE_ASSET);
-        insta::assert_snapshot!(asset(&test_asset()));
+        insta::assert_snapshot!("native_asset_key", NATIVE_ASSET);
+        insta::assert_snapshot!("asset_key_test_asset", asset(&test_asset()));
     }
 
     #[test]

diff --git a/...eys__tests__keys_should_not_change-2.snap → ...e__keys__tests__asset_key_test_asset.snap b/...eys__tests__keys_should_not_change-2.snap → ...e__keys__tests__asset_key_test_asset.snap
@@ -1,6 +1,5 @@
 ---
 source: crates/astria-sequencer/src/assets/storage/keys.rs
-assertion_line: 87
 expression: asset(&test_asset())
 ---
 assets/ibc/be429a02d00837245167a2616674a979a2ac6f9806468b48a975b156ad711320
diff --git a/..._keys__tests__keys_should_not_change.snap → ...orage__keys__tests__native_asset_key.snap b/..._keys__tests__keys_should_not_change.snap → ...orage__keys__tests__native_asset_key.snap
@@ -1,6 +1,5 @@
 ---
 source: crates/astria-sequencer/src/assets/storage/keys.rs
-assertion_line: 84
-expression: NATIVE_ASSET_KEY
+expression: NATIVE_ASSET
 ---
 assets/native_asset
diff --git a/crates/astria-sequencer/src/authority/storage/keys.rs b/crates/astria-sequencer/src/authority/storage/keys.rs
@@ -10,9 +10,9 @@ mod tests {
 
     #[test]
     fn keys_should_not_change() {
-        insta::assert_snapshot!(SUDO);
-        insta::assert_snapshot!(VALIDATOR_SET);
-        insta::assert_snapshot!(VALIDATOR_UPDATES);
+        insta::assert_snapshot!("sudo_address_key", SUDO);
+        insta::assert_snapshot!("validator_set_key", VALIDATOR_SET);
+        insta::assert_snapshot!("validator_updates_key", VALIDATOR_UPDATES);
     }
 
     #[test]

diff --git a/..._keys__tests__keys_should_not_change.snap → ...orage__keys__tests__sudo_address_key.snap b/..._keys__tests__keys_should_not_change.snap → ...orage__keys__tests__sudo_address_key.snap
@@ -1,6 +1,5 @@
 ---
 source: crates/astria-sequencer/src/authority/storage/keys.rs
-assertion_line: 15
-expression: SUDO_STORAGE_KEY
+expression: SUDO
 ---
 authority/sudo
diff --git a/...eys__tests__keys_should_not_change-2.snap → ...rage__keys__tests__validator_set_key.snap b/...eys__tests__keys_should_not_change-2.snap → ...rage__keys__tests__validator_set_key.snap
@@ -1,6 +1,5 @@
 ---
 source: crates/astria-sequencer/src/authority/storage/keys.rs
-assertion_line: 16
-expression: VALIDATOR_SET_STORAGE_KEY
+expression: VALIDATOR_SET
 ---
 authority/validator_set
diff --git a/...eys__tests__keys_should_not_change-3.snap → ...__keys__tests__validator_updates_key.snap b/...eys__tests__keys_should_not_change-3.snap → ...__keys__tests__validator_updates_key.snap
@@ -1,6 +1,5 @@
 ---
 source: crates/astria-sequencer/src/authority/storage/keys.rs
-assertion_line: 15
 expression: VALIDATOR_UPDATES
 ---
 authority/validator_updates
diff --git a/crates/astria-sequencer/src/bridge/storage/keys.rs b/crates/astria-sequencer/src/bridge/storage/keys.rs
@@ -91,14 +91,26 @@ mod tests {
 
     #[test]
     fn keys_should_not_change() {
-        insta::assert_snapshot!(DEPOSITS_EPHEMERAL);
-        insta::assert_snapshot!(rollup_id(&address()));
-        insta::assert_snapshot!(asset_id(&address()));
-        insta::assert_snapshot!(bridge_account_sudo_address(&address()));
-        insta::assert_snapshot!(bridge_account_withdrawer_address(&address()));
-        insta::assert_snapshot!(bridge_account_withdrawal_event(&address(), "the-event"));
-        insta::assert_snapshot!(deposit(&[1; 32], &RollupId::new([2; 32])));
-        insta::assert_snapshot!(last_transaction_id_for_bridge_account(&address()));
+        insta::assert_snapshot!("ephemeral_deposits_key", DEPOSITS_EPHEMERAL);
+        insta::assert_snapshot!("rollup_id_key", rollup_id(&address()));
+        insta::assert_snapshot!("asset_id_key", asset_id(&address()));
+        insta::assert_snapshot!(
+            "bridge_sudo_address_key",
+            bridge_account_sudo_address(&address())
+        );
+        insta::assert_snapshot!(
+            "bridge_withdrawer_address_key",
+            bridge_account_withdrawer_address(&address())
+        );
+        insta::assert_snapshot!(
+            "bridge_withdrawal_event_key",
+            bridge_account_withdrawal_event(&address(), "the-event")
+        );
+        insta::assert_snapshot!("deposit_key", deposit(&[1; 32], &RollupId::new([2; 32])));
+        insta::assert_snapshot!(
+            "last_tx_id_for_bridge_acct_key",
+            last_transaction_id_for_bridge_account(&address())
+        );
     }
 
     #[test]

diff --git a/...eys__tests__keys_should_not_change-3.snap → ...__storage__keys__tests__asset_id_key.snap b/...eys__tests__keys_should_not_change-3.snap → ...__storage__keys__tests__asset_id_key.snap
diff --git a/...eys__tests__keys_should_not_change-4.snap → ...keys__tests__bridge_sudo_address_key.snap b/...eys__tests__keys_should_not_change-4.snap → ...keys__tests__bridge_sudo_address_key.snap
diff --git a/...eys__tests__keys_should_not_change-6.snap → ...__tests__bridge_withdrawal_event_key.snap b/...eys__tests__keys_should_not_change-6.snap → ...__tests__bridge_withdrawal_event_key.snap
diff --git a/...eys__tests__keys_should_not_change-5.snap → ...tests__bridge_withdrawer_address_key.snap b/...eys__tests__keys_should_not_change-5.snap → ...tests__bridge_withdrawer_address_key.snap
diff --git a/...eys__tests__keys_should_not_change-7.snap → ...e__storage__keys__tests__deposit_key.snap b/...eys__tests__keys_should_not_change-7.snap → ...e__storage__keys__tests__deposit_key.snap
diff --git a/..._keys__tests__keys_should_not_change.snap → ..._keys__tests__ephemeral_deposits_key.snap b/..._keys__tests__keys_should_not_change.snap → ..._keys__tests__ephemeral_deposits_key.snap
diff --git a/...eys__tests__keys_should_not_change-8.snap → ...ests__last_tx_id_for_bridge_acct_key.snap b/...eys__tests__keys_should_not_change-8.snap → ...ests__last_tx_id_for_bridge_acct_key.snap
diff --git a/...eys__tests__keys_should_not_change-2.snap → ..._storage__keys__tests__rollup_id_key.snap b/...eys__tests__keys_should_not_change-2.snap → ..._storage__keys__tests__rollup_id_key.snap
diff --git a/crates/astria-sequencer/src/grpc/storage/keys.rs b/crates/astria-sequencer/src/grpc/storage/keys.rs
@@ -50,12 +50,21 @@ mod tests {
 
     #[test]
     fn keys_should_not_change() {
-        insta::assert_snapshot!(block_hash_by_height(42));
-        insta::assert_snapshot!(sequencer_block_header_by_hash(&HASH));
-        insta::assert_snapshot!(rollup_data_by_hash_and_rollup_id(&HASH, &ROLLUP_ID));
-        insta::assert_snapshot!(rollup_ids_by_hash(&HASH));
-        insta::assert_snapshot!(rollup_transactions_proof_by_hash(&HASH));
-        insta::assert_snapshot!(rollup_ids_proof_by_hash(&HASH));
+        insta::assert_snapshot!("block_hash_by_height_key", block_hash_by_height(42));
+        insta::assert_snapshot!(
+            "sequencer_block_header_by_hash_key",
+            sequencer_block_header_by_hash(&HASH)
+        );
+        insta::assert_snapshot!(
+            "rollup_data_by_has_and_id_key",
+            rollup_data_by_hash_and_rollup_id(&HASH, &ROLLUP_ID)
+        );
+        insta::assert_snapshot!("rollup_ids_by_hash_key", rollup_ids_by_hash(&HASH));
+        insta::assert_snapshot!(
+            "rollup_transactions_proof_by_hash_key",
+            rollup_transactions_proof_by_hash(&HASH)
+        );
+        insta::assert_snapshot!("rollup_ids_proof_by_hash", rollup_ids_proof_by_hash(&HASH));
     }
 
     #[test]

diff --git a/..._keys__tests__keys_should_not_change.snap → ...eys__tests__block_hash_by_height_key.snap b/..._keys__tests__keys_should_not_change.snap → ...eys__tests__block_hash_by_height_key.snap
@@ -1,6 +1,5 @@
 ---
 source: crates/astria-sequencer/src/grpc/storage/keys.rs
-assertion_line: 44
 expression: block_hash_by_height(42)
 ---
 grpc/block_hash/42
diff --git a/...eys__tests__keys_should_not_change-3.snap → ...tests__rollup_data_by_has_and_id_key.snap b/...eys__tests__keys_should_not_change-3.snap → ...tests__rollup_data_by_has_and_id_key.snap
@@ -1,6 +1,5 @@
 ---
 source: crates/astria-sequencer/src/grpc/storage/keys.rs
-assertion_line: 46
 expression: "rollup_data_by_hash_and_rollup_id(&HASH, &ROLLUP_ID)"
 ---
 grpc/rollup_data/AQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQE=/AgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgI=
diff --git a/...eys__tests__keys_should_not_change-4.snap → ..._keys__tests__rollup_ids_by_hash_key.snap b/...eys__tests__keys_should_not_change-4.snap → ..._keys__tests__rollup_ids_by_hash_key.snap
@@ -1,6 +1,5 @@
 ---
 source: crates/astria-sequencer/src/grpc/storage/keys.rs
-assertion_line: 49
 expression: rollup_ids_by_hash(&HASH)
 ---
 grpc/rollup_ids/AQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQE=
diff --git a/...eys__tests__keys_should_not_change-6.snap → ...eys__tests__rollup_ids_proof_by_hash.snap b/...eys__tests__keys_should_not_change-6.snap → ...eys__tests__rollup_ids_proof_by_hash.snap
@@ -1,6 +1,5 @@
 ---
 source: crates/astria-sequencer/src/grpc/storage/keys.rs
-assertion_line: 51
 expression: rollup_ids_proof_by_hash(&HASH)
 ---
 grpc/rollup_ids_proof/AQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQE=
diff --git a/...eys__tests__keys_should_not_change-5.snap → ...ollup_transactions_proof_by_hash_key.snap b/...eys__tests__keys_should_not_change-5.snap → ...ollup_transactions_proof_by_hash_key.snap
@@ -1,6 +1,5 @@
 ---
 source: crates/astria-sequencer/src/grpc/storage/keys.rs
-assertion_line: 50
 expression: rollup_transactions_proof_by_hash(&HASH)
 ---
 grpc/rollup_txs_proof/AQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQE=
diff --git a/...eys__tests__keys_should_not_change-2.snap → ...__sequencer_block_header_by_hash_key.snap b/...eys__tests__keys_should_not_change-2.snap → ...__sequencer_block_header_by_hash_key.snap
@@ -1,6 +1,5 @@
 ---
 source: crates/astria-sequencer/src/grpc/storage/keys.rs
-assertion_line: 45
 expression: sequencer_block_header_by_hash(&HASH)
 ---
 grpc/block_header/AQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQEBAQE=
-Original file line number
+Diff line change
@@ Expand Up / @@ -621,7 +621,7 @@ mod tests { @@
                 body_bytes: body.to_raw().encode_to_vec().into(),
             };
-            insta::assert_json_snapshot!(tx.id().to_raw());
+            insta::assert_json_snapshot!("transaction_id", tx.id().to_raw());
         }
         #[test]
@@ Expand Down @@