On Wed, Oct 24, 2018 at 10:33:07AM -0400, Masayoshi Mizuma wrote:
> From: Masayoshi Mizuma <m.mizuma@xxxxxxxxxxxxxx>
>
> If we run tests using '*', for example './check xfs/*', we will
> get following error messages. This patch introduces to avoid
> the messages.
>
> ./check xfs/*
> xfs/001.out - unknown test, ignored
> xfs/002.out - unknown test, ignored
> xfs/003.out - unknown test, ignored
> xfs/004.out - unknown test, ignored
> ...
Hmm, that's just not the recommended way to run tests. Not all tests are
valid (yes, there're broken tests) and sometimes we may forget to add x
permission on a test (though I have a local script to catch that, but it
did happen before).
Please either use a more precise pattern to match the tests you want to
run, or even better, run tests by specifying the groups, e.g.
./check -g xfs/log
to run all tests in 'log' group in 'xfs' directory.
Thanks,
Eryu
>
> Signed-off-by: Masayoshi Mizuma <m.mizuma@xxxxxxxxxxxxxx>
> ---
> check | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/check b/check
> index f3c8021d..96cb222f 100755
> --- a/check
> +++ b/check
> @@ -339,6 +339,10 @@ if $have_test_arg; then
> test_dir=`dirname $t`
> test_dir=${test_dir#$SRC_DIR/*}
> test_name=`basename $t`
> + if ! file -bi $SRC_DIR/$test_dir/$test_name | \
> + egrep -q 'text/x-shellscript'; then
> + continue
> + fi
> group_file=$SRC_DIR/$test_dir/group
>
> if egrep -q "^$test_name" $group_file; then
> --
> 2.18.0
>
