LBRYCommunity/lbrycrd
1
0
Fork 0
Code Issues 29 Pull requests 1 Projects Releases 32 Packages Wiki Activity

Merge #14672: tests: Send fewer spam messages in p2p_invalid_messages

Browse source 
3d305e3b89 Send fewer spam messages in p2p_invalid_messages (James O'Beirne)

Pull request description:

  Builds on travis are failing because the test node isn't
  able to drop all the bad messages sent within the given
  timeout. Reduce the number of bad messages we're sending
  and increase the timeout to avoid failures on travis.

Tree-SHA512: 11c389619d9590caf7eca74e0efe6d950469415d34220072770689024b350cc08a2d5ec90634237d87ff71ba8b638c1152b8a45ffbb2815a48bde6a88fbb8fc6
This commit is contained in:
MarcoFalke 2018-11-06 15:14:38 -05:00
commit 976583e109
No known key found for this signature in database
GPG key ID: D2EA4850E7528B25
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
test/functional/p2p_invalid_messages.py
View file

@ -72,7 +72,7 @@ class InvalidMessagesTest(BitcoinTestFramework):
"memory exhaustion. May take a bit...") "memory exhaustion. May take a bit...")
# Run a bunch of times to test for memory exhaustion. # Run a bunch of times to test for memory exhaustion.
for _ in range(200): for _ in range(80):
node.p2p.send_message(msg_at_size) node.p2p.send_message(msg_at_size)
# Check that, even though the node is being hammered by nonsense from one # Check that, even though the node is being hammered by nonsense from one
@ -82,7 +82,7 @@ class InvalidMessagesTest(BitcoinTestFramework):
# Peer 1, despite serving up a bunch of nonsense, should still be connected. # Peer 1, despite serving up a bunch of nonsense, should still be connected.
self.log.info("Waiting for node to drop junk messages.") self.log.info("Waiting for node to drop junk messages.")
node.p2p.sync_with_ping(timeout=8) node.p2p.sync_with_ping(timeout=30)
assert node.p2p.is_connected assert node.p2p.is_connected
# #