SamWheating commented on a change in pull request #14306:
URL: https://github.com/apache/airflow/pull/14306#discussion_r579304547
##########
File path: tests/api_connexion/endpoints/test_dag_endpoint.py
##########
@@ -274,6 +279,7 @@ def test_should_response_200_for_null_start_date(self):
"fileloc": __file__,
"file_token": FILE_TOKEN,
"is_paused": None,
+ "is_active": None,
Review comment:
I think that in this case the DAG object hasn't been sync'd to the DB,
which is why it returns None (the same happens with `is_paused` here, which
should also never be None).
From my understanding it should never be None in the real world, but in the
current testing set up this makes sense?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
