From c8fd5f2338c743f0bfd468999820c318a45ab737 Mon Sep 17 00:00:00 2001 From: Andreas Dilger Date: Wed, 30 Aug 2017 02:05:25 -0400 Subject: [PATCH] tests: report if a test is taking a long time Print out a message if a test takes longer than 60s, with a reminder to potentially add is_slow_test to that test, so that it is easier to find which tests are taking a long time, especially if running with "make -j check" or similar. Add an exclusion for r_expand_full on MacOS since HFS+ does not support sparse devices and doesn't like large test images. Signed-off-by: Andreas Dilger Signed-off-by: Theodore Ts'o --- tests/r_expand_full/script | 6 ++++++ tests/test_one.in | 6 ++++++ 2 files changed, 12 insertions(+) diff --git a/tests/r_expand_full/script b/tests/r_expand_full/script index 861a68a..672829f 100644 --- a/tests/r_expand_full/script +++ b/tests/r_expand_full/script @@ -23,6 +23,12 @@ cat > $CONF << ENDL } ENDL +if [ $(uname -s) = "Darwin" ]; then + # creates a 786MB filesystem + echo "$test_name: $DESCRIPTION: skipped for HFS+ (no sparse files)" + return 0 +fi + echo "resize2fs test" > $OUT MKE2FS_CONFIG=$CONF $MKE2FS -F -T ext4h $TMPFILE 786432 >> $OUT 2>&1 diff --git a/tests/test_one.in b/tests/test_one.in index 94f56e7..b20e207 100644 --- a/tests/test_one.in +++ b/tests/test_one.in @@ -63,6 +63,7 @@ fi rm -f $test_name.ok $test_name.failed #echo -e -n "$test_name: $test_description:\r" +start=$SECONDS if [ -f $test_dir/script ]; then . $test_dir/script else @@ -74,6 +75,11 @@ else echo "$test_name: Missing test script $default_script!" fi fi +elapsed=$((SECONDS - start)) +if [ $elapsed -gt 60 -a ! -f $test_dir/is_slow_test ]; then + echo "$test_name: *** took $elapsed seconds to finish ***" + echo "$test_name: consider adding $test_dir/is_slow_test" +fi if [ "$SKIP_UNLINK" != "true" ] ; then rm -f $TMPFILE -- 1.8.3.1