-
Notifications
You must be signed in to change notification settings - Fork 2
orchestration: add edge-case tests for DagScheduler concurrency-deferral #1516
Copy link
Copy link
Closed
Labels
enhancementNew feature or requestNew feature or request
Description
Context
PR #1514 added 2 unit tests for the concurrency-limit deferral path in DagScheduler::record_spawn_failure(). The tester identified 3 missing edge cases worth covering:
Missing tests
- Multi-task graph: 2 tasks, one Running, one hits concurrency limit — verify the Running task is unaffected and the deferred task reverts to Ready
- max_concurrent=0: every spawn is rejected — verify the scheduler does not enter a Spawn → failure → Spawn infinite loop across ticks
- All tasks deferred: empty running set + all tasks deferred to Ready — verify no premature
Done(Failed)is emitted and the graph staysRunning
Location
crates/zeph-core/src/orchestration/scheduler.rs, mod tests
Related
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request