)]}'
{"tempest/tests/services/compute/base.py":[{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"5cd4dc4fe90437b5db8dfd4dd80ffaf81f90510b","unresolved":false,"context_lines":[{"line_number":23,"context_line":"class BaseComputeServiceTest(base.TestCase):"},{"line_number":24,"context_line":"    def create_response(self, body, to_utf\u003dFalse, status\u003d200):"},{"line_number":25,"context_line":"        json_body \u003d {}"},{"line_number":26,"context_line":"        if body:"},{"line_number":27,"context_line":"            json_body \u003d json.dumps(body)"},{"line_number":28,"context_line":"            if to_utf:"},{"line_number":29,"context_line":"                json_body \u003d json_body.encode(\u0027utf-8\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"da20952f_32976b3b","line":26,"updated":"2015-09-07 09:37:30.000000000","message":"do you actually need this for your patch? I didn\u0027t see any call with body\u003dNone.\nI think we could do that in a separate patch and fix it in all of the other unit tests right away.","commit_id":"c555aa24b73a86d4e208c399373a00a30b96447e"},{"author":{"_account_id":11278,"name":"Anusha Ramineni","email":"anusha.ramineni@india.nec.com","username":"anusha08"},"change_message_id":"018fbc689c272f3ca59fe3d7c4466d2db6b0dcee","unresolved":false,"context_lines":[{"line_number":23,"context_line":"class BaseComputeServiceTest(base.TestCase):"},{"line_number":24,"context_line":"    def create_response(self, body, to_utf\u003dFalse, status\u003d200):"},{"line_number":25,"context_line":"        json_body \u003d {}"},{"line_number":26,"context_line":"        if body:"},{"line_number":27,"context_line":"            json_body \u003d json.dumps(body)"},{"line_number":28,"context_line":"            if to_utf:"},{"line_number":29,"context_line":"                json_body \u003d json_body.encode(\u0027utf-8\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"da20952f_f241f3a9","line":26,"in_reply_to":"da20952f_32976b3b","updated":"2015-09-07 09:42:53.000000000","message":"Marc, it is used in delete_security_group, as the response will be {}.\nBut +1 to do it in separate patch, as it can be used by other modules as well.","commit_id":"c555aa24b73a86d4e208c399373a00a30b96447e"}],"tempest/tests/services/compute/test_security_groups_client.py":[{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"5bfedd477a3cce7962850cd6c66e8ed7a1b5387b","unresolved":false,"context_lines":[{"line_number":69,"context_line":"            \u0027tempest.common.service_client.ServiceClient.get\u0027,"},{"line_number":70,"context_line":"            return_value\u003dmocked_resp))"},{"line_number":71,"context_line":"        resp \u003d self.client.show_security_group(\u0027fake-id\u0027)"},{"line_number":72,"context_line":"        self.assertEqual(expected, resp)"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"    def test_show_security_group_with_str_body(self):"},{"line_number":75,"context_line":"        self._test_show_security_group()"}],"source_content_type":"text/x-python","patch_set":1,"id":"da20952f_48e594d9","line":72,"updated":"2015-09-07 06:46:38.000000000","message":"Duplicate code with other tests.","commit_id":"964d662e06892387be014193c7674f13c63fce7c"},{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"5bfedd477a3cce7962850cd6c66e8ed7a1b5387b","unresolved":false,"context_lines":[{"line_number":89,"context_line":"            \u0027tempest.common.service_client.ServiceClient.post\u0027,"},{"line_number":90,"context_line":"            return_value\u003dmocked_resp))"},{"line_number":91,"context_line":"        resp \u003d self.client.create_security_group(**post_body)"},{"line_number":92,"context_line":"        self.assertEqual(expected, resp)"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"    def test_create_security_group_with_str_body(self):"},{"line_number":95,"context_line":"        self._test_create_security_group()"}],"source_content_type":"text/x-python","patch_set":1,"id":"da20952f_c8d9a423","line":92,"updated":"2015-09-07 06:46:38.000000000","message":"See above","commit_id":"964d662e06892387be014193c7674f13c63fce7c"},{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"5bfedd477a3cce7962850cd6c66e8ed7a1b5387b","unresolved":false,"context_lines":[{"line_number":109,"context_line":"            \u0027tempest.common.service_client.ServiceClient.put\u0027,"},{"line_number":110,"context_line":"            return_value\u003dmocked_resp))"},{"line_number":111,"context_line":"        resp \u003d self.client.update_security_group(\u0027fake-id\u0027, **req_body)"},{"line_number":112,"context_line":"        self.assertEqual(expected, resp)"},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"    def test_update_security_group_with_str_body(self):"},{"line_number":115,"context_line":"        self._test_update_security_group()"}],"source_content_type":"text/x-python","patch_set":1,"id":"da20952f_a809d096","line":112,"updated":"2015-09-07 06:46:38.000000000","message":"see above","commit_id":"964d662e06892387be014193c7674f13c63fce7c"}]}