)]}'
{"tests/unit/services/image/test_glance_v1.py":[{"author":{"_account_id":21528,"name":"chenhb","email":"chen.haibing1@zte.com.cn","username":"chenhb"},"change_message_id":"09ea2644303cb9a79cf7f77763f82962932dfa68","unresolved":false,"context_lines":[{"line_number":68,"context_line":"                     \"min_ram\": 0,"},{"line_number":69,"context_line":"                     \"properties\": properties}"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"        if location.startswith(\"/\"):"},{"line_number":72,"context_line":"            call_args[\"data\"] \u003d mock_open.return_value"},{"line_number":73,"context_line":"            mock_open.assert_called_once_with(location, \"rb\")"},{"line_number":74,"context_line":"            mock_open.return_value.close.assert_called_once_with()"},{"line_number":75,"context_line":"        else:"},{"line_number":76,"context_line":"            call_args[\"copy_from\"] \u003d location"},{"line_number":77,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_c93f91d4","side":"PARENT","line":74,"range":{"start_line":71,"start_character":0,"end_line":74,"end_character":66},"updated":"2019-06-04 00:58:02.000000000","message":"why to remove these line?","commit_id":"902359a11d1f4d1852327453280bd0938829b70a"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"513438ee9bf0975297a677a662a712b596a11ede","unresolved":false,"context_lines":[{"line_number":68,"context_line":"                     \"min_ram\": 0,"},{"line_number":69,"context_line":"                     \"properties\": properties}"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"        if location.startswith(\"/\"):"},{"line_number":72,"context_line":"            call_args[\"data\"] \u003d mock_open.return_value"},{"line_number":73,"context_line":"            mock_open.assert_called_once_with(location, \"rb\")"},{"line_number":74,"context_line":"            mock_open.return_value.close.assert_called_once_with()"},{"line_number":75,"context_line":"        else:"},{"line_number":76,"context_line":"            call_args[\"copy_from\"] \u003d location"},{"line_number":77,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_160299b5","side":"PARENT","line":74,"range":{"start_line":71,"start_character":0,"end_line":74,"end_character":66},"in_reply_to":"9fb8cfa7_c93f91d4","updated":"2019-06-04 13:19:34.000000000","message":"Since we are using a fixture \u0027open\u0027 is no longer called.  This actually reminds me the mock_open isn\u0027t needed either.","commit_id":"902359a11d1f4d1852327453280bd0938829b70a"},{"author":{"_account_id":21528,"name":"chenhb","email":"chen.haibing1@zte.com.cn","username":"chenhb"},"change_message_id":"09ea2644303cb9a79cf7f77763f82962932dfa68","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        properties \u003d {\"fakeprop\": \"fake\"}"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"        # override the location with a private temp file"},{"line_number":57,"context_line":"        if not is_public:"},{"line_number":58,"context_line":"            location \u003d self._get_temp_file_name()"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"        image \u003d self.service.create_image("}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_cc0d5f38","line":57,"updated":"2019-06-04 00:58:02.000000000","message":"is_public is for the image status, we could try to make a difference at \"location\" key.","commit_id":"e2c083cb82ef79501991ea300a37cb0bd34a9bd2"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"513438ee9bf0975297a677a662a712b596a11ede","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        properties \u003d {\"fakeprop\": \"fake\"}"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"        # override the location with a private temp file"},{"line_number":57,"context_line":"        if not is_public:"},{"line_number":58,"context_line":"            location \u003d self._get_temp_file_name()"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"        image \u003d self.service.create_image("}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_96276965","line":57,"in_reply_to":"9fb8cfa7_cc0d5f38","updated":"2019-06-04 13:19:34.000000000","message":"Done","commit_id":"e2c083cb82ef79501991ea300a37cb0bd34a9bd2"}],"tests/unit/services/image/test_glance_v2.py":[{"author":{"_account_id":21528,"name":"chenhb","email":"chen.haibing1@zte.com.cn","username":"chenhb"},"change_message_id":"09ea2644303cb9a79cf7f77763f82962932dfa68","unresolved":false,"context_lines":[{"line_number":50,"context_line":""},{"line_number":51,"context_line":"        self.service.upload_data(image_id, image_location\u003dlocation)"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"        if location.startswith(\"/\"):"},{"line_number":54,"context_line":"            mock_open.assert_called_once_with(location, \"rb\")"},{"line_number":55,"context_line":"            mock_open.return_value.close.assert_called_once_with()"},{"line_number":56,"context_line":"            self.gc.images.upload.assert_called_once_with("},{"line_number":57,"context_line":"                image_id, mock_open.return_value)"},{"line_number":58,"context_line":"        else:"},{"line_number":59,"context_line":"            mock_requests_get.assert_called_once_with(location, stream\u003dTrue)"},{"line_number":60,"context_line":"            self.gc.images.upload.assert_called_once_with("},{"line_number":61,"context_line":"                image_id, mock_requests_get.return_value.raw)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_692465c7","side":"PARENT","line":58,"range":{"start_line":53,"start_character":0,"end_line":58,"end_character":13},"updated":"2019-06-04 00:58:02.000000000","message":"the same here","commit_id":"902359a11d1f4d1852327453280bd0938829b70a"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"513438ee9bf0975297a677a662a712b596a11ede","unresolved":false,"context_lines":[{"line_number":50,"context_line":""},{"line_number":51,"context_line":"        self.service.upload_data(image_id, image_location\u003dlocation)"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"        if location.startswith(\"/\"):"},{"line_number":54,"context_line":"            mock_open.assert_called_once_with(location, \"rb\")"},{"line_number":55,"context_line":"            mock_open.return_value.close.assert_called_once_with()"},{"line_number":56,"context_line":"            self.gc.images.upload.assert_called_once_with("},{"line_number":57,"context_line":"                image_id, mock_open.return_value)"},{"line_number":58,"context_line":"        else:"},{"line_number":59,"context_line":"            mock_requests_get.assert_called_once_with(location, stream\u003dTrue)"},{"line_number":60,"context_line":"            self.gc.images.upload.assert_called_once_with("},{"line_number":61,"context_line":"                image_id, mock_requests_get.return_value.raw)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_16f059c9","side":"PARENT","line":58,"range":{"start_line":53,"start_character":0,"end_line":58,"end_character":13},"in_reply_to":"9fb8cfa7_692465c7","updated":"2019-06-04 13:19:34.000000000","message":"Same answer as the other file, \u0027open\u0027 is no longer called.","commit_id":"902359a11d1f4d1852327453280bd0938829b70a"}]}