)]}'
{"tempest/tests/lib/services/volume/v2/test_transfers_client.py":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"7a2db7b38fd0b871c7162ca84162d7a47ec6d51e","unresolved":false,"context_lines":[{"line_number":67,"context_line":"            resp_body,"},{"line_number":68,"context_line":"            to_utf\u003dbytes_body,"},{"line_number":69,"context_line":"            status\u003d202,"},{"line_number":70,"context_line":"            transfer_id\u003d\"0e89cdd1-6249-421b-96d8-25fac0623d42\")"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"    def _test_show_volume_transfer(self, bytes_body\u003dFalse):"},{"line_number":73,"context_line":"        resp_body \u003d self.FAKE_VOLUME_TRANSFER_INFO"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f1d235d_4a908a95","line":70,"updated":"2017-07-06 16:19:21.000000000","message":"According to API docs this also takes required auth_key kwarg.","commit_id":"19c05f263196f113a706253c58194cb16acf93f8"},{"author":{"_account_id":23081,"name":"Jeremy Zhang","email":"zhangjun_inspur@163.com","username":"jeremy.zhang"},"change_message_id":"0fe5bf5f8060f4c47b50b0e7f9b4cc68321236dd","unresolved":false,"context_lines":[{"line_number":67,"context_line":"            resp_body,"},{"line_number":68,"context_line":"            to_utf\u003dbytes_body,"},{"line_number":69,"context_line":"            status\u003d202,"},{"line_number":70,"context_line":"            transfer_id\u003d\"0e89cdd1-6249-421b-96d8-25fac0623d42\")"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"    def _test_show_volume_transfer(self, bytes_body\u003dFalse):"},{"line_number":73,"context_line":"        resp_body \u003d self.FAKE_VOLUME_TRANSFER_INFO"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f1d235d_0106e920","line":70,"in_reply_to":"3f1d235d_4a908a95","updated":"2017-07-07 01:23:59.000000000","message":"Hi, Felipe.\nI think the \u0027auth_key\u0027 kwarg should be provided in request body, not the url path, :)","commit_id":"19c05f263196f113a706253c58194cb16acf93f8"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"7a2db7b38fd0b871c7162ca84162d7a47ec6d51e","unresolved":false,"context_lines":[{"line_number":88,"context_line":"            resp_body,"},{"line_number":89,"context_line":"            to_utf\u003dbytes_body)"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    def _test_list_volume_transfers_with_detail(self, bytes_body\u003dFalse):"},{"line_number":92,"context_line":"        resp_body \u003d {\"transfers\": [self.FAKE_VOLUME_TRANSFER_INFO[\u0027transfer\u0027]]}"},{"line_number":93,"context_line":"        self.check_service_client_function("},{"line_number":94,"context_line":"            self.client.list_volume_transfers,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f1d235d_6acd8683","line":91,"range":{"start_line":91,"start_character":8,"end_line":91,"end_character":47},"updated":"2017-07-06 16:19:21.000000000","message":"nit: You could combine this with _test_list_volume_transfers above.","commit_id":"19c05f263196f113a706253c58194cb16acf93f8"},{"author":{"_account_id":23081,"name":"Jeremy Zhang","email":"zhangjun_inspur@163.com","username":"jeremy.zhang"},"change_message_id":"0fe5bf5f8060f4c47b50b0e7f9b4cc68321236dd","unresolved":false,"context_lines":[{"line_number":88,"context_line":"            resp_body,"},{"line_number":89,"context_line":"            to_utf\u003dbytes_body)"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"    def _test_list_volume_transfers_with_detail(self, bytes_body\u003dFalse):"},{"line_number":92,"context_line":"        resp_body \u003d {\"transfers\": [self.FAKE_VOLUME_TRANSFER_INFO[\u0027transfer\u0027]]}"},{"line_number":93,"context_line":"        self.check_service_client_function("},{"line_number":94,"context_line":"            self.client.list_volume_transfers,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f1d235d_9c049e16","line":91,"range":{"start_line":91,"start_character":8,"end_line":91,"end_character":47},"in_reply_to":"3f1d235d_6acd8683","updated":"2017-07-07 01:23:59.000000000","message":"Yes, it will be done, thanks.","commit_id":"19c05f263196f113a706253c58194cb16acf93f8"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"c91bb53a639bd133e235f4c3629b380e4f4db0a8","unresolved":false,"context_lines":[{"line_number":56,"context_line":"            \u0027tempest.lib.common.rest_client.RestClient.post\u0027,"},{"line_number":57,"context_line":"            resp_body,"},{"line_number":58,"context_line":"            to_utf\u003dbytes_body,"},{"line_number":59,"context_line":"            status\u003d202)"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"    def _test_accept_volume_transfer(self, bytes_body\u003dFalse):"},{"line_number":62,"context_line":"        resp_body \u003d copy.deepcopy(self.FAKE_VOLUME_TRANSFER_INFO)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3f1d235d_0210284d","line":59,"range":{"start_line":59,"start_character":22,"end_line":59,"end_character":23},"updated":"2017-07-07 01:51:49.000000000","message":"ditto, we can pass and check request param also.","commit_id":"15fbe7235c66f4e3bc45ac8ce34ad9877085a665"},{"author":{"_account_id":23081,"name":"Jeremy Zhang","email":"zhangjun_inspur@163.com","username":"jeremy.zhang"},"change_message_id":"5cc98f41ef1c4fa187566692ea0114ede1d6d87c","unresolved":false,"context_lines":[{"line_number":56,"context_line":"            \u0027tempest.lib.common.rest_client.RestClient.post\u0027,"},{"line_number":57,"context_line":"            resp_body,"},{"line_number":58,"context_line":"            to_utf\u003dbytes_body,"},{"line_number":59,"context_line":"            status\u003d202)"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"    def _test_accept_volume_transfer(self, bytes_body\u003dFalse):"},{"line_number":62,"context_line":"        resp_body \u003d copy.deepcopy(self.FAKE_VOLUME_TRANSFER_INFO)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3f1d235d_e252ccdd","line":59,"range":{"start_line":59,"start_character":22,"end_line":59,"end_character":23},"in_reply_to":"3f1d235d_0210284d","updated":"2017-07-07 01:59:54.000000000","message":"Ok, it will be updated too, :)","commit_id":"15fbe7235c66f4e3bc45ac8ce34ad9877085a665"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"c91bb53a639bd133e235f4c3629b380e4f4db0a8","unresolved":false,"context_lines":[{"line_number":67,"context_line":"            resp_body,"},{"line_number":68,"context_line":"            to_utf\u003dbytes_body,"},{"line_number":69,"context_line":"            status\u003d202,"},{"line_number":70,"context_line":"            transfer_id\u003d\"0e89cdd1-6249-421b-96d8-25fac0623d42\")"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"    def _test_show_volume_transfer(self, bytes_body\u003dFalse):"},{"line_number":73,"context_line":"        resp_body \u003d self.FAKE_VOLUME_TRANSFER_INFO"}],"source_content_type":"text/x-python","patch_set":3,"id":"3f1d235d_6207dc19","line":70,"range":{"start_line":70,"start_character":62,"end_line":70,"end_character":63},"updated":"2017-07-07 01:51:49.000000000","message":"as pointed by Felipe in PS2, you can pass request param with kwargs and then for checking it with mock_args \n\n-https://github.com/openstack/tempest/blob/21dd8a5ee2ab5a068cbb20d0468bd5f444fef59a/tempest/tests/lib/services/base.py#L32","commit_id":"15fbe7235c66f4e3bc45ac8ce34ad9877085a665"},{"author":{"_account_id":23081,"name":"Jeremy Zhang","email":"zhangjun_inspur@163.com","username":"jeremy.zhang"},"change_message_id":"5cc98f41ef1c4fa187566692ea0114ede1d6d87c","unresolved":false,"context_lines":[{"line_number":67,"context_line":"            resp_body,"},{"line_number":68,"context_line":"            to_utf\u003dbytes_body,"},{"line_number":69,"context_line":"            status\u003d202,"},{"line_number":70,"context_line":"            transfer_id\u003d\"0e89cdd1-6249-421b-96d8-25fac0623d42\")"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"    def _test_show_volume_transfer(self, bytes_body\u003dFalse):"},{"line_number":73,"context_line":"        resp_body \u003d self.FAKE_VOLUME_TRANSFER_INFO"}],"source_content_type":"text/x-python","patch_set":3,"id":"3f1d235d_42cd00b5","line":70,"range":{"start_line":70,"start_character":62,"end_line":70,"end_character":63},"in_reply_to":"3f1d235d_6207dc19","updated":"2017-07-07 01:59:54.000000000","message":"Thanks, Ghanshyam, I will update it in next patch","commit_id":"15fbe7235c66f4e3bc45ac8ce34ad9877085a665"}]}