fstests: fix group list generation for whacky test names
authorDave Chinner <dchinner@redhat.com>
Tue, 17 May 2022 07:01:03 +0000 (17:01 +1000)
committerZorro Lang <zlang@kernel.org>
Sat, 21 May 2022 18:51:35 +0000 (02:51 +0800)
Darrick noticed that tests/xfs/191-input-validation didn't get
generated properly. Fix the regex to handle this.

$ grep -I -R "^_begin_fstest" tests/xfs | \
  sed -e 's/^.*\/\([0-9]*\):_begin_fstest/\1/' |grep 191
tests/xfs/191-input-validation:_begin_fstest auto quick mkfs realtime
$
$ grep -I -R "^_begin_fstest" tests/xfs | \
  sed -e 's/^.*\/\([0-9]*\).*:_begin_fstest/\1/ ' |grep 191
191 auto quick mkfs realtime
$

Use the regexes for matching test names defined in common/test_names
rather than trying to open code it.

Signed-off-by: Dave Chinner <dchinner@redhat.com>
tools/mkgroupfile

index 244358989774f9aca19d1b4de06472cc8b95ded7..414cb538c21a9ec91989b7a40371a3ff1a2ff669 100755 (executable)
@@ -19,6 +19,8 @@ if [ ! -x ../../check ]; then
        exit 1
 fi
 
        exit 1
 fi
 
+. ../../common/test_names
+
 cleanup()
 {
        rm -f $new_groups.check
 cleanup()
 {
        rm -f $new_groups.check
@@ -60,7 +62,8 @@ ENDL
 
        # Aggregate the groups each test belongs to for the group file
        grep -I -R "^_begin_fstest" $test_dir/ | \
 
        # Aggregate the groups each test belongs to for the group file
        grep -I -R "^_begin_fstest" $test_dir/ | \
-               sed -e 's/^.*\/\([0-9]*\):_begin_fstest/\1/' >> $new_groups
+               sed -e "s/^.*\/\($VALID_TEST_NAME\):_begin_fstest/\1/" \
+               >> $new_groups
 
        # Create the list of unique groups for existence checking
        grep -I -R "^_begin_fstest" $test_dir/ | \
 
        # Create the list of unique groups for existence checking
        grep -I -R "^_begin_fstest" $test_dir/ | \