zach-overflow commented on code in PR #63353:
URL: https://github.com/apache/airflow/pull/63353#discussion_r2919432494
##########
airflow-core/tests/unit/jobs/test_triggerer_job.py:
##########
@@ -1129,6 +1137,8 @@ async def
test_trigger_can_fetch_dag_run_count_ti_count_in_deferrable(session, d
}
[email protected](reason="Obsolete method with API refactor")
[email protected](reason="Obsolete method with API refactor")
Review Comment:
I'm not sure all of these tests with skip decorators are actually obsolete.
They may need to be refactored to remain compatible with the proposed changes,
but several of the tests this PR skips are important edge cases for how trigger
assignment, trigger failures, and trigger capacity are handled.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
