Kami commented on PR #1886: URL: https://github.com/apache/libcloud/pull/1886#issuecomment-1659035952
I've added a basic unit test in ebecb9af9593ab639298b3f17373efa3056fb765. This test is not the most ideal since it only exercises the logic of `_set_up_connection_info()` method, but it doesn't actually verify it end to end to make sure the underlying HTTP connection is actually re-used. That would require an integration test / similar. It's still better to have this unit test than having no test though. Change has now been merged into trunk. Thanks -- 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: notifications-unsubscr...@libcloud.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org