From 64645d872731ae51504bbcfc7e9986b26603c1ca Mon Sep 17 00:00:00 2001
From: David Glasser <glasser@meteor.com>
Date: Tue, 16 Dec 2014 17:04:53 -0800
Subject: [PATCH] Fix pathwatcher fallback tests

- Future.wait does not throw
- any non-empty env var counts as true
---
 tools/tests/old.js | 19 +++++++++++--------
 1 file changed, 11 insertions(+), 8 deletions(-)

diff --git a/tools/tests/old.js b/tools/tests/old.js
index 55a1d6e15c..e1cf2ce9de 100644
--- a/tools/tests/old.js
+++ b/tools/tests/old.js
@@ -27,7 +27,7 @@ var runOldTest = function (filename, extraEnv) {
       METEOR_TOOL_PATH: s.execPath
     }, extraEnv))
   });
-  run.waitSecs(1000);
+  run.waitSecs(40);
   run.expectExit(0);
 };
 
@@ -47,16 +47,19 @@ var runOldTest = function (filename, extraEnv) {
 //
 selftest.define("watch", ["slow"], function () {
   var runFuture = runOldTest.future();
-
-  Future.wait(
-    runFuture('test-watch.js', {
-      METEOR_WATCH_FORCE_POLLING: 0
-    }),
-
+  var futures = [
+    // Run with pathwatcher (if possible)
+    runFuture('test-watch.js'),
+    // Run with fs.watchFile fallback
     runFuture('test-watch.js', {
       METEOR_WATCH_FORCE_POLLING: 1
     })
-  );
+  ];
+  Future.wait(futures);
+  // Throw if any threw.
+  _.each(futures, function (f) {
+    f.get();
+  });
 });
 
 selftest.define("bundler-assets", ["checkout"], function () {