Skip to content

Commit e2e50c5

Browse files
committed
test: tighten flake comment in test_actor_charge_limit
1 parent 09a56d2 commit e2e50c5

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

tests/e2e/test_actor_charge.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -142,9 +142,8 @@ async def test_actor_charge_limit(
142142
) -> None:
143143
run = await run_actor(ppe_actor, max_total_charge_usd=Decimal('0.2'))
144144

145-
# Reaching `max_total_charge_usd` makes the platform abort the run automatically, and that abort races with the
146-
# Actor's own clean exit — so the terminal status is either SUCCEEDED or ABORTED. Both are valid here; the
147-
# behavior under test is that the charge limit capped the run at exactly 2 of the 4 attempted events.
145+
# Reaching `max_total_charge_usd` makes the platform auto-abort the run, racing with the Actor's clean exit, so
146+
# the terminal status is either SUCCEEDED or ABORTED. What matters is that the limit capped it at 2 events.
148147
terminal_statuses = {ActorJobStatus.SUCCEEDED, ActorJobStatus.ABORTED}
149148

150149
# Refetch until the charged event counts propagate on the platform.

0 commit comments

Comments
 (0)