From 5d0f4aaa99f09aa7d4efe12cdcb5b76a69ed29aa Mon Sep 17 00:00:00 2001 From: Denis Vlasenko Date: Sun, 27 Apr 2008 13:34:43 +0000 Subject: runtest: even more fixes from Cristian --- testsuite/runtest | 28 +++++++++++++++------------- 1 file changed, 15 insertions(+), 13 deletions(-) (limited to 'testsuite') diff --git a/testsuite/runtest b/testsuite/runtest index 5496395fa..34f0de2b3 100755 --- a/testsuite/runtest +++ b/testsuite/runtest @@ -16,16 +16,16 @@ run_applet_testcase() local uc_applet=$(echo "$applet" | tr a-z A-Z) local testname="$testcase" - testname=${testname##*/} # take basename - if grep -q "^# CONFIG_$uc_applet is not set$" "$bindir/.config"; then + testname="${testname##*/}" # take basename + if grep "^# CONFIG_$uc_applet is not set$" "$bindir/.config" >/dev/null; then echo "UNTESTED: $testname" return 0 fi - if grep -q "^# FEATURE: " "$testcase"; then + if grep "^# FEATURE: " "$testcase" >/dev/null; then local feature=$(sed -ne 's/^# FEATURE: //p' "$testcase") - if grep -q "^# $feature is not set$" "$bindir/.config"; then + if grep "^# $feature is not set$" "$bindir/.config" >/dev/null; then echo "UNTESTED: $testname" return 0 fi @@ -56,7 +56,7 @@ run_applet_testcase() # Run all old-style tests for given applet run_applet_tests() { - local applet=$1 + local applet="$1" local status=0 for testcase in "$tsdir/$applet"/*; do @@ -82,8 +82,8 @@ run_applet_tests() lcwd=$(pwd) -[ "$tsdir" ] || tsdir="$lcwd" -[ "$bindir" ] || bindir="${lcwd%/*}" # one directory up from $lcwd +[ x"$tsdir" = x ] || tsdir="$lcwd" +[ x"$bindir" = x ] || bindir="${lcwd%/*}" # one directory up from $lcwd PATH="$bindir:$PATH" if [ x"$VERBOSE" = x ]; then @@ -116,15 +116,17 @@ LINKSDIR="$bindir/runtest-tempdir-links" rm -rf "$LINKSDIR" 2>/dev/null mkdir "$LINKSDIR" for i in $implemented; do - ln -s "$bindir/busybox" "$LINKSDIR"/$i + ln -s "$bindir/busybox" "$LINKSDIR/$i" done # Set up option flags so tests can be selective. -export OPTIONFLAGS=:$(sed -nr 's/^CONFIG_//p' $bindir/.config | sed 's/=.*//' | xargs | sed 's/ /:/g') +export OPTIONFLAGS=:$(sed -nr 's/^CONFIG_//p' "$bindir/.config" | sed 's/=.*//' | xargs | sed 's/ /:/g') status=0 for applet in $applets; do - if [ "$applet" = "links" ]; then continue; fi + if [ "$applet" = "links" ]; then + continue + fi # Any old-style tests for this applet? if [ "$applet" != "CVS" -a -d "$tsdir/$applet" ]; then @@ -133,7 +135,7 @@ for applet in $applets; do fi # Is this a new-style test? - if [ -f "${applet}.tests" ]; then + if [ -f "$applet.tests" ]; then if [ ! -h "$LINKSDIR/$applet" ]; then # (avoiding bash'ism "${applet:0:4}") if ! echo "$applet" | grep "^all_" >/dev/null; then @@ -141,8 +143,8 @@ for applet in $applets; do continue fi fi -# echo "Running test ${tsdir:-.}/${applet}.tests" - PATH="$LINKSDIR:$tsdir:$bindir:$PATH" "${tsdir:-.}/${applet}.tests" +# echo "Running test ${tsdir:-.}/$applet.tests" + PATH="$LINKSDIR:$tsdir:$bindir:$PATH" "${tsdir:-.}/$applet.tests" test $? -eq 0 || status=1 fi done -- cgit v1.2.3