Add shellcheck to Travis tests and fix existing shell lint errors.

Author: tswast

.travis.yml

@@ -17,8 +17,13 @@ jdk:
 - oraclejdk8
 addons:
   apt:
+    sources:
+    # Grab shellcheck from the Debian repo.
+    # https://github.com/koalaman/shellcheck/wiki/TravisCI
+    - debian-sid
     packages:
     - expect
+    - shellcheck
 before_install:
 - openssl aes-256-cbc -K $encrypted_d17a6ce39290_key -iv $encrypted_d17a6ce39290_iv
   -in gae-firebase-secrets.json.enc -out appengine/firebase-event-proxy/gae-firebase-event-proxy/src/main/webapp/gae-firebase-secrets.json

speech/grpc/bin/speech-sample-async.sh

@@ -14,5 +14,5 @@
 # limitations under the License.
 
 SRC_DIR=$(cd "$(dirname "$0")/.."; pwd)
-java -cp ${SRC_DIR}/target/grpc-sample-1.0-jar-with-dependencies.jar \
+java -cp "${SRC_DIR}/target/grpc-sample-1.0-jar-with-dependencies.jar" \
     com.examples.cloud.speech.AsyncRecognizeClient "$@"

speech/grpc/bin/speech-sample-streaming.sh

@@ -14,5 +14,5 @@
 # limitations under the License.
 
 SRC_DIR=$(cd "$(dirname "$0")/.."; pwd)
-java -cp ${SRC_DIR}/target/grpc-sample-1.0-jar-with-dependencies.jar \
+java -cp "${SRC_DIR}/target/grpc-sample-1.0-jar-with-dependencies.jar" \
     com.examples.cloud.speech.StreamingRecognizeClient "$@"

speech/grpc/bin/speech-sample-sync.sh

@@ -14,5 +14,5 @@
 # limitations under the License.
 
 SRC_DIR=$(cd "$(dirname "$0")/.."; pwd)
-java -cp ${SRC_DIR}/target/grpc-sample-1.0-jar-with-dependencies.jar \
+java -cp "${SRC_DIR}/target/grpc-sample-1.0-jar-with-dependencies.jar" \
     com.examples.cloud.speech.SyncRecognizeClient "$@"

travis.sh

@@ -17,9 +17,10 @@ set -e
 set -x
 # Set pipefail so that `egrep` does not eat the exit code.
 set -o pipefail
+shopt -s globstar
 
 SKIP_TESTS=false
-if [ -z "$GOOGLE_APPLICATION_CREDENTIALS"]; then
+if [ -z "$GOOGLE_APPLICATION_CREDENTIALS" ] ; then
   SKIP_TESTS=true
 fi
 mvn --batch-mode clean verify -DskipTests=$SKIP_TESTS | egrep -v "(^\[INFO\] Download|^\[INFO\].*skipping)"
@@ -32,7 +33,10 @@ devserver_tests=(
     appengine/datastore/indexes-exploding
     appengine/datastore/indexes-perfect
 )
-for testdir in ${devserver_tests[@]} ; do
+for testdir in "${devserver_tests[@]}" ; do
   ./java-repo-tools/scripts/test-localhost.sh appengine "${testdir}"
 done
 
+# Check that all shell scripts in this repo (including this one) pass the
+# Shell Check linter.
+shellcheck ./**/*.sh