Improve MaxScale startup and shutdown in NPM tests

The tests are now less violent in how they stop MaxScale. Also the logs
are now removed before a new test run is executed.

test/maxscale_test.cnf

@@ -1,6 +1,5 @@
 [maxscale]
 threads=4
-log_info=1
 libdir=@CMAKE_INSTALL_PREFIX@/@MAXSCALE_LIBDIR@
 logdir=@CMAKE_INSTALL_PREFIX@/log/maxscale/
 datadir=@CMAKE_INSTALL_PREFIX@/lib/maxscale

test/maxscale_test_secondary.cnf

@@ -1,6 +1,5 @@
 [maxscale]
 threads=4
-log_info=1
 libdir=@CMAKE_INSTALL_PREFIX@/@MAXSCALE_LIBDIR@
 logdir=@CMAKE_INSTALL_PREFIX@/secondary/log/maxscale/
 datadir=@CMAKE_INSTALL_PREFIX@/secondary/lib/maxscale

test/run_npm_test.sh

@@ -18,6 +18,8 @@ testdir=$3
 
 maxscaledir=$PWD/maxscale_test/
 
+rm -f $maxscaledir/maxscale{,1,2}.output $maxscaledir/{,secondary/}log/maxscale/maxscale.log
+
 # Create the test directories
 mkdir -p $maxscaledir $testdir
 

test/stop_double_maxscale.sh

@@ -1,20 +1,3 @@
 #!/bin/bash
 
-#
+./stop_maxscale.sh
-# This script is run after each test block. It kills the two MaxScale processes.
-#
-
-maxscaledir=$MAXSCALE_DIR
-
-test -z "$MAXSCALE_DIR" && exit 1
-
-for ((i=0;i<200;i++))
-do
-    pkill '^maxscale$' || break
-    sleep 0.1
-done
-
-# If it wasn't dead before, now it is
-pkill -9 '^maxscale$'
-
-exit 0

test/stop_maxscale.sh

@@ -8,13 +8,15 @@ maxscaledir=$MAXSCALE_DIR
 
 test -z "$MAXSCALE_DIR" && exit 1
 
-for ((i=0;i<200;i++))
+pkill '^maxscale$'
+
+for ((i=0;i<100;i++))
 do
-    pkill '^maxscale$' || break
+    pgrep '^maxscale$' &> /dev/null || break
     sleep 0.1
 done
 
 # If it wasn't dead before, now it is
-pkill -9 '^maxscale$'
+pgrep '^maxscale$' &> /dev/null && pkill -11 '^maxscale$'
 
 exit 0