Skip to content
This repository has been archived by the owner on Jul 5, 2024. It is now read-only.

Refactor: merge test_ctx and test_ctx2 into one #1651

Open
KimiWu123 opened this issue Oct 11, 2023 · 0 comments
Open

Refactor: merge test_ctx and test_ctx2 into one #1651

KimiWu123 opened this issue Oct 11, 2023 · 0 comments
Labels
T-feature Type: new features

Comments

@KimiWu123
Copy link
Contributor

KimiWu123 commented Oct 11, 2023

Describe the feature you would like

To support Beacon chain withdrawals, we added TestContext2 to support 3 generic inputs (number of accounts, number of txs and number of withdrawals). However, it's quite duplicated so we should find a better way to merge them into one. For more details, see discussion here: #1634 (comment)

Additional context

No response

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-feature Type: new features
Projects
Status: No status
Development

No branches or pull requests

1 participant