Skip to content

fix: filecoin chain notify API and tests - #7493

Merged
akaladarshi merged 5 commits into
mainfrom
akaladarshi/fix-chain-notify-and-tests
Aug 20, 2026
Merged

fix: filecoin chain notify API and tests#7493
akaladarshi merged 5 commits into
mainfrom
akaladarshi/fix-chain-notify-and-tests

test(rpc): unify subscription test timeouts

efe5fa5
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Aug 20, 2026 in 0s

67.13% (+0.33%) compared to 78cc905

View this Pull Request on Codecov

67.13% (+0.33%) compared to 78cc905

Details

Codecov Report

❌ Patch coverage is 97.69231% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 67.13%. Comparing base (78cc905) to head (efe5fa5).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/rpc/channel.rs 96.95% 3 Missing and 2 partials ⚠️
src/rpc/methods/chain.rs 98.23% 2 Missing and 2 partials ⚠️
Additional details and impacted files
Files with missing lines Coverage Δ
src/rpc/methods/chain.rs 60.65% <98.23%> (+6.84%) ⬆️
src/rpc/channel.rs 92.99% <96.95%> (+72.29%) ⬆️

... and 12 files with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 78cc905...efe5fa5. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.