)]}'
{"nova/tests/unit/virt/libvirt/test_driver.py":[{"author":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"change_message_id":"6f98b58c58f80bf2ed8fd4df6675fedabb6da5e4","unresolved":false,"context_lines":[{"line_number":8190,"context_line":""},{"line_number":8191,"context_line":"        instance \u003d objects.Instance(self.context, **self.test_instance)"},{"line_number":8192,"context_line":"        drvr.destroy(self.context, instance, {})"},{"line_number":8193,"context_line":"        mock_save.assert_called_once_with()"},{"line_number":8194,"context_line":""},{"line_number":8195,"context_line":"    def _test_destroy_removes_disk(self, volume_fail\u003dFalse):"},{"line_number":8196,"context_line":"        instance \u003d objects.Instance(self.context, **self.test_instance)"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_422673cf","line":8193,"updated":"2015-02-11 00:21:06.000000000","message":"assert_called_once_with() looks a bit odd since save does not have any parameters","commit_id":"9dafd17c8208a507d97c5aa3ca9f22062b521f07"},{"author":{"_account_id":642,"name":"Matt Dietz","email":"matthew.dietz@gmail.com","username":"cerberus"},"change_message_id":"5c00f4253ac5f351e962a936341ccf99c2563900","unresolved":false,"context_lines":[{"line_number":8190,"context_line":""},{"line_number":8191,"context_line":"        instance \u003d objects.Instance(self.context, **self.test_instance)"},{"line_number":8192,"context_line":"        drvr.destroy(self.context, instance, {})"},{"line_number":8193,"context_line":"        mock_save.assert_called_once_with()"},{"line_number":8194,"context_line":""},{"line_number":8195,"context_line":"    def _test_destroy_removes_disk(self, volume_fail\u003dFalse):"},{"line_number":8196,"context_line":"        instance \u003d objects.Instance(self.context, **self.test_instance)"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_d7b5a332","line":8193,"in_reply_to":"da86d52c_1b5ccf18","updated":"2015-02-11 16:50:02.000000000","message":"And even then, I\u0027d say you want both the call count and the assert_called_once_with to show it was called with no parameters","commit_id":"9dafd17c8208a507d97c5aa3ca9f22062b521f07"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"6ac8630edb2cbfa176bb1a5382d67419c22370a6","unresolved":false,"context_lines":[{"line_number":8190,"context_line":""},{"line_number":8191,"context_line":"        instance \u003d objects.Instance(self.context, **self.test_instance)"},{"line_number":8192,"context_line":"        drvr.destroy(self.context, instance, {})"},{"line_number":8193,"context_line":"        mock_save.assert_called_once_with()"},{"line_number":8194,"context_line":""},{"line_number":8195,"context_line":"    def _test_destroy_removes_disk(self, volume_fail\u003dFalse):"},{"line_number":8196,"context_line":"        instance \u003d objects.Instance(self.context, **self.test_instance)"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_1b5ccf18","line":8193,"in_reply_to":"da86d52c_422673cf","updated":"2015-02-11 14:44:32.000000000","message":"Yeah, but the alternative is something like:\n\nself.assertEqual(1, mock_save.call_count)","commit_id":"9dafd17c8208a507d97c5aa3ca9f22062b521f07"}]}