ephraimbuddy commented on a change in pull request #15002:
URL: https://github.com/apache/airflow/pull/15002#discussion_r603116588



##########
File path: tests/providers/microsoft/azure/log/test_wasb_task_handler.py
##########
@@ -98,7 +98,7 @@ def test_wasb_log_exists(self, mock_hook):
 
     @mock.patch("airflow.providers.microsoft.azure.hooks.wasb.WasbHook")
     def test_wasb_read(self, mock_hook):
-        mock_hook.return_value.read_file.return_value = 'Log line'
+        mock_hook.return_value.read_file.return_value = b'Log line'

Review comment:
       You don't need to install azure-storage-blob by yourself.
   To install wasb on your airflow instance, just run `pip install 
apache-airflow-providers-microsoft-azure` on the instance. The latest version 
will be installed on the instance




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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to