)]}'
{"nova/tests/unit/consoleauth/test_consoleauth.py":[{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"25a764ff6d0b34b460ddf38b824a2108f63ac2f4","unresolved":false,"context_lines":[{"line_number":155,"context_line":"                                           self.u_instance)"},{"line_number":156,"context_line":"            mock_get.assert_called_once_with(mock.ANY)"},{"line_number":157,"context_line":"            expected_calls \u003d [mock.call(mock.ANY, mock.ANY, mock.ANY),"},{"line_number":158,"context_line":"                              mock.call(mock.ANY, mock.ANY)]"},{"line_number":159,"context_line":"            mock_set.assert_has_calls(expected_calls)"},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"    def test_check_token_encoding(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a68dd71_d3428e67","line":158,"updated":"2016-01-25 21:28:48.000000000","message":"I\u0027m not sure I understand why this is a useful test at this point (or before even), as it\u0027s so mocked out. It seems like we should know at least what params should match up, right?","commit_id":"fdef2e54ca5e0d758756e74cd6c03856f83878f7"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"17a9471c6135413ae412f382b3cd6cd7f373c49f","unresolved":false,"context_lines":[{"line_number":155,"context_line":"                                           self.u_instance)"},{"line_number":156,"context_line":"            mock_get.assert_called_once_with(mock.ANY)"},{"line_number":157,"context_line":"            expected_calls \u003d [mock.call(mock.ANY, mock.ANY, mock.ANY),"},{"line_number":158,"context_line":"                              mock.call(mock.ANY, mock.ANY)]"},{"line_number":159,"context_line":"            mock_set.assert_has_calls(expected_calls)"},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"    def test_check_token_encoding(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a68dd71_a22bcf93","line":158,"in_reply_to":"9a68dd71_d3428e67","updated":"2016-01-26 19:35:51.000000000","message":"I guess they are just check how many arguments are used to do the call, I can add check for the string ...thanks","commit_id":"fdef2e54ca5e0d758756e74cd6c03856f83878f7"},{"author":{"_account_id":12299,"name":"Paweł Koniszewski","email":"pawel.koniszewski@gmail.com","username":"pawel.koniszewski"},"change_message_id":"e5e2a7ed456dc40b7e189130b176fd80c696974c","unresolved":false,"context_lines":[{"line_number":175,"context_line":"        with mock.patch.object(self.manager.mc, \u0027get\u0027) as mock_get:"},{"line_number":176,"context_line":"            mock_get.return_value \u003d None"},{"line_number":177,"context_line":"            self.manager.check_token(self.context, self.u_token)"},{"line_number":178,"context_line":"            mock_get.assert_called_once_with(mock.ANY)"},{"line_number":179,"context_line":""},{"line_number":180,"context_line":"    def test_delete_tokens_for_instance_encoding(self):"},{"line_number":181,"context_line":"        with test.nested("}],"source_content_type":"text/x-python","patch_set":3,"id":"dae33548_c3de793e","line":178,"range":{"start_line":178,"start_character":45,"end_line":178,"end_character":53},"updated":"2016-02-16 18:33:32.000000000","message":"You replaced mox.IsA(str) with mock.ANY which is a lot weaker. You should use nova.test.MatchType(type) there","commit_id":"9081ede62687af79acfc7531fecbdc46f122cc80"}]}