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

Reply via email to