)]}'
{"keystone/tests/unit/common/test_notifications.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"a675af91676cc19785ad34f3f0d981a204293d76","unresolved":false,"context_lines":[{"line_number":28,"context_line":"from keystone.tests import unit"},{"line_number":29,"context_line":"from keystone.tests.unit import test_v3"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"CONF \u003d cfg.CONF"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"EXP_RESOURCE_TYPE \u003d uuid.uuid4().hex"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a2fa921_70ef7b87","side":"PARENT","line":31,"updated":"2015-10-07 20:51:01.000000000","message":"what\u0027s the whitespace change for?","commit_id":"5e92e68431e075998f896a11972cffd4fb1532f0"},{"author":{"_account_id":11387,"name":"Sam Leong","email":"chio-fai-sam.leong@hpe.com","username":"Chioleong"},"change_message_id":"5662c981b97796c2a3798d10b407ba072b89dfbc","unresolved":false,"context_lines":[{"line_number":28,"context_line":"from keystone.tests import unit"},{"line_number":29,"context_line":"from keystone.tests.unit import test_v3"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"CONF \u003d cfg.CONF"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"EXP_RESOURCE_TYPE \u003d uuid.uuid4().hex"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a2fa921_d613b5b6","side":"PARENT","line":31,"in_reply_to":"7a2fa921_70ef7b87","updated":"2015-10-08 21:01:09.000000000","message":"fixed it. accidentally removed :-)","commit_id":"5e92e68431e075998f896a11972cffd4fb1532f0"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"a675af91676cc19785ad34f3f0d981a204293d76","unresolved":false,"context_lines":[{"line_number":644,"context_line":"        super(V2Notifications, self).setUp()"},{"line_number":645,"context_line":"        self.config_fixture.config(notification_format\u003d\u0027cadf\u0027)"},{"line_number":646,"context_line":""},{"line_number":647,"context_line":"    def _assert_initiator_data_is_set(self, action, typeURI):"},{"line_number":648,"context_line":"        self.assertTrue(len(self._audits) \u003e 0)"},{"line_number":649,"context_line":"        audit \u003d self._audits[-1]"},{"line_number":650,"context_line":"        payload \u003d audit[\u0027payload\u0027]"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a2fa921_107407cd","line":647,"updated":"2015-10-07 20:51:01.000000000","message":"is this v2 specific?","commit_id":"02c65da485d20adbbd6076784a1aafa65872360d"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"73f4218c65ee36c19d58419c337ade452ea2f145","unresolved":false,"context_lines":[{"line_number":644,"context_line":"        super(V2Notifications, self).setUp()"},{"line_number":645,"context_line":"        self.config_fixture.config(notification_format\u003d\u0027cadf\u0027)"},{"line_number":646,"context_line":""},{"line_number":647,"context_line":"    def _assert_initiator_data_is_set(self, action, typeURI):"},{"line_number":648,"context_line":"        self.assertTrue(len(self._audits) \u003e 0)"},{"line_number":649,"context_line":"        audit \u003d self._audits[-1]"},{"line_number":650,"context_line":"        payload \u003d audit[\u0027payload\u0027]"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a2fa921_e697518e","line":647,"in_reply_to":"7a2fa921_107407cd","updated":"2015-10-07 21:34:08.000000000","message":"no it\u0027s not, maybe move this to BaseNotificationTest?\n\nthis way we can change lines 635-639 in a follow up patch :)","commit_id":"02c65da485d20adbbd6076784a1aafa65872360d"},{"author":{"_account_id":11387,"name":"Sam Leong","email":"chio-fai-sam.leong@hpe.com","username":"Chioleong"},"change_message_id":"5662c981b97796c2a3798d10b407ba072b89dfbc","unresolved":false,"context_lines":[{"line_number":644,"context_line":"        super(V2Notifications, self).setUp()"},{"line_number":645,"context_line":"        self.config_fixture.config(notification_format\u003d\u0027cadf\u0027)"},{"line_number":646,"context_line":""},{"line_number":647,"context_line":"    def _assert_initiator_data_is_set(self, action, typeURI):"},{"line_number":648,"context_line":"        self.assertTrue(len(self._audits) \u003e 0)"},{"line_number":649,"context_line":"        audit \u003d self._audits[-1]"},{"line_number":650,"context_line":"        payload \u003d audit[\u0027payload\u0027]"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a2fa921_e02707b4","line":647,"in_reply_to":"7a2fa921_e697518e","updated":"2015-10-08 21:01:09.000000000","message":"yep, you got it!","commit_id":"02c65da485d20adbbd6076784a1aafa65872360d"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"73f4218c65ee36c19d58419c337ade452ea2f145","unresolved":false,"context_lines":[{"line_number":667,"context_line":"            },"},{"line_number":668,"context_line":"            token\u003dtoken,"},{"line_number":669,"context_line":"        )"},{"line_number":670,"context_line":"        self._assert_initiator_data_is_set(\u0027created.user\u0027,"},{"line_number":671,"context_line":"                                           cadftaxonomy.SECURITY_ACCOUNT_USER)"},{"line_number":672,"context_line":""},{"line_number":673,"context_line":"    def test_role(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a2fa921_46e165fd","line":670,"updated":"2015-10-07 21:34:08.000000000","message":"being picky here, but can you pass these in as two options? so we can re-use CREATED_OPERATION\n\n  self._assert_initiator_data(CREATED_OPERATION, \u0027user\u0027, cadftaxonomy.SECURITY_ACCOUNT_USER)\n\nand in the function, just set event type like the following:\n\n  action \u003d \u0027%s.%s\u0027 % (operation, resource_type)","commit_id":"02c65da485d20adbbd6076784a1aafa65872360d"},{"author":{"_account_id":11387,"name":"Sam Leong","email":"chio-fai-sam.leong@hpe.com","username":"Chioleong"},"change_message_id":"5662c981b97796c2a3798d10b407ba072b89dfbc","unresolved":false,"context_lines":[{"line_number":667,"context_line":"            },"},{"line_number":668,"context_line":"            token\u003dtoken,"},{"line_number":669,"context_line":"        )"},{"line_number":670,"context_line":"        self._assert_initiator_data_is_set(\u0027created.user\u0027,"},{"line_number":671,"context_line":"                                           cadftaxonomy.SECURITY_ACCOUNT_USER)"},{"line_number":672,"context_line":""},{"line_number":673,"context_line":"    def test_role(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a2fa921_b63dd9de","line":670,"in_reply_to":"7a2fa921_46e165fd","updated":"2015-10-08 21:01:09.000000000","message":"done!","commit_id":"02c65da485d20adbbd6076784a1aafa65872360d"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"73f4218c65ee36c19d58419c337ade452ea2f145","unresolved":false,"context_lines":[{"line_number":703,"context_line":"        self._assert_initiator_data_is_set(\u0027created.service\u0027,"},{"line_number":704,"context_line":"                                           cadftaxonomy.SECURITY_SERVICE)"},{"line_number":705,"context_line":""},{"line_number":706,"context_line":"    def test_endpoint(self):"},{"line_number":707,"context_line":"        token \u003d self.get_scoped_token()"},{"line_number":708,"context_line":"        resp \u003d self.admin_request("},{"line_number":709,"context_line":"            method\u003d\u0027POST\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a2fa921_46ce45b5","line":706,"updated":"2015-10-07 21:34:08.000000000","message":"maybe have 1 test case here?\n\n  test_service_and_endpoint()?\n\nthe code for creating the service is duplicated, and not really necessary","commit_id":"02c65da485d20adbbd6076784a1aafa65872360d"},{"author":{"_account_id":11387,"name":"Sam Leong","email":"chio-fai-sam.leong@hpe.com","username":"Chioleong"},"change_message_id":"5662c981b97796c2a3798d10b407ba072b89dfbc","unresolved":false,"context_lines":[{"line_number":703,"context_line":"        self._assert_initiator_data_is_set(\u0027created.service\u0027,"},{"line_number":704,"context_line":"                                           cadftaxonomy.SECURITY_SERVICE)"},{"line_number":705,"context_line":""},{"line_number":706,"context_line":"    def test_endpoint(self):"},{"line_number":707,"context_line":"        token \u003d self.get_scoped_token()"},{"line_number":708,"context_line":"        resp \u003d self.admin_request("},{"line_number":709,"context_line":"            method\u003d\u0027POST\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a2fa921_38ebf714","line":706,"in_reply_to":"7a2fa921_46ce45b5","updated":"2015-10-08 21:01:09.000000000","message":"done","commit_id":"02c65da485d20adbbd6076784a1aafa65872360d"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"6a3ca880cfce342082e08eab77fbe20b2b28036e","unresolved":false,"context_lines":[{"line_number":647,"context_line":"                                           \u0027domain\u0027,"},{"line_number":648,"context_line":"                                           cadftaxonomy.SECURITY_DOMAIN)"},{"line_number":649,"context_line":""},{"line_number":650,"context_line":"class V2Notifications(BaseNotificationTest):"},{"line_number":651,"context_line":"    def setUp(self):"},{"line_number":652,"context_line":"        super(V2Notifications, self).setUp()"},{"line_number":653,"context_line":"        self.config_fixture.config(notification_format\u003d\u0027cadf\u0027)"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a2fa921_ad049f49","line":650,"updated":"2015-10-08 22:04:05.000000000","message":"need another new line between the classes","commit_id":"51a4edd3c68985107905c8924dd87c8763996740"}]}