]> git.apps.os.sepia.ceph.com Git - xfstests-dev.git/commitdiff
xfstests: use _notrun for tape checks
authorDave Chinner <dchinner@redhat.com>
Fri, 15 Mar 2013 12:28:09 +0000 (12:28 +0000)
committerRich Johnston <rjohnston@sgi.com>
Wed, 27 Mar 2013 02:53:50 +0000 (21:53 -0500)
_require_tape is failing tests when no TAPE_DEV is specified. It
should be using _notrun for these cases rather than trying to open
code the $seqres.notrun magic file manipulations.

Signed-off-by: Dave Chinner <dchinner@redhat.com>
Reviewed-by: Phil White <pwhite@sgi.com>
Signed-off-by: Rich Johnston <rjohnston@sgi.com>
common/dump

index 3df619125018c945e0b3a2793906623e47f8e3ea..5d4d2aea954e23d79af4e2f122eff063fcfb1f06 100644 (file)
@@ -139,9 +139,7 @@ _check_onl()
        echo "ERROR: mt -f $dumptape failed"
        cat $tmp.status
 
-       echo "mt -f $dumptape failed" >$seq.notrun
-       status=$NOTRUNSTS
-       exit
+       _notrun "mt -f $dumptape failed"
     fi
 
 
@@ -151,9 +149,7 @@ _check_onl()
        echo "ERROR: $dumptape is not online"
        cat $tmp.status
 
-       echo "dumptape, $dumptape, is not online" >$seq.notrun
-       status=$NOTRUNSTS
-       exit
+       _notrun "dumptape, $dumptape, is not online"
     fi
 }
 
@@ -229,10 +225,7 @@ _require_tape()
     dumptape=$1
 
     if [ -z "$dumptape" -o "@" == "$dumptape" ]; then
-        echo "This test requires a dump tape - none was specified"
-       echo "No dump tape specified" >$seqres.notrun
-       status=$NOTRUNSTS
-       exit
+       _notrun "No dump tape specified"
     fi
 
     _check_onl