)]}'
{"tempest/common/rest_client.py":[{"author":{"_account_id":8911,"name":"Sergey Vilgelm","email":"sergey@vilgelm.info","username":"sergey.vilgelm"},"change_message_id":"fcc0dc8068342bd109bff2ea8fb3b6b6c01a3116","unresolved":false,"context_lines":[{"line_number":451,"context_line":"                    # exception."},{"line_number":452,"context_line":"                    raise exceptions.InvalidHTTPResponseBody(message)"},{"line_number":453,"context_line":"                else:"},{"line_number":454,"context_line":"                    if isinstance(resp_body, dict):"},{"line_number":455,"context_line":"                        # I\u0027m seeing both computeFault"},{"line_number":456,"context_line":"                        # and cloudServersFault come back."},{"line_number":457,"context_line":"                        # Will file a bug to fix, but leave as is for now."}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAU3%2F%2Ffgw%3D","line":454,"updated":"2014-02-26 10:09:46.000000000","message":"Why not \"elif\"?","commit_id":"e91e67d0717bd81591a02baaa30f047c7d59a4aa"},{"author":{"_account_id":8911,"name":"Sergey Vilgelm","email":"sergey@vilgelm.info","username":"sergey.vilgelm"},"change_message_id":"f88ad4b9fb5cad520bc3cbc407aebeb40c7869c7","unresolved":false,"context_lines":[{"line_number":451,"context_line":"                    # exception."},{"line_number":452,"context_line":"                    raise exceptions.InvalidHTTPResponseBody(message)"},{"line_number":453,"context_line":"                else:"},{"line_number":454,"context_line":"                    if isinstance(resp_body, dict):"},{"line_number":455,"context_line":"                        # I\u0027m seeing both computeFault"},{"line_number":456,"context_line":"                        # and cloudServersFault come back."},{"line_number":457,"context_line":"                        # Will file a bug to fix, but leave as is for now."}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAU3%2F%2FfWQ%3D","line":454,"in_reply_to":"AAAAU3%2F%2FfX8%3D","updated":"2014-02-26 10:27:45.000000000","message":"sorry","commit_id":"e91e67d0717bd81591a02baaa30f047c7d59a4aa"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"75c4652a8a70d0874c51bb6548b8c5ff90db9598","unresolved":false,"context_lines":[{"line_number":451,"context_line":"                    # exception."},{"line_number":452,"context_line":"                    raise exceptions.InvalidHTTPResponseBody(message)"},{"line_number":453,"context_line":"                else:"},{"line_number":454,"context_line":"                    if isinstance(resp_body, dict):"},{"line_number":455,"context_line":"                        # I\u0027m seeing both computeFault"},{"line_number":456,"context_line":"                        # and cloudServersFault come back."},{"line_number":457,"context_line":"                        # Will file a bug to fix, but leave as is for now."}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAU3%2F%2FfX8%3D","line":454,"in_reply_to":"AAAAU3%2F%2Ffgw%3D","updated":"2014-02-26 10:26:05.000000000","message":"\u0027else\u0027 from string 453 is related to try-block. There is no \u0027elif\u0027 for try-block.\n\nIn string 447 we can get dict or string, and here we set condition what is it exactly.","commit_id":"e91e67d0717bd81591a02baaa30f047c7d59a4aa"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"8c6bb9b6a6dc9d67ed0e80418941832f94207c71","unresolved":false,"context_lines":[{"line_number":467,"context_line":"            raise exceptions.ServerFault(message)"},{"line_number":468,"context_line":""},{"line_number":469,"context_line":"        if resp.status \u003e\u003d 400:"},{"line_number":470,"context_line":"            if parse_resp:"},{"line_number":471,"context_line":"                resp_body \u003d self._parse_resp(resp_body)"},{"line_number":472,"context_line":"            raise exceptions.RestClientException(str(resp.status))"},{"line_number":473,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAVn%2F%2FQr4%3D","side":"PARENT","line":470,"updated":"2014-03-07 07:56:01.000000000","message":"Removed condition as redundant code, because exception will be raised and this \"resp_body\" won\u0027t be used.","commit_id":"be392f2d7ce2940aa5c92c253b28344eb52210f9"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"8c6bb9b6a6dc9d67ed0e80418941832f94207c71","unresolved":false,"context_lines":[{"line_number":409,"context_line":"        elif ctype.lower() in TXT_ENC:"},{"line_number":410,"context_line":"            parse_resp \u003d False"},{"line_number":411,"context_line":"        else:"},{"line_number":412,"context_line":"            raise exceptions.InvalidContentType(str(resp.status))"},{"line_number":413,"context_line":""},{"line_number":414,"context_line":"        if resp.status \u003d\u003d 401 or resp.status \u003d\u003d 403:"},{"line_number":415,"context_line":"            raise exceptions.Unauthorized()"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAVn%2F%2FQq8%3D","line":412,"updated":"2014-03-07 07:56:01.000000000","message":"This Exception was moved to \"base\" module, replaced it with appropriate new Exception.","commit_id":"da5717299cafaa90b296ed29cb942f42fc30dea2"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"8c6bb9b6a6dc9d67ed0e80418941832f94207c71","unresolved":false,"context_lines":[{"line_number":470,"context_line":"            raise exceptions.ServerFault(message)"},{"line_number":471,"context_line":""},{"line_number":472,"context_line":"        if resp.status \u003e\u003d 400:"},{"line_number":473,"context_line":"            raise exceptions.UnexpectedResponseCode(str(resp.status))"},{"line_number":474,"context_line":""},{"line_number":475,"context_line":"    def is_absolute_limit(self, resp, resp_body):"},{"line_number":476,"context_line":"        if (not isinstance(resp_body, collections.Mapping) or"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAVn%2F%2FQq4%3D","line":473,"updated":"2014-03-07 07:56:01.000000000","message":"This Exception was moved to \"base\" module, replaced it with appropriate new Exception.","commit_id":"da5717299cafaa90b296ed29cb942f42fc30dea2"}],"tempest/tests/test_rest_client.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"8a7debfd92aeb69e11d1fee9413b3d4c4900e898","unresolved":false,"context_lines":[{"line_number":235,"context_line":"class TestRestClientErrorCheckerJSON(base.TestCase):"},{"line_number":236,"context_line":"    c_type \u003d \"application/json\""},{"line_number":237,"context_line":""},{"line_number":238,"context_line":"    def set_data(self, r_code, enc\u003dNone, r_body\u003dNone):"},{"line_number":239,"context_line":"        if enc is None:"},{"line_number":240,"context_line":"            enc \u003d self.c_type"},{"line_number":241,"context_line":"        resp_dict \u003d {\u0027status\u0027: r_code, \u0027content-type\u0027: enc}"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAVn%2F%2FRIQ%3D","line":238,"updated":"2014-03-07 06:16:44.000000000","message":"\"enc\" argument seems unnecessary. we can specify self.c_type to \u0027content-type\u0027 directly instead.","commit_id":"e91e67d0717bd81591a02baaa30f047c7d59a4aa"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"279660fcc94a2284af9b36400c7e49ef783ebc83","unresolved":false,"context_lines":[{"line_number":235,"context_line":"class TestRestClientErrorCheckerJSON(base.TestCase):"},{"line_number":236,"context_line":"    c_type \u003d \"application/json\""},{"line_number":237,"context_line":""},{"line_number":238,"context_line":"    def set_data(self, r_code, enc\u003dNone, r_body\u003dNone):"},{"line_number":239,"context_line":"        if enc is None:"},{"line_number":240,"context_line":"            enc \u003d self.c_type"},{"line_number":241,"context_line":"        resp_dict \u003d {\u0027status\u0027: r_code, \u0027content-type\u0027: enc}"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAVn%2F%2FQsU%3D","line":238,"in_reply_to":"AAAAVn%2F%2FRIQ%3D","updated":"2014-03-07 07:46:01.000000000","message":"This is useful for test added in patch-set 3.","commit_id":"e91e67d0717bd81591a02baaa30f047c7d59a4aa"}]}