)]}'
{"cinder/tests/unit/volume/test_snapshot.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"4f539b4fca8a9631b3830d2d58421b4d2be87d2f","unresolved":false,"context_lines":[{"line_number":499,"context_line":""},{"line_number":500,"context_line":"        # cleanup resource"},{"line_number":501,"context_line":"        snapshot.destroy()"},{"line_number":502,"context_line":"        db.volume_destroy(self.context, volume_id)"},{"line_number":503,"context_line":""},{"line_number":504,"context_line":"    def test_delete_busy_snapshot(self):"},{"line_number":505,"context_line":"        \"\"\"Test snapshot can be created and deleted.\"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"df7087c5_446cd49b","line":502,"updated":"2018-03-19 16:22:38.000000000","message":"Supposedly you can pass testcase_instance\u003dself to create_volume() and then you don\u0027t need this.","commit_id":"a8bd01d06b0ebab7da60911f07ebd5bf77a6e3e2"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"dc70c0dc51543d114f6737b1d0b4d36e70ccc096","unresolved":false,"context_lines":[{"line_number":499,"context_line":""},{"line_number":500,"context_line":"        # cleanup resource"},{"line_number":501,"context_line":"        snapshot.destroy()"},{"line_number":502,"context_line":"        db.volume_destroy(self.context, volume_id)"},{"line_number":503,"context_line":""},{"line_number":504,"context_line":"    def test_delete_busy_snapshot(self):"},{"line_number":505,"context_line":"        \"\"\"Test snapshot can be created and deleted.\"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"df7087c5_41fe01af","line":502,"in_reply_to":"df7087c5_446cd49b","updated":"2018-03-19 18:21:42.000000000","message":"Yes, that would be a good cleanup, and to do it right would hit a lot of places in this file. The test_utils.create_snapshot() also supports this, but this file uses its own create_snapshot(), and the scope of the cleanup effort would get fuzzy.","commit_id":"a8bd01d06b0ebab7da60911f07ebd5bf77a6e3e2"}]}