ext4/044: Add missing check for return value

When calling _within_tolerance either return value or text output
should be checked. Otherwise the function is useless for a test.

Signed-off-by: Pavel Reichl <preichl@redhat.com>
Reviewed-by: Zorro Lang <zlang@redhat.com>
Signed-off-by: Zorro Lang <zlang@kernel.org>

diff --git a/tests/ext4/044 b/tests/ext4/044
index 158e2ac50889856350265e22022268df16d1bd0d..50de5a40bdc78558f163c33e9d94ae6247751db3 100755 (executable)
--- a/tests/ext4/044
+++ b/tests/ext4/044
@@ -18,8 +18,6 @@ _require_scratch
 _require_test_program "t_get_file_time"
 _require_metadata_journaling
 
-echo "Silence is golden"
-
 echo "Test timestamps with 256 inode size one device $SCRATCH_DEV" >$seqres.full
 _scratch_mkfs -t ext3 -I 256 >> $seqres.full 2>&1
 _scratch_mount
@@ -49,9 +47,9 @@ nsec_mtime: $nsec_mtime, nsec_ctime: $nsec_ctime, cur_time[ns]: $nsec)"
 fi
 
 # Check difference between file time and current time
-_within_tolerance "sec_atime" $sec_atime $sec 1
-_within_tolerance "sec_mtime" $sec_mtime $sec 1
-_within_tolerance "sec_ctime" $sec_ctime $sec 1
+_within_tolerance "sec_atime" $sec_atime $sec 1 -v
+_within_tolerance "sec_mtime" $sec_mtime $sec 1 -v
+_within_tolerance "sec_ctime" $sec_ctime $sec 1 -v
 
 _scratch_unmount >> $seqres.full 2>&1
 

diff --git a/tests/ext4/044.out b/tests/ext4/044.out
index 12a61dc44583e1dbd1b296d032df8f817203fbce..8a3a80723b7515631fd217722ad8d269f2034919 100644 (file)
--- a/tests/ext4/044.out
+++ b/tests/ext4/044.out
@@ -1,2 +1,4 @@
 QA output created by 044
-Silence is golden
+sec_atime is in range
+sec_mtime is in range
+sec_ctime is in range