Skip to content

fix: cover gravity hardfork integration test in CI - #374

Merged
ByteYue merged 3 commits into
mainfrom
codex/fix-hardfork-test-ci
Jul 5, 2026
Merged

fix: cover gravity hardfork integration test in CI#374
ByteYue merged 3 commits into
mainfrom
codex/fix-hardfork-test-ci

Conversation

@ByteYue

@ByteYue ByteYue commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add reth-ethereum-forks as a direct dev-dependency for reth-pipe-exec-layer-ext-v2 so gravity_hardfork_test can compile under the crate's integration test build.
  • Include gravity_hardfork_test in the focused gravity pipe integration workflow.
  • Refresh the workflow comment now that the hardfork test is intentionally covered.

Fixes Galxe/gravity-audit#673.

Validation

  • cargo check -p reth-pipe-exec-layer-ext-v2 --tests
  • cargo check --locked -p reth-pipe-exec-layer-ext-v2 --tests

Both checks pass. Existing warnings remain in reth-pipe-exec-layer-ext-v2 tests/lib code.

@ByteYue ByteYue changed the title [codex] Fix hardfork test CI coverage fix: cover gravity hardfork integration test in CI Jul 2, 2026
@ByteYue
ByteYue marked this pull request as ready for review July 2, 2026 08:13
ByteYue added 2 commits July 5, 2026 00:01
…st-ci

# Conflicts:
#	.github/workflows/integration.yml
…st-ci

# Conflicts:
#	.github/workflows/integration.yml
@ByteYue
ByteYue merged commit a379a86 into main Jul 5, 2026
30 checks passed
nekomoto911 added a commit to nekomoto911/gravity-reth that referenced this pull request Jul 10, 2026
…onto v2.3.0 API base

Merge Galxe/gravity-reth main (a379a86) into the v2.3.0 migration branch:
9 conflicted files / 18 hunks resolved, 3 auto-merged files needed revm-40
semantic fixes (ResultGas, AccountInfo::account_id, EvmOverrides signature),
system_caller_migration.rs ported to the new Account/AccountInfo API.

Upstream commits: Galxe#367 gas-exempt system txs + zero SYSTEM_CALLER at Alpha,
Galxe#370 unconditional BLS pop-verify registration, Galxe#371 randomness equivalence
tests, Galxe#374 hardfork test CI, Galxe#376 gravity-api-types bump, Galxe#377
debug_traceTransaction target-tx gas-exempt gap.

Resolution ledger + shutdown-race investigation:
docs/merge-v2.3.0/upstream-sync-2026-07-09.md

Verified: cargo check --workspace --all-features = 0 err; fmt clean;
gravity_pipe_test smoke 5/5 green (one flaky pre-existing shutdown race
observed once, root-caused as reth-tasks Arc-owned-runtime teardown race,
documented, not merge-introduced).
nekomoto911 added a commit that referenced this pull request Jul 10, 2026
… API base

Linear replay of the Galxe main sync (a379a86): 9 conflicted files /
18 hunks resolved, 3 auto-merged files needed revm-40 semantic fixes
(ResultGas, AccountInfo::account_id, EvmOverrides signature),
system_caller_migration.rs ported to the new Account/AccountInfo API.

Upstream commits: #367 gas-exempt system txs + zero SYSTEM_CALLER at Alpha,
#370 unconditional BLS pop-verify registration, #371 randomness equivalence
tests, #374 hardfork test CI, #376 gravity-api-types bump, #377
debug_traceTransaction target-tx gas-exempt gap.

Resolution ledger + shutdown-race investigation:
docs/merge-v2.3.0/upstream-sync-2026-07-09.md

Verified: cargo check --workspace --all-features = 0 err; fmt clean;
gravity_pipe_test smoke 5/5 green.
nekomoto911 added a commit that referenced this pull request Jul 20, 2026
Advances the merge-base so PR #396 no longer conflicts with main. Brings in the
4 commits that landed after the 2026-07-09 sync: #383 (randomness from canonical
headers), #385 (EIP-7702 cross-account nonce halt + recoverable-error panic
hardening), #382 (mint precompile gas guard), #390 (withdrawal-invariant docs).

The 11 conflicts are all v2.3.0/revm40 API-form divergence against the
already-adapted #367-#374 batch; resolved take-ours (revm40 accessor methods,
PrecompileHalt, AccountInfo::account_id, no StateCacheDbRefMutWrapper). #382's
gas guard re-adapted from revm36 PrecompileError to revm40 PrecompileHalt.

Verified: cargo check --workspace --all-features = 0 err; fmt clean; pipe-exec
and gravity-storage --tests clean.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants