EricGao888 commented on pull request #21992: URL: https://github.com/apache/airflow/pull/21992#issuecomment-1059132125
> * At present, unit tests of oss hook in alibaba-provider uses real OSS(remote object storage service), which is not a good practice for unit test and could cause stability issues. Furthermore, most of test cases for OSSHook object are absent. > * This pr refactors the previous oss hook unit tests and uses unittest.mock to get rid of communications with real OSS. > * This pr also adds test cases to cover all functions for OSSHook object. > * related: [Switch Alibaba tests to use Mocks #17617](https://github.com/apache/airflow/issues/17617) This commit fixed unit tests in oss hook. If it works, I will fix those in oss_task_handler, oss operator and oss sensor. -- 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]
