keith991001 opened a new pull request, #73153: URL: https://github.com/apache/airflow/pull/73153
Provider batch filling the six remaining unclaimed `OVERLOOKED_TESTS` entries for `cncf.kubernetes` (grouped per provider as the maintainers requested when closing #35442; the entries claimed by #72717 and #72461 are left untouched). **What each entry needed turned out to differ:** - `test_k8s_model.py` — **new**: `K8SModel` abstractness, `append_to_pod` no-op/ordering semantics - `test_kube_config.py` — **new**: `KubeConfig` parsing (`kube_image` composition and `None` fallback, fatal-container-state-reason splitting, multi-namespace list, `_request_timeout` list→tuple conversion, non-dict JSON rejection for `kube_client_request_args` / `delete_option_kwargs`) - `executors/test_kubernetes_executor_types.py` — **new**: stability of pod annotation/label keys that are persisted on live pods, `KubernetesJob.kube_image` default, `FailureDetails` total=False contract - `test_secret.py` — **moved + extended**: tests existed under `models/test_secret.py`, a path the source layout does not mirror, so the guard reported the module untested. Moved to the mirroring path and added the uncovered surface: constructor validation, env-target uppercasing, `attach_to_pod` env/env-from paths and deep-copy semantics, equality/hash/repr - `test_kube_client.py` — **stale exemption + extended**: the file already exists on `main` but its exemption was never removed (the stale-entry check compares `Path` objects against strings and never fires — #71985 fixes the mechanism). Extended it with the untested client-side request-timeout logic (`_get_request_timeout` boundaries, `_TimeoutK8sApiClient` timeout injection and `disable_verify_ssl`) - `utils/test_k8s_hashlib_wrapper.py` — **stale exemption removed**: the source module `k8s_hashlib_wrapper.py` no longer exists 48 tests in the touched modules pass locally, as do the `test_providers_modules_should_have_tests` guard and mypy on the five test files. --- ##### 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]
