potiuk commented on code in PR #49978:
URL: https://github.com/apache/airflow/pull/49978#discussion_r2100722960
##########
airflow-core/tests/unit/api_fastapi/core_api/routes/public/test_dag_report.py:
##########
@@ -23,6 +23,7 @@
from airflow.utils.file import list_py_file_paths
+import system.standard
Review Comment:
More explanation. This is the way how we - currently - introduce
"namespacing" in our tests - we had discussion about it before.
```
tests.
unit.provider id
system.provider id
integration/ provider id
```
Example:
And the root PYTHONPATH is `tests` for each provider and airflow core.
One of the reasons for that was the problem with importing things like
`smtp` or `imap` or a number of others - in many of those providers, the
"library" used to import things start with the same `smtp` or `imap`. So adding
`tests.smtp` like it was a problem.
We could have more sub-folders.
Maybe we should rethink it. I proposed iniitlllly to have `provider_tests`
as top-level package - that would be more reasonable - but adds more nesting.
--
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]
