)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9171,"name":"Vipin Balachandran","email":"vipin.bl@gmail.com","username":"vbala"},"change_message_id":"cd959cc669063eaf9a3fd557509e3896fd08a4be","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix calls to assert_not_called() and assert_called_once()"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There is no assert_not_called() and assert_called_once()"},{"line_number":10,"context_line":"method in mock, as described here:"},{"line_number":11,"context_line":"https://code.google.com/p/mock/issues/detail?id\u003d159"},{"line_number":12,"context_line":"Fix these calls to use assertFalse() and"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3aaa91ec_8aedc7f1","line":9,"range":{"start_line":9,"start_character":9,"end_line":9,"end_character":31},"updated":"2016-06-22 15:04:08.000000000","message":"Then how did https://review.openstack.org/#/c/328189/ pass all tests? If this is indeed a problem, there are other places where we call this and should be fixed.\n\nIn fact, I started using assert_not_called after seeing Gorka\u0027s comment here:\nhttps://review.openstack.org/#/c/320686/1/cinder/tests/unit/test_hpelefthand.py","commit_id":"b3576f86a20ea6f02d9254dc9512f28c87dc3004"},{"author":{"_account_id":9171,"name":"Vipin Balachandran","email":"vipin.bl@gmail.com","username":"vbala"},"change_message_id":"cd959cc669063eaf9a3fd557509e3896fd08a4be","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There is no assert_not_called() and assert_called_once()"},{"line_number":10,"context_line":"method in mock, as described here:"},{"line_number":11,"context_line":"https://code.google.com/p/mock/issues/detail?id\u003d159"},{"line_number":12,"context_line":"Fix these calls to use assertFalse() and"},{"line_number":13,"context_line":"assert_called_once_with() attribute instead."},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3aaa91ec_caa4efba","line":11,"range":{"start_line":11,"start_character":0,"end_line":11,"end_character":51},"updated":"2016-06-22 15:04:08.000000000","message":"I think this is not valid.\n\nSee https://github.com/testing-cabal/mock/blob/2.0.0/mock/mock.py#L890\n\nand \n\nhttps://github.com/testing-cabal/mock/blob/2.0.0/mock/mock.py#L908","commit_id":"b3576f86a20ea6f02d9254dc9512f28c87dc3004"},{"author":{"_account_id":16066,"name":"xiexs","email":"xiexs@cn.fujitsu.com","username":"simen"},"change_message_id":"c4326e827bda51b40559dea9ceea4b6b110a9462","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There is no assert_not_called() and assert_called_once()"},{"line_number":10,"context_line":"method in mock, as described here:"},{"line_number":11,"context_line":"https://code.google.com/p/mock/issues/detail?id\u003d159"},{"line_number":12,"context_line":"Fix these calls to use assertFalse() and"},{"line_number":13,"context_line":"assert_called_once_with() attribute instead."},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3aaa91ec_1561eb39","line":11,"range":{"start_line":11,"start_character":0,"end_line":11,"end_character":51},"in_reply_to":"3aaa91ec_caa4efba","updated":"2016-06-22 18:48:03.000000000","message":"Sorry and yes you\u0027re right. It is supported in the newest mock version.\nhttps://docs.python.org/dev/whatsnew/changelog.html\n\nI\u0027ll abandon it. Thanks.","commit_id":"b3576f86a20ea6f02d9254dc9512f28c87dc3004"}]}