Merge pull request #1212 from nats-io/travis_coveralls

Run coverage, but push to coverall only after_success

.travis.yml

@@ -42,6 +42,10 @@ script:
     go test -v -p=1 ./... -count=1 -sql=false -failfast;
   fi
 - set +e
+after_success:
+- if [[ "$TRAVIS_OS_NAME" == "linux" ]]; then
+    if [[ ! $TRAVIS_TAG ]]; then if [[ "$TRAVIS_GO_VERSION" =~ 1.15 ]]; then $HOME/gopath/bin/goveralls -coverprofile=acc.out -service travis-ci; fi; fi;
+  fi
 
 deploy:
   provider: script

scripts/cov.sh

@@ -23,10 +23,9 @@ go test -v -failfast -covermode=atomic -coverprofile=./cov/util.out ./util
 gocovmerge ./cov/*.out > acc.out
 rm -rf ./cov
 
-# If we have an arg, assume travis run and push to coveralls. Otherwise launch browser results
-if [[ -n $1 ]]; then
-    $HOME/gopath/bin/goveralls -coverprofile=acc.out
-    rm -rf ./acc.out
-else
+# Without argument, launch browser results. We are going to push to coveralls only
+# from Travis.yml and after success of the build (and result of pushing will not affect
+# build result).
+if [[ $1 == "" ]]; then
     go tool cover -html=acc.out
 fi