)]}'
{"manila_tempest_tests/tests/api/admin/test_admin_actions.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"f62e6ce13b497d3ae6591dba4ceb1d7bc89208d1","unresolved":true,"context_lines":[{"line_number":134,"context_line":"        # Verify that the share has been deleted."},{"line_number":135,"context_line":"        self.assertRaises(lib_exc.NotFound,"},{"line_number":136,"context_line":"                          self.shares_v2_client.get_share,"},{"line_number":137,"context_line":"                          share[\u0027id\u0027])"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"    @decorators.idempotent_id(\u0027d5a48182-ecd7-463e-a31a-148c81d3c5ed\u0027)"},{"line_number":140,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"}],"source_content_type":"text/x-python","patch_set":1,"id":"b617a8ef_5f4d7d32","line":137,"range":{"start_line":137,"start_character":37,"end_line":137,"end_character":38},"updated":"2021-06-11 04:37:27.000000000","message":"This is redundant, since the \"wait_for_resource_deletion\" method bails out safely only when it encounters  a NotFound exception after a resource has been deleted. However, its harmless if it improves readability.","commit_id":"ac53004eddfa5be7b95b2264cb181dbe2d97ec8c"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"62599bbef63192175dbd895537021c26dfe474ae","unresolved":true,"context_lines":[{"line_number":134,"context_line":"        # Verify that the share has been deleted."},{"line_number":135,"context_line":"        self.assertRaises(lib_exc.NotFound,"},{"line_number":136,"context_line":"                          self.shares_v2_client.get_share,"},{"line_number":137,"context_line":"                          share[\u0027id\u0027])"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"    @decorators.idempotent_id(\u0027d5a48182-ecd7-463e-a31a-148c81d3c5ed\u0027)"},{"line_number":140,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"}],"source_content_type":"text/x-python","patch_set":1,"id":"ee7b2ab2_ebd85093","line":137,"range":{"start_line":137,"start_character":37,"end_line":137,"end_character":38},"in_reply_to":"b617a8ef_5f4d7d32","updated":"2021-06-11 08:40:16.000000000","message":"Thanks Goutham!\nIt\u0027s not really redundant.\nThis verification makes sure that the share resource has been deleted after share instance deletion during the test and not in the teardownd level.\nAnd it does matter because \"create_share\" method also tries to delete the share in the teardown stage.","commit_id":"ac53004eddfa5be7b95b2264cb181dbe2d97ec8c"}]}
