erigontech / erigontech/erigon

Remove ExecV3: Extend BAL test with varied transaction types

Open
#19,632 2 comments 0 reactions 0 assignees View on GitHub
Glamsterdam tech debt reduction
Dominant language
Go
Stars
3.6k
Forks
1.5k
Avg merge
1d 16h
Merged PRs (30d)
455

Description

## Sub-task of #19318

### Phase 1d: BAL test extension

**File**: `execution/engineapi/engine_api_bal_test.go`

Extend `TestEngineApiGeneratedPayloadIncludesBlockAccessList` with additional transaction types to ensure the refactoring doesn't break BAL tracking:

- Contract creation (code changes tracked in BAL)
- Storage writes via contract calls (storage changes tracked)
- Multiple transaction types in same block

The existing test only covers a simple transfer. Varying the transaction types ensures all BAL tracking paths (balance, nonce, code, storage) produce correct results after the refactoring.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.