jscheffl commented on code in PR #57320: URL: https://github.com/apache/airflow/pull/57320#discussion_r2484573724
########## airflow-core/tests/unit/dag_processing/test_dagbag.py: ########## @@ -41,6 +42,7 @@ from airflow.models.dag import DagModel from airflow.models.dagwarning import DagWarning, DagWarningType from airflow.models.serialized_dag import SerializedDagModel +from airflow.providers.standard import example_dags as standard_example_dags Review Comment: Similar as above but a bit pmore problematic - if we make this a global import the python module loader needs to have the dependency available at moment of collecting tests already. Can you make this a local import of a small utility function in the test or better to be a fixture that loads it at test runtime to prevent a file level import? (Alongside with code in line 56 and then use it in line 216 and 482 etc) -- 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]
