erigontech / erigontech/erigon
Remove ExecV3: Extend BAL test with varied transaction types
Open
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
Assessment
This issue has not been assessed yet.