Skip to content

Lower constant in acceptance test, given recent tighter processing limit. #5394

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.

Sign up for GitHub

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

Merged
sisuresh merged 2 commits into from
Jul 30, 2026
+3 −3
Merged

Lower constant in acceptance test, given recent tighter processing limit. #5394

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
8b8c03e
Lower constant in acceptance test, given recent tighter processing li…
graydon Jul 29, 2026
763a369
Fix comments to match change
graydon Jul 29, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions src/overlay/test/FloodTests.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ TEST_CASE("Flooding", "[flood][overlay][acceptance]")
Hash networkID = sha256(getTestConfig().NETWORK_PASSPHRASE);
Simulation::pointer simulation;

int const nbTx = 100;
int const nbTx = 20;
Comment thread
graydon marked this conversation as resolved.

std::vector<TestAccount> sources;
SequenceNumber expectedSeq = 0;
Expand Down Expand Up @@ -362,8 +362,8 @@ TEST_CASE("Flooding", "[flood][overlay][acceptance]")
messagesUnfulfilled);

// There are four peers.
// Each node starts with 25 txns.
// For every node to get all the 100 txns,
// Each node starts with 5 txns.
// For every node to get all the 20 txns,
// every node has to advertise, demand, and fulfill demands
// at least once.
// No node should do so more than 3 * nbTx times.
Expand Down
Loading