keith991001 opened a new pull request, #73229: URL: https://github.com/apache/airflow/pull/73229
Two of the four `standard` provider entries in `OVERLOOKED_TESTS` turned out to be misfiled rather than missing: `operators/branch.py` and `operators/latest_only.py` are both well tested, but the test files carry an `_operator` suffix (`test_branch_operator.py` / `test_latest_only_operator.py`) that the source layout does not mirror, so the guard reports them as untested — and the stale-entry check never fires (#71985 fixes that mechanism). - Rename both files to the mirroring paths (`git mv`, history preserved) and drop the two exemptions - Close the two gaps the audit surfaced in the branch suite: `BaseBranchOperator.choose_branch` raising `NotImplementedError`, and a branch operator returning `None` skipping every direct downstream task (covered for both the `DownstreamTasksSkipped` path on Airflow 3.0.1+ and the legacy state-assertion path) The other two `standard` entries are intentionally left: `test_empty.py` is claimed by #72603, and `sensors/external_task.py` (~720 lines) deserves its own PR. 13 tests pass locally across both renamed modules, as do the `test_providers_modules_should_have_tests` guard and mypy. --- ##### Was generative AI tooling used to co-author this PR? - [X] Yes — Claude Code (Fable 5) Generated-by: Claude Code (Fable 5) following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions) -- 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]
