fix: fix error messages which appear in system tests (#21)

Author: nolanmar511

package.json

@@ -11,12 +11,12 @@
     "check": "gts check",
     "clean": "gts clean && node-gyp clean",
     "codecov": "nyc report --reporter=json && codecov -f coverage/*.json",
-    "compile": "tsc -p . && node-pre-gyp build",
+    "compile": "tsc -p .",
     "fix": "gts fix",
     "lint": "gts check",
     "docs": "echo 'no docs yet'",
     "prepare": "npm run compile",
-    "pretest": "npm run compile",
+    "pretest": "npm run compile && node-pre-gyp build",
     "posttest": "npm run check && npm run license-check",
     "proto": "npm run proto:profile",
     "proto:profile": "mkdir -p proto && pbjs -t static-module -w commonjs -o proto/profile.js third_party/proto/profile.proto && pbts -o proto/profile.d.ts proto/profile.js",

system-test/system_test.go

@@ -52,6 +52,9 @@ retry() {
 # Display commands being run.
 set -x
 
+# Fail on any error.
+set -eo pipefail
+
 # Note directory from which test is being run.
 BASE_DIR=$(pwd)
 

tools/build.sh

@@ -35,3 +35,8 @@ do
   cp -r build/stage/* "${ARTIFACTS_OUT}/"
   rm -rf build
 done
+
+# Remove node_modules directory. When this script is run in a docker container
+# with  user root, then a system test running after this script cannot run npm
+# install.
+rm -r node_modules