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

Author: tswast

.travis.yml

@@ -19,6 +19,7 @@ addons:
   apt:
     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

@@ -19,7 +19,7 @@ set -x
 set -o pipefail
 
 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 +32,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