]> granicus.if.org Git - pdns/commitdiff
Test services should log into their own log file
authorThomas D <whissi@whissi.de>
Fri, 13 Feb 2015 01:02:47 +0000 (02:02 +0100)
committerThomas D <whissi@whissi.de>
Tue, 17 Feb 2015 09:15:03 +0000 (10:15 +0100)
Each test has a general log file in the following schema:

  remotebackend_<module>.log

When testrunner.sh spins up a service, the service logs its output
into the general log file. But test-driver from automake, which is
using the same log file and starts after the test service is up and
running, will overwrite the log file.

So in case of a server failure we will miss important log data.

Using a dedicated log file for the services we start will solve
this problem. The new schema for server log files will be

  remotebackend_<module>_server.log

modules/remotebackend/Makefile.am
modules/remotebackend/testrunner.sh

index c6b0cd6147a363d760fc1a0014bc1a1b73b74693..4aa0a548bb1682976c4c92c7292c5f1613cab486 100644 (file)
@@ -71,6 +71,13 @@ TESTS = \
        remotebackend_json.test \
        remotebackend_zeromq.test
 
+RECHECK_LOGS = \
+       $(TEST_LOGS) \
+       remotebackend_http_server.log \
+       remotebackend_post_server.log \
+       remotebackend_json_server.log \
+       remotebackend_zeromq_server.log
+
 ## The http, post and json test are using the same TCP port.
 ## To prevent "Address already in use - bind(2) (Errno::EADDRINUSE)"
 ## errors when running `make check` in parallel, we need to specify
index cf2b8c05fe7e5dbeec189d68bb913aef401cd84c..ae4aa8e51745c47d859e8ee2055f61321c2de39d 100755 (executable)
@@ -22,7 +22,7 @@ zeromq_pid=""
 socat=$(which socat)
 
 function start_web() {
- ./unittest_$1.rb >> $mode.log 2>&1 & 
+ ./unittest_$1.rb >> ${mode%\.test}_server.log 2>&1 &
  webrick_pid=$!
  loopcount=0
  while [ $loopcount -lt 20 ]; do
@@ -49,7 +49,7 @@ function stop_web() {
 
 function start_zeromq() {
   if [ x"$REMOTEBACKEND_ZEROMQ" == "xyes" ]; then
-   ./unittest_zeromq.rb >> $mode.log 2>&1 &
+   ./unittest_zeromq.rb >> ${mode%\.test}_server.log 2>&1 &
    zeromq_pid=$!
    # need to wait a moment
    sleep 5