kalluripradeep commented on code in PR #61019:
URL: https://github.com/apache/airflow/pull/61019#discussion_r3665538185
##########
airflow-core/tests/unit/dag_processing/test_manager.py:
##########
@@ -1040,6 +1041,38 @@ def
test_deactivate_stale_dags_marks_dags_in_inactive_bundles(self, session):
)
assert is_stale_by_dag == {"dag_in_inactive_bundle": True,
"dag_in_active_bundle": False}
+ def test_deactivate_stale_dags_marks_dags_with_null_bundle_name(self):
Review Comment:
Reworked in `9425646` — it now uses the `session` fixture and patches
`session.execute`, matching `test_deactivate_stale_dags_handles_lock_timeout`
right below it. No `create_engine`, no `__table__` mocking.
Worth flagging why it cannot simply insert the row: migration 0082 made
`dag.bundle_name` NOT NULL and backfills existing NULLs, so a NULL row raises
`IntegrityError` on every backend — the branch only fires for a database still
on the 3.0.x schema.
--
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]
