xfs
[Top] [All Lists]

Re: [PATCH 08/25] xfstests: remove test expunge file support

To: Dave Chinner <david@xxxxxxxxxxxxx>
Subject: Re: [PATCH 08/25] xfstests: remove test expunge file support
From: Phil White <pwhite@xxxxxxx>
Date: Sat, 23 Mar 2013 03:19:47 -0700
Cc: xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <1363350489-22257-9-git-send-email-david@xxxxxxxxxxxxx>
References: <1363350489-22257-1-git-send-email-david@xxxxxxxxxxxxx> <1363350489-22257-9-git-send-email-david@xxxxxxxxxxxxx>
User-agent: Mutt/1.5.21 (2010-09-15)
Looks OK.

Reviewed-by: Phil White <pwhite@xxxxxxx>

On Fri, Mar 15, 2013 at 11:27:52PM +1100, Dave Chinner wrote:
> From: Dave Chinner <dchinner@xxxxxxxxxx>
> 
> Check supports an expunge file for tests - a way of marking tests
> not to run even if you specify it to run. Use is like this:
> 
> $ echo 002 > expunged
> test-2:~/src/xfstests-dev$ sudo ./check 001-003
> FSTYP         -- xfs (debug)
> PLATFORM      -- Linux/x86_64 test-2 3.5.0-rc1-dgc+
> MKFS_OPTIONS  -- -f -bsize=4096 /dev/vdb
> MOUNT_OPTIONS -- /dev/vdb /mnt/scratch
> 
> 001 4s ... 4s
> 002 - expunged
> 003 0s ... 0s
> Ran: 001 002 003
> Not run:1
> Passed all 2 tests
> 
> This is arguably useful(*), but gets in the way of splitting up the
> tests into multiple directories. Remove it for now, but such
> functionality should be considered for re-implementation at a later
> date.
> 
> (*) e.g. having a different "expunged" file for each distro release
> you have to test to avoid the tests known to fail due to fixes or
> features that will never be back ported to older releases....
> 
> Signed-off-by: Dave Chinner <dchinner@xxxxxxxxxx>
> ---
>  check |   18 ------------------
>  1 file changed, 18 deletions(-)
> 
> diff --git a/check b/check
> index d69f948..8a998c4 100755
> --- a/check
> +++ b/check
> @@ -30,7 +30,6 @@ notrun=""
>  interrupt=true
>  diff="diff -u"
>  showme=false
> -expunge=true
>  have_test_arg=false
>  randomize=false
>  here=`pwd`
> @@ -106,10 +105,6 @@ BEGIN    { for (t='$start'; t<='$end'; t++) printf 
> "%03d\n",t }' \
>               if grep -s "^$id " group >/dev/null ; then
>                       # in group file ... OK
>                       echo $id >>$tmp.list
> -             elif [ -f expunged ] && $expunge && \
> -                             egrep "^$id([   ]|\$)" expunged >/dev/null ; 
> then
> -                     # expunged ... will be reported, but not run, later
> -                     echo $id >>$tmp.list
>               else
>                       # oops
>                       echo "$id - unknown test, ignored"
> @@ -293,13 +288,6 @@ END      { if (NR > 0) {
>           mv $tmp.out check.time
>       fi
>  
> -     if [ -f $tmp.expunged ]
> -     then
> -         notrun=`wc -l <$tmp.expunged | sed -e 's/  *//g'`
> -         n_try=`expr $n_try - $notrun`
> -         list=`echo "$list" | sed -f $tmp.expunged`
> -     fi
> -
>       echo "" >>check.log
>       date >>check.log
>       echo $list | fmt | sed -e 's/^/    /' >>check.log
> @@ -393,15 +381,9 @@ do
>      then
>       echo
>       continue
> -    elif [ -f expunged ] && $expunge && egrep "^$seq([       ]|\$)" expunged 
> >/dev/null
> -    then
> -     echo " - expunged"
> -     rm -f $seq.out.bad
> -     echo "/^$seq\$/d" >>$tmp.expunged
>      elif [ ! -f $seq ]
>      then
>       echo " - no such test?"
> -     echo "/^$seq\$/d" >>$tmp.expunged
>      else
>       # really going to try and run this one
>       #
> -- 
> 1.7.10.4
> 
> _______________________________________________
> xfs mailing list
> xfs@xxxxxxxxxxx
> http://oss.sgi.com/mailman/listinfo/xfs

<Prev in Thread] Current Thread [Next in Thread>