From a4d007ec1dd00f78a2b58416c37bf31bee2edf3e Mon Sep 17 00:00:00 2001
From: Rainer Gerhards <rgerhards@adiscon.com>
Date: Wed, 30 Nov 2016 14:25:32 +0100
Subject: [PATCH] travis: output testbench detail result

---
 .travis.yml               | 5 ++++-
 configure.ac              | 4 ++--
 tests/Makefile.am         | 9 +++++----
 tests/perf_to_string.test | 1 +
 4 files changed, 12 insertions(+), 7 deletions(-)

diff --git a/.travis.yml b/.travis.yml
index e5bde71..0ccaf48 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -28,7 +28,9 @@ install:
   - sh autogen.sh
 
 before_script:
-  - sudo apt-get -qq --force-yes install valgrind
+  - sudo add-apt-repository ppa:adiscon/v8-stable -y
+  - sudo apt-get update -qq
+  - sudo apt-get -qq --force-yes install valgrind libfastjson
   - # note: valgrind is only available on Linux
   - if [ "$CC" == "gcc" ] && [ "$TRAVIS_OS_NAME" == "linux" ]; then export ENA_VG="--enable-valgrind"; fi
   - if [ "$CC" == "clang" ] && [ "$TRAVIS_OS_NAME" == "linux" ]; then export SANITIZER="-fsanitize=address" ; fi
@@ -42,5 +44,6 @@ script:
   - valgrind --tool=callgrind ls
   - export VERBOSE=1
   - make check
+  - cat tests/test-suite.log
   - cat tests/perf_to_string.trs
   - make distcheck
diff --git a/configure.ac b/configure.ac
index 8cb8b04..d73f1d9 100644
--- a/configure.ac
+++ b/configure.ac
@@ -90,8 +90,8 @@ AC_ARG_ENABLE(performance_testbench,
         [enable_performance_testbench="no"]
 )
 if test "$enable_performance_testbench" = "yes"; then
-        AC_CHECK_PROG(VALGRIND, [valgrind], [valgrind], [no])
-	if test "x$VALGRIND" = "xno"; then
+        AC_CHECK_PROG(VALGRIND_CALLGRIND, [valgrind], [valgrind], [no])
+	if test "x$VALGRIND_CALLGRIND" = "xno"; then
 		AC_MSG_ERROR([--enable-performance-testbench given, but valgrind is not present on system])
 	fi
 fi
diff --git a/tests/Makefile.am b/tests/Makefile.am
index 231cc06..dbcd3ab 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -32,8 +32,7 @@ check_PROGRAMS += $(TESTS:.test=)
 
 # some programs that do internal checking
 check_PROGRAMS += chk_version \
-	cr_obj_multi \
-	perf_to_string
+	cr_obj_multi
 
 TESTS += chk_version
 
@@ -62,9 +61,10 @@ test_object_object_add_exFormatted_SOURCES = test_object_object_add_ex.c parse_f
 test_object_object_add_exFormatted_CPPFLAGS = -DTEST_FORMATTED
 
 # performancebaseline tests
-# TODO: guard via configure switch! -- rgerhards, 2016-11-29
+if ENABLE_PERFORMANCE_TESTBENCH
+check_PROGRAMS += perf_to_string
 TESTS += perf_to_string.test
-
+endif
 
 
 EXTRA_DIST=
@@ -97,6 +97,7 @@ EXTRA_DIST += test_object_object_add_exFormatted_pretty.expected
 EXTRA_DIST += test_object_object_add_exFormatted_spaced.expected
 EXTRA_DIST += test_many_subobj.expected
 EXTRA_DIST += test_obj_obj_get_ex-null.expected
+EXTRA_DIST += perf_to_string.c
 
 testsubdir=testSubDir
 TESTS_ENVIRONMENT       = top_builddir=$(top_builddir)
diff --git a/tests/perf_to_string.test b/tests/perf_to_string.test
index 441276d..a088ffe 100755
--- a/tests/perf_to_string.test
+++ b/tests/perf_to_string.test
@@ -13,6 +13,7 @@ valgrind --tool=callgrind --callgrind-out-file=callgrind.out .libs/perf_to_strin
 if [ $? -ne 0 ]; then
     echo "Error obtaining performance baseline, assuming system is not able to"
     echo "support this test - skipping"
+    rm callgrind.out
     exit 77
 fi
 export RETCODE=$?