)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":18337,"name":"Sujitha","email":"sujitha.neti@intel.com","username":"Sujitha"},"change_message_id":"d1a67a106d06acf245646614fe757e37b014f765","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Not using mox stubs in nova.tests.unit.consoleauth"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"remove stubs with newly introduced stub_out"},{"line_number":10,"context_line":"function also use mock to replace old mox in consoleauth subfolder."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Part of blueprint remove-mox-newton"},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1aa78d24_64949542","line":10,"updated":"2016-07-06 21:22:11.000000000","message":"stub_out method is not being used to replace stubs.set here.\n\nPlease change the commit message or use stub_out to replace.","commit_id":"160bfc56662bdc0cbc2cc8af45f836cd7a644c17"}],"nova/tests/unit/consoleauth/test_consoleauth.py":[{"author":{"_account_id":18602,"name":"Gábor Antal","email":"antal@inf.u-szeged.hu","username":"antal"},"change_message_id":"a670a7b9afaa7584e71a293200f7220f2560a859","unresolved":false,"context_lines":[{"line_number":61,"context_line":"        timeutils.advance_time_seconds(1)"},{"line_number":62,"context_line":"        self.assertIsNone(self.manager_api.check_token(self.context, token))"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    def _stub_validate_console_port(self, result):"},{"line_number":65,"context_line":"        self.manager.compute_rpcapi.validate_console_port \u003d ("},{"line_number":66,"context_line":"            mock.Mock(return_value\u003dresult))"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"    @mock.patch(\u0027nova.objects.instance.Instance.get_by_uuid\u0027)"},{"line_number":69,"context_line":"    def test_multiple_tokens_for_instance(self, mock_get):"}],"source_content_type":"text/x-python","patch_set":3,"id":"bacf61ea_2cfde23a","line":66,"range":{"start_line":64,"start_character":0,"end_line":66,"end_character":43},"updated":"2016-07-28 13:46:26.000000000","message":"Why it is needed here?","commit_id":"160bfc56662bdc0cbc2cc8af45f836cd7a644c17"},{"author":{"_account_id":18602,"name":"Gábor Antal","email":"antal@inf.u-szeged.hu","username":"antal"},"change_message_id":"a670a7b9afaa7584e71a293200f7220f2560a859","unresolved":false,"context_lines":[{"line_number":172,"context_line":"                    [mock.call(\u0027instance\u0027, mock.ANY)])"},{"line_number":173,"context_line":""},{"line_number":174,"context_line":"    def test_check_token_encoding(self):"},{"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(test.MatchType(str))"}],"source_content_type":"text/x-python","patch_set":3,"id":"bacf61ea_6cc7da12","line":175,"range":{"start_line":175,"start_character":8,"end_line":175,"end_character":67},"updated":"2016-07-28 13:46:26.000000000","message":"I think it would be more readable if you would use decorator patching","commit_id":"160bfc56662bdc0cbc2cc8af45f836cd7a644c17"}]}