Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cmd/devp2p/internal/ethtest: add more tx propagation tests #22630

Merged
merged 2 commits into from
Apr 23, 2021

Conversation

renaynay
Copy link
Contributor

@renaynay renaynay commented Apr 7, 2021

This PR adds a large transaction request test to the eth test suite as well as a new pooled tx hashes announcement test.

It depends on #22698.

@renaynay renaynay force-pushed the add-largeTxRequest-ethtest branch 5 times, most recently from 67ec51c to 8596bc3 Compare April 21, 2021 12:51
@renaynay renaynay changed the title [DRAFT] cmd/devp2p: add large tx request test to eth test suite cmd/devp2p: add more tx tests to eth test suite Apr 21, 2021
@renaynay renaynay requested a review from fjl April 21, 2021 12:57
@renaynay renaynay marked this pull request as ready for review April 21, 2021 12:57
@renaynay renaynay changed the title cmd/devp2p: add more tx tests to eth test suite [ WIP ] cmd/devp2p: add more tx tests to eth test suite Apr 21, 2021
@fjl fjl changed the title [ WIP ] cmd/devp2p: add more tx tests to eth test suite cmd/devp2p/internal/ethtest: add more tx propagation tests Apr 23, 2021
@fjl fjl merged commit cac1b21 into ethereum:master Apr 23, 2021
@fjl fjl added this to the 1.10.3 milestone Apr 23, 2021
atif-konasl pushed a commit to frozeman/pandora-execution-engine that referenced this pull request Oct 15, 2021
…22630)

This adds a test for large tx announcement messages, as well as a test to
check that announced tx hashes are requested by the node.
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.

2 participants