mysql-test-run.sh Added spit-polish. Also --verbose is default now.

mysql-test/mysql-test-run.sh:
  Added spit-polish. Also --verbose is default now.

mysql-test/mysql-test-run.sh

@@ -2,6 +2,7 @@
 # mysql-test-run - originally written by Matt Wagner <matt@mysql.com>
 # modified by Sasha Pachev <sasha@mysql.com>
 # Sligtly updated by Monty
+# Cleaned up again by Matt
 
 #++
 # Access Definitions
@@ -25,16 +26,13 @@ else
  if [ -f ./mysql-test-run ] && [ -d ../sql ] ; then
  SOURCE_DIST=1
  else
-  echo "If you are using binary distribution, run me from install root as"
-  echo "scripts/mysql-test-run. On source distribution run me from source root"
-  echo "as mysql-test/mysql-test-run or from mysql-test as ./mysql-test-run"
+  $ECHO "If you are using binary distribution, run from install root as"
+  $ECHO "scripts/mysql-test-run. On source distribution run from source root"
+  $ECHO "as mysql-test/mysql-test-run or from mysql-test as ./mysql-test-run"
   exit 1
  fi
- 
 fi
 
-
-
 #++
 # Misc. Definitions
 #--
@@ -112,8 +110,8 @@ XARGS=`which xargs | head -1`
 
 [ -z "$COLUMNS" ] && COLUMNS=80
 E=`$EXPR $COLUMNS - 8`
-#DASH72=`expr substr '________________________________________________________________________' 1 $E`
-DASH72=`$ECHO '________________________________________________________________________'|$CUT -c 1-$E`
+#DASH72=`expr substr '------------------------------------------------------------------------' 1 $E`
+DASH72=`$ECHO '------------------------------------------------------------------------'|$CUT -c 1-$E`
 
 # on source dist, we pick up freshly build executables
 # on binary, use what is installed
@@ -132,8 +130,7 @@ fi
 
 SLAVE_MYSQLD=$MYSQLD #this will be changed later if we are doing gcov
 
-
-MYSQL_TEST="$MYSQL_TEST --no-defaults --socket=$MASTER_MYSOCK --database=$DB --user=$DBUSER --password=$DBPASSWD --silent"
+MYSQL_TEST="$MYSQL_TEST --no-defaults --socket=$MASTER_MYSOCK --database=$DB --user=$DBUSER --password=$DBPASSWD --silent -v"
 GDB_MASTER_INIT=/tmp/gdbinit.master
 GDB_SLAVE_INIT=/tmp/gdbinit.slave
 
@@ -141,22 +138,21 @@ while test $# -gt 0; do
   case "$1" in
     --force ) FORCE=1 ;;
     --record ) RECORD=1 ;;
-    --verbose) MYSQL_TEST="$MYSQL_TEST -v";;
     --gcov )
       if [ x$BINARY_DIST = x1 ] ; then
-	echo "Cannot do coverage test without the source - please use source dist"
+	$ECHO "Cannot do coverage test without the source - please use source dist"
 	exit 1
       fi
       DO_GCOV=1
       ;;
     --gdb )
       if [ x$BINARY_DIST = x1 ] ; then
-	echo "Note: you will get more meaningful output on a source distribution compiled with debugging option when running tests with -gdb option"
+	$ECHO "Note: you will get more meaningful output on a source distribution compiled with debugging option when running tests with -gdb option"
       fi
       DO_GDB=1
       ;;
     -- )  shift; break ;;
-    --* ) echo "Unrecognized option: $1"; exit 1 ;;
+    --* ) $ECHO "Unrecognized option: $1"; exit 1 ;;
     * ) break ;;
   esac
   shift
@@ -168,19 +164,22 @@ done
 
 prompt_user ()
 {
- echo $1
+ $ECHO $1
  read unused
 }
 
 
 error () {
-
     $ECHO  "Error:  $1"
     exit 1
 }
 
