sjyangkevin commented on PR #52382: URL: https://github.com/apache/airflow/pull/52382#issuecomment-3016023833
> > > 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. > > It ensures that for every file in providers/, there exists a corresponding test file. 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, which is why they are added to the list and thus not checked for :) Hope that helps? It's really helpful! Appreciate it -- 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]
