)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"a906fb2d0a53fd2666f43293997195f90d926510","unresolved":true,"context_lines":[{"line_number":1,"context_line":"Parent:     79de1371 (Merge \"identity: Add missing user argument\")"},{"line_number":2,"context_line":"Author:     deveoper \u003cdimsss0607@gmail.com\u003e"},{"line_number":3,"context_line":"AuthorDate: 2025-05-18 14:50:28 +0900"},{"line_number":4,"context_line":"Commit:     djp \u003cdimsss0607@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2025-05-18 06:03:58 +0000"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"54359829_c6a0d2c6","line":2,"updated":"2025-05-19 11:31:05.000000000","message":"If you would like to ensure your name is associated with this change, I would suggest setting your name correctly in Git. You can do this like so:\n\n```\ngit commit --global user.name \u003cyour name here\u003e\n```\n\nWhat you\u0027ve done is fine but I assume you would like credit for your hard work 😊","commit_id":"816f26adea122a081e53c770f84b89f1d5cfdb75"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"fc048f3d6a5f93fea5bdad9dff841c9e3ca705fb","unresolved":false,"context_lines":[{"line_number":1,"context_line":"Parent:     79de1371 (Merge \"identity: Add missing user argument\")"},{"line_number":2,"context_line":"Author:     deveoper \u003cdimsss0607@gmail.com\u003e"},{"line_number":3,"context_line":"AuthorDate: 2025-05-18 14:50:28 +0900"},{"line_number":4,"context_line":"Commit:     djp \u003cdimsss0607@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2025-05-18 06:03:58 +0000"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"f23cb92c_f61a1fb2","line":2,"in_reply_to":"082795ab_0c5dff95","updated":"2025-05-19 12:33:57.000000000","message":"Done","commit_id":"816f26adea122a081e53c770f84b89f1d5cfdb75"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"b5623fb8bd270524161b30824ce5d865b67fdeda","unresolved":true,"context_lines":[{"line_number":1,"context_line":"Parent:     79de1371 (Merge \"identity: Add missing user argument\")"},{"line_number":2,"context_line":"Author:     deveoper \u003cdimsss0607@gmail.com\u003e"},{"line_number":3,"context_line":"AuthorDate: 2025-05-18 14:50:28 +0900"},{"line_number":4,"context_line":"Commit:     djp \u003cdimsss0607@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2025-05-18 06:03:58 +0000"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"082795ab_0c5dff95","line":2,"in_reply_to":"54359829_c6a0d2c6","updated":"2025-05-19 11:32:43.000000000","message":"Oh, and once you\u0027ve done that, you can update this field by doing the following (with this change checked out):\n\n```\ngit commit --amend --reset-author\n```","commit_id":"816f26adea122a081e53c770f84b89f1d5cfdb75"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"a906fb2d0a53fd2666f43293997195f90d926510","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     djp \u003cdimsss0607@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2025-05-18 06:03:58 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Replace assignment of `Mock` objects to methods that are already mocked in test_image.py"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Replace assignment of `Mock` objects to methods that are already mocked"},{"line_number":10,"context_line":"in tests/unit/v1/test_image.py, tests/unit/v2/test_image.py"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"4a153e2e_56185824","line":7,"updated":"2025-05-19 11:31:05.000000000","message":"This is a little verbose. How about something simpler?\n\n```\ntests: Simplify mocking in image tests\n```","commit_id":"816f26adea122a081e53c770f84b89f1d5cfdb75"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"fc048f3d6a5f93fea5bdad9dff841c9e3ca705fb","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     djp \u003cdimsss0607@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2025-05-18 06:03:58 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Replace assignment of `Mock` objects to methods that are already mocked in test_image.py"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Replace assignment of `Mock` objects to methods that are already mocked"},{"line_number":10,"context_line":"in tests/unit/v1/test_image.py, tests/unit/v2/test_image.py"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"eba49b22_ad01fcfe","line":7,"in_reply_to":"3a6d73a2_d1970530","updated":"2025-05-19 12:33:57.000000000","message":"Done","commit_id":"816f26adea122a081e53c770f84b89f1d5cfdb75"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8700ef69e513bda3a910634efbab206bde5963ac","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     djp \u003cdimsss0607@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2025-05-18 06:03:58 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Replace assignment of `Mock` objects to methods that are already mocked in test_image.py"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Replace assignment of `Mock` objects to methods that are already mocked"},{"line_number":10,"context_line":"in tests/unit/v1/test_image.py, tests/unit/v2/test_image.py"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3a6d73a2_d1970530","line":7,"in_reply_to":"4a153e2e_56185824","updated":"2025-05-19 12:33:19.000000000","message":"\u003e This is a little verbose. How about something simpler?\n\u003e \n\u003e ```\n\u003e tests: Simplify mocking in image tests\n\u003e ```","commit_id":"816f26adea122a081e53c770f84b89f1d5cfdb75"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8700ef69e513bda3a910634efbab206bde5963ac","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"7a00c3f6_6bf86f4b","updated":"2025-05-19 12:33:19.000000000","message":"This is looking better. You just missed one. Also, please mark comments as Done when you have addressed them, please and thank you 😊","commit_id":"ee129a340eb7a9db2f3c51d89d02130da2574260"}],"openstackclient/tests/unit/image/v1/test_image.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"9c7fcc9d72824553eea6408d98f3f6a29d8baaee","unresolved":true,"context_lines":[{"line_number":54,"context_line":"    def setUp(self):"},{"line_number":55,"context_line":"        super().setUp()"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"        self.image_client.create_image \u003d mock.Mock(return_value\u003dself.new_image)"},{"line_number":58,"context_line":"        self.image_client.find_image.return_value \u003d self.new_image"},{"line_number":59,"context_line":"        self.image_client.update_image \u003d mock.Mock(return_image\u003dself.new_image)"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"        # Get the command object to test"},{"line_number":62,"context_line":"        self.cmd \u003d image.CreateImage(self.app, None)"}],"source_content_type":"text/x-python","patch_set":2,"id":"69f85238_665e9fd5","line":59,"range":{"start_line":57,"start_character":0,"end_line":59,"end_character":79},"updated":"2025-05-19 09:54:05.000000000","message":"Could you fix all of these while you\u0027re here?\n\n```suggestion\n        self.image_client.create_image.return_value \u003d self.new_image\n        self.image_client.find_image.return_value \u003d self.new_image\n        self.image_client.update_image.return_image \u003d self.new_image\n```\n\nditto for the other calls below","commit_id":"816f26adea122a081e53c770f84b89f1d5cfdb75"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8700ef69e513bda3a910634efbab206bde5963ac","unresolved":false,"context_lines":[{"line_number":54,"context_line":"    def setUp(self):"},{"line_number":55,"context_line":"        super().setUp()"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"        self.image_client.create_image \u003d mock.Mock(return_value\u003dself.new_image)"},{"line_number":58,"context_line":"        self.image_client.find_image.return_value \u003d self.new_image"},{"line_number":59,"context_line":"        self.image_client.update_image \u003d mock.Mock(return_image\u003dself.new_image)"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"        # Get the command object to test"},{"line_number":62,"context_line":"        self.cmd \u003d image.CreateImage(self.app, None)"}],"source_content_type":"text/x-python","patch_set":2,"id":"46c9a98f_ff0322b9","line":59,"range":{"start_line":57,"start_character":0,"end_line":59,"end_character":79},"in_reply_to":"69f85238_665e9fd5","updated":"2025-05-19 12:33:19.000000000","message":"Done","commit_id":"816f26adea122a081e53c770f84b89f1d5cfdb75"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"9c7fcc9d72824553eea6408d98f3f6a29d8baaee","unresolved":true,"context_lines":[{"line_number":211,"context_line":"    def setUp(self):"},{"line_number":212,"context_line":"        super().setUp()"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"        # This is the return value for utils.find_resource()"},{"line_number":215,"context_line":"        self.image_client.find_image.return_value \u003d self._image"},{"line_number":216,"context_line":"        self.image_client.delete_image \u003d mock.Mock(return_value\u003dNone)"},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"        # Get the command object to test"},{"line_number":219,"context_line":"        self.cmd \u003d image.DeleteImage(self.app, None)"}],"source_content_type":"text/x-python","patch_set":2,"id":"89760a32_42304a25","line":216,"range":{"start_line":214,"start_character":0,"end_line":216,"end_character":69},"updated":"2025-05-19 09:54:05.000000000","message":"```suggestion\n        self.image_client.find_image.return_value \u003d self._image\n        self.image_client.delete_image.return_value \u003d None\n```","commit_id":"816f26adea122a081e53c770f84b89f1d5cfdb75"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8700ef69e513bda3a910634efbab206bde5963ac","unresolved":false,"context_lines":[{"line_number":211,"context_line":"    def setUp(self):"},{"line_number":212,"context_line":"        super().setUp()"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"        # This is the return value for utils.find_resource()"},{"line_number":215,"context_line":"        self.image_client.find_image.return_value \u003d self._image"},{"line_number":216,"context_line":"        self.image_client.delete_image \u003d mock.Mock(return_value\u003dNone)"},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"        # Get the command object to test"},{"line_number":219,"context_line":"        self.cmd \u003d image.DeleteImage(self.app, None)"}],"source_content_type":"text/x-python","patch_set":2,"id":"bbee1dea_5dab716d","line":216,"range":{"start_line":214,"start_character":0,"end_line":216,"end_character":69},"in_reply_to":"89760a32_42304a25","updated":"2025-05-19 12:33:19.000000000","message":"Done","commit_id":"816f26adea122a081e53c770f84b89f1d5cfdb75"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"9c7fcc9d72824553eea6408d98f3f6a29d8baaee","unresolved":true,"context_lines":[{"line_number":440,"context_line":"    def setUp(self):"},{"line_number":441,"context_line":"        super().setUp()"},{"line_number":442,"context_line":""},{"line_number":443,"context_line":"        # This is the return value for utils.find_resource()"},{"line_number":444,"context_line":"        self.image_client.find_image.return_value \u003d self._image"},{"line_number":445,"context_line":"        self.image_client.update_image \u003d mock.Mock(return_value\u003dself._image)"},{"line_number":446,"context_line":""},{"line_number":447,"context_line":"        # Get the command object to test"},{"line_number":448,"context_line":"        self.cmd \u003d image.SetImage(self.app, None)"}],"source_content_type":"text/x-python","patch_set":2,"id":"a9037fec_7415c0d0","line":445,"range":{"start_line":443,"start_character":0,"end_line":445,"end_character":76},"updated":"2025-05-19 09:54:05.000000000","message":"```suggestion\n        self.image_client.find_image.return_value \u003d self._image\n        self.image_client.update_image.return_value \u003d self._image\n```","commit_id":"816f26adea122a081e53c770f84b89f1d5cfdb75"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8700ef69e513bda3a910634efbab206bde5963ac","unresolved":false,"context_lines":[{"line_number":440,"context_line":"    def setUp(self):"},{"line_number":441,"context_line":"        super().setUp()"},{"line_number":442,"context_line":""},{"line_number":443,"context_line":"        # This is the return value for utils.find_resource()"},{"line_number":444,"context_line":"        self.image_client.find_image.return_value \u003d self._image"},{"line_number":445,"context_line":"        self.image_client.update_image \u003d mock.Mock(return_value\u003dself._image)"},{"line_number":446,"context_line":""},{"line_number":447,"context_line":"        # Get the command object to test"},{"line_number":448,"context_line":"        self.cmd \u003d image.SetImage(self.app, None)"}],"source_content_type":"text/x-python","patch_set":2,"id":"32d4d0ff_59115284","line":445,"range":{"start_line":443,"start_character":0,"end_line":445,"end_character":76},"in_reply_to":"a9037fec_7415c0d0","updated":"2025-05-19 12:33:19.000000000","message":"Done","commit_id":"816f26adea122a081e53c770f84b89f1d5cfdb75"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8700ef69e513bda3a910634efbab206bde5963ac","unresolved":true,"context_lines":[{"line_number":261,"context_line":"    def setUp(self):"},{"line_number":262,"context_line":"        super().setUp()"},{"line_number":263,"context_line":""},{"line_number":264,"context_line":"        self.image_client.images \u003d mock.Mock()"},{"line_number":265,"context_line":"        self.image_client.images.side_effect \u003d ["},{"line_number":266,"context_line":"            [self._image],"},{"line_number":267,"context_line":"            [],"}],"source_content_type":"text/x-python","patch_set":5,"id":"dea43514_1b73be9d","line":264,"updated":"2025-05-19 12:33:19.000000000","message":"You missed one","commit_id":"ee129a340eb7a9db2f3c51d89d02130da2574260"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"db40e846a1273f3f4d56b2dcc60c759ffe6fba7c","unresolved":false,"context_lines":[{"line_number":261,"context_line":"    def setUp(self):"},{"line_number":262,"context_line":"        super().setUp()"},{"line_number":263,"context_line":""},{"line_number":264,"context_line":"        self.image_client.images \u003d mock.Mock()"},{"line_number":265,"context_line":"        self.image_client.images.side_effect \u003d ["},{"line_number":266,"context_line":"            [self._image],"},{"line_number":267,"context_line":"            [],"}],"source_content_type":"text/x-python","patch_set":5,"id":"f3d03717_0fab8898","line":264,"in_reply_to":"dea43514_1b73be9d","updated":"2025-05-19 15:01:11.000000000","message":"Done","commit_id":"ee129a340eb7a9db2f3c51d89d02130da2574260"}]}