From fa3a38a7f269f7a33c6176975f171b93025f05d7 Mon Sep 17 00:00:00 2001
From: MarcoFalke <falke.marco@gmail.com>
Date: Mon, 30 Nov 2015 14:53:07 +0100
Subject: [PATCH] [qa] pull-tester: Cleanup (run keypool, tidy stdout)

* Run keypool (takes 6 seconds)
* Print duration of each rpc test
* Structure output (bold, new lines)
---
 qa/pull-tester/rpc-tests.py | 19 ++++++++++++++-----
 1 file changed, 14 insertions(+), 5 deletions(-)

diff --git a/qa/pull-tester/rpc-tests.py b/qa/pull-tester/rpc-tests.py
index 5004b09c18..7ac89e9bc1 100755
--- a/qa/pull-tester/rpc-tests.py
+++ b/qa/pull-tester/rpc-tests.py
@@ -22,6 +22,7 @@ For a description of arguments recognized by test scripts, see
 """
 
 import os
+import time
 import shutil
 import sys
 import subprocess
@@ -47,6 +48,10 @@ opts = set()
 passOn = ""
 p = re.compile("^--")
 
+bold = ("","")
+if (os.name == 'posix'):
+    bold = ('\033[0m', '\033[1m')
+
 for arg in sys.argv[1:]:
     if arg == '--coverage':
         ENABLE_COVERAGE = 1
@@ -92,6 +97,7 @@ testScripts = [
     'blockchain.py',
     'disablewallet.py',
     'sendheaders.py',
+    'keypool.py',
 ]
 testScriptsExt = [
     'bip65-cltv.py',
@@ -105,7 +111,6 @@ testScriptsExt = [
     'pruning.py',
     'forknotify.py',
     'invalidateblock.py',
-    'keypool.py',
 #    'rpcbind_test.py', #temporary, bug in libevent, see #6655
     'smartfees.py',
     'maxblocksinflight.py',
@@ -126,7 +131,7 @@ def runtests():
 
     if ENABLE_COVERAGE:
         coverage = RPCCoverage()
-        print("Initializing coverage directory at %s" % coverage.dir)
+        print("Initializing coverage directory at %s\n" % coverage.dir)
 
     if(ENABLE_WALLET == 1 and ENABLE_UTILS == 1 and ENABLE_BITCOIND == 1):
         rpcTestDir = buildDir + '/qa/rpc-tests/'
@@ -141,10 +146,12 @@ def runtests():
                     or run_extended
                     or testScripts[i] in opts
                     or re.sub(".py$", "", testScripts[i]) in opts ):
-                print("Running testscript " + testScripts[i] + "...")
 
+                print("Running testscript %s%s%s ..." % (bold[1], testScripts[i], bold[0]))
+                time0 = time.time()
                 subprocess.check_call(
                     rpcTestDir + testScripts[i] + flags, shell=True)
+                print("Duration: %s s\n" % (int(time.time() - time0)))
 
                 # exit if help is called so we print just one set of
                 # instructions
@@ -156,12 +163,14 @@ def runtests():
         for i in range(len(testScriptsExt)):
             if (run_extended or testScriptsExt[i] in opts
                     or re.sub(".py$", "", testScriptsExt[i]) in opts):
+
                 print(
                     "Running 2nd level testscript "
-                    + testScriptsExt[i] + "...")
-
+                    + "%s%s%s ..." % (bold[1], testScriptsExt[i], bold[0]))
+                time0 = time.time()
                 subprocess.check_call(
                     rpcTestDir + testScriptsExt[i] + flags, shell=True)
+                print("Duration: %s s\n" % (int(time.time() - time0)))
 
         if coverage:
             coverage.report_rpc_coverage()