+error_is () {
+    $ECHO `$CAT $TIMEFILE` | $SED -e 's/.* At line .*\: \(.*\)Command .*$/   \>\> Error: \1<\</'
+}
+
 prefix_to_8() {
- echo "        $1" | $SED -e 's:.*\(........\)$:\1:'
+ $ECHO "        $1" | $SED -e 's:.*\(........\)$:\1:'
 }
 
 pass_inc () {
@@ -209,22 +208,22 @@ report_stats () {
 	whole=`$PRINTF %.2s $raw`         
 	xwhole=`$EXPR $whole \* 100`      
 	deci=`$EXPR $raw - $xwhole`       
-	$ECHO  "Failed ${TOT_FAIL}/${TOT_TEST} tests ${whole}.${deci}% successful."
+	$ECHO  "Failed ${TOT_FAIL}/${TOT_TEST} tests, ${whole}.${deci}% successful."
     fi
 }
 
 mysql_install_db () {
-    echo "Removing stale files from previous run"
+    $ECHO "Removing Stale Files"
     $RM -rf $MASTER_MYDDIR $SLAVE_MYDDIR $SLAVE_MYLOG $MASTER_MYLOG \
      $SLAVE_MYERR $MASTER_MYERR
     [ -d $MYRUN_DIR ] || mkdir -p $MYRUN_DIR
-    echo "installing master databases"
+    $ECHO "Installing Master Databases"
     $INSTALL_DB
     if [ $? != 0 ]; then
 	error "Could not install master test DBs"
 	exit 1
     fi
-    echo "Installing slave databases"
+    $ECHO "Installing Slave Databases"
     $INSTALL_DB -slave
     if [ $? != 0 ]; then
 	error "Could not install slave test DBs"
@@ -270,7 +269,7 @@ start_master()
 	    --language=english $EXTRA_MASTER_OPT"
     if [ x$DO_GDB = x1 ]
     then
-      echo "set args $master_args" > $GDB_MASTER_INIT
+      $ECHO "set args $master_args" > $GDB_MASTER_INIT
       xterm -title "Master" -e gdb -x $GDB_MASTER_INIT $MYSQLD &
       prompt_user "Hit enter to continue after you've started the master"
     else	    
@@ -305,7 +304,7 @@ start_slave()
             --language=english $EXTRA_SLAVE_OPT"
     if [ x$DO_GDB = x1 ]
     then
-      echo "set args $slave_args" > $GDB_SLAVE_INIT
+      $ECHO "set args $slave_args" > $GDB_SLAVE_INIT
       xterm -title "Slave" -e gdb -x $GDB_SLAVE_INIT $SLAVE_MYSQLD &
       prompt_user "Hit enter to continue after you've started the slave"
     else
@@ -315,6 +314,7 @@ start_slave()
 }
 
 mysql_start () {
+    $ECHO "Starting MySQL daemon"
     start_master
     start_slave
     cd $MYSQL_TEST_DIR
@@ -327,15 +327,15 @@ stop_slave ()
   then
     $MYSQLADMIN --no-defaults --socket=$SLAVE_MYSOCK -u root shutdown
     if [ $? != 0 ] ; then # try harder!
-     echo "slave not cooperating with mysqladmin, will try manual kill"
+     $ECHO "slave not cooperating with mysqladmin, will try manual kill"
      kill `cat $SLAVE_MYPID`
      sleep 2
      if [ -f $SLAVE_MYPID ] ; then
-       echo "slave refused to die, resorting to SIGKILL murder"
+       $ECHO "slave refused to die, resorting to SIGKILL murder"
        kill -9 `cat $SLAVE_MYPID`
        $RM -f $SLAVE_MYPID
      else
-      echo "slave responded to SIGTERM " 
+      $ECHO "slave responded to SIGTERM " 
      fi
     fi
     SLAVE_RUNNING=0
@@ -348,15 +348,15 @@ stop_master ()
   then
     $MYSQLADMIN --no-defaults --socket=$MASTER_MYSOCK -u root shutdown
     if [ $? != 0 ] ; then # try harder!
-     echo "master not cooperating with mysqladmin, will try manual kill"
+     $ECHO "master not cooperating with mysqladmin, will try manual kill"
      kill `cat $MASTER_MYPID`
      sleep 2
      if [ -f $MASTER_MYPID ] ; then
-       echo "master refused to die, resorting to SIGKILL murder"
+       $ECHO "master refused to die, resorting to SIGKILL murder"
        kill -9 `cat $MASTER_MYPID`
        $RM -f $MASTER_MYPID
      else
-      echo "master responded to SIGTERM " 
+      $ECHO "master responded to SIGTERM " 
      fi
     fi
     MASTER_RUNNING=0
@@ -365,6 +365,9 @@ stop_master ()
 
 mysql_stop ()
 {
+ $ECHO  "Ending Tests"
+ $ECHO  "Shutting-down MySQL daemon"
+ $ECHO
  stop_master
  stop_slave
  return 1
@@ -473,28 +476,29 @@ run_testcase ()
     if [ $res == 0 ]; then
       total_inc
       pass_inc
-      echo "$RES_SPACE [ pass ]"
+      $ECHO "$RES_SPACE [ pass ]"
     else
       if [ $res == 1 ]; then
 	total_inc
         fail_inc
-	echo "$RES_SPACE [ fail ]"
-        $ECHO "failed output"
-	$CAT $TIMEFILE
+	$ECHO "$RES_SPACE [ fail ]"
         $ECHO
+	error_is
 	$ECHO
 	if [ x$FORCE != x1 ] ; then
-	 echo "Aborting, if you want to continue, re-run with --force"
+	 $ECHO "Aborting. To continue, re-run with '--force'."
+	 $ECHO
 	 mysql_stop
 	 exit 1
 	fi
 	 
-	echo "Restarting mysqld"
+#	$ECHO "Restarting mysqld"
 	mysql_restart
-	echo "Resuming Tests"
+	$ECHO "Resuming Tests"
+	$ECHO
       else
         pass_inc
-	echo "$RES_SPACE [ skipped ]"
+	$ECHO "$RES_SPACE [ skipped ]"
       fi
     fi
   fi  
@@ -505,21 +509,20 @@ run_testcase ()
 
 [ "$DO_GCOV" ] && gcov_prepare 
 
-echo "Installing test databases"
+$ECHO "Installing Test Databases"
 mysql_install_db
 
 #do not automagically start deamons if we are in gdb or running only one test
 #case
 if [ -z "$DO_GDB" ] && [ -z "$1" ]
 then
- $ECHO  "Starting mysqld for Testing" 
  mysql_start
 fi
 
-$ECHO  "Loading Standard Test Database"
+$ECHO  "Loading Standard Test Databases"
 mysql_loadstd
 
-$ECHO  "Starting Tests for MySQL daemon"
+$ECHO  "Starting Tests"
 
 $ECHO
 $ECHO " TEST                         USER   SYSTEM  ELAPSED        RESULT"
@@ -528,7 +531,7 @@ $ECHO $DASH72
 if [ -z "$1" ] ;
 then
  if [ x$RECORD = x1 ]; then
-  echo "Will not run in record mode without a specific test case"
+  $ECHO "Will not run in record mode without a specific test case."
  else
   for tf in $TESTDIR/*.$TESTSUFFIX
   do
@@ -541,23 +544,20 @@ else
  if [ -f $tf ] ; then
   run_testcase $tf
  else
-   echo "Test case $tf does not exist"
+   $ECHO "Test case $tf does not exist."
  fi
 fi
 
 $ECHO $DASH72
 $ECHO
-$ECHO  "Ending Tests for MySQL daemon"
+
 $RM -f $TIMEFILE
 
 if [ -z "$DO_GDB" ] ;
 then
-    $ECHO  "Shutdown mysqld"
     mysql_stop
 fi
 
-
-$ECHO
 report_stats
 $ECHO