-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Beat [5/4]: fix itests for blockbeat
#9227
base: yy-blockbeat-end
Are you sure you want to change the base?
Beat [5/4]: fix itests for blockbeat
#9227
Conversation
Important Review skippedAuto reviews are limited to specific labels. 🏷️ Labels to auto review (1)
Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
0ab9846
to
a5d0e99
Compare
dc84315
to
e543c74
Compare
6819393
to
ac8ce82
Compare
e543c74
to
8bc5dbc
Compare
d2b0ffa
to
0ddd693
Compare
8bc5dbc
to
339941c
Compare
96474de
to
c7747a2
Compare
339941c
to
e202c05
Compare
a4f226a
to
9f740cc
Compare
0502f30
to
75d1510
Compare
Starting from this commit, we begin the process of flattening the multi-hop itests to make them easier to be maintained. The tests are refactored into their own test cases, with each test focusing on testing one channel type. This is necessary to save effort for future development. These tests are also updated to reflect the new `blockbeat` behavior.
This commit simplfies the test since we only test the preimage extraction logic in the htlc timeout resolver, there's no need to test it for all different channel types as the resolver is made to be oblivious about them.
Also remove unused code.
Also flatten the tests to make them easier to be maintained.
The method `AssertActiveHtlcs` is now removed due to it's easy to be misused. To assert a given htlc, use `AssertOutgoingHTLCActive` and `AssertIncomingHTLCActive` instead for ensuring the HTLC exists in the right direction. Although often the case `AssertNumActiveHtlcs` would be enough as it implicitly checks the forwarding behavior for an intermediate node by asserting there are always num_payment*2 HTLCs.
0339e40
to
03cb629
Compare
75d1510
to
eabf0a5
Compare
This PR is meant to fix all the itests that are failing due to the new behaviors introduced from
blockbeat
. There are still other non-blockbeat-related flakes, which are fixed in the final PR.Major changes in this PR,