vineoy commented on PR #75694: URL: https://github.com/apache/airflow/pull/75694#issuecomment-6489913161
Done in `5028fba` per your review: dropped all 24 tests that only assert on values of class attributes (`key`, `name`) — every `test_class_attributes` across the three files — and kept only tests that verify actual behavior: - `test_dataproc.py`: 14 tests (persist XCom payloads, `get_link` formatting incl. the empty-XCom fallback, deprecation warnings, the module-level `__getattr__` shim, and the unknown-attribute error path) - `test_kubernetes_engine.py`: 8 tests (`_format_link` deep links, persist with dict/object payloads, and the ValueError path) - `test_translate.py`: 17 tests (`_format_link`, `extract_output_uri_prefix`/persist round-trips) That's 39 behavior tests in total, down from 63. Thanks for the pointer to #74924. -- 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]
