)]}'
{"nova/tests/unit/virt/libvirt/test_imagebackend.py":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"ae1caf462f362af4977aecf39fc788c96d7e58c1","unresolved":false,"context_lines":[{"line_number":217,"context_line":"    @mock.patch.object(os.path, \u0027exists\u0027)"},{"line_number":218,"context_line":"    def test_cache(self, mock_exists, mock_ensure):"},{"line_number":219,"context_line":"        self.stub_out(\u0027nova.virt.libvirt.imagebackend.Flat.correct_format\u0027,"},{"line_number":220,"context_line":"                       lambda _: None)"},{"line_number":221,"context_line":"        mock_exists.side_effect \u003d [False, False, False]"},{"line_number":222,"context_line":"        exist_calls \u003d [mock.call(self.TEMPLATE_DIR),"},{"line_number":223,"context_line":"                       mock.call(self.PATH), mock.call(self.TEMPLATE_PATH)]"}],"source_content_type":"text/x-python","patch_set":8,"id":"7a77a97e_892134fe","line":220,"updated":"2016-11-16 12:02:43.000000000","message":"Nit: I would rather we just used the mock decorator this this one was well.","commit_id":"dfe4259cedc114f3fdfc200e5c502744cb1ceeef"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"2eb301d986e1641a8bc14c9c6b4e1e93c1eebd04","unresolved":false,"context_lines":[{"line_number":217,"context_line":"    @mock.patch.object(os.path, \u0027exists\u0027)"},{"line_number":218,"context_line":"    def test_cache(self, mock_exists, mock_ensure):"},{"line_number":219,"context_line":"        self.stub_out(\u0027nova.virt.libvirt.imagebackend.Flat.correct_format\u0027,"},{"line_number":220,"context_line":"                       lambda _: None)"},{"line_number":221,"context_line":"        mock_exists.side_effect \u003d [False, False, False]"},{"line_number":222,"context_line":"        exist_calls \u003d [mock.call(self.TEMPLATE_DIR),"},{"line_number":223,"context_line":"                       mock.call(self.PATH), mock.call(self.TEMPLATE_PATH)]"}],"source_content_type":"text/x-python","patch_set":8,"id":"7a77a97e_0fad5988","line":220,"in_reply_to":"7a77a97e_892134fe","updated":"2016-11-21 22:05:06.000000000","message":"yeah, it is nice to use decorator instead for readable code, but I am OK to merge current one.","commit_id":"dfe4259cedc114f3fdfc200e5c502744cb1ceeef"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"ae1caf462f362af4977aecf39fc788c96d7e58c1","unresolved":false,"context_lines":[{"line_number":266,"context_line":"                      lambda _: None)"},{"line_number":267,"context_line":"        mock_exists.side_effect \u003d [True, False, True]"},{"line_number":268,"context_line":"        exist_calls \u003d [mock.call(self.TEMPLATE_DIR),"},{"line_number":269,"context_line":"                       mock.call(self.PATH), mock.call(self.TEMPLATE_PATH)]"},{"line_number":270,"context_line":"        image \u003d self.image_class(self.INSTANCE, self.NAME)"},{"line_number":271,"context_line":"        self.mock_create_image(image)"},{"line_number":272,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"7a77a97e_a9eef0b1","line":269,"updated":"2016-11-16 12:02:43.000000000","message":"Nit: the exist calls should be defined at the bottom near the assert. As with all the ones above.","commit_id":"dfe4259cedc114f3fdfc200e5c502744cb1ceeef"}]}