Skip to content
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

Modify test to show that update_witnesses may disrupt LN channel operations #29

Merged
merged 1 commit into from
Jan 13, 2025

Conversation

St333p
Copy link
Contributor

@St333p St333p commented Jan 13, 2025

This PR parametrizes the ln_transfers test to show that calls to update_witnesses may disrupt LN channel operations if they happen during a channel update. Details in RGB-WG/rgb-std#292.

Copy link

codecov bot commented Jan 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 36.92%. Comparing base (1163628) to head (d6c1603).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #29      +/-   ##
==========================================
- Coverage   37.26%   36.92%   -0.34%     
==========================================
  Files         279      279              
  Lines       42807    42804       -3     
==========================================
- Hits        15950    15805     -145     
- Misses      26857    26999     +142     
Flag Coverage Δ
rust 36.92% <ø> (-0.34%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@St333p St333p force-pushed the ln_update_witnesses branch from 71c257a to d6c1603 Compare January 13, 2025 14:23
@zoedberg zoedberg merged commit d6c1603 into RGB-WG:master Jan 13, 2025
12 checks passed
Sign up for free to 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.

2 participants