Skip to content

(Very) marginally expand test_monitor_update_fail_claim - #3126

Merged
G8XSU merged 1 commit into
lightningdevkit:mainfrom
TheBlueMatt:2024-06-preimage-removal-blocked-by-claim-event
Jun 18, 2024
Merged

(Very) marginally expand test_monitor_update_fail_claim#3126
G8XSU merged 1 commit into
lightningdevkit:mainfrom
TheBlueMatt:2024-06-preimage-removal-blocked-by-claim-event

Conversation

@TheBlueMatt

@TheBlueMattTheBlueMatt commented Jun 13, 2024

Copy link
Copy Markdown
Collaborator

For some reason I wasn't sure this happened yet, so making the test for no-PaymentClaimed-event-before-preimage-on-disk explicit is nice.

@TheBlueMattTheBlueMatt mentioned this pull request Jun 13, 2024
24 tasks
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.90%. Comparing base (07f3380) to head (130464e).

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@ Coverage Diff @@## main #3126 +/- ##
==========================================
- Coverage 89.90% 89.90% -0.01% 
==========================================
Files 121 121 Lines 99170 99175 +5 Branches 99170 99175 +5 ==========================================
+ Hits 89159 89162 +3 + Misses 7408 7405 -3 - Partials 2603 2608 +5 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Comment threadlightning/src/ln/chanmon_update_fail_tests.rs
G8XSU
G8XSU previously approved these changes Jun 18, 2024
@TheBlueMatt
TheBlueMattforce-pushed the 2024-06-preimage-removal-blocked-by-claim-event branch from 130464e to b1cf58eCompareJune 18, 2024 13:58
For some reason I wasn't sure this happened yet, so making the
test for no-`PaymentClaimed`-event-before-preimage-on-disk explicit
is nice.
@TheBlueMatt
TheBlueMattforce-pushed the 2024-06-preimage-removal-blocked-by-claim-event branch from b1cf58e to 68af6d0CompareJune 18, 2024 17:59
@G8XSU
G8XSU merged commit 87fc324 into lightningdevkit:mainJun 18, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@TheBlueMatt@codecov-commenter@G8XSU