)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"ff90958d580cda37bb88ce0ece7ae694b2b5121c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"05c99b32_68f739c1","updated":"2022-06-22 09:20:06.000000000","message":"thanks for the test, although I believe we could just extend an existing test which already tests create-update-delete of a router","commit_id":"0209bbd9d799dd7a2cd3c7583ce900543a8e944b"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"4bc7ce668d375163004b74b5ded8005aff2dd9df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a3a0079a_eeb7ca6b","updated":"2022-08-26 18:43:18.000000000","message":"yes, please check description in existing test or it can be done at functional or unit test at project side.","commit_id":"0209bbd9d799dd7a2cd3c7583ce900543a8e944b"}],"tempest/api/network/test_routers.py":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"ff90958d580cda37bb88ce0ece7ae694b2b5121c","unresolved":true,"context_lines":[{"line_number":53,"context_line":"    @decorators.idempotent_id(\u0027f64403e2-8483-4b34-8ccd-b09a87bcc68c\u0027)"},{"line_number":54,"context_line":"    @testtools.skipUnless(CONF.network.public_network_id,"},{"line_number":55,"context_line":"                          \u0027The public_network_id option must be specified.\u0027)"},{"line_number":56,"context_line":"    def test_create_show_list_update_delete_router(self):"},{"line_number":57,"context_line":"        \"\"\"Test create/show/list/update/delete of a router\"\"\""},{"line_number":58,"context_line":"        # Create a router"},{"line_number":59,"context_line":"        router_name \u003d data_utils.rand_name(self.__class__.__name__ + \u0027-router\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"20b99e3d_8d500044","side":"PARENT","line":56,"range":{"start_line":56,"start_character":8,"end_line":56,"end_character":50},"updated":"2022-06-22 09:20:06.000000000","message":"what about just extending this test so that it sets and updates also the router description?","commit_id":"d720340c44e67856274235a47ebae38efd0b7625"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"ff90958d580cda37bb88ce0ece7ae694b2b5121c","unresolved":true,"context_lines":[{"line_number":154,"context_line":"        router \u003d self.create_router(description\u003dr_description)"},{"line_number":155,"context_line":"        self.addCleanup(self.delete_router, router)"},{"line_number":156,"context_line":"        self.assertEqual(router[\u0027description\u0027], r_description)"},{"line_number":157,"context_line":"        # Update the descirption of router and verify if it is updated"},{"line_number":158,"context_line":"        updated_description \u003d \u0027updated\u0027 + router[\u0027description\u0027]"},{"line_number":159,"context_line":"        router_update \u003d self.routers_client.update_router("},{"line_number":160,"context_line":"            router[\u0027id\u0027], description\u003dupdated_description)[\u0027router\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"95cded05_b84273f1","line":157,"range":{"start_line":157,"start_character":21,"end_line":157,"end_character":32},"updated":"2022-06-22 09:20:06.000000000","message":"typo: description","commit_id":"0209bbd9d799dd7a2cd3c7583ce900543a8e944b"}]}
