)]}'
{"openstackclient/tests/unit/volume/test_find_resource.py":[{"author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"change_message_id":"042f671e5311b7ffd74fae04a30266e931fb061f","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        resp \u003d mock.Mock()"},{"line_number":48,"context_line":"        body \u003d {\"volumes\": [{\"id\": ID, \u0027display_name\u0027: NAME}]}"},{"line_number":49,"context_line":"        api.client.get.side_effect \u003d [Exception(\"Not found\"),"},{"line_number":50,"context_line":"                                      Exception(\"Not found\"),"},{"line_number":51,"context_line":"                                      (resp, body)]"},{"line_number":52,"context_line":"        self.manager \u003d volumes.VolumeManager(api)"},{"line_number":53,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"1f1a1f67_0d73a582","line":50,"range":{"start_line":50,"start_character":37,"end_line":50,"end_character":60},"updated":"2017-07-12 10:36:36.000000000","message":"https://review.openstack.org/#/c/476523/ add \"if\" logic to avoid calling get() twice, for these test cases, kwargs is empty dict in find_resource(), so the second call to get() is in manager.find(), we can just remove the second exception, make (resp, body) returning","commit_id":"c592facfd7f2ac15ee120127e7d506091dfe9bbd"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"f80674cc8eb2495ae9f3680e00ff939a6626cf84","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        resp \u003d mock.Mock()"},{"line_number":48,"context_line":"        body \u003d {\"volumes\": [{\"id\": ID, \u0027display_name\u0027: NAME}]}"},{"line_number":49,"context_line":"        api.client.get.side_effect \u003d [Exception(\"Not found\"),"},{"line_number":50,"context_line":"                                      Exception(\"Not found\"),"},{"line_number":51,"context_line":"                                      (resp, body)]"},{"line_number":52,"context_line":"        self.manager \u003d volumes.VolumeManager(api)"},{"line_number":53,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"1f1a1f67_dfef8ee7","line":50,"range":{"start_line":50,"start_character":37,"end_line":50,"end_character":60},"in_reply_to":"1f1a1f67_0d73a582","updated":"2017-07-12 12:10:59.000000000","message":"Great. I will import your finding in the next patch set.","commit_id":"c592facfd7f2ac15ee120127e7d506091dfe9bbd"}]}