kaxil commented on issue #35442: URL: https://github.com/apache/airflow/issues/35442#issuecomment-5636749567
Closing this as obsolete. The checklist in the description predates the provider restructure. All 182 paths in it use the old `tests/providers/...` layout, which no longer exists, so neither the checked nor the unchecked entries map onto files in the repo today. The gap itself is not closed: `OVERLOOKED_TESTS` currently carries 267 entries, more than this list ever tracked. That in-code list is the live source of truth and `test_providers_modules_should_have_tests` enforces it, so it cannot go stale the way this description did: https://github.com/apache/airflow/blob/d076a57dcb24303c69d8505ea4816417003ffb34/airflow-core/tests/unit/always/test_project_structure.py#L57-L63 Anyone who wants to fill one of these in can read the current list there. Open PRs already doing so are not affected by this closure. One request for anyone picking these up: group them. A module at a time, claimed in a comment and opened as its own PR, means one review and one CI run per file, and they are all the same review. Batch a provider's worth of missing test modules into a single PR instead. -- 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]
