)]}'
{"keystone/tests/unit/common/test_notifications.py":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"dda5c08a88e6cacc0222e659d4efc93f19b6fa39","unresolved":false,"context_lines":[{"line_number":633,"context_line":""},{"line_number":634,"context_line":"    def setUp(self):"},{"line_number":635,"context_line":"        super(CADFNotificationsForEntities, self).setUp()"},{"line_number":636,"context_line":"        self.config_fixture.config(notification_format\u003d\u0027cadf\u0027)"},{"line_number":637,"context_line":""},{"line_number":638,"context_line":"    def test_initiator_data_is_set(self):"},{"line_number":639,"context_line":"        ref \u003d self.new_domain_ref()"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a80dd14_1aed3cf0","line":636,"updated":"2015-03-10 18:14:04.000000000","message":"i\u0027ll bite... why do you test all the above (in NotificationsForEntities)? backward compatibility?\n\nmore importantly, how is this passing now? i would assume [initiator][id] is cadf specific?","commit_id":"eec14955137013fc4b2bbc6b7a8ce933fb1d5c34"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"560e487b56929b6f067be40eed9968ac422df258","unresolved":false,"context_lines":[{"line_number":633,"context_line":""},{"line_number":634,"context_line":"    def setUp(self):"},{"line_number":635,"context_line":"        super(CADFNotificationsForEntities, self).setUp()"},{"line_number":636,"context_line":"        self.config_fixture.config(notification_format\u003d\u0027cadf\u0027)"},{"line_number":637,"context_line":""},{"line_number":638,"context_line":"    def test_initiator_data_is_set(self):"},{"line_number":639,"context_line":"        ref \u003d self.new_domain_ref()"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a80dd14_b9a35d4e","line":636,"in_reply_to":"9a80dd14_1aed3cf0","updated":"2015-03-10 19:04:06.000000000","message":"we test all the above because the deployer can choose to use cadf formatted notifications or the old style (just resource_info). this is handled with a config option. so yeah, kinda for backwards compat.\n\nit\u0027s passing now because self._assert_last_note is designed to verify the payload in the case of the old style, and self._assert_last_audit is designed to verify the payload of a cadf notification. both check the config option that is set before verifying.","commit_id":"eec14955137013fc4b2bbc6b7a8ce933fb1d5c34"}]}