From bbd3f18178da3d6ff6c7de5adff76440c4ceb95d Mon Sep 17 00:00:00 2001
From: Kevin O'Connor <kevin@koconnor.net>
Date: Thu, 5 Jul 2018 11:49:10 -0400
Subject: [PATCH] test: Clean up travis-ci build output

Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
---
 scripts/travis-build.sh | 36 ++++++++++++++++++++++++++----------
 1 file changed, 26 insertions(+), 10 deletions(-)

diff --git a/scripts/travis-build.sh b/scripts/travis-build.sh
index 4c4a185b..be1e4336 100755
--- a/scripts/travis-build.sh
+++ b/scripts/travis-build.sh
@@ -2,7 +2,7 @@
 # Test script for travis-ci.org continuous integration.
 
 # Stop script early on any error; check variables; be verbose
-set -eux
+set -eu
 
 # Paths to tools installed by travis-install.sh
 MAIN_DIR=${PWD}
@@ -12,12 +12,30 @@ export PATH=${BUILD_DIR}/pru-gcc/bin:${PATH}
 PYTHON=${BUILD_DIR}/python-env/bin/python
 
 
+######################################################################
+# Travis CI helpers
+######################################################################
+
+start_test()
+{
+    echo "travis_fold:start:$1"
+    echo "=============== $2"
+    set -x
+}
+
+finish_test()
+{
+    set +x
+    echo "=============== Finished $2"
+    echo "travis_fold:end:$1"
+}
+
+
 ######################################################################
 # Check for whitespace errors
 ######################################################################
 
-echo "travis_fold:start:check_whitespace"
-echo "=============== Check whitespace"
+start_test check_whitespace "Check whitespace"
 WS_DIRS="config/ docs/ klippy/ scripts/ src/ test/"
 WS_EXCLUDE="-path scripts/kconfig -prune"
 WS_FILES="-o -iname '*.[csh]' -o -name '*.py' -o -name '*.sh'"
@@ -25,7 +43,7 @@ WS_FILES="$WS_FILES -o -name '*.md' -o -name '*.cfg'"
 WS_FILES="$WS_FILES -o -name '*.test' -o -name '*.config'"
 WS_FILES="$WS_FILES -o -iname '*.lds' -o -iname 'Makefile' -o -iname 'Kconfig'"
 eval find $WS_DIRS $WS_EXCLUDE $WS_FILES | xargs ./scripts/check_whitespace.py
-echo "travis_fold:end:check_whitespace"
+finish_test check_whitespace "Check whitespace"
 
 
 ######################################################################
@@ -36,8 +54,7 @@ DICTDIR=${BUILD_DIR}/dict
 mkdir -p ${DICTDIR}
 
 for TARGET in test/configs/*.config ; do
-    echo "travis_fold:start:mcu_compile $TARGET"
-    echo "=============== Test compile $TARGET"
+    start_test mcu_compile "$TARGET"
     make clean
     make distclean
     unset CC
@@ -45,7 +62,7 @@ for TARGET in test/configs/*.config ; do
     make olddefconfig
     make V=1
     cp out/klipper.dict ${DICTDIR}/$(basename ${TARGET} .config).dict
-    echo "travis_fold:end:mcu_compile $TARGET"
+    finish_test mcu_compile "$TARGET"
 done
 
 
@@ -56,7 +73,6 @@ done
 HOSTDIR=${BUILD_DIR}/hosttest
 mkdir -p ${HOSTDIR}
 
-echo "travis_fold:start:klippy"
-echo "=============== Test invoke klippy"
+start_test klippy "Test invoke klippy"
 $PYTHON scripts/test_klippy.py -d ${DICTDIR} test/klippy/*.test
-echo "travis_fold:end:klippy"
+finish_test klippy "Test invoke klippy"