Skip to content

Commit 244f2cd

Browse files
committed
test(op-revm): run deposit gas tests under ARSIA (Osaka) spec to match production
The 7 end-to-end deposit gas/behavior tests were pinned to OpSpecId::ISTHMUS (eth_spec PRAGUE), but block 59294 — and all Mantle hoodi-qa2 deposits — execute under OpSpecId::ARSIA (eth_spec OSAKA). ISTHMUS exercises a different EVM gas regime and leaves the is_arsia branches uncovered. Switch deposit_to_eoa_with_calldata_no_compensation_matches_geth (block 59294 replay) plus the 6 other end-to-end gas tests to ARSIA so they validate under the deployed spec. ARSIA superset of OSAKA, so one ARSIA spec covers the Osaka EVM rules plus Mantle ARSIA features. Mechanism/cold-flag tests and the mainnet fixture replays (path_1/2/3, spec-bound to their historical blocks) stay on ISTHMUS. cargo test -p op-revm: 121 passed, 0 failed (gas values unchanged; 21320 floor holds under Osaka).
1 parent 4c215fb commit 244f2cd

1 file changed

Lines changed: 7 additions & 7 deletions

File tree

crates/op-revm/src/transaction/bvm_eth.rs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -693,7 +693,7 @@ mod tests {
693693
.with_db(InMemoryDB::default())
694694
.with_chain(l1_block_info)
695695
.with_block(block_env)
696-
.modify_cfg_chained(|cfg| cfg.spec = OpSpecId::ISTHMUS)
696+
.modify_cfg_chained(|cfg| cfg.spec = OpSpecId::ARSIA)
697697
.with_tx(op_tx);
698698
let mut evm = ctx.build_op();
699699
let mut handler = OpHandler::<
@@ -1141,7 +1141,7 @@ mod tests {
11411141
let ctx = Context::op()
11421142
.with_db(db)
11431143
.with_chain(l1_block_info)
1144-
.modify_cfg_chained(|cfg| cfg.spec = OpSpecId::ISTHMUS)
1144+
.modify_cfg_chained(|cfg| cfg.spec = OpSpecId::ARSIA)
11451145
.modify_tx_chained(|tx| {
11461146
tx.base.caller = caller;
11471147
tx.base.kind = TxKind::Call(target);
@@ -1210,7 +1210,7 @@ mod tests {
12101210
let ctx = Context::op()
12111211
.with_db(db)
12121212
.with_chain(l1_block_info)
1213-
.modify_cfg_chained(|cfg| cfg.spec = OpSpecId::ISTHMUS)
1213+
.modify_cfg_chained(|cfg| cfg.spec = OpSpecId::ARSIA)
12141214
.modify_tx_chained(|tx| {
12151215
tx.base.caller = caller;
12161216
tx.base.kind = TxKind::Call(BvmEth::ADDRESS);
@@ -1278,7 +1278,7 @@ mod tests {
12781278
let ctx = Context::op()
12791279
.with_db(db)
12801280
.with_chain(l1_block_info)
1281-
.modify_cfg_chained(|cfg| cfg.spec = OpSpecId::ISTHMUS)
1281+
.modify_cfg_chained(|cfg| cfg.spec = OpSpecId::ARSIA)
12821282
.modify_tx_chained(|tx| {
12831283
tx.base.caller = caller;
12841284
tx.base.kind = TxKind::Call(BvmEth::ADDRESS);
@@ -1350,7 +1350,7 @@ mod tests {
13501350
let ctx = Context::op()
13511351
.with_db(db)
13521352
.with_chain(l1_block_info)
1353-
.modify_cfg_chained(|cfg| cfg.spec = OpSpecId::ISTHMUS)
1353+
.modify_cfg_chained(|cfg| cfg.spec = OpSpecId::ARSIA)
13541354
.modify_tx_chained(|tx| {
13551355
tx.base.caller = caller;
13561356
tx.base.kind = TxKind::Call(target);
@@ -1425,7 +1425,7 @@ mod tests {
14251425
let ctx = Context::op()
14261426
.with_db(db)
14271427
.with_chain(l1_block_info)
1428-
.modify_cfg_chained(|cfg| cfg.spec = OpSpecId::ISTHMUS)
1428+
.modify_cfg_chained(|cfg| cfg.spec = OpSpecId::ARSIA)
14291429
.modify_tx_chained(|tx| {
14301430
tx.base.caller = caller;
14311431
tx.base.kind = TxKind::Call(target);
@@ -1775,7 +1775,7 @@ mod tests {
17751775
let ctx = Context::op()
17761776
.with_db(db)
17771777
.with_chain(l1_block_info)
1778-
.modify_cfg_chained(|cfg| cfg.spec = OpSpecId::ISTHMUS)
1778+
.modify_cfg_chained(|cfg| cfg.spec = OpSpecId::ARSIA)
17791779
.modify_tx_chained(|tx| {
17801780
tx.base.caller = caller;
17811781
tx.base.kind = TxKind::Call(outer);

0 commit comments

Comments
 (0)