You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "vincbeck (via GitHub)" <gi...@apache.org> on 2023/07/20 17:27:35 UTC

[GitHub] [airflow] vincbeck commented on pull request #32664: Add endpoint_url in test_connection

vincbeck commented on PR #32664:
URL: https://github.com/apache/airflow/pull/32664#issuecomment-1644313378

   > @vincbeck After making the mentioned change, the test for `test_hook_connection_test_failed` in `test/providers/amazon/aws/hooks/test_base_aws.py` started asserting an error due to the absence of the `test_endpoint_url` in `mock_sts_client`. To resolve this, I will make the necessary modifications. By the way, Would it be better to include a unit test specifically for test_endpoint_url to ensure its validity?
   
   That would be great :)


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org