)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"c7c9e71691c1f7aac2e48a0ca1671bec51d03a3b","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"tests: Clean up flavors tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Ahead of adding additional tests."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I2cac4cc79288f7b3bacc4a63a1d36d4cf12013d7"},{"line_number":12,"context_line":"Signed-off-by: Stephen Finucane \u003cstephenfin@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ac32ef23_de77fd06","line":9,"updated":"2025-08-28 11:18:35.000000000","message":"with would have been nice to summerise the changes\nin this commit\n\n\n```\n\ntests: Refactor flavor API tests for\n  improved maintainability\n\n  Modernize flavor test classes by:\n  - Replacing complex HTTP request\n  simulation with direct controller calls\n  - Updating to modern Python super() syntax\n  - Consolidating imports and removing\n  unused code\n  - Standardizing test class naming\n  conventions (V2_55 -\u003e V255)\n  - Simplifying test helper methods and \n  assertions\n\n  This cleanup reduces technical debt and \n  improves test readability\n  without changing test coverage or \n  behavior.\n\n  Change-Id:I2cac4cc79288f7b3bacc4a63a1d36d4cf12013d7\n  Signed-off-by: Stephen Finucane \u003cstephenfin@redhat.com\u003e\n```\n\nclaude summereises it as that. which i tend to agree with\n\nif you update can you incorrperate some of all of this into the commit message.\n\nit certenly makes reviewing it easier.","commit_id":"851887ef2570d6e83c0b1ff26bf6e1ca6202a2a4"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8e0b07bc813fbb1c8422fe5c34a2243e27fcfb29","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"tests: Clean up flavors tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Ahead of adding additional tests."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I2cac4cc79288f7b3bacc4a63a1d36d4cf12013d7"},{"line_number":12,"context_line":"Signed-off-by: Stephen Finucane \u003cstephenfin@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"4a6dc5c3_d73fe289","line":9,"in_reply_to":"4fd369bc_5f294216","updated":"2025-09-01 08:58:41.000000000","message":"Done","commit_id":"851887ef2570d6e83c0b1ff26bf6e1ca6202a2a4"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f2fa895a12929a2a92092b22bdff566808e1d131","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"tests: Clean up flavors tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Ahead of adding additional tests."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I2cac4cc79288f7b3bacc4a63a1d36d4cf12013d7"},{"line_number":12,"context_line":"Signed-off-by: Stephen Finucane \u003cstephenfin@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"4fd369bc_5f294216","line":9,"in_reply_to":"87d77141_f190bbaa","updated":"2025-08-28 16:17:53.000000000","message":"to be clear i think stephen is actully demonstarting a very good progarming practice here.\n\nthey are about to modify something so they took the time to first clean up some technical debt in the code they are about to change and submitted that as a seprate cleanup patch first. then they are actully doing the code change later keep the refactoring sepreat form the functional change.\n\nso we shoudl not discusage things like this.\n\nnote also that stephen didnt as for review on this series, i did because i tought it woudl be nice to have given its small and will avoid havign to reappove the spec again next cycle but im also not pushing for us to land it if we identify issue.\n\ni ask for folks to take a look if they had time.\n\nno pressure either ways.","commit_id":"851887ef2570d6e83c0b1ff26bf6e1ca6202a2a4"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"b73b72348f420c7c2b1c61eec241a5d4e30bd1bc","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"tests: Clean up flavors tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Ahead of adding additional tests."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I2cac4cc79288f7b3bacc4a63a1d36d4cf12013d7"},{"line_number":12,"context_line":"Signed-off-by: Stephen Finucane \u003cstephenfin@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"87d77141_f190bbaa","line":9,"in_reply_to":"ac32ef23_de77fd06","updated":"2025-08-28 13:49:00.000000000","message":"that\u0027s my main concern : we are very close to the Feature Freeze (litterally today EOB), this series just appears 6 hours before I have to stop my day and then I find lots of unncessary modifications for that series.\n\nSorry, please understand it takes time for reviews and if I take more time for this patch for reviewing it while it\u0027s not needed, then other series from other contributors wouldn\u0027t have that time so they couldn\u0027t be reviewed.","commit_id":"851887ef2570d6e83c0b1ff26bf6e1ca6202a2a4"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"b73b72348f420c7c2b1c61eec241a5d4e30bd1bc","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2159406a_ec0a4963","updated":"2025-08-28 13:49:00.000000000","message":"-1 for the test coverage but I\u0027m very afraid of the volume of the modifications while we\u0027re short in time (and while they\u0027re absolutely not required for the feature implementation, right?)","commit_id":"851887ef2570d6e83c0b1ff26bf6e1ca6202a2a4"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"e3cbbfa7a029a777b915c3fea8400b57cd04ae71","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4163a77d_d4132abb","in_reply_to":"2159406a_ec0a4963","updated":"2025-09-01 08:58:50.000000000","message":"Done","commit_id":"851887ef2570d6e83c0b1ff26bf6e1ca6202a2a4"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"0ce358ef791370b1b1828b2351ff21879fc4abef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0c1033dc_b9e6e246","updated":"2025-12-12 16:52:07.000000000","message":"im still ok with this and the commit message has been improved since v1 so i think we should still proceed with this cleanup","commit_id":"29ac6cb6b4cde6836a4145527166cef2e644e573"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"195652e65c52a4e1dac78ab88467674ba8069b30","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"8df94e69_9b6d966f","updated":"2026-01-08 16:21:16.000000000","message":"recheck tox test timeout","commit_id":"04dab12beac32303ed11fb96ff72b1a752dc1445"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"7f201452217195e21e6771cc7a8f482a08339fea","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"6bb45a54_7a438011","updated":"2026-02-26 11:15:24.000000000","message":"Test-only change that simplifies our tests and is blocking later work. Let\u0027s get it merged.","commit_id":"61c1ce6c8ed8bab8bc4ca84fb9f0c57c1eea7bcc"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"778e83857b7426f4323857f617a6951f82ecf99f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"d4a56e2f_549f4dec","updated":"2026-02-25 17:03:43.000000000","message":"recheck nova.tests.functional.regressions.test_bug_1938326.TestMigrateFromDownHost.test_migrate_from_disabled_host seam to be flakey","commit_id":"61c1ce6c8ed8bab8bc4ca84fb9f0c57c1eea7bcc"}],"nova/tests/unit/api/openstack/compute/test_flavor_manage.py":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"dc7a07854c813fc30fda571f72916a5e0fe1ad38","unresolved":true,"context_lines":[{"line_number":336,"context_line":"        self.assertRaises(exception.InvalidInput, flavors.create, \"abcdef\","},{"line_number":337,"context_line":"                          512, 2, None, 1, 1234, 512, 1, True)"},{"line_number":338,"context_line":"        self.assertRaises(exception.InvalidInput, flavors.create, \"abcdef\","},{"line_number":339,"context_line":"                          \"test_memory_mb\", 2, None, 1, 1234, 512, 1, True)"},{"line_number":340,"context_line":""},{"line_number":341,"context_line":"    def test_create_with_description(self):"},{"line_number":342,"context_line":"        \"\"\"With microversion \u003c2.55 this should return a failure.\"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"b7d9986a_636006e5","side":"PARENT","line":339,"updated":"2025-08-28 12:51:13.000000000","message":"Do we loose test coverage by removing these test cases?","commit_id":"2f72d4535667d19b8d4fa41759d37dc96f742cd7"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8fb9983dff0c1332dfa7044c5ea0e3b952b95ba3","unresolved":false,"context_lines":[{"line_number":336,"context_line":"        self.assertRaises(exception.InvalidInput, flavors.create, \"abcdef\","},{"line_number":337,"context_line":"                          512, 2, None, 1, 1234, 512, 1, True)"},{"line_number":338,"context_line":"        self.assertRaises(exception.InvalidInput, flavors.create, \"abcdef\","},{"line_number":339,"context_line":"                          \"test_memory_mb\", 2, None, 1, 1234, 512, 1, True)"},{"line_number":340,"context_line":""},{"line_number":341,"context_line":"    def test_create_with_description(self):"},{"line_number":342,"context_line":"        \"\"\"With microversion \u003c2.55 this should return a failure.\"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"31bc2b2d_9c0b7c43","side":"PARENT","line":339,"in_reply_to":"8dce9a2b_8fff0390","updated":"2025-08-29 09:27:07.000000000","message":"Apologies, I definitely needed a better commit message. Most of these tests are already handled by the following tests:\n\n* `test_create_without_ram`\n* `test_create_with_0_ram`\n* `test_create_with_ram_exceed_max_limit`\n\nand we have the same tests for `vcpus` and `disk`. Those seemed good enough. Probably no harm to add a test for negative though.","commit_id":"2f72d4535667d19b8d4fa41759d37dc96f742cd7"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"b73b72348f420c7c2b1c61eec241a5d4e30bd1bc","unresolved":true,"context_lines":[{"line_number":336,"context_line":"        self.assertRaises(exception.InvalidInput, flavors.create, \"abcdef\","},{"line_number":337,"context_line":"                          512, 2, None, 1, 1234, 512, 1, True)"},{"line_number":338,"context_line":"        self.assertRaises(exception.InvalidInput, flavors.create, \"abcdef\","},{"line_number":339,"context_line":"                          \"test_memory_mb\", 2, None, 1, 1234, 512, 1, True)"},{"line_number":340,"context_line":""},{"line_number":341,"context_line":"    def test_create_with_description(self):"},{"line_number":342,"context_line":"        \"\"\"With microversion \u003c2.55 this should return a failure.\"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"dce08048_3e68f1c2","side":"PARENT","line":339,"in_reply_to":"b7d9986a_636006e5","updated":"2025-08-28 13:49:00.000000000","message":"agreed with gibi, I don\u0027t see where we cover this.","commit_id":"2f72d4535667d19b8d4fa41759d37dc96f742cd7"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f2fa895a12929a2a92092b22bdff566808e1d131","unresolved":true,"context_lines":[{"line_number":336,"context_line":"        self.assertRaises(exception.InvalidInput, flavors.create, \"abcdef\","},{"line_number":337,"context_line":"                          512, 2, None, 1, 1234, 512, 1, True)"},{"line_number":338,"context_line":"        self.assertRaises(exception.InvalidInput, flavors.create, \"abcdef\","},{"line_number":339,"context_line":"                          \"test_memory_mb\", 2, None, 1, 1234, 512, 1, True)"},{"line_number":340,"context_line":""},{"line_number":341,"context_line":"    def test_create_with_description(self):"},{"line_number":342,"context_line":"        \"\"\"With microversion \u003c2.55 this should return a failure.\"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"8dce9a2b_8fff0390","side":"PARENT","line":339,"in_reply_to":"dce08048_3e68f1c2","updated":"2025-08-28 16:17:53.000000000","message":"we do have schema check for this but i guess you can use the older micro version.\n\nso i guess your right if we we dont have these elsewehre then this would be a loss of coverage.\n\nthese shoudl really be in anyway test_flavors.py so this change should move them not delete them.","commit_id":"2f72d4535667d19b8d4fa41759d37dc96f742cd7"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"b73b72348f420c7c2b1c61eec241a5d4e30bd1bc","unresolved":true,"context_lines":[{"line_number":356,"context_line":"        flavor \u003d self.controller.create(req, body\u003dself.request_body)[\u0027flavor\u0027]"},{"line_number":357,"context_line":"        self.assertEqual(\u0027test description\u0027, flavor[\u0027description\u0027])"},{"line_number":358,"context_line":"        mock_get.return_value \u003d self.get_flavor(flavor)"},{"line_number":359,"context_line":""},{"line_number":360,"context_line":"        # Now null out the flavor description."},{"line_number":361,"context_line":"        flavor \u003d self.controller.update("},{"line_number":362,"context_line":"            self._get_http_request(), flavor[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":1,"id":"a81227ea_401176e2","line":359,"updated":"2025-08-28 13:49:00.000000000","message":"unnecessary update","commit_id":"851887ef2570d6e83c0b1ff26bf6e1ca6202a2a4"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"6bb664a828c8b58f8a15f0a1a5592ca8fb6be427","unresolved":false,"context_lines":[{"line_number":356,"context_line":"        flavor \u003d self.controller.create(req, body\u003dself.request_body)[\u0027flavor\u0027]"},{"line_number":357,"context_line":"        self.assertEqual(\u0027test description\u0027, flavor[\u0027description\u0027])"},{"line_number":358,"context_line":"        mock_get.return_value \u003d self.get_flavor(flavor)"},{"line_number":359,"context_line":""},{"line_number":360,"context_line":"        # Now null out the flavor description."},{"line_number":361,"context_line":"        flavor \u003d self.controller.update("},{"line_number":362,"context_line":"            self._get_http_request(), flavor[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":1,"id":"6612453b_2d57d6a2","line":359,"in_reply_to":"a81227ea_401176e2","updated":"2025-08-29 09:30:08.000000000","message":"It\u0027s a cleanup patch","commit_id":"851887ef2570d6e83c0b1ff26bf6e1ca6202a2a4"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"b73b72348f420c7c2b1c61eec241a5d4e30bd1bc","unresolved":true,"context_lines":[{"line_number":422,"context_line":"                body\u003d{\u0027flavor\u0027: {\u0027description\u0027: description}})"},{"line_number":423,"context_line":""},{"line_number":424,"context_line":""},{"line_number":425,"context_line":"class FlavorManageTestV261(FlavorManageTestV255):"},{"line_number":426,"context_line":"    \"\"\"Run the same tests as we would for v2.55 but with a extra_specs.\"\"\""},{"line_number":427,"context_line":"    microversion \u003d \u00272.61\u0027"},{"line_number":428,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3c33c69b_f4a0e6e9","line":425,"range":{"start_line":425,"start_character":6,"end_line":425,"end_character":26},"updated":"2025-08-28 13:49:00.000000000","message":"I don\u0027t understand why we need to change the name","commit_id":"851887ef2570d6e83c0b1ff26bf6e1ca6202a2a4"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f2fa895a12929a2a92092b22bdff566808e1d131","unresolved":true,"context_lines":[{"line_number":422,"context_line":"                body\u003d{\u0027flavor\u0027: {\u0027description\u0027: description}})"},{"line_number":423,"context_line":""},{"line_number":424,"context_line":""},{"line_number":425,"context_line":"class FlavorManageTestV261(FlavorManageTestV255):"},{"line_number":426,"context_line":"    \"\"\"Run the same tests as we would for v2.55 but with a extra_specs.\"\"\""},{"line_number":427,"context_line":"    microversion \u003d \u00272.61\u0027"},{"line_number":428,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"460252c8_78d22ab1","line":425,"range":{"start_line":425,"start_character":6,"end_line":425,"end_character":26},"in_reply_to":"3c33c69b_f4a0e6e9","updated":"2025-08-28 16:17:53.000000000","message":"we dont need too this is just cleaning up the nameing to follow the convetion we have elsewhere in nova so this is just for code consitency.","commit_id":"851887ef2570d6e83c0b1ff26bf6e1ca6202a2a4"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"6bb664a828c8b58f8a15f0a1a5592ca8fb6be427","unresolved":false,"context_lines":[{"line_number":422,"context_line":"                body\u003d{\u0027flavor\u0027: {\u0027description\u0027: description}})"},{"line_number":423,"context_line":""},{"line_number":424,"context_line":""},{"line_number":425,"context_line":"class FlavorManageTestV261(FlavorManageTestV255):"},{"line_number":426,"context_line":"    \"\"\"Run the same tests as we would for v2.55 but with a extra_specs.\"\"\""},{"line_number":427,"context_line":"    microversion \u003d \u00272.61\u0027"},{"line_number":428,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"f11002f6_6cbae8f4","line":425,"range":{"start_line":425,"start_character":6,"end_line":425,"end_character":26},"in_reply_to":"460252c8_78d22ab1","updated":"2025-08-29 09:30:08.000000000","message":"It\u0027s a cleanup patch","commit_id":"851887ef2570d6e83c0b1ff26bf6e1ca6202a2a4"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"b73b72348f420c7c2b1c61eec241a5d4e30bd1bc","unresolved":true,"context_lines":[{"line_number":427,"context_line":"    microversion \u003d \u00272.61\u0027"},{"line_number":428,"context_line":""},{"line_number":429,"context_line":"    def get_flavor(self, flavor):"},{"line_number":430,"context_line":"        return super().get_flavor("},{"line_number":431,"context_line":"            flavor, extra_specs\u003d{\"key1\": \"value1\"})"},{"line_number":432,"context_line":""},{"line_number":433,"context_line":"    def setUp(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"25d27430_c62e57c1","line":430,"updated":"2025-08-28 13:49:00.000000000","message":"unnnecessary update","commit_id":"851887ef2570d6e83c0b1ff26bf6e1ca6202a2a4"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f2fa895a12929a2a92092b22bdff566808e1d131","unresolved":true,"context_lines":[{"line_number":427,"context_line":"    microversion \u003d \u00272.61\u0027"},{"line_number":428,"context_line":""},{"line_number":429,"context_line":"    def get_flavor(self, flavor):"},{"line_number":430,"context_line":"        return super().get_flavor("},{"line_number":431,"context_line":"            flavor, extra_specs\u003d{\"key1\": \"value1\"})"},{"line_number":432,"context_line":""},{"line_number":433,"context_line":"    def setUp(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"7daf1bd6_1d29d74f","line":430,"in_reply_to":"25d27430_c62e57c1","updated":"2025-08-28 16:17:53.000000000","message":"correct but this is a dedicated cleanup patch which is exactly when this type of change shoudl be done.","commit_id":"851887ef2570d6e83c0b1ff26bf6e1ca6202a2a4"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8fb9983dff0c1332dfa7044c5ea0e3b952b95ba3","unresolved":false,"context_lines":[{"line_number":427,"context_line":"    microversion \u003d \u00272.61\u0027"},{"line_number":428,"context_line":""},{"line_number":429,"context_line":"    def get_flavor(self, flavor):"},{"line_number":430,"context_line":"        return super().get_flavor("},{"line_number":431,"context_line":"            flavor, extra_specs\u003d{\"key1\": \"value1\"})"},{"line_number":432,"context_line":""},{"line_number":433,"context_line":"    def setUp(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"cdaf6f72_f7feaae0","line":430,"in_reply_to":"7daf1bd6_1d29d74f","updated":"2025-08-29 09:27:07.000000000","message":"What Sean said","commit_id":"851887ef2570d6e83c0b1ff26bf6e1ca6202a2a4"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"b73b72348f420c7c2b1c61eec241a5d4e30bd1bc","unresolved":true,"context_lines":[{"line_number":431,"context_line":"            flavor, extra_specs\u003d{\"key1\": \"value1\"})"},{"line_number":432,"context_line":""},{"line_number":433,"context_line":"    def setUp(self):"},{"line_number":434,"context_line":"        super().setUp()"},{"line_number":435,"context_line":"        self.expected_flavor \u003d copy.deepcopy(self.request_body)"},{"line_number":436,"context_line":"        self.expected_flavor[\u0027flavor\u0027][\u0027extra_specs\u0027] \u003d {}"},{"line_number":437,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1026337d_2d24bdc0","line":434,"updated":"2025-08-28 13:49:00.000000000","message":"ditto","commit_id":"851887ef2570d6e83c0b1ff26bf6e1ca6202a2a4"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8fb9983dff0c1332dfa7044c5ea0e3b952b95ba3","unresolved":false,"context_lines":[{"line_number":431,"context_line":"            flavor, extra_specs\u003d{\"key1\": \"value1\"})"},{"line_number":432,"context_line":""},{"line_number":433,"context_line":"    def setUp(self):"},{"line_number":434,"context_line":"        super().setUp()"},{"line_number":435,"context_line":"        self.expected_flavor \u003d copy.deepcopy(self.request_body)"},{"line_number":436,"context_line":"        self.expected_flavor[\u0027flavor\u0027][\u0027extra_specs\u0027] \u003d {}"},{"line_number":437,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"be736f6a_b31aa496","line":434,"in_reply_to":"1026337d_2d24bdc0","updated":"2025-08-29 09:27:07.000000000","message":"It\u0027s a cleanup patch 😅","commit_id":"851887ef2570d6e83c0b1ff26bf6e1ca6202a2a4"}]}