BF: moved running all the scripts via coverage into a separate helper tool

Author: yarikoptic

.travis.yml

@@ -23,7 +23,7 @@ script:
   # Run tests
   - PYTHONPATH=. nosetests -s -v --with-doctest --with-cov --cover-package . --logging-level=INFO -v .
   # Actually run all the scripts, contributing to coverage
-  - bash -c 'failed=""; for f in [^_]*py; do python `which coverage` run -a $f || failed+=" $f"; echo "I: done $f. Exit code $?"; done; [ -z "$failed" ] || { echo "Failed: $failed"; exit 1; }'
+  - ./run_all.sh
   # for now failure in flaking is ignored
   - flake8 *py || echo "PEP8 the code"
 

run_all.sh

@@ -0,0 +1,22 @@
+#!/bin/bash
+#
+# Little helper to run all the scripts, under python coverage if coverage is available
+#
+
+set -eu
+failed=""
+
+if which coverage > /dev/null; then
+    COVERAGE="`which coverage` run -a"
+else
+    COVERAGE=''
+fi
+for f in [^_]*py; do
+    python $COVERAGE $f || failed+=" $f"
+    echo "I: done $f. Exit code $?"
+done;
+
+if [ ! -z "$failed" ]; then
+    echo "Failed: $failed";
+    exit 1
+fi