Skip to content

ci: add Autobahn EVM Module (CON-256)#3468

Draft
wen-coding wants to merge 2 commits into
mainfrom
wen/add_autobahn_evm_hardhat_tests
Draft

ci: add Autobahn EVM Module (CON-256)#3468
wen-coding wants to merge 2 commits into
mainfrom
wen/add_autobahn_evm_hardhat_tests

Conversation

@wen-coding
Copy link
Copy Markdown
Contributor

Mirrors the existing "EVM Module" CI job with AUTOBAHN=true plus the
GIGA_EXECUTOR/GIGA_STORAGE/GIGA_OCC flag set used by the "Autobahn
EVM GIGA Module" job. Runs evm_tests.sh under Autobahn:

  • EVMCompatabilityTest.js (broad EVM semantics: deployment, gas,
    variable types, assembly, trace, log
    filtering, edge cases — 78 cases)
  • EVMPrecompileTest.js (Sei precompiles: bank, addr, gov,
    distribution, staking, oracle, wasm)
  • SeiEndpointsTest.js (sei_*ExcludeTraceFail JSON-RPC paths)
  • AssociateTest.js (account association mechanics)

Complements the narrower "Autobahn EVM GIGA Module" (smoke-tests
EVMGigaTest.js on native+ERC20 transfers only) by adding broader
EVM-compat + Sei-precompile + sei-endpoint + association coverage
that aren't exercised elsewhere under Autobahn.

Deliberately omits the flatkv post-steps from the CometBFT "EVM
Module" job; flatkv-under-Autobahn is its own concern and would
make per-job failure triage harder.

Verified locally on a 4-node Autobahn docker cluster: 101 passing,
1 pending, 0 failing.

Autobahn EVM Interoperability deferred until #3406 (lib.js
response-reading helper migration) lands; without it, the
suite's CW-deploy setup hooks depend on unmigrated -b block
helpers and CI passing would be accidental rather than verified.

Co-Authored-By: Claude Opus 4.7 (1M context) noreply@anthropic.com

@cursor
Copy link
Copy Markdown

cursor Bot commented May 20, 2026

PR Summary

Low Risk
Low risk: CI-only change that adds an additional integration test matrix row; main risk is increased CI time/flakiness due to extra EVM test coverage under Autobahn flags.

Overview
Adds a new "Autobahn EVM Module" entry to the integration-test.yml matrix that runs evm_tests.sh with AUTOBAHN=true plus GIGA_EXECUTOR/GIGA_STORAGE/GIGA_OCC enabled.

Unlike the existing EVM Module job, this new variant intentionally omits the FlatKV deploy/verify follow-up steps (left as a TODO), so it focuses on running the core EVM test suite under Autobahn configuration.

Reviewed by Cursor Bugbot for commit 6accbac. Bugbot is set up for automated code reviews on this repo. Configure here.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 20, 2026

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedMay 20, 2026, 3:50 AM

@github-actions
Copy link
Copy Markdown

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedMay 20, 2026, 12:20 AM

@wen-coding wen-coding force-pushed the wen/add_autobahn_evm_hardhat_tests branch 2 times, most recently from f3976e9 to 6accbac Compare May 20, 2026 00:22
@wen-coding wen-coding marked this pull request as draft May 20, 2026 00:46
@wen-coding
Copy link
Copy Markdown
Contributor Author

This should succeed after PR#3452 lands.

Mirrors the existing "EVM Module" CI job with AUTOBAHN=true plus the
GIGA_EXECUTOR/GIGA_STORAGE/GIGA_OCC flag set used by the "Autobahn
EVM GIGA Module" job. Same 4 steps as the CometBFT job:

  - evm_tests.sh                          (EVMCompatabilityTest,
                                           EVMPrecompileTest,
                                           SeiEndpointsTest,
                                           AssociateTest)
  - deploy_flatkv_evm_fixture.sh          (seeds historical EVM
                                           state: balance, storage,
                                           code)
  - flatkv_evm_test.yaml                  (historical EVM RPC reads
                                           via cast: balance, storage,
                                           code at past block heights)
  - verify_flatkv_evm_store.sh            (seidb dump-flatkv smoke
                                           check on the on-disk
                                           storage bucket)

Complements the narrower "Autobahn EVM GIGA Module" (smoke-tests
EVMGigaTest.js on native+ERC20 transfers only) by adding broader
EVM-compat + Sei-precompile + sei-endpoint + association coverage
plus historical-state correctness, none of which are exercised
elsewhere under Autobahn.

Verified locally on a 4-node Autobahn docker cluster:
  evm_tests.sh:                  101 passing, 1 pending, 0 failing
  deploy_flatkv_evm_fixture.sh:  fixture seeded at heights 17567/17571
  (flatkv yaml + verify steps deferred to CI; the 3 flatkv components
  have zero -b block usages and use only EVM RPC + seid queries, so
  they're Autobahn-clean.)

Autobahn EVM Interoperability deferred until #3406 (lib.js
response-reading helper migration) lands; without it, the
suite's CW-deploy setup hooks depend on unmigrated -b block
helpers and CI passing would be accidental rather than verified.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@wen-coding wen-coding force-pushed the wen/add_autobahn_evm_hardhat_tests branch from 6accbac to a9bb50e Compare May 20, 2026 00:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant