)]}'
{"nova/tests/unit/objects/test_security_group.py":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"0ab0e503ea242e7b37f7424a097acb985d0d602c","unresolved":false,"context_lines":[{"line_number":40,"context_line":"        return dict(db_secgroup.items(), deleted\u003dFalse)"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    @mock.patch.object(db, \u0027security_group_get\u0027)"},{"line_number":43,"context_line":"    def test_get(self, get_mock):"},{"line_number":44,"context_line":"        get_mock.return_value \u003d fake_secgroup"},{"line_number":45,"context_line":"        secgroup \u003d security_group.SecurityGroup.get(self.context, 1)"},{"line_number":46,"context_line":"        self.assertEqual(self._fix_deleted(fake_secgroup),"}],"source_content_type":"text/x-python","patch_set":6,"id":"bab6814e_2d315fed","line":43,"range":{"start_line":43,"start_character":23,"end_line":43,"end_character":31},"updated":"2016-05-24 16:15:34.000000000","message":"most of the test uses the naming convention mock_* instead of *_mock","commit_id":"de00bab0c3c4cc25e6c714bf5768b4b7f8552259"},{"author":{"_account_id":15888,"name":"Zhenyu Zheng","email":"zheng.zhenyu@outlook.com","username":"Kevin_Zheng"},"change_message_id":"6117f69040ec49116e1508e94fe2c5b7e5e8f49a","unresolved":false,"context_lines":[{"line_number":40,"context_line":"        return dict(db_secgroup.items(), deleted\u003dFalse)"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    @mock.patch.object(db, \u0027security_group_get\u0027)"},{"line_number":43,"context_line":"    def test_get(self, get_mock):"},{"line_number":44,"context_line":"        get_mock.return_value \u003d fake_secgroup"},{"line_number":45,"context_line":"        secgroup \u003d security_group.SecurityGroup.get(self.context, 1)"},{"line_number":46,"context_line":"        self.assertEqual(self._fix_deleted(fake_secgroup),"}],"source_content_type":"text/x-python","patch_set":6,"id":"bab6814e_252321a9","line":43,"range":{"start_line":43,"start_character":23,"end_line":43,"end_character":31},"in_reply_to":"bab6814e_2d315fed","updated":"2016-05-25 01:17:33.000000000","message":"Done","commit_id":"de00bab0c3c4cc25e6c714bf5768b4b7f8552259"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"0ab0e503ea242e7b37f7424a097acb985d0d602c","unresolved":false,"context_lines":[{"line_number":49,"context_line":"        get_mock.assert_called_once_with(self.context, 1)"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    @mock.patch.object(db, \u0027security_group_get_by_name\u0027)"},{"line_number":52,"context_line":"    def test_get_by_name(self, get_mock):"},{"line_number":53,"context_line":"        get_mock.return_value \u003d fake_secgroup"},{"line_number":54,"context_line":"        secgroup \u003d security_group.SecurityGroup.get_by_name(self.context,"},{"line_number":55,"context_line":"                                                            \u0027fake-project\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"bab6814e_8d9a0bc4","line":52,"range":{"start_line":52,"start_character":31,"end_line":52,"end_character":39},"updated":"2016-05-24 16:15:34.000000000","message":"ditto","commit_id":"de00bab0c3c4cc25e6c714bf5768b4b7f8552259"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"0ab0e503ea242e7b37f7424a097acb985d0d602c","unresolved":false,"context_lines":[{"line_number":62,"context_line":"                                         \u0027fake-name\u0027)"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    @mock.patch.object(db, \u0027security_group_in_use\u0027)"},{"line_number":65,"context_line":"    def test_in_use(self, inuse_mock):"},{"line_number":66,"context_line":"        inuse_mock.return_value \u003d True"},{"line_number":67,"context_line":"        secgroup \u003d security_group.SecurityGroup(context\u003dself.context)"},{"line_number":68,"context_line":"        secgroup.id \u003d 123"}],"source_content_type":"text/x-python","patch_set":6,"id":"bab6814e_cda4130a","line":65,"range":{"start_line":65,"start_character":26,"end_line":65,"end_character":36},"updated":"2016-05-24 16:15:34.000000000","message":"ditto","commit_id":"de00bab0c3c4cc25e6c714bf5768b4b7f8552259"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"4188978d621cfe7854e7eca4b1d4b6256fd229ba","unresolved":false,"context_lines":[{"line_number":89,"context_line":"        secgroup \u003d security_group.SecurityGroup._from_db_object("},{"line_number":90,"context_line":"            self.context, security_group.SecurityGroup(),"},{"line_number":91,"context_line":"            fake_secgroup)"},{"line_number":92,"context_line":"        secgroup.save()"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"    @mock.patch.object(db, \u0027security_group_get\u0027)"},{"line_number":95,"context_line":"    def test_refresh(self, mock_get):"}],"source_content_type":"text/x-python","patch_set":8,"id":"9abb7d3a_2c9a6f6e","line":92,"updated":"2016-06-01 11:17:32.000000000","message":"self.assertFalse(mock_update.called) seems to be needed here.","commit_id":"eebf981ca674dd37e49d49dbc780d1ebf754187e"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"4188978d621cfe7854e7eca4b1d4b6256fd229ba","unresolved":false,"context_lines":[{"line_number":132,"context_line":"            self.assertEqual(fake_secgroups[i][\u0027id\u0027],"},{"line_number":133,"context_line":"                             secgroup_list[i][\u0027id\u0027])"},{"line_number":134,"context_line":"            self.assertEqual(secgroup_list[i]._context, self.context)"},{"line_number":135,"context_line":"            mock_get.assert_called_once_with(self.context)"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"    @mock.patch.object(db, \u0027security_group_get_by_project\u0027,"},{"line_number":138,"context_line":"                       return_value\u003dfake_secgroups)"}],"source_content_type":"text/x-python","patch_set":8,"id":"9abb7d3a_8cd6fb25","line":135,"updated":"2016-06-01 11:17:32.000000000","message":"hmm, its a little odd that works, but I guess its Ok.","commit_id":"eebf981ca674dd37e49d49dbc780d1ebf754187e"}]}