Skip to content

Commit ee06a90

Browse files
committed
Update comments in full_stack_target test for reduced breakage
The previous two changes should materially reduce how finicky the `full_stack_target` tests are, which we reflect in the comments here.
1 parent dadac03 commit ee06a90

File tree

1 file changed

+7
-8
lines changed

1 file changed

+7
-8
lines changed

fuzz/src/full_stack.rs

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1626,10 +1626,10 @@ mod tests {
16261626
#[test]
16271627
fn test_no_existing_test_breakage() {
16281628
// To avoid accidentally causing all existing fuzz test cases to be useless by making minor
1629-
// changes (such as requesting feerate info in a new place), we run a pretty full
1630-
// step-through with two peers and HTLC forwarding here. Obviously this is pretty finicky,
1631-
// so this should be updated pretty liberally, but at least we'll know when changes occur.
1632-
// If nothing else, this test serves as a pretty great initial full_stack_target seed.
1629+
// changes, we run a pretty full step-through with two peers and HTLC forwarding here.
1630+
// Obviously this can be somewhat finicky, so this should be updated pretty liberally, but
1631+
// at least we'll know when changes occur. If nothing else, this test serves as a pretty
1632+
// great initial full_stack_target seed.
16331633

16341634
let test = super::two_peer_forwarding_seed();
16351635

@@ -1662,10 +1662,9 @@ mod tests {
16621662
#[test]
16631663
fn test_gossip_exchange_breakage() {
16641664
// To avoid accidentally causing all existing fuzz test cases to be useless by making minor
1665-
// changes (such as requesting feerate info in a new place), we exchange some gossip
1666-
// messages. Obviously this is pretty finicky, so this should be updated pretty liberally,
1667-
// but at least we'll know when changes occur.
1668-
// This test serves as a pretty good full_stack_target seed.
1665+
// changes, we exchange some gossip messages. Obviously this is somewhat finicky, so this
1666+
// should be updated pretty liberally, but at least we'll know when changes occur.
1667+
// This test serves as a helpful additional full_stack_target seed.
16691668

16701669
let test = super::gossip_exchange_seed();
16711670

0 commit comments

Comments
 (0)