Velocity ci patches (reactioncommerce#1046)

* add bin/ci.sh

wraps velocity test and watch for tests passed output.  ci will timeout
if this doesn’t exit in 10 minutes.

* add npm.depends for bunyan testing

kludge fix, add npm.depends specifically for testing

* clean up extra param

remove extra VELOCITY_TEST_PACKAGES=1

* add export

the cleanup that became dirty.. add export

bin/ci.sh

@@ -0,0 +1,15 @@
+#!/bin/bash
+
+set -u
+set -e
+# test using velocity
+export VELOCITY_TEST_PACKAGES=1
+# send testing output to log
+meteor test-packages --driver-package velocity:html-reporter --velocity >& logfile.log &
+# tail log for passed tests notice
+# circle ci test will timeout if we don't pass
+tail -f logfile.log | while read LOGLINE
+do
+   echo ${LOGLINE}
+   [[ "${LOGLINE}" == *"tests passed"* ]] && pkill -P $$ tail
+done

circle.yml

@@ -21,7 +21,7 @@ dependencies:
 test:
   override:
     - cd $HOME/reaction
-    - VELOCITY_TEST_PACKAGES=1 meteor test-packages --driver-package velocity:html-reporter --velocity
+    - bin/ci.sh
     - docker-compose -f docker/docker-compose.test.yml up -d; sleep 10
     - curl --retry 10 --retry-delay 5 -v http://localhost
 

packages/reaction-logger/.npm/package/.gitignore

@@ -0,0 +1 @@
+node_modules

packages/reaction-logger/.npm/package/README

@@ -0,0 +1,7 @@
+This directory and the files immediately inside it are automatically generated
+when you change this package's NPM dependencies. Commit the files in this
+directory (npm-shrinkwrap.json, .gitignore, and this README) to source control
+so that others run the same versions of sub-dependencies.
+
+You should NOT check in the node_modules directory that Meteor automatically
+creates; if you are using git, the .gitignore file tells git to ignore it.

packages/reaction-logger/package.js

@@ -5,6 +5,11 @@ Package.describe({
   version: "0.2.1"
 });
 
+Npm.depends({
+  "bunyan": "1.8.1",
+  "bunyan-format": "0.2.1"
+});
+
 Package.onUse(function (api) {
   api.versionsFrom("[email protected]");
   api.use("tmeasday:[email protected]");

packages/reaction-logger/tests/bunyan-test.js

@@ -1,4 +1,7 @@
-Tinytest.add('bunyan', function(test) {
-  test.isNotNull(logger, 'logger should be available');
-  test.isNotNull(bunyan, 'bunyan should be available');
+bunyan = Npm.require("bunyan");
+bunyanFormat = Npm.require("bunyan-format");
+
+Tinytest.add("bunyan", function (test) {
+  test.isNotNull(logger, "logger should be available");
+  test.isNotNull(bunyan, "bunyan should be available");
 });