)]}'
{"nova/tests/compute/test_compute_mgr.py":[{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"ed4d9e367e0d861d4480ab407a7a9db5ab6cf448","unresolved":false,"context_lines":[{"line_number":570,"context_line":"            get_nw_info_for_instance"},{"line_number":571,"context_line":"          ):"},{"line_number":572,"context_line":"            self.compute._init_instance(self.context, instance)"},{"line_number":573,"context_line":"            instance_save.assert_called_once()"},{"line_number":574,"context_line":"            self.assertEqual(None, instance.task_state)"},{"line_number":575,"context_line":"            self.assertEqual(vm_states.ACTIVE, instance.vm_state)"},{"line_number":576,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAXH%2F%2FvUw%3D","line":573,"updated":"2014-04-17 19:43:15.000000000","message":"assert_callled_once() is not a method of mock.Mock() :) The above is actually just calling a stubbed magic mock method.\n\nShould be:\n\n instance_save.assert_called_once_with()","commit_id":"8273cdb5cfcfb5537a9afe73dadad0e4e356df79"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"7fd3838dca220123ab21ad95d8acd118217923e0","unresolved":false,"context_lines":[{"line_number":570,"context_line":"            get_nw_info_for_instance"},{"line_number":571,"context_line":"          ):"},{"line_number":572,"context_line":"            self.compute._init_instance(self.context, instance)"},{"line_number":573,"context_line":"            instance_save.assert_called_once()"},{"line_number":574,"context_line":"            self.assertEqual(None, instance.task_state)"},{"line_number":575,"context_line":"            self.assertEqual(vm_states.ACTIVE, instance.vm_state)"},{"line_number":576,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAXH%2F%2FvJk%3D","line":573,"in_reply_to":"AAAAXH%2F%2FvUw%3D","updated":"2014-04-17 20:07:38.000000000","message":"gah, I just caught this same thing in another review last week.  Good catch :)","commit_id":"8273cdb5cfcfb5537a9afe73dadad0e4e356df79"}]}
