)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"cb3d1ae0dacec14d5df32bf7c4c3f51423ff5cbb","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2015-07-22 16:29:30 -0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"NotificationsTestCase running in isolation"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"They were excluded lines in setUp:"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"-fixture \u003d self.useFixture (config_fixture.Config (CONF))"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a50d1a3_42335dab","line":8,"updated":"2015-07-22 19:52:42.000000000","message":"You don\u0027t need to copy the lines you deleted here, but I\u0027d like to see an explanation of why they were causing the failure.","commit_id":"67b90351a9e54e76c2a4b3a3c92e2591445dd1da"},{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"cb3d1ae0dacec14d5df32bf7c4c3f51423ff5cbb","unresolved":false,"context_lines":[{"line_number":11,"context_line":"-fixture \u003d self.useFixture (config_fixture.Config (CONF))"},{"line_number":12,"context_line":"-fixture.config (rpc_backend \u003d \u0027fake\u0027, notification_driver \u003d [\u0027fake\u0027])"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"correcting the test \"test_send_notification\" in NotificationsTestCase class and not changing the behavior of other tests when it runs in conjunction with the file too \"test_notifications.py\""},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I730dff62470b20875b6237d661145d53a734cc45"},{"line_number":17,"context_line":"Closes-Bug: 1474490"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a50d1a3_82b96517","line":14,"updated":"2015-07-22 19:52:42.000000000","message":"This line is too long. See https://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure for reference. On a side note I\u0027m not sure it\u0027s valuable if you address my comment above.","commit_id":"67b90351a9e54e76c2a4b3a3c92e2591445dd1da"},{"author":{"_account_id":13055,"name":"Alexander Makarov","email":"ajiekcahdp.makapob@gmail.com","username":"amakarov"},"change_message_id":"c265c454631278530a4e59943c863ed867e24d75","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"NotificationsTestCase running in isolation"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Calls to the object \"fixture\" were removed because they are not being used by the class."},{"line_number":10,"context_line":"And when alone run the test \"send_notification\" the fixture is not configured correctly,"},{"line_number":11,"context_line":"it does not have the attributes \"rpc_backend\" and \"notification_driver\";"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1a4dcd0f_3241e438","line":9,"updated":"2015-08-05 20:22:24.000000000","message":"Please adjust this text to have lines \u003c 80 chars","commit_id":"dba3abfe475ccbb640562c50dd4a6d269a7fab2e"},{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"9bc83293152363bc34f661a9ce3a0ffbb95bbf2e","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"NotificationsTestCase running in isolation"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Calls to the object \"fixture\" were removed because they are not being used by the class."},{"line_number":10,"context_line":"And when alone run the test \"send_notification\" the fixture is not configured correctly,"},{"line_number":11,"context_line":"it does not have the attributes \"rpc_backend\" and \"notification_driver\";"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1a4dcd0f_65eae7dc","line":9,"in_reply_to":"1a4dcd0f_3241e438","updated":"2015-08-07 15:04:09.000000000","message":"The commit message should wrap at 72 characters. The summary line should be 50 or under.\n\nhttps://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"dba3abfe475ccbb640562c50dd4a6d269a7fab2e"}],"keystone/tests/unit/common/test_notifications.py":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"73b1344766f81cbf05a11c5c29dc2aaddd24e12f","unresolved":false,"context_lines":[{"line_number":175,"context_line":""},{"line_number":176,"context_line":"class NotificationsTestCase(unit.BaseTestCase):"},{"line_number":177,"context_line":"    def setUp(self):"},{"line_number":178,"context_line":"        super(NotificationsTestCase, self).setUp()"},{"line_number":179,"context_line":""},{"line_number":180,"context_line":"    def test_send_notification(self):"},{"line_number":181,"context_line":"        \"\"\"Test the private method _send_notification to ensure event_type,"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a4dcd0f_85a13396","line":178,"updated":"2015-08-07 15:07:12.000000000","message":"this whole method should be removed since it\u0027s not doing anything anymore.","commit_id":"2df5833c910aafca1e39cb871db8f997eeaad56b"},{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"a0513b8ffbc36226a07c29ba65b9b9f1bebb6120","unresolved":false,"context_lines":[{"line_number":175,"context_line":""},{"line_number":176,"context_line":"class NotificationsTestCase(unit.BaseTestCase):"},{"line_number":177,"context_line":"    def setUp(self):"},{"line_number":178,"context_line":"        super(NotificationsTestCase, self).setUp()"},{"line_number":179,"context_line":""},{"line_number":180,"context_line":"    def test_send_notification(self):"},{"line_number":181,"context_line":"        \"\"\"Test the private method _send_notification to ensure event_type,"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a4dcd0f_c50dfb8f","line":178,"in_reply_to":"1a4dcd0f_85a13396","updated":"2015-08-07 15:10:50.000000000","message":"Done","commit_id":"2df5833c910aafca1e39cb871db8f997eeaad56b"}]}