Skip to content

Conversation

dajneem23
Copy link

This PR starts addressing #4093 by consolidating duplicate test utilities

Description

  • deduplicate produceBlockWith4844Tx -> move to helpers.ts with extra params.

This is still a work in progress

@dajneem23 dajneem23 changed the title client: deduplicate produceBlockWith4844Tx client: duplicate test utilities Sep 4, 2025
@dajneem23 dajneem23 marked this pull request as draft September 4, 2025 02:58
@dajneem23 dajneem23 marked this pull request as ready for review September 4, 2025 02:58
@dajneem23
Copy link
Author

dajneem23 commented Sep 4, 2025

Hi, I've started my first PR for EthereumJS.
Kindly help me review and comment if any changes are needed

Copy link

codecov bot commented Sep 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.31%. Comparing base (aa039fd) to head (fa666b9).

Additional details and impacted files

Impacted file tree graph

Flag Coverage Δ
block 84.33% <ø> (ø)
blockchain 89.32% <ø> (ø)
common 97.50% <ø> (ø)
evm 72.77% <ø> (ø)
mpt 90.00% <ø> (+0.56%) ⬆️
statemanager 69.06% <ø> (ø)
static 99.77% <ø> (ø)
tx 89.83% <ø> (?)
util 88.70% <ø> (ø)
vm 55.31% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 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.

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.

2 participants