Taragolis commented on code in PR #39025:
URL: https://github.com/apache/airflow/pull/39025#discussion_r1565554055
##########
tests/providers/amazon/aws/hooks/test_s3.py:
##########
@@ -467,7 +466,6 @@ async def
test_s3_key_hook_get_head_object_with_error_async(self, mock_client):
@async_mock.patch("airflow.providers.amazon.aws.triggers.s3.S3Hook.async_conn")
@pytest.mark.asyncio
- @unittest.expectedFailure
Review Comment:
This one was added into the initial commit:
https://github.com/apache/airflow/pull/31018
So there is two option:
1. It was add by a mistake
2. The test is broken from the initial commit, which are not good.
There is no error when I tried to run it locally, however better to check it
into the CI
--
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]
