From bcc53c089f34bce6587ba1145ee116d0c533edc9 Mon Sep 17 00:00:00 2001
From: Scott Kyle <scott@appden.com>
Date: Tue, 26 Jan 2016 16:12:26 -0800
Subject: [PATCH] Cleanup test script

This should make it cleanup artifacts more reliably, and gives the script a consistent style.
---
 scripts/test.sh | 138 +++++++++++++++++++++++++++---------------------
 1 file changed, 77 insertions(+), 61 deletions(-)

diff --git a/scripts/test.sh b/scripts/test.sh
index ee9dbeda..3fd851f0 100755
--- a/scripts/test.sh
+++ b/scripts/test.sh
@@ -3,11 +3,17 @@
 set -o pipefail
 set -e
 
-TARGET=$1
-CONFIGURATION=${2:-"Debug"}
+TARGET="$1"
+CONFIGURATION="${2:-"Debug"}"
+DESTINATION=
+PATH="/opt/android-sdk-linux/platform-tools:$PATH"
+SRCROOT=$(cd "$(dirname "$0")/.." && pwd)
 
-if [ "$TARGET" != "react-tests-android" ]; then
-while pgrep -q Simulator; do
+# Start current working directory at the root of the project.
+cd "$SRCROOT"
+
+if [[ $TARGET != *-android ]]; then
+  while pgrep -q Simulator; do
     # Kill all the current simulator processes as they may be from a
     # different Xcode version
     pkill Simulator 2>/dev/null || true
@@ -17,104 +23,114 @@ while pgrep -q Simulator; do
   DESTINATION="-destination id=$(xcrun simctl list devices | grep -v unavailable | grep -m 1 -o '[0-9A-F\-]\{36\}')"
 fi
 
-PACKAGER_OUT="packager_out.txt"
+PACKAGER_OUT="$SRCROOT/packager_out.txt"
+LOGCAT_OUT="$SRCROOT/logcat_out.txt"
 
-function start_packager()
-{
-  rm -f $PACKAGER_OUT
-  ./node_modules/react-native/packager/packager.sh | tee $PACKAGER_OUT &
-  while :;
-  do
-  if grep -Fxq "React packager ready." $PACKAGER_OUT
-  then
-    break
-  else
-    echo "Waiting for packager."
-    sleep 2
-  fi
+cleanup() {
+  # Kill all child processes.
+  pkill -P $$ || true
+  pkill node || true
+  rm -f "$PACKAGER_OUT" "$LOGCAT_OUT"
+}
+
+open_chrome() {
+  local dir
+  for dir in "$HOME/Applications" "/Applications"; do
+    if [ -d "$dir/Google Chrome.app" ]; then
+      open "$dir/Google Chrome.app"
+      break
+    fi
   done
 }
 
-function unlock_device()
-{
-  /opt/android-sdk-linux/platform-tools/adb shell input keyevent 82
+start_packager() {
+  # Starting with --nonPersistent will avoid starting a file watcher.
+  ./node_modules/react-native/packager/packager.sh --nonPersistent | tee "$PACKAGER_OUT" &
+
+  while :; do
+    if grep -Fxq "React packager ready." "$PACKAGER_OUT"; then
+      break
+    else
+      echo "Waiting for packager."
+      sleep 2
+    fi
+  done
 }
 
-# kill old packagers
-pkill node || true
+unlock_device() {
+  adb shell input keyevent 82
+}
 
-if [ "$TARGET" = "realmjs" ]; then
+# Cleanup now and also cleanup when this script exits.
+cleanup
+trap cleanup EXIT
+
+case "$TARGET" in
+"realmjs")
   xcodebuild -scheme RealmJS -configuration "$CONFIGURATION" -sdk iphonesimulator $DESTINATION build test
-elif [ "$TARGET" = "react-tests" ]; then
+  ;;
+"react-tests")
   pushd tests/react-test-app
 
-  if [ -d  ~/Applications/Google\ Chrome.app ]; then
-    open ~/Applications/Google\ Chrome.app
-  fi
-
   if [ -f ../../target=node_modules/react_tests_node_modules.zip ]; then
       unzip -q ../../target=node_modules/react_tests_node_modules.zip
   fi
+
   npm update react-native
+  open_chrome
   start_packager
   popd
 
   xcodebuild -scheme RealmReact -configuration "$CONFIGURATION" -sdk iphonesimulator $DESTINATION build test
-elif [ "$TARGET" = "react-example" ]; then
+  ;;
+"react-example")
   pushd examples/ReactExample
+
   if [ -f ../../target=node_modules/react_example_node_modules.zip ]; then
     unzip -q ../../target=node_modules/react_example_node_modules.zip
   fi
+
   npm update react-native
   start_packager
 
   pushd ios
   xcodebuild -scheme ReactExample -configuration "$CONFIGURATION" -sdk iphonesimulator build $DESTINATION
-  popd
-elif [ "$TARGET" = "react-tests-android" ]; then
+  ;;
+"react-tests-android")
   [ -s "${HOME}/.nvm/nvm.sh" ] && . "${HOME}/.nvm/nvm.sh"
   nvm use 5.4.0
+
   pushd react-native/android
   ./gradlew installarchives
   popd
 
   pushd tests/react-test-app
 
-  if [ -d  ~/Applications/Google\ Chrome.app ]; then
-    open ~/Applications/Google\ Chrome.app
-  fi
-
   npm install
+  open_chrome
   start_packager
   unlock_device
   ./run-android.sh
 
-  LOGCAT_OUT="logcat_out.txt"
-  rm -f $LOGCAT_OUT
+  adb logcat -c
+  adb logcat | tee "$LOGCAT_OUT" &
 
-  /opt/android-sdk-linux/platform-tools/adb logcat -c
-  /opt/android-sdk-linux/platform-tools/adb logcat | tee $LOGCAT_OUT &
-  while :;
-  do
-  if grep -q "__REALM_REACT_ANDROID_TESTS_COMPLETED__" $LOGCAT_OUT
-  then
-    break
-  else
-    echo "Waiting for tests."
-    sleep 2
-  fi
+  while :; do
+    if grep -q "__REALM_REACT_ANDROID_TESTS_COMPLETED__" "$LOGCAT_OUT"; then
+      break
+    else
+      echo "Waiting for tests."
+      sleep 2
+    fi
   done
 
-  /opt/android-sdk-linux/platform-tools/adb pull /sdcard/tests.xml . || true
-  echo "********* TESTS COMPLETED *********";
-  echo "********* File location: `pwd`/tests.xml *********";
-  more tests.xml
-else
-  echo "Invalid target '${TARGET}'"
-fi
+  adb pull /sdcard/tests.xml . || true
 
-# kill all children
-pkill -P $$ || true
-pkill node || true
-rm -f $PACKAGER_OUT
-rm -f $LOGCAT_OUT
+  echo "********* TESTS COMPLETED *********";
+  echo "********* File location: $(pwd)/tests.xml *********";
+  more tests.xml
+  ;;
+*)
+  echo "Invalid target '${TARGET}'"
+  exit 1
+esac