ShousenZHANG opened a new pull request, #72970:
URL: https://github.com/apache/airflow/pull/72970

   `providers/google/tests/unit/google/cloud/links/test_dataproc.py` was listed 
in `OVERLOOKED_TESTS`, so its 6 link classes had no coverage.
   
   Adds the checks the sibling link tests use, for each class:
   
   - `test_class_attributes` — `key`, `name`, `format_str`
   - `test_persist` — the XCom payload pushed by `BaseGoogleLink.persist`
   - `test_format_link` — the `_format_link` output
   
   This module builds its link constants on top of `BASE_LINK`, so the 
formatted URL is already absolute and `_format_link` returns it instead of 
prefixing the console base a second time. The `_format_link` assertions pin 
that.
   
   `DATAPROC_CLUSTER_LINK_DEPRECATED` and `DATAPROC_JOB_LINK_DEPRECATED` are 
reachable only through this module's `__getattr__` deprecation shim and no link 
class references them, so there is nothing to assert about them here.
   
   Same shape as `test_bigquery.py` (added in #68066) in the same directory, 
and the `OVERLOOKED_TESTS` entry is removed in this PR.
   
   related: #35442
   
   ### Was generative AI tooling used to co-author this PR?
   
   - [x] Yes
   
   `Generated-by: Claude Code`
   
   The test bodies were written by an AI assistant following the existing 
`test_bigquery.py` pattern in the same directory. I verified them myself:
   
   - `pytest providers/google/tests/unit/google/cloud/links/test_dataproc.py` — 
18 passed
   - `pytest airflow-core/tests/unit/always/test_project_structure.py` — 10 
passed, 1 xfailed
   - `ruff check` / `ruff format --check` on the test file — clean
   - `prek run --files 
providers/google/tests/unit/google/cloud/links/test_dataproc.py 
airflow-core/tests/unit/always/test_project_structure.py` — 43 passed, 212 
skipped, 0 failed
   


-- 
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]

Reply via email to