jedcunningham commented on code in PR #47592:
URL: https://github.com/apache/airflow/pull/47592#discussion_r2237566957
##########
airflow-core/tests/unit/api_fastapi/common/db/test_dags.py:
##########
@@ -49,14 +50,14 @@ def setup_teardown(self):
self._clear_db()
@pytest.fixture
- def dag_with_queued_run(self, session):
+ def dag_with_queued_run(self, session, testing_dag_bundle):
"""Returns a DAG with a QUEUED DagRun and null start_date."""
-
dag_id = "dag_with_queued_run"
# Create DagModel
dag_model = DagModel(
dag_id=dag_id,
+ bundle_name="testing",
Review Comment:
That fixture doesnt return the model, but it could. My 2c, we have so much
of this pattern already, keeping consistent and refactoring separately, if we
wanted to, would be the way to go.
--
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: commits-unsubscr...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
