)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9914,"name":"Ade Lee","email":"alee@redhat.com","username":"alee"},"change_message_id":"4c4572eb712cc88704893ace42ef32590d717796","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove test_cafile_certfile_keyfile_kwargs_set_correctly"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This test needs to mock many calls to keystone, so it makes"},{"line_number":10,"context_line":"more sense to have it as a functional test instead of a unit test."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I6e06c2a4da8c39fe0dad32aec95e75ac2307f379"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"22eb3271_574e1e6a","line":10,"range":{"start_line":9,"start_character":0,"end_line":10,"end_character":66},"updated":"2022-11-24 11:27:50.000000000","message":"The reason this test needs to mock a bunch of keystone calls is because - with the addition of the microversion patch - we now have to mock the return from keystone for a version discovery call.\n\nBut we need to add unit tests to confirm that the version discovery code is working.  And so,after that mocking code is added - this test can be rewritten accordingly.\n\nI think there is still value in making sure that if you pass in cert/key/cacert command line options, then these are actually used by the client.","commit_id":"2e5846aacdcf946dcaafbd13da0a84ae636c2bbf"}]}