dominikhei commented on PR #52382: URL: https://github.com/apache/airflow/pull/52382#issuecomment-3016023046
> > You need to add `test_version_compat.py` to the OVERLOOKED_TESTS in `airflow-core/tests/unit/alway/test_project_structure.py`. > > Yes. I checked the latest instruction early today, and working on making the changes. Thanks for the feedback! I also noticed the file is not actually present in the test folder. I would appreciate if I could get your help to understand more why we need the `OVERLOOKED_TESTS` list for this change. This test ensures that for every Python module under airflow/providers/, there exists a corresponding unit test file under tests/unit/ For example, if you create a new provider xyz and add an XYZOperator, the test checks that a matching file like test_xyz_operator.py exists and that the operator's functionality is covered. However some files inside the providers folder are "support" files, not directly part of the provider logic, and thus don't necessarily require unit tests, just like version_compat.py :) Hope that helps? -- 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]
