On 7/28/26 20:53, Kevin Wolf wrote:

diff --git a/tests/qemu-iotests/161 b/tests/qemu-iotests/161
index f25effab936..0b8ba0b19e2 100755
--- a/tests/qemu-iotests/161
+++ b/tests/qemu-iotests/161
@@ -106,6 +106,7 @@ echo
  echo "*** Commit and then change an option on the backing file"
  echo
  # Create the images again
+rm -f -- "$TEST_IMG.base" "$TEST_IMG.int"
  TEST_IMG="$TEST_IMG.base" _make_test_img $IMG_SIZE
  TEST_IMG="$TEST_IMG.int" _make_test_img -b "$TEST_IMG.base" -F $IMGFMT
  _make_test_img -b "$TEST_IMG.int" -F $IMGFMT

This can only work for the file protocol, but the test also supports
fuse, which is now broken.

I haven't noticed _rm_test_img function.  I wonder maybe we can add
this rm into _make_test_img directly, and fix this for all tests?

_make_test_img already calls _rm_test_img - for fuse protocol exactly.
It can call _rm_test_img unconditionally, I guess.

I'll give it a try tomorrow.

Thanks,

/mjt

Reply via email to