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
itest: fix wrong node queried for balance #5348
itest: fix wrong node queried for balance #5348
Conversation
The itest fails, with the following line, |
17e0f05
to
efc0af3
Compare
Turns out that was the miner fee. |
efc0af3
to
46b0cd8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice find and fix! LGTM 💯
Very weird this ever worked. I assume because Bob just always had a high balance that was larger than craolExpectedBalance
?
46b0cd8
to
140a35e
Compare
Yep I think so. When we set up the test we |
140a35e
to
a60db53
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, very nice! 🥇
Was looking at the channel force close test and found the wrong node was used to query the balance. (weird as it should have been caught by the test?)
testChannelForceClosure
has only two relevant participants,Alice
(notnet.Alice
) andCarol
, seems likenet.Bob
doesn't belong here.