keith991001 opened a new pull request, #73111: URL: https://github.com/apache/airflow/pull/73111
Batch of missing test modules for the `edge3` provider, filling all six `OVERLOOKED_TESTS` entries for the provider in one PR — grouped per provider as requested by the maintainers when closing the old tracking issue #35442 (all further context there). **New test modules (80 tests total across the change):** - `models/test_edge_job.py` — `EdgeJobModel`: `key` → `TaskInstanceKey` mapping, `queued_dttm` default, `last_update_t` fallback, DB round trip, per-try rows - `models/test_edge_logs.py` — `EdgeLogsModel`: constructor field mapping, DB round trip, incremental chunks as separate rows - `worker_api/test_datamodels.py` — `EdgeJobBase.key`, `EdgeJobFetched` command coercion + `identifier`, `WorkerStateBody`/`WorkerQueuesBase` defaults and enum coercion, `PushLogsBody` timestamp parsing, `versions_match` defaulting to `False` on both return models - `worker_api/test_datamodels_ui.py` — `Worker`/`Job` defaults and required fields, collection responses, `ConcurrencyRequest` `gt=0` bound, request-body validation errors - `cli/test_example_extended_sysinfo.py` — measured sysinfo fields, INFO/WARNING/ERROR thresholds for CPU and free disk, status-file override paths **One special case:** `models/test_edge_worker.py` already exists on `main`, but its `OVERLOOKED_TESTS` exemption was never removed — the stale-entry check compares `Path` objects against strings and never fires (#71985 fixes that mechanism). The existing file only covered metrics and host queries, so this PR extends it with the untested surface (`EdgeWorkerModel` queue handling and the maintenance/shutdown/remove/queue/concurrency lifecycle operations, 33 new tests) and removes the stale exemption alongside the five real ones. All 80 tests pass locally; the `test_providers_modules_should_have_tests` guard passes with the six entries removed. --- ##### 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]
