Travis: move coverage processing to after_success

Move the processing of coverage information to the `after_success` step
to ensure it cannot alter the state of the build anymore. It would do
this on private forks if people don't have a coveralls setup for their
private fork.

Author: muggenhor

.travis.yml

@@ -50,4 +50,19 @@ install:
 
 script: ./travis.sh
 
+after_success:
+ - if [ -n "${COVERALLS_SERVICE_NAME}" ]; then
+      lcov --rc lcov_branch_coverage=1 --directory build --capture --output-file coverage.info &&
+      lcov --rc lcov_branch_coverage=1 --remove coverage.info
+          "${PWD}/3rdparty/*"
+          "${PWD}/build/*"
+          "${PWD}/examples/*"
+          "${PWD}/features/*"
+          "${PWD}/tests/*"
+          "/usr/*"
+          --output-file coverage.info &&
+      lcov --rc lcov_branch_coverage=1 --list coverage.info &&
+      coveralls-lcov coverage.info ;
+  fi
+
 # default notifications: https://docs.travis-ci.com/user/notifications#Notifications

travis.sh

@@ -91,17 +91,3 @@ if [ -n "${XVFBPID:-}" ]; then
     kill $XVFBPID
     wait
 fi
-
-if [ -n "${COVERALLS_SERVICE_NAME}" ]; then
-    lcov --rc lcov_branch_coverage=1 --directory build --capture --output-file coverage.info
-    lcov --rc lcov_branch_coverage=1 --remove coverage.info \
-        "${PWD}/3rdparty/*" \
-        "${PWD}/build/*" \
-        "${PWD}/examples/*" \
-        "${PWD}/features/*" \
-        "${PWD}/tests/*" \
-        "/usr/*" \
-        --output-file coverage.info
-    lcov --rc lcov_branch_coverage=1 --list coverage.info
-    coveralls-lcov coverage.info
-fi