btrfs-progs: tests: fix run_mustfail in cli-tests/007-check-force
authorDavid Sterba <dsterba@suse.com>
Mon, 11 Sep 2017 15:29:03 +0000 (17:29 +0200)
committerDavid Sterba <dsterba@suse.com>
Mon, 25 Sep 2017 13:17:06 +0000 (15:17 +0200)
The sanity check in run_mustfail does not work as expected and allowed
the command to continue without the missing parameter

Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=196893
Signed-off-by: David Sterba <dsterba@suse.com>
tests/cli-tests/007-check-force/test.sh

index ecee016..8b60033 100755 (executable)
@@ -12,10 +12,13 @@ prepare_test_dev
 
 run_check "$TOP/mkfs.btrfs" -f "$TEST_DEV"
 run_check_mount_test_dev
-run_mustfail $SUDO_HELPER "$TOP/btrfs" check "$TEST_DEV"
+run_mustfail "checking mounted filesystem without --force" \
+       $SUDO_HELPER "$TOP/btrfs" check "$TEST_DEV"
 run_check $SUDO_HELPER "$TOP/btrfs" check --force "$TEST_DEV"
-run_mustfail $SUDO_HELPER "$TOP/btrfs" check --force --repair "$TEST_DEV"
+run_mustfail "checking mounted filesystem with --force --repair" \
+       $SUDO_HELPER "$TOP/btrfs" check --force --repair "$TEST_DEV"
 run_check_umount_test_dev
 run_check $SUDO_HELPER "$TOP/btrfs" check "$TEST_DEV"
 run_check $SUDO_HELPER "$TOP/btrfs" check --force "$TEST_DEV"
-run_mustfail $SUDO_HELPER "$TOP/btrfs" check --force --repair "$TEST_DEV"
+run_mustfail "--force --repair on unmounted filesystem" \
+       $SUDO_HELPER "$TOP/btrfs" check --force --repair "$TEST_DEV"