)]}'
{"nova/tests/unit/compute/test_compute_mgr.py":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"a7781693c23664831a02646cbd86cc00c116303f","unresolved":false,"context_lines":[{"line_number":452,"context_line":"        def _do_mock_calls(mock_inst_init,"},{"line_number":453,"context_line":"                           mock_destroy, mock_admin_ctxt, mock_host_get,"},{"line_number":454,"context_line":"                           mock_filter_off, mock_filter_on, mock_init_host,"},{"line_number":455,"context_line":"                           defer_iptables_apply\u003dFalse):"},{"line_number":456,"context_line":"            mock_admin_ctxt.return_value \u003d self.context"},{"line_number":457,"context_line":"            inst_list \u003d _make_instance_list(startup_instances)"},{"line_number":458,"context_line":"            mock_host_get.return_value \u003d inst_list"}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_ee393ff4","line":455,"range":{"start_line":455,"start_character":48,"end_line":455,"end_character":53},"updated":"2016-06-09 10:03:06.000000000","message":"Nit: not sure we need a default value here.","commit_id":"c00e32ba0e4f04f6cdacf060df96ef8babd06856"},{"author":{"_account_id":19741,"name":"Hieu LE","email":"hieulq2@viettel.com.vn","username":"hieulq"},"change_message_id":"6c37de426904450748274909f5baade1e70450ae","unresolved":false,"context_lines":[{"line_number":452,"context_line":"        def _do_mock_calls(mock_inst_init,"},{"line_number":453,"context_line":"                           mock_destroy, mock_admin_ctxt, mock_host_get,"},{"line_number":454,"context_line":"                           mock_filter_off, mock_filter_on, mock_init_host,"},{"line_number":455,"context_line":"                           defer_iptables_apply\u003dFalse):"},{"line_number":456,"context_line":"            mock_admin_ctxt.return_value \u003d self.context"},{"line_number":457,"context_line":"            inst_list \u003d _make_instance_list(startup_instances)"},{"line_number":458,"context_line":"            mock_host_get.return_value \u003d inst_list"}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_7457e026","line":455,"range":{"start_line":455,"start_character":48,"end_line":455,"end_character":53},"in_reply_to":"7aa08908_ee393ff4","updated":"2016-06-09 10:39:59.000000000","message":"Done","commit_id":"c00e32ba0e4f04f6cdacf060df96ef8babd06856"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"a7781693c23664831a02646cbd86cc00c116303f","unresolved":false,"context_lines":[{"line_number":456,"context_line":"            mock_admin_ctxt.return_value \u003d self.context"},{"line_number":457,"context_line":"            inst_list \u003d _make_instance_list(startup_instances)"},{"line_number":458,"context_line":"            mock_host_get.return_value \u003d inst_list"},{"line_number":459,"context_line":"            self.compute.init_host()"},{"line_number":460,"context_line":"            if defer_iptables_apply:"},{"line_number":461,"context_line":"                self.assertTrue(mock_filter_on.called)"},{"line_number":462,"context_line":"            mock_destroy.assert_called_once_with(self.context)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_4e54d320","line":459,"updated":"2016-06-09 10:03:06.000000000","message":"Its not very clear this is the line of code thats under test here.\n\nMaybe we should add a new line either side of this line?","commit_id":"c00e32ba0e4f04f6cdacf060df96ef8babd06856"},{"author":{"_account_id":19741,"name":"Hieu LE","email":"hieulq2@viettel.com.vn","username":"hieulq"},"change_message_id":"6c37de426904450748274909f5baade1e70450ae","unresolved":false,"context_lines":[{"line_number":456,"context_line":"            mock_admin_ctxt.return_value \u003d self.context"},{"line_number":457,"context_line":"            inst_list \u003d _make_instance_list(startup_instances)"},{"line_number":458,"context_line":"            mock_host_get.return_value \u003d inst_list"},{"line_number":459,"context_line":"            self.compute.init_host()"},{"line_number":460,"context_line":"            if defer_iptables_apply:"},{"line_number":461,"context_line":"                self.assertTrue(mock_filter_on.called)"},{"line_number":462,"context_line":"            mock_destroy.assert_called_once_with(self.context)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7aa08908_54546420","line":459,"in_reply_to":"7aa08908_4e54d320","updated":"2016-06-09 10:39:59.000000000","message":"Done","commit_id":"c00e32ba0e4f04f6cdacf060df96ef8babd06856"}]}