13.08.2019 12:10, Kevin Wolf wrote: > Am 12.08.2019 um 20:11 hat Vladimir Sementsov-Ogievskiy geschrieben: >> Check that it is fixed by previous commit >> >> Signed-off-by: Vladimir Sementsov-Ogievskiy <vsement...@virtuozzo.com> >> --- >> tests/qemu-iotests/263 | 46 ++++++++++++++++++++++++++++++++++++++ >> tests/qemu-iotests/263.out | 12 ++++++++++ >> tests/qemu-iotests/group | 1 + >> 3 files changed, 59 insertions(+) >> create mode 100755 tests/qemu-iotests/263 >> create mode 100644 tests/qemu-iotests/263.out >> >> diff --git a/tests/qemu-iotests/263 b/tests/qemu-iotests/263 >> new file mode 100755 >> index 0000000000..dbe38e6c6c >> --- /dev/null >> +++ b/tests/qemu-iotests/263 >> @@ -0,0 +1,46 @@ >> +#!/usr/bin/env python >> +# >> +# Test mirroring qcow2 under raw-format >> +# >> +# Copyright (c) 2019 Virtuozzo International GmbH. >> +# >> +# This program is free software; you can redistribute it and/or modify >> +# it under the terms of the GNU General Public License as published by >> +# the Free Software Foundation; either version 2 of the License, or >> +# (at your option) any later version. >> +# >> +# This program is distributed in the hope that it will be useful, >> +# but WITHOUT ANY WARRANTY; without even the implied warranty of >> +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the >> +# GNU General Public License for more details. >> +# >> +# You should have received a copy of the GNU General Public License >> +# along with this program. If not, see <http://www.gnu.org/licenses/>. >> +# >> + >> +import iotests >> +from iotests import qemu_img_create, qemu_io, filter_qemu_io, file_path, log >> + >> +iotests.verify_image_format(supported_fmts=['qcow2']) >> + >> +base, top, target = file_path('base', 'top', 'target') >> +size = 1024 * 1024 >> + >> +qemu_img_create('-f', iotests.imgfmt, base, str(size)) >> +log(filter_qemu_io(qemu_io('-f', iotests.imgfmt, >> + '-c', 'write -P 1 0 1M', base))) >> +qemu_img_create('-f', iotests.imgfmt, '-b', base, top, str(size)) > > Who deletes these files at the end of the test? (There's a reason why > all the other test cases uses 'with FilePath(...)'.) >
file_path sets corresponding handling, it's alternative to FilePath -- Best regards, Vladimir