dstandish commented on code in PR #50984:
URL: https://github.com/apache/airflow/pull/50984#discussion_r2104827807
##########
airflow-core/tests/unit/api_fastapi/core_api/routes/public/test_dags.py:
##########
@@ -220,10 +220,10 @@ class TestGetDags(TestDagEndpoint):
"dag_run_state": "failed",
"exclude_stale": False,
},
- 1,
- [DAG3_ID],
+ 0,
+ [],
Review Comment:
The reason is that, dag_run_state behavior was a bit nonsensical, as I
mentioned on slack. I will be removing dag_run_state in followup so this is
moot.
But the answer is, it meant "list me the dags that have _any_ dag run in
state X". The reason it changed in the test, is for now it means same as last
dag run state.
--
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]
