The GitHub Actions job "Tests (AMD)" on 
airflow.git/fix/airflow-fail-in-retry-callback has failed.
Run started by GitHub user 1fanwang (triggered by 1fanwang).

Head commit for run:
f59182e05c1c2d1a7e16f1bedb189374aff66f23 / 1fanwang <[email protected]>
Address review: drop redundant test docstring, assert callback state transitions

- Drop the docstring on 
`test_airflow_fail_exception_in_on_retry_callback_fails_task`;
  the test name already describes the assertion, and the project style guide
  prefers no docstrings in tests that merely repeat the function name.
- Assert the recorded callback states directly
  (`retry_callback_calls == [UP_FOR_RETRY]`, `failure_callback_calls == 
[FAILED]`)
  instead of the call counts. Pins both the count AND the state at the moment
  each callback ran -- proving on_retry_callback fired while the task was
  UP_FOR_RETRY and on_failure_callback fired after AirflowFailException
  promoted it to FAILED.

Signed-off-by: 1fanwang <[email protected]>

Report URL: https://github.com/apache/airflow/actions/runs/25785121574

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to