Skip to content

test(pipeline): add stage timeout tests (#1423)#1554

Merged
OneStepAt4time merged 1 commit intodevelopfrom
fix/pipeline-stage-timeout-1423
Apr 9, 2026
Merged

test(pipeline): add stage timeout tests (#1423)#1554
OneStepAt4time merged 1 commit intodevelopfrom
fix/pipeline-stage-timeout-1423

Conversation

@OneStepAt4time
Copy link
Copy Markdown
Owner

Summary

  • Add 4 unit tests for the stageTimeoutMs feature in PipelineManager
  • The implementation (stageTimeoutMs on PipelineStage, timeout check in pollPipelines(), transition to failed with reason stage_timeout) was already present in src/pipeline.ts
  • Tests cover: timeout exceeded, not yet elapsed, no config set, and completion before timeout

Closes #1423

Aegis version

Developed with: v0.3.2-alpha

Test plan

  • npx tsc --noEmit passes
  • npm run build passes
  • npm test — all 322 tests pass (4 new)

Generated by Hephaestus (Aegis dev agent)

Covers timeout exceeded, not yet elapsed, no config set, and
completion before timeout scenarios for the stageTimeoutMs feature.

Generated by Hephaestus (Aegis dev agent)
@OneStepAt4time
Copy link
Copy Markdown
Owner Author

🔧 PR #1554 ready for review: fix(pipeline): add stage timeout support (E4-2) (#1423). CI CLEAN. Please review.

@OneStepAt4time OneStepAt4time merged commit 1156818 into develop Apr 9, 2026
9 checks passed
@OneStepAt4time OneStepAt4time deleted the fix/pipeline-stage-timeout-1423 branch April 9, 2026 19:11
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.

1 participant