Hi!
> +     local get_size=$(blockdev --getsize64 ${TST_DEVICE})
> +     # 20971520 = 20 * 1024 * 1024 = 20M
> +     if [ ${get_size} != "20971520" ]; then
> +             tst_brkm TCONF "${TST_DEVICE} size is ${get_size}, excepted 20M"
> +     fi

Why can't we calculate the expected free space based on the size of the
device? Does the space taken by the ext2 internal data structures
vary unpredictedly?

> +     ROD_SILENT mkfs.ext2 ${TST_DEVICE}
> +
> +     ROD_SILENT mkdir -p mntpoint
> +
> +     ROD_SILENT mount ${TST_DEVICE} mntpoint
> +}
> +
> +cleanup()
> +{
> +     grep -q mntpoint /proc/self/mounts
> +     if [ $? -eq 0 ]; then
> +             umount mntpoint
> +             if [ $? -ne 0 ];then
> +                     tst_resm TWARN "'umount mntpoint' failed"
> +             fi
> +     else
> +             tst_resm TINFO "mntpoint is not mounted"
> +     fi
> +
> +     tst_release_device
> +
> +     tst_rmdir
> +}
> +
> +df_test()
> +{
> +     $1 >${TCID}.temp 2>&1

You do tst_tmpdir in the setup, which creates unique directory for the
testcase. There is no need to construct complicated names for temporary
files. Simple $1 > out 2>&1 should be fine.

> +     if [ $? -ne 0 ]; then
> +             grep -q -E "unrecognized option | invalid option" ${TCID}.temp
> +             if [ $? -eq 0 ]; then
> +                     tst_resm TCONF "'$1' not supported."
> +                     return
> +             else
> +                     tst_resm TFAIL "'$1' failed."
> +                     return
> +             fi
> +     fi
> +
> +     grep ${TST_DEVICE} ${TCID}.temp | grep mntpoint | grep -q $2
> +     if [ $? -eq 0 ]; then
> +             tst_resm TPASS "'$1' passed."
> +     else
> +             tst_resm TFAIL "'$1' failed."
> +     fi
> +}
> +
> +test1()
> +{
> +     df_test "df" "19827"
> +}
> +
> +test2()
> +{
> +     df_test "df -a" "19827"
> +}
> +
> +test3()
> +{
> +     df_test "df -h" "20M"
> +}
> +
> +test4()
> +{
> +     df_test "df -H" "21M"
> +}
> +
> +test5()
> +{
> +     df_test "df -i" "5136"
> +}
> +
> +test6()
> +{
> +     df_test "df -k" "19827"
> +}
> +
> +test7()
> +{
> +     df_test "df -m" "20"
> +}
> +
> +test8()
> +{
> +     df_test "df -t ext2" "19827"
> +}
> +
> +test9()
> +{
> +     df_test "df -T" "19827"
> +}
> +
> +test10()
> +{
> +     df_test "df -v mntpoint" "19827"
> +}
> +
> +test11()
> +{
> +     df -x ext2 >${TCID}.temp 2>&1
> +     if [ $? -ne 0 ]; then
> +             grep -q -E "unrecognized option | invalid option" ${TCID}.temp
> +             if [ $? -eq 0 ]; then
> +                     tst_resm TCONF "'df -x ext2' not supported."
> +                     return
> +             else
> +                     tst_resm TFAIL "'df -x ext2' failed."
> +                     return
> +             fi
> +     fi
> +
> +     grep ${TST_DEVICE} ${TCID}.temp | grep -q mntpoint
> +     if [ $? -ne 0 ]; then
> +             tst_resm TPASS "'df -x ext2' passed."
> +     else
> +             tst_resm TFAIL "'df -x ext2' failed."
> +     fi
> +}
> +
> +test12()
> +{
> +     df --version >${TCID}.temp 2>&1
> +     if [ $? -ne 0 ]; then
> +             grep -q -E "unrecognized option | invalid option" ${TCID}.temp
> +             if [ $? -eq 0 ]; then
> +                     tst_resm TCONF "'df --version' not supported."
> +                     return
> +             else
> +                     tst_resm TFAIL "'df --version' failed."
> +                     return
> +             fi
> +     else
> +             tst_resm TPASS "'df --version' passed."
> +     fi
> +}
> +
> +TST_CLEANUP="cleanup"
> +setup
> +
> +for i in $(seq 1 ${TST_TOTAL})
> +do
> +     test$i
> +done
> +
> +tst_exit
> -- 
> 1.9.3
> 

-- 
Cyril Hrubis
chru...@suse.cz

------------------------------------------------------------------------------
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to