Skip to content

Tests: rpc initial setup and rpc tests phase 1#3520

Open
kollegian wants to merge 14 commits into
mainfrom
tests/ev_rpc_tests
Open

Tests: rpc initial setup and rpc tests phase 1#3520
kollegian wants to merge 14 commits into
mainfrom
tests/ev_rpc_tests

Conversation

@kollegian
Copy link
Copy Markdown

Describe your changes and provide context

Testing performed to validate your change

@cursor
Copy link
Copy Markdown

cursor Bot commented May 29, 2026

PR Summary

Low Risk
Adds integration tests and CI wiring only; no production RPC or node logic changes.

Overview
Adds a self-contained integration_test/rpc_tests module that compares Sei’s EVM JSON-RPC to a local geth --dev reference (happy path, schema, empty/null handling, and byte-for-byte error parity via raw JSON-RPC). A one-time bootstrap deploys shared fixtures (TestERC20, gas burner, EIP-7702 target), mirrors ERC20 on geth, pre-funds a signer pool, and writes runtime/runtime.json for parallel Mocha specs under eth/.

CI: New integration matrix job “EVM RPC Parity (geth reference)” runs scripts/run-ci.sh (GIGA_STORAGE=true); workflow always uploads the merged mochawesome report from reports/merged (ignores if missing).

Phase 1 specs cover core eth_* methods (accounts, block number/hash, chain id, coinbase, call, estimateGas, feeHistory, gasPrice, getBalance, getBlockByNumber/Hash), including Sei-only behavior (precompiles, sei_* helpers) and documented divergences from geth where intentional.

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

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 29, 2026

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

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedJun 3, 2026, 12:44 PM

Comment thread integration_test/rpc_tests/config/endpoints.ts Outdated
Comment thread integration_test/rpc_tests/eth/eth_accounts.spec.ts Outdated
Comment thread integration_test/rpc_tests/eth/eth_call.spec.ts Outdated
@codecov
Copy link
Copy Markdown

codecov Bot commented May 29, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 58.17%. Comparing base (2151ba0) to head (699aa8d).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3520      +/-   ##
==========================================
- Coverage   59.12%   58.17%   -0.96%     
==========================================
  Files        2213     2129      -84     
  Lines      182814   173668    -9146     
==========================================
- Hits       108096   101035    -7061     
+ Misses      64993    63643    -1350     
+ Partials     9725     8990     -735     
Flag Coverage Δ
sei-db 70.41% <ø> (ø)
sei-db-state-db ?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 146 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Comment thread integration_test/rpc_tests/utils/evmUtils.ts
Comment thread integration_test/rpc_tests/utils/testUtils.ts
Comment thread integration_test/rpc_tests/eth/eth_accounts.spec.ts
Comment thread integration_test/rpc_tests/_start/00_bootstrap.spec.ts Outdated
Comment thread integration_test/rpc_tests/eth/eth_gasPrice.spec.ts Outdated
Comment thread integration_test/rpc_tests/_start/00_bootstrap.spec.ts
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit c188432. Configure here.

Comment thread integration_test/rpc_tests/utils/feeHistoryUtils.ts
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