)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"change_message_id":"f9b1f87c3afee77852cd038bb65f1d5177ef03cc","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Hu Yupeng \u003chuyp@inspur.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-07-11 18:30:27 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fixes Bug1600725"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: If5d84d3beb42fb57f31e2759bf8572863fdd161f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1aa78d24_3377c73b","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":16},"updated":"2016-07-11 10:48:23.000000000","message":"please describe here the change, not bug number it fixes","commit_id":"50b23d1fbd759226a15b21b07e4b221b6cc37fb0"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"5473e3ed0af326e66b9390fde18d79c6161c7b16","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Hu Yupeng \u003chuyp@inspur.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-07-11 18:30:27 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fixes Bug1600725"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: If5d84d3beb42fb57f31e2759bf8572863fdd161f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1aa78d24_f3a37f9c","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":16},"updated":"2016-07-11 10:42:33.000000000","message":"the commit message should contain some details about the review. Please refer [1] for more details.\n\nAlso, check the comment left in the bug report.\n\n[1] https://wiki.openstack.org/wiki/GitCommitMessages","commit_id":"50b23d1fbd759226a15b21b07e4b221b6cc37fb0"},{"author":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"change_message_id":"f9b1f87c3afee77852cd038bb65f1d5177ef03cc","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2016-07-11 18:30:27 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fixes Bug1600725"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: If5d84d3beb42fb57f31e2759bf8572863fdd161f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1aa78d24_9385db08","line":8,"updated":"2016-07-11 10:48:23.000000000","message":"it should be\nCloses-Bug: 1600725","commit_id":"50b23d1fbd759226a15b21b07e4b221b6cc37fb0"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"5473e3ed0af326e66b9390fde18d79c6161c7b16","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2016-07-11 18:30:27 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fixes Bug1600725"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: If5d84d3beb42fb57f31e2759bf8572863fdd161f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1aa78d24_33c5073c","line":8,"updated":"2016-07-11 10:42:33.000000000","message":"the commit messages from reviews that are intended to close a bug should have:\n\n  Closes-Bug: ######\n\nIn this specific case, something like:\n\n  Closes-Bug: 1600725\n\nRefer the link in the comment above for more details","commit_id":"50b23d1fbd759226a15b21b07e4b221b6cc37fb0"}],"keystone/tests/unit/identity/test_controllers.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"9037ec8f29d45e715c53051b64d75893111dbf7d","unresolved":false,"context_lines":[{"line_number":65,"context_line":"        # If the above doesn\u0027t fail then this is successful."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"class GroupV3TestCase(unit.TestCase):"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"    def setUp(self):"},{"line_number":71,"context_line":"        super(GroupV3TestCase, self).setUp()"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_9bab201e","line":68,"updated":"2016-07-11 15:42:03.000000000","message":"This seems like something we should put in https://github.com/openstack/keystone/blob/master/keystone/tests/unit/test_v3_identity.py","commit_id":"50b23d1fbd759226a15b21b07e4b221b6cc37fb0"},{"author":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"change_message_id":"f9b1f87c3afee77852cd038bb65f1d5177ef03cc","unresolved":false,"context_lines":[{"line_number":85,"context_line":"        self.assertRaises("},{"line_number":86,"context_line":"            exception.GroupNotFound,"},{"line_number":87,"context_line":"            self.groupv3_controller.get_group,"},{"line_number":88,"context_line":"            self.make_request(is_admin\u003dTrue), group_id)"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"    def test_update_group(self):"},{"line_number":91,"context_line":"        group_id \u003d uuid.uuid4().hex"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_93a2fb49","line":88,"updated":"2016-07-11 10:48:23.000000000","message":"this is a negative test case. There should be a test showing that getting the group works.","commit_id":"50b23d1fbd759226a15b21b07e4b221b6cc37fb0"},{"author":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"change_message_id":"f9b1f87c3afee77852cd038bb65f1d5177ef03cc","unresolved":false,"context_lines":[{"line_number":93,"context_line":"        self.assertRaises("},{"line_number":94,"context_line":"            exception.GroupNotFound,"},{"line_number":95,"context_line":"            self.groupv3_controller.update_group,"},{"line_number":96,"context_line":"            self.make_request(is_admin\u003dTrue), group_id, group)"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"    def test_delete_group(self):"},{"line_number":99,"context_line":"        group_id \u003d uuid.uuid4().hex"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_d3abe31c","line":96,"updated":"2016-07-11 10:48:23.000000000","message":"same here. There should be a test showing that updating the group works","commit_id":"50b23d1fbd759226a15b21b07e4b221b6cc37fb0"},{"author":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"change_message_id":"f9b1f87c3afee77852cd038bb65f1d5177ef03cc","unresolved":false,"context_lines":[{"line_number":100,"context_line":"        self.assertRaises("},{"line_number":101,"context_line":"            exception.GroupNotFound,"},{"line_number":102,"context_line":"            self.groupv3_controller.delete_group,"},{"line_number":103,"context_line":"            self.make_request(is_admin\u003dTrue), group_id)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_93703bde","line":103,"updated":"2016-07-11 10:48:23.000000000","message":"same here. There should be a test showing that deleting the group works.","commit_id":"50b23d1fbd759226a15b21b07e4b221b6cc37fb0"}]}