)]}'
{"openstack/compute/v2/server.py":[{"author":{"_account_id":8246,"name":"Qiming Teng","email":"tengqm@outlook.com","username":"tengqm"},"change_message_id":"6b2198b3b681d8e403ebffa55e37ff9cc7061490","unresolved":false,"context_lines":[{"line_number":339,"context_line":"    def get_console_output(self, session, length\u003dNone):"},{"line_number":340,"context_line":"        body \u003d {\"os-getConsoleOutput\": None}"},{"line_number":341,"context_line":"        if length is not None:"},{"line_number":342,"context_line":"            body[\"os-getConsoleOutput\"][\"length\"] \u003d length"},{"line_number":343,"context_line":"        resp \u003d self._action(session, body)"},{"line_number":344,"context_line":"        return resp.json()"},{"line_number":345,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"ff0f0b1f_4b3f37b5","line":342,"range":{"start_line":342,"start_character":12,"end_line":342,"end_character":58},"updated":"2017-05-26 01:10:36.000000000","message":"this won\u0027t work, it will raise TypeError:\n\nTypeError: \u0027NoneType\u0027 object does not support item assignment","commit_id":"3f19c012e70e97296ca8205b5e407d8c37621d0c"}],"openstack/tests/unit/compute/v2/test_proxy.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"6d2e9a490b29a39f3ba839e22c27bea7bc44c660","unresolved":false,"context_lines":[{"line_number":405,"context_line":"        self._verify(\"openstack.compute.v2.server.Server.get_console_output\","},{"line_number":406,"context_line":"                     self.proxy.get_server_console_output,"},{"line_number":407,"context_line":"                     method_args\u003d[\"value\"],"},{"line_number":408,"context_line":"                     expected_kwargs\u003d{\"length\": None})"},{"line_number":409,"context_line":""},{"line_number":410,"context_line":"    def test_availability_zones(self):"},{"line_number":411,"context_line":"        self.verify_list_no_kwargs(self.proxy.availability_zones,"}],"source_content_type":"text/x-python","patch_set":4,"id":"df140735_b3cbe5a3","line":408,"updated":"2017-05-29 13:33:19.000000000","message":"It is nice if we have a test coverage for non-None length.","commit_id":"6467c765475b265412159ac1c5d4ec8911d2c60f"}],"openstack/tests/unit/compute/v2/test_server.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"6d2e9a490b29a39f3ba839e22c27bea7bc44c660","unresolved":false,"context_lines":[{"line_number":617,"context_line":"        body \u003d {\"os-getConsoleOutput\": {}}"},{"line_number":618,"context_line":"        headers \u003d {\u0027Accept\u0027: \u0027\u0027}"},{"line_number":619,"context_line":"        self.sess.post.assert_called_with("},{"line_number":620,"context_line":"            url, endpoint_filter\u003dsot.service, json\u003dbody, headers\u003dheaders)"}],"source_content_type":"text/x-python","patch_set":4,"id":"df140735_13e3b135","line":620,"updated":"2017-05-29 13:33:19.000000000","message":"suggestion: it is nice to have a test coverage for non-None length parameter.","commit_id":"6467c765475b265412159ac1c5d4ec8911d2c60f"}]}