summary refs log tree commit diff stats
diff options
context:
space:
mode:
authorPaolo Bonzini <pbonzini@redhat.com>2021-10-27 15:31:44 +0200
committerPaolo Bonzini <pbonzini@redhat.com>2022-01-28 11:13:33 +0100
commit18c1cdd21d318adf2d02d90e25e9c04f33db76e8 (patch)
treebd8ac1b7bcd3a5311b236e28bb78e6eedaeba5e5
parent98487b9035d8540376024df74ab5510fdc37f12e (diff)
downloadfocaccia-qemu-18c1cdd21d318adf2d02d90e25e9c04f33db76e8.tar.gz
focaccia-qemu-18c1cdd21d318adf2d02d90e25e9c04f33db76e8.zip
qemu-iotests: require at least an argument to check-block.sh
This is anyway how check-block.sh is used in practice, and by removing the
list of formats in the script we avoid duplication between meson.build
and check-block.sh.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
-rwxr-xr-xtests/check-block.sh17
1 files changed, 7 insertions, 10 deletions
diff --git a/tests/check-block.sh b/tests/check-block.sh
index 9afeea5275..88e02453d2 100755
--- a/tests/check-block.sh
+++ b/tests/check-block.sh
@@ -1,21 +1,18 @@
 #!/bin/sh
 
+if [ "$#" -eq 0 ]; then
+    echo "Usage: $0 fmt..." >&2
+    exit 99
+fi
+
 # Honor the SPEED environment variable, just like we do it for "meson test"
-if [ "$SPEED" = "slow" ]; then
-    format_list="raw qcow2"
-    group=
-elif [ "$SPEED" = "thorough" ]; then
-    format_list="raw qcow2 qed vmdk vpc"
+format_list="$@"
+if [ "$SPEED" = "slow" ] || [ "$SPEED" = "thorough" ]; then
     group=
 else
-    format_list=qcow2
     group="-g auto"
 fi
 
-if [ "$#" -ne 0 ]; then
-    format_list="$@"
-fi
-
 skip() {
     echo "$*"
     exit 77