Skip to content

test(module44): add tests for open-ended streams and runway top-up (closes #610) - #689

Open
Ranjeet2063 wants to merge 1 commit into
conduit-protocol:mainfrom
Ranjeet2063:test/issue-610-open-ended-stream-coverage
Open

test(module44): add tests for open-ended streams and runway top-up (closes #610)#689
Ranjeet2063 wants to merge 1 commit into
conduit-protocol:mainfrom
Ranjeet2063:test/issue-610-open-ended-stream-coverage

Conversation

@Ranjeet2063

Copy link
Copy Markdown

Summary

Closes #610.

This pull request provides comprehensive test coverage for open-ended streams (endTime === 0) across the Module44 runway calculator and assessment suite.

Changes Made

  • Open-Ended Risk Assessment Coverage:
    • Added explicit tests for active open-ended streams asserting runwaySecs === null and riskLevel === 'healthy'.
    • Tested open-ended streams with zero rate, paused state, and cancelled state asserting proper fallback to runwaySecs === 0 and riskLevel === 'inactive'.
    • Asserted cache retrieval preserves null runway and healthy risk classification without corruption.
  • Top-Up Runway Estimation Coverage:
    • Added comprehensive assertions for estimateTopUpNeeded on active open-ended streams across varied short, medium, and long target runway thresholds (10s to 1,000,000,000s) returning 0n.
    • Verified 0n return for paused, cancelled, zero-rate, and far-future timestamped open-ended streams.
  • Batch Processing Parity:
    • Added mixed batch test in assessBatch verifying concurrent handling of open-ended, bounded critical, and paused streams.

Verification

  • All 34/34 tests in src/tests/module44.test.ts pass cleanly.
  • Full project build passes without errors (npm run build).

Copilot AI lite review requested due to automatic review settings September 6, 2026 00:46

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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.

Module44 runway calculator: tests for open-ended streams

2 participants