Skip to content

Commit 1c2475c

Browse files
committed
f clarify test change comment
1 parent 496d497 commit 1c2475c

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

lightning/src/ln/functional_tests.rs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9222,8 +9222,10 @@ fn test_duplicate_chan_id() {
92229222

92239223
let funding_created = {
92249224
let mut a_channel_lock = nodes[0].node.channel_state.lock().unwrap();
9225-
// Note that we drop the channel entirely here as its in a somewhat nonsense state (leaving
9226-
// the channelmanager in a possibly nonsense state instead).
9225+
// Once we call `get_outbound_funding_created` the channel has a duplicate channel_id as
9226+
// another channel in the ChannelManager - an invalid state. Thus, we'd panic later when we
9227+
// try to create another channel. Instead, we drop the channel entirely here (leaving the
9228+
// channelmanager in a possibly nonsense state instead).
92279229
let mut as_chan = a_channel_lock.by_id.remove(&open_chan_2_msg.temporary_channel_id).unwrap();
92289230
let logger = test_utils::TestLogger::new();
92299231
as_chan.get_outbound_funding_created(tx.clone(), funding_outpoint, &&logger).unwrap()

0 commit comments

Comments
 (0)