)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"7cce6bc17652bf23f34ee197b6a17b8ea67096cb","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Test the NotificationFixture"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch moves the test of the fake_notifier to target"},{"line_number":10,"context_line":"NotificaitonFixture"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ib433a90746f597d9c34d6b43e64478468d6be6e1"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1f621f24_c7266e17","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":19},"updated":"2020-10-28 11:38:06.000000000","message":"Notification","commit_id":"cb2b3427f86b8e56be1376c2d419b4c69a7587d5"}],"nova/tests/unit/test_fake_notifier.py":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"4654cf36095e1f8b57aa6b89700b4a978a3a4aaf","unresolved":true,"context_lines":[{"line_number":15,"context_line":"from nova import context"},{"line_number":16,"context_line":"from nova import exception_wrapper"},{"line_number":17,"context_line":"from nova import test"},{"line_number":18,"context_line":"from nova.tests import fixtures"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"class FakeVersionedNotifierTestCase(test.NoDBTestCase):"}],"source_content_type":"text/x-python","patch_set":5,"id":"4201e3c5_17eab28a","line":18,"updated":"2021-06-09 00:45:24.000000000","message":"It feels odd IMHO to be doing this without moving the content of this file into nova/tests/unit/test_fixtures.py and s/FakeVersionedNotifierTestCase/TestNotificationFixture/\n\nAnd I see that in the next patch you will delete nova/tests/unit/fake_notifier.py and move its content into nova/tests/fixtures/notifications.py, so it seems even more so that this should move, either in this patch or the next one. What do you think?","commit_id":"7a2c28c7111b98145e367e87e6d09b54397c63f2"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"a002ce6e27b8f2985439efcfc2de3d9104b607fe","unresolved":false,"context_lines":[{"line_number":15,"context_line":"from nova import context"},{"line_number":16,"context_line":"from nova import exception_wrapper"},{"line_number":17,"context_line":"from nova import test"},{"line_number":18,"context_line":"from nova.tests import fixtures"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"class FakeVersionedNotifierTestCase(test.NoDBTestCase):"}],"source_content_type":"text/x-python","patch_set":5,"id":"f99ecb89_e4caefbb","line":18,"in_reply_to":"4201e3c5_17eab28a","updated":"2021-06-10 16:10:10.000000000","message":"sure I moved this to the proper place.","commit_id":"7a2c28c7111b98145e367e87e6d09b54397c63f2"}]}