From 076b88134a3e77441c35076f1c2a9329d47fb6b7 Mon Sep 17 00:00:00 2001
From: Erik Arvidsson <erik.arvidsson@gmail.com>
Date: Wed, 25 Jan 2017 20:14:56 -0800
Subject: [PATCH] Integration test: Just disable samples/js/fs

samples/js/fs fails with

```
+ go test -v github.com/attic-labs/noms/samples/js/fs
=== RUN   TestIntegration
== Calling npmInstall
yarn install v0.19.1
$ cp ../../../js/noms/.babelrc ../.eslintrc.js ../../../js/noms/.flowconfig .
[1/4] Resolving packages...
success Already up-to-date.
$ yarn run build
yarn run v0.19.1
$ BABEL_ENV=production babel -d dist src
Build step 'Execute shell' marked build as failure
```

We also have:

```
+ ls -al /var/workspace/src/github.com/attic-labs/noms/samples/js/fs
total 124
drwxrwxrwx 3 112 116  4096 Jan 26 04:08 .
drwxrwxrwx 8 112 116  4096 Jan 26 04:08 ..
-rw-rw-rw- 1 112 116    59 Jan 26 04:08 .gitignore
-rw-rw-rw- 1 112 116   244 Jan 26 04:08 dummy.go
-rw-rw-rw- 1 112 116  1162 Jan 26 04:08 integration_test.go
-rw-rw-rw- 1 112 116   666 Jan 26 04:08 package.json
drwxrwxrwx 2 112 116  4096 Jan 26 04:08 src
-rw-rw-rw- 1 112 116    13 Jan 26 04:08 test-data.txt
-rw-rw-rw- 1 112 116 93335 Jan 26 04:08 yarn.lock
```

But it is not clear why this is different than the other samples.
---
 tools/run_pr_builder.sh | 44 ++++++++++++++++++-----------------------
 1 file changed, 19 insertions(+), 25 deletions(-)

diff --git a/tools/run_pr_builder.sh b/tools/run_pr_builder.sh
index 73fb2ba6de..1ce6ea8cb2 100755
--- a/tools/run_pr_builder.sh
+++ b/tools/run_pr_builder.sh
@@ -14,38 +14,32 @@ NOMS_DIR=${WORKSPACE}/src/github.com/attic-labs/noms
 go version
 node --version
 
-# # go list is expensive, only do it once.
-# GO_LIST="$(go list ./... | grep -v /vendor/ | grep -v /samples/js/)"
-# go build ${GO_LIST}
+# go list is expensive, only do it once.
+GO_LIST="$(go list ./... | grep -v /vendor/ | grep -v /samples/js/)"
+go build ${GO_LIST}
 
-# # go test plus build coverage data for upload codecov.io
-# rm -rf coverage.txt
-# touch coverage.txt
-# for d in ${GO_LIST}; do
-#     go test -coverprofile=profile.out -covermode=atomic $d
-#     if [ -f profile.out ]; then
-#         cat profile.out >> coverage.txt
-#         rm profile.out
-#     fi
-# done
+# go test plus build coverage data for upload codecov.io
+rm -rf coverage.txt
+touch coverage.txt
+for d in ${GO_LIST}; do
+    go test -coverprofile=profile.out -covermode=atomic $d
+    if [ -f profile.out ]; then
+        cat profile.out >> coverage.txt
+        rm profile.out
+    fi
+done
 
-#pushd ${NOMS_DIR}
-#python tools/run-all-js-tests.py
-#popd
-
-pushd ${NOMS_DIR}/js/noms
-yarn
+pushd ${NOMS_DIR}
+python tools/run-all-js-tests.py
 popd
 
 # The integration test only works after the node tests because the node tests sets up samples/js/node_modules
-echo "== Integration tests"
 SAMPLES_JS=github.com/attic-labs/noms/samples/js
-go test -v ${SAMPLES_JS}/aggregate
-go test -v ${SAMPLES_JS}/counter
-ls -al ${NOMS_DIR}/samples/js/fs
+go test ${SAMPLES_JS}/aggregate
+go test ${SAMPLES_JS}/counter
+# This test fails
 #go test -v ${SAMPLES_JS}/fs
-go test -v ${SAMPLES_JS}/url_fetch
-echo "== Integration tests done"
+go test ${SAMPLES_JS}/url_fetch
 
 python -m unittest discover -p "*_test.py" -s $GOPATH/src/github.com/attic-labs/noms/tools