From: Vojtěch Eichler Date: Wed, 8 Sep 2021 10:36:19 +0000 (+0200) Subject: tests: make use of subtests X-Git-Tag: v2.38-rc1~260^2~1 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=99120a8bded6493a27d7dce7f51704fb3e9e54e3;p=thirdparty%2Futil-linux.git tests: make use of subtests --- diff --git a/tests/expected/mount/fstab-devname b/tests/expected/mount/fstab-devname index 35821117c8..e69de29bb2 100644 --- a/tests/expected/mount/fstab-devname +++ b/tests/expected/mount/fstab-devname @@ -1 +0,0 @@ -Success diff --git a/tests/expected/mount/fstab-label b/tests/expected/mount/fstab-label index 35821117c8..e69de29bb2 100644 --- a/tests/expected/mount/fstab-label +++ b/tests/expected/mount/fstab-label @@ -1 +0,0 @@ -Success diff --git a/tests/expected/mount/fstab-label2devname b/tests/expected/mount/fstab-label2devname index 35821117c8..e69de29bb2 100644 --- a/tests/expected/mount/fstab-label2devname +++ b/tests/expected/mount/fstab-label2devname @@ -1 +0,0 @@ -Success diff --git a/tests/expected/mount/fstab-label2uuid b/tests/expected/mount/fstab-label2uuid index 35821117c8..e69de29bb2 100644 --- a/tests/expected/mount/fstab-label2uuid +++ b/tests/expected/mount/fstab-label2uuid @@ -1 +0,0 @@ -Success diff --git a/tests/expected/mount/fstab-uuid b/tests/expected/mount/fstab-uuid index 35821117c8..e69de29bb2 100644 --- a/tests/expected/mount/fstab-uuid +++ b/tests/expected/mount/fstab-uuid @@ -1 +0,0 @@ -Success diff --git a/tests/expected/mount/fstab-uuid2devname b/tests/expected/mount/fstab-uuid2devname index 35821117c8..e69de29bb2 100644 --- a/tests/expected/mount/fstab-uuid2devname +++ b/tests/expected/mount/fstab-uuid2devname @@ -1 +0,0 @@ -Success diff --git a/tests/expected/mount/fstab-uuid2label b/tests/expected/mount/fstab-uuid2label index 35821117c8..e69de29bb2 100644 --- a/tests/expected/mount/fstab-uuid2label +++ b/tests/expected/mount/fstab-uuid2label @@ -1 +0,0 @@ -Success diff --git a/tests/expected/mount/label b/tests/expected/mount/label index 35821117c8..e69de29bb2 100644 --- a/tests/expected/mount/label +++ b/tests/expected/mount/label @@ -1 +0,0 @@ -Success diff --git a/tests/expected/mount/uuid b/tests/expected/mount/uuid index 35821117c8..e69de29bb2 100644 --- a/tests/expected/mount/uuid +++ b/tests/expected/mount/uuid @@ -1 +0,0 @@ -Success diff --git a/tests/ts/mount/fstab-devname b/tests/ts/mount/fstab-devname index 4e4961edec..660c5e7d06 100755 --- a/tests/ts/mount/fstab-devname +++ b/tests/ts/mount/fstab-devname @@ -36,19 +36,23 @@ mkfs.ext2 $DEVICE &> /dev/null || ts_die "Cannot make ext2 on $DEVICE" ts_device_has "TYPE" "ext2" $DEVICE || ts_die "Cannot find ext2 on $DEVICE" [ -d "$TS_MOUNTPOINT" ] || mkdir -p $TS_MOUNTPOINT +# Use one mountpoint for all subtests +MOUNTPOINT=$TS_MOUNTPOINT ts_fstab_add $DEVICE -# variant A) -$TS_CMD_MOUNT $TS_MOUNTPOINT >> $TS_OUTPUT 2>> $TS_ERRLOG -ts_is_mounted $DEVICE || ts_die "A) Cannot find $DEVICE in /proc/mounts" -$TS_CMD_UMOUNT $DEVICE || ts_die "A) Cannot umount $DEVICE" +ts_init_subtest "mountpoint" +$TS_CMD_MOUNT $MOUNTPOINT >> $TS_OUTPUT 2>> $TS_ERRLOG +ts_is_mounted $DEVICE || ts_log "Cannot find $DEVICE in /proc/mounts" +$TS_CMD_UMOUNT $DEVICE || >> $TS_OUTPUT 2>> $TS_ERRLOG +ts_finalize_subtest -# variant B) +ts_init_subtest "device-name" $TS_CMD_MOUNT $DEVICE >> $TS_OUTPUT 2>> $TS_ERRLOG -ts_is_mounted $DEVICE || ts_die "B) Cannot find $DEVICE in /proc/mounts" -$TS_CMD_UMOUNT $DEVICE || ts_die "B) Cannot umount $DEVICE" +ts_is_mounted $DEVICE || ts_log "Cannot find $DEVICE in /proc/mounts" +$TS_CMD_UMOUNT $DEVICE >> $TS_OUTPUT 2>> $TS_ERRLOG +ts_finalize_subtest ts_fstab_clean -ts_log "Success" + ts_finalize diff --git a/tests/ts/mount/fstab-label b/tests/ts/mount/fstab-label index a86bdf6c07..6ec06414ab 100755 --- a/tests/ts/mount/fstab-label +++ b/tests/ts/mount/fstab-label @@ -35,29 +35,29 @@ DEVICE=$TS_LODEV mkfs.ext2 -L $LABEL $DEVICE &> /dev/null || ts_die "Cannot make ext2 on $DEVICE" ts_device_has "LABEL" $LABEL $DEVICE \ || ts_die "Cannot find LABEL '$LABEL' on $DEVICE" - [ -d "$TS_MOUNTPOINT" ] || mkdir -p $TS_MOUNTPOINT +MOUNTPOINT=$TS_MOUNTPOINT ts_fstab_add "LABEL=$LABEL" ts_udevadm_settle "$DEVICE" "LABEL" -# variant A) -$TS_CMD_MOUNT $TS_MOUNTPOINT >> $TS_OUTPUT 2>> $TS_ERRLOG -ts_is_mounted $DEVICE || ts_die "A) Cannot find $DEVICE in /proc/mounts" -$TS_CMD_UMOUNT $DEVICE || ts_die "A) Cannot umount $DEVICE" +ts_init_subtest "no-option" +$TS_CMD_MOUNT $MOUNTPOINT >> $TS_OUTPUT 2>> $TS_ERRLOG +ts_is_mounted $DEVICE || ts_log "Cannot find $DEVICE in /proc/mounts" +$TS_CMD_UMOUNT $DEVICE >> $TS_OUTPUT 2>> $TS_ERRLOG +ts_finalize_subtest -# variant B) +ts_init_subtest "L-option" $TS_CMD_MOUNT -L $LABEL >> $TS_OUTPUT 2>> $TS_ERRLOG -ts_is_mounted $DEVICE || ts_die "B) Cannot find $DEVICE in /proc/mounts" -$TS_CMD_UMOUNT $DEVICE || ts_die "B) Cannot umount $DEVICE" +ts_is_mounted $DEVICE || ts_log "Cannot find $DEVICE in /proc/mounts" +$TS_CMD_UMOUNT $DEVICE >> $TS_OUTPUT 2>> $TS_ERRLOG +ts_finalize_subtest -# variant C) +ts_init_subtest "LABEL-option" $TS_CMD_MOUNT LABEL=$LABEL >> $TS_OUTPUT 2>> $TS_ERRLOG -ts_is_mounted $DEVICE || ts_die "C) Cannot find $DEVICE in /proc/mounts" -$TS_CMD_UMOUNT $DEVICE || ts_die "C) Cannot umount $DEVICE" +ts_is_mounted $DEVICE || ts_log "Cannot find $DEVICE in /proc/mounts" +$TS_CMD_UMOUNT $DEVICE >> $TS_OUTPUT 2>> $TS_ERRLOG +ts_finalize_subtest -ts_fstab_clean -ts_log "Success" ts_finalize - diff --git a/tests/ts/mount/fstab-label2devname b/tests/ts/mount/fstab-label2devname index 181352aef5..5b184edc75 100755 --- a/tests/ts/mount/fstab-label2devname +++ b/tests/ts/mount/fstab-label2devname @@ -36,23 +36,25 @@ mkfs.ext2 -L $LABEL $DEVICE &> /dev/null || ts_die "Cannot make ext2 on $DEVICE" ts_device_has "LABEL" $LABEL $DEVICE \ || ts_die "Cannot find LABEL '$LABEL' on $DEVICE" -[ -d "$TS_MOUNTPOINT" ] || mkdir -p $TS_MOUNTPOINT - ts_fstab_add "$DEVICE" ts_udevadm_settle "$DEVICE" "LABEL" -# variant A) +# Mountpoint has to be created here rather than in each subtest since ts_fstab_add writes +# this mountpoint to the /etc/fstab file and the mountpoint is needed in subtests +[ -d "$TS_MOUNTPOINT" ] || mkdir -p $TS_MOUNTPOINT + +ts_init_subtest "L-option" $TS_CMD_MOUNT -L $LABEL >> $TS_OUTPUT 2>> $TS_ERRLOG -ts_is_mounted $DEVICE || ts_die "A) Cannot find $DEVICE in /proc/mounts" -$TS_CMD_UMOUNT $DEVICE || ts_die "A) Cannot umount $DEVICE" +ts_is_mounted $DEVICE || ts_log "Cannot find $DEVICE in /proc/mounts" +$TS_CMD_UMOUNT $DEVICE >> $TS_OUTPUT 2>> $TS_ERRLOG +ts_finalize_subtest -# variant B) +ts_init_subtest "LABEL-option" $TS_CMD_MOUNT "LABEL=$LABEL" >> $TS_OUTPUT 2>> $TS_ERRLOG -ts_is_mounted $DEVICE || ts_die "B) Cannot find $DEVICE in /proc/mounts" -$TS_CMD_UMOUNT $DEVICE || ts_die "B) Cannot umount $DEVICE" +ts_is_mounted $DEVICE || ts_log "Cannot find $DEVICE in /proc/mounts" +$TS_CMD_UMOUNT $DEVICE >> $TS_OUTPUT 2>> $TS_ERRLOG +ts_finalize_subtest ts_fstab_clean -ts_log "Success" ts_finalize - diff --git a/tests/ts/mount/fstab-label2uuid b/tests/ts/mount/fstab-label2uuid index 027bf8059c..b0ef960d94 100755 --- a/tests/ts/mount/fstab-label2uuid +++ b/tests/ts/mount/fstab-label2uuid @@ -38,23 +38,26 @@ ts_device_has "LABEL" $LABEL $DEVICE \ || ts_die "Cannot find LABEL '$LABEL' on $DEVICE" UUID=$(ts_uuid_by_devname "$DEVICE") || ts_die "Cannot find UUID on $DEVICE" -[ -d "$TS_MOUNTPOINT" ] || mkdir -p $TS_MOUNTPOINT - ts_fstab_add "UUID=$UUID" ts_udevadm_settle "$DEVICE" "LABEL" "UUID" -# variant A) +# Mountpoint has to be created here rather than in each subtest since ts_fstab_add writes +# this mountpoint to the /etc/fstab file and the mountpoint is needed in subtests +[ -d "$TS_MOUNTPOINT" ] || mkdir -p $TS_MOUNTPOINT + +ts_init_subtest "L-option" $TS_CMD_MOUNT -L $LABEL >> $TS_OUTPUT 2>> $TS_ERRLOG -ts_is_mounted $DEVICE || ts_die "A) Cannot find $DEVICE in /proc/mounts" -$TS_CMD_UMOUNT $DEVICE || ts_die "A) Cannot umount $DEVICE" +ts_is_mounted $DEVICE || ts_log "Cannot find $DEVICE in /proc/mounts" +$TS_CMD_UMOUNT $DEVICE >> $TS_OUTPUT 2>> $TS_ERRLOG +ts_finalize_subtest -# variant B) +ts_init_subtest "LABEL-option" $TS_CMD_MOUNT "LABEL=$LABEL" >> $TS_OUTPUT 2>> $TS_ERRLOG -ts_is_mounted $DEVICE || ts_die "B) Cannot find $DEVICE in /proc/mounts" -$TS_CMD_UMOUNT $DEVICE || ts_die "B) Cannot umount $DEVICE" +ts_is_mounted $DEVICE || ts_log "Cannot find $DEVICE in /proc/mounts" +$TS_CMD_UMOUNT $DEVICE >> $TS_OUTPUT 2>> $TS_ERRLOG +ts_finalize_subtest ts_fstab_clean -ts_log "Success" ts_finalize diff --git a/tests/ts/mount/fstab-uuid b/tests/ts/mount/fstab-uuid index ce4c86ea85..f1cb4bdf41 100755 --- a/tests/ts/mount/fstab-uuid +++ b/tests/ts/mount/fstab-uuid @@ -35,27 +35,30 @@ mkfs.ext2 $DEVICE &> /dev/null || ts_die "Cannot make ext2 on $DEVICE" UUID=$(ts_uuid_by_devname "$DEVICE") || ts_die "Cannot find UUID on $DEVICE" [ -d "$TS_MOUNTPOINT" ] || mkdir -p $TS_MOUNTPOINT +# Use one mountpoint for all tests +MOUNTPOINT=$TS_MOUNTPOINT ts_fstab_add "UUID=$UUID" ts_udevadm_settle "$DEVICE" "UUID" -# variant A) -$TS_CMD_MOUNT $TS_MOUNTPOINT >> $TS_OUTPUT 2>> $TS_ERRLOG -ts_is_mounted $DEVICE || ts_die "A) Cannot find $DEVICE in /proc/mounts" -$TS_CMD_UMOUNT $DEVICE || ts_die "A) Cannot umount $DEVICE" +ts_init_subtest "no-option" +$TS_CMD_MOUNT $MOUNTPOINT >> $TS_OUTPUT 2>> $TS_ERRLOG +ts_is_mounted $DEVICE || ts_log "Cannot find $DEVICE in /proc/mounts" +$TS_CMD_UMOUNT $DEVICE >> $TS_OUTPUT 2>> $TS_ERRLOG +ts_finalize_subtest -# variant B) +ts_init_subtest "U-option" $TS_CMD_MOUNT -U $UUID >> $TS_OUTPUT 2>> $TS_ERRLOG -ts_is_mounted $DEVICE || ts_die "B) Cannot find $DEVICE in /proc/mounts" -$TS_CMD_UMOUNT $DEVICE || ts_die "B) Cannot umount $DEVICE" +ts_is_mounted $DEVICE || ts_log "Cannot find $DEVICE in /proc/mounts" +$TS_CMD_UMOUNT $DEVICE >> $TS_OUTPUT 2>> $TS_ERRLOG +ts_finalize_subtest -# variant C) +ts_init_subtest "UUID-option" $TS_CMD_MOUNT UUID=$UUID >> $TS_OUTPUT 2>> $TS_ERRLOG -ts_is_mounted $DEVICE || ts_die "C) Cannot find $DEVICE in /proc/mounts" -$TS_CMD_UMOUNT $DEVICE || ts_die "C) Cannot umount $DEVICE" +ts_is_mounted $DEVICE || ts_log "Cannot find $DEVICE in /proc/mounts" +$TS_CMD_UMOUNT $DEVICE >> $TS_OUTPUT 2>> $TS_ERRLOG +ts_finalize_subtest ts_fstab_clean -ts_log "Success" ts_finalize - diff --git a/tests/ts/mount/fstab-uuid2devname b/tests/ts/mount/fstab-uuid2devname index 4ce72556c0..cd51a2f97c 100755 --- a/tests/ts/mount/fstab-uuid2devname +++ b/tests/ts/mount/fstab-uuid2devname @@ -34,23 +34,26 @@ DEVICE=$TS_LODEV mkfs.ext2 $DEVICE &> /dev/null || ts_die "Cannot make ext2 on $DEVICE" UUID=$(ts_uuid_by_devname "$DEVICE") || ts_die "Cannot find UUID on $DEVICE" -[ -d "$TS_MOUNTPOINT" ] || mkdir -p $TS_MOUNTPOINT - ts_fstab_add "$DEVICE" ts_udevadm_settle "$DEVICE" "UUID" -# variant A) +# Mountpoint has to be created here rather than in each subtest since ts_fstab_add writes +# this mountpoint to the /etc/fstab file and the mountpoint is needed in subtests +[ -d "$TS_MOUNTPOINT" ] || mkdir -p $TS_MOUNTPOINT + +ts_init_subtest "U-option" $TS_CMD_MOUNT -U $UUID >> $TS_OUTPUT 2>> $TS_ERRLOG -ts_is_mounted $DEVICE || ts_die "A) Cannot find $DEVICE in /proc/mounts" -$TS_CMD_UMOUNT $DEVICE || ts_die "A) Cannot umount $DEVICE" +ts_is_mounted $DEVICE || ts_log "Cannot find $DEVICE in /proc/mounts" +$TS_CMD_UMOUNT $DEVICE >> $TS_OUTPUT 2>> $TS_ERRLOG +ts_finalize_subtest -# variant B) +ts_init_subtest "UUID-option" $TS_CMD_MOUNT "UUID=$UUID" >> $TS_OUTPUT 2>> $TS_ERRLOG -ts_is_mounted $DEVICE || ts_die "B) Cannot find $DEVICE in /proc/mounts" -$TS_CMD_UMOUNT $DEVICE || ts_die "B) Cannot umount $DEVICE" +ts_is_mounted $DEVICE || ts_log "Cannot find $DEVICE in /proc/mounts" +$TS_CMD_UMOUNT $DEVICE >> $TS_OUTPUT 2>> $TS_ERRLOG +ts_finalize_subtest ts_fstab_clean -ts_log "Success" ts_finalize diff --git a/tests/ts/mount/fstab-uuid2label b/tests/ts/mount/fstab-uuid2label index b1f983df1c..e7d4db07b1 100755 --- a/tests/ts/mount/fstab-uuid2label +++ b/tests/ts/mount/fstab-uuid2label @@ -37,23 +37,25 @@ ts_device_has "LABEL" $LABEL $DEVICE \ || ts_die "Cannot find LABEL '$LABEL' on $DEVICE" UUID=$(ts_uuid_by_devname "$DEVICE") || ts_die "Cannot find UUID on $DEVICE" -[ -d "$TS_MOUNTPOINT" ] || mkdir -p $TS_MOUNTPOINT - ts_fstab_add "LABEL=$LABEL" ts_udevadm_settle "$DEVICE" "LABEL" "UUID" -# variant A) +# Mountpoint has to be created here rather than in each subtest since ts_fstab_add writes +# this mountpoint to the /etc/fstab file and the mountpoint is needed in subtests +[ -d "$TS_MOUNTPOINT" ] || mkdir -p $TS_MOUNTPOINT + +ts_init_subtest "U-option" $TS_CMD_MOUNT -U $UUID >> $TS_OUTPUT 2>> $TS_ERRLOG -ts_is_mounted $DEVICE || ts_die "A) Cannot find $DEVICE in /proc/mounts" -$TS_CMD_UMOUNT $DEVICE || ts_die "A) Cannot umount $DEVICE" +ts_is_mounted $DEVICE || ts_log "Cannot find $DEVICE in /proc/mounts" +$TS_CMD_UMOUNT $DEVICE >> $TS_OUTPUT 2>> $TS_ERRLOG +ts_finalize_subtest -# variant B) +ts_init_subtest "UUID-option" $TS_CMD_MOUNT "UUID=$UUID" >> $TS_OUTPUT 2>> $TS_ERRLOG -ts_is_mounted $DEVICE || ts_die "B) Cannot find $DEVICE in /proc/mounts" -$TS_CMD_UMOUNT $DEVICE || ts_die "B) Cannot umount $DEVICE" +ts_is_mounted $DEVICE || ts_log "Cannot find $DEVICE in /proc/mounts" +$TS_CMD_UMOUNT $DEVICE >> $TS_OUTPUT 2>>$TS_ERRLOG +ts_finalize_subtest ts_fstab_clean -ts_log "Success" ts_finalize - diff --git a/tests/ts/mount/label b/tests/ts/mount/label index a5b9622dfb..cd177841c5 100755 --- a/tests/ts/mount/label +++ b/tests/ts/mount/label @@ -37,19 +37,20 @@ mkfs.ext2 -L $LABEL $DEVICE &> /dev/null || ts_die "Cannot make ext2 on $DEVICE" ts_device_has "LABEL" $LABEL $DEVICE \ || ts_die "Cannot find LABEL '$LABEL' on $DEVICE" -[ -d "$TS_MOUNTPOINT" ] || mkdir -p $TS_MOUNTPOINT ts_udevadm_settle "$DEVICE" "LABEL" -# variant A) +ts_init_subtest "L-option" +[ -d "$TS_MOUNTPOINT" ] || mkdir -p $TS_MOUNTPOINT $TS_CMD_MOUNT -L $LABEL $TS_MOUNTPOINT >> $TS_OUTPUT 2>> $TS_ERRLOG -ts_is_mounted $DEVICE || ts_die "A) Cannot find $DEVICE in /proc/mounts" -$TS_CMD_UMOUNT $DEVICE || ts_die "A) Cannot umount $DEVICE" +ts_is_mounted $DEVICE || ts_log "Cannot find $DEVICE in /proc/mounts" +$TS_CMD_UMOUNT $DEVICE >> $TS_OUTPUT 2>> $TS_ERRLOG +ts_finalize_subtest -# variant B) +ts_init_subtest "LABEL-option" +[ -d "$TS_MOUNTPOINT" ] || mkdir -p $TS_MOUNTPOINT $TS_CMD_MOUNT LABEL=$LABEL $TS_MOUNTPOINT >> $TS_OUTPUT 2>> $TS_ERRLOG -ts_is_mounted $DEVICE || ts_die "B) Cannot find $DEVICE in /proc/mounts" -$TS_CMD_UMOUNT $DEVICE || ts_die "B) Cannot umount $DEVICE" +ts_is_mounted $DEVICE || ts_log "Cannot find $DEVICE in /proc/mounts" +$TS_CMD_UMOUNT $DEVICE >> $TS_OUTPUT 2>> $TS_ERRLOG +ts_finalize_subtest -ts_log "Success" ts_finalize - diff --git a/tests/ts/mount/uuid b/tests/ts/mount/uuid index ddf05f2497..db0376642a 100755 --- a/tests/ts/mount/uuid +++ b/tests/ts/mount/uuid @@ -34,19 +34,20 @@ DEVICE=$TS_LODEV mkfs.ext2 $DEVICE &> /dev/null || ts_die "Cannot make ext2 on $DEVICE" UUID=$(ts_uuid_by_devname "$DEVICE") || ts_die "Cannot find UUID on $DEVICE" -[ -d "$TS_MOUNTPOINT" ] || mkdir -p $TS_MOUNTPOINT ts_udevadm_settle "$DEVICE" "UUID" -# variant A) +ts_init_subtest "U-option" +[ -d "$TS_MOUNTPOINT" ] || mkdir -p $TS_MOUNTPOINT $TS_CMD_MOUNT -U $UUID $TS_MOUNTPOINT >> $TS_OUTPUT 2>> $TS_ERRLOG -ts_is_mounted $DEVICE || ts_die "A) Cannot find $DEVICE in /proc/mounts" -$TS_CMD_UMOUNT $DEVICE || ts_die "A) Cannot umount $DEVICE" +ts_is_mounted $DEVICE || ts_log "Cannot find $DEVICE in /proc/mounts" +$TS_CMD_UMOUNT $DEVICE >> $TS_OUTPUT 2>> $TS_ERRLOG +ts_finalize_subtest -# variant B) +ts_init_subtest "UUID-option" +[ -d "$TS_MOUNTPOINT" ] || mkdir -p $TS_MOUNTPOINT $TS_CMD_MOUNT UUID=$UUID $TS_MOUNTPOINT >> $TS_OUTPUT 2>> $TS_ERRLOG -ts_is_mounted $DEVICE || ts_die "B) Cannot find $DEVICE in /proc/mounts" -$TS_CMD_UMOUNT $DEVICE || ts_die "B) Cannot umount $DEVICE" +ts_is_mounted $DEVICE || ts_log "Cannot find $DEVICE in /proc/mounts" +$TS_CMD_UMOUNT $DEVICE >> $TS_OUTPUT 2>> $TS_ERRLOG +ts_finalize_subtest -ts_log "Success" ts_finalize -