)]}'
{"zun/tests/unit/container/docker/test_docker_driver.py":[{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"935e9b54e999c901f31bc9f4b3ace79d775f0a7e","unresolved":false,"context_lines":[{"line_number":141,"context_line":"            self.mock_docker.remove_container \u003d mock.Mock("},{"line_number":142,"context_line":"                side_effect\u003derrors.APIError(\u0027Error\u0027, \u0027\u0027, \u0027\u0027))"},{"line_number":143,"context_line":"            db_container \u003d db_utils.create_test_container(context\u003dself.context)"},{"line_number":144,"context_line":"            self.driver.delete(db_container, \u0027test\u0027)"},{"line_number":145,"context_line":"            self.mock_docker.remove_container.assert_called_once_with("},{"line_number":146,"context_line":"                db_container.container_id, force\u003d\u0027test\u0027)"},{"line_number":147,"context_line":"            self.assertEqual(1, mock_init.call_count)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a71b18c_b394728b","line":144,"range":{"start_line":144,"start_character":45,"end_line":144,"end_character":51},"updated":"2016-12-09 21:51:02.000000000","message":"nit: pass True/False instead.","commit_id":"499bc0c002ce0ec3689fb2ab1b97885850c73dc4"},{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"935e9b54e999c901f31bc9f4b3ace79d775f0a7e","unresolved":false,"context_lines":[{"line_number":143,"context_line":"            db_container \u003d db_utils.create_test_container(context\u003dself.context)"},{"line_number":144,"context_line":"            self.driver.delete(db_container, \u0027test\u0027)"},{"line_number":145,"context_line":"            self.mock_docker.remove_container.assert_called_once_with("},{"line_number":146,"context_line":"                db_container.container_id, force\u003d\u0027test\u0027)"},{"line_number":147,"context_line":"            self.assertEqual(1, mock_init.call_count)"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"    def test_delete_fail_raise_error(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a71b18c_13ea860f","line":146,"range":{"start_line":146,"start_character":50,"end_line":146,"end_character":54},"updated":"2016-12-09 21:51:02.000000000","message":"Ditto: It should be either True or False","commit_id":"499bc0c002ce0ec3689fb2ab1b97885850c73dc4"},{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"935e9b54e999c901f31bc9f4b3ace79d775f0a7e","unresolved":false,"context_lines":[{"line_number":415,"context_line":"            name\u003d\u0027test_sanbox_name\u0027, image\u003d\u0027kubernetes/pause\u0027,"},{"line_number":416,"context_line":"            flavor\u003d\u0027m1.small\u0027, key_name\u003dNone,"},{"line_number":417,"context_line":"            nics\u003d\u0027auto\u0027)"},{"line_number":418,"context_line":"        ensure_active_mock.assert_called_once_with(nova_client_instance,"},{"line_number":419,"context_line":"                                                   \u0027server_instance\u0027)"},{"line_number":420,"context_line":"        find_container_by_server_name_mock.assert_called_once_with("},{"line_number":421,"context_line":"            \u0027test_sanbox_name\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a71b18c_13de0611","line":418,"range":{"start_line":418,"start_character":8,"end_line":418,"end_character":26},"updated":"2016-12-09 21:51:02.000000000","message":"nit: it read better if it is renamed to mock_ensure_active","commit_id":"499bc0c002ce0ec3689fb2ab1b97885850c73dc4"},{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"935e9b54e999c901f31bc9f4b3ace79d775f0a7e","unresolved":false,"context_lines":[{"line_number":417,"context_line":"            nics\u003d\u0027auto\u0027)"},{"line_number":418,"context_line":"        ensure_active_mock.assert_called_once_with(nova_client_instance,"},{"line_number":419,"context_line":"                                                   \u0027server_instance\u0027)"},{"line_number":420,"context_line":"        find_container_by_server_name_mock.assert_called_once_with("},{"line_number":421,"context_line":"            \u0027test_sanbox_name\u0027)"},{"line_number":422,"context_line":"        self.assertEqual(result_sandbox_id, \u0027test_container_name_id\u0027)"},{"line_number":423,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3a71b18c_f3d50ae5","line":420,"range":{"start_line":420,"start_character":8,"end_line":420,"end_character":42},"updated":"2016-12-09 21:51:02.000000000","message":"Ditto. It read better with name mock_find_container_by_server_name","commit_id":"499bc0c002ce0ec3689fb2ab1b97885850c73dc4"}]}