Vamsi-klu commented on PR #69398: URL: https://github.com/apache/airflow/pull/69398#issuecomment-4889070580
Could you move `get_remote_os` to a module-level import in `encryption_utils`, or patch the symbol where it is actually imported? The tests patch `airflow.providers.teradata.utils.encryption_utils.get_remote_os`, but this PR imports it inside `decrypt_remote_file_to_string`, so that attribute is not available on the module and the test patch should fail before exercising the Unix/Windows branches. --- Drafted-by: Codex (GPT-5); reviewed by @Vamsi-klu before posting -- 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]
