Address review: remove unused global, fix assertion messages

This commit is contained in:
J. Nick Koston
2026-04-06 20:54:01 -10:00
parent c79e7f4018
commit cf704b6b91
2 changed files with 3 additions and 7 deletions
@@ -17,10 +17,6 @@ globals:
- id: slow_count
type: int
initial_value: "0"
# Track ordering: set when medium fires before fast has fired 5 times
- id: order_ok
type: bool
initial_value: "true"
# Track interval that cancels itself
- id: self_cancel_count
type: int
@@ -88,7 +88,7 @@ async def test_scheduler_interval_reschedule(
# Fast runs at 50ms, medium at 100ms, so fast should be ~2x medium
# Allow some slack for scheduling jitter
assert fast_at_medium_5 >= 7, (
f"Fast interval should have fired ~10 times when medium hit 5, "
f"Fast interval should have fired at least 7 times when medium hit 5, "
f"but only fired {fast_at_medium_5} times"
)
@@ -102,11 +102,11 @@ async def test_scheduler_interval_reschedule(
fast_at_slow_3, medium_at_slow_3 = result
# At 600ms: fast ~12, medium ~6, slow 3
assert fast_at_slow_3 >= 8, (
f"Fast should have fired ~12 times when slow hit 3, "
f"Fast should have fired at least 8 times when slow hit 3, "
f"but only fired {fast_at_slow_3}"
)
assert medium_at_slow_3 >= 4, (
f"Medium should have fired ~6 times when slow hit 3, "
f"Medium should have fired at least 4 times when slow hit 3, "
f"but only fired {medium_at_slow_3}"
)