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

   `providers/google/tests/unit/google/cloud/links/test_dataprep.py` was listed 
in `OVERLOOKED_TESTS`, so `DataprepFlowLink` and `DataprepJobGroupLink` had no 
coverage.
   
   Adds class-attribute and `persist` checks for both, plus a `_format_link` 
case that pins a behaviour specific to this module:
   
   Dataprep defines its **own** absolute `BASE_LINK` 
(`https://clouddataprep.com`), unlike the other link modules which build 
relative paths on top of `links/base.py`'s Google Cloud console `BASE_LINK`. 
`BaseGoogleLink._format_link` branches on `formatted_str.startswith("http")`, 
so Dataprep links are returned as-is with no console prefix. The test asserts 
exactly that, which is the part a future refactor of `_format_link` could 
silently break.
   
   The `OVERLOOKED_TESTS` entry is removed in this PR.
   
   Follows the same shape as #72949 (`test_bigquery_dts.py`).
   
   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_dataprep.py` — 
6 passed
   - `pytest airflow-core/tests/unit/always/test_project_structure.py` — 10 
passed, 1 pre-existing xfail
   - `prek run --files <both files>` — 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