Skip to content

test(evm): split archived StakingRewards suite#1033

Open
branarakic wants to merge 1 commit into
mainfrom
codex/split-staking-rewards-test
Open

test(evm): split archived StakingRewards suite#1033
branarakic wants to merge 1 commit into
mainfrom
codex/split-staking-rewards-test

Conversation

@branarakic

Copy link
Copy Markdown
Contributor

Summary

  • split the 4k-line archived StakingRewards test into 11 skipped suite part files
  • move the shared V8 fixture/helpers into StakingRewards.shared.ts
  • keep each generated .test.ts file below 500 lines

Notes

  • StakingRewards.shared.ts remains larger because it preserves the original archived fixture as shared setup; further splitting that fixture would be a separate refactor.

Verification

  • compared ordered skipped test titles between the original file and split files
  • ./node_modules/.bin/hardhat test --no-compile --config hardhat.node.config.ts test/archive/integration/StakingRewards.part-*.test.ts

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Codex review skipped: filtered diff is 8991 lines (cap: 5,000). Please consider splitting this into smaller PRs for reviewability.

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.

1 participant