Skip to content

chore: fix test compilation and precision issues for CI#878

Merged
edehvictor merged 3 commits into
edehvictor:mainfrom
ritik4ever:fix-issues-811-812-813-814
Jun 26, 2026
Merged

chore: fix test compilation and precision issues for CI#878
edehvictor merged 3 commits into
edehvictor:mainfrom
ritik4ever:fix-issues-811-812-813-814

Conversation

@ritik4ever

Copy link
Copy Markdown
Contributor

Summary

This PR adds missing regression tests and addresses test coverage gaps across the backend keeper modules to ensure robustness under edge cases.

Changes Included

  • Compound Scheduler: Added catch-up behavior tests to ensure missed intervals after downtime are handled deterministically without duplicate execution.
  • Vault Monitor: Introduced capacity forecasting tests covering rising utilization, sudden inflows, and saturation/safety warnings.
  • Keeper Queues: Added test coverage for poison-message isolation and retry caps to verify that bad jobs stop retrying and healthy jobs continue independently.
  • Redis State: Added Redis reconnect and TTL expiry regression tests to validate transient disconnections and prevent stale keeper state revival.

Issue Links

Closes #811
Closes #812
Closes #813
Closes #814

Verification

  • Local build completes successfully.
  • Added tests verify required behavior.
  • All CI/CD pipelines and required checks are expected to pass.

@ritik4ever ritik4ever requested a review from edehvictor as a code owner June 26, 2026 17:46
@vercel

vercel Bot commented Jun 26, 2026

Copy link
Copy Markdown

@ritik4ever is attempting to deploy a commit to the Edeh Victor's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Jun 26, 2026

Copy link
Copy Markdown

@ritik4ever Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@edehvictor edehvictor left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice contribution!

@edehvictor edehvictor merged commit 1724e05 into edehvictor:main Jun 26, 2026
9 of 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

2 participants