From e41276fd5237d0fd40883dacc93095e373f4e1c8 Mon Sep 17 00:00:00 2001 From: filipenevola Date: Thu, 14 Jan 2021 16:04:19 -0400 Subject: [PATCH] Revert "Adds timeout as an option to Sandbox.createApp" This reverts commit dc2b2ca1 --- tools/tests/update-tests.js | 3 +-- tools/tool-testing/sandbox.js | 4 ++-- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/tools/tests/update-tests.js b/tools/tests/update-tests.js index 3da4f8bfe0..035246d3d9 100644 --- a/tools/tests/update-tests.js +++ b/tools/tests/update-tests.js @@ -35,8 +35,7 @@ selftest.define("'meteor update' alters constraints in `.meteor/packages`", () = }); s.createApp("myapp", "very-simple-app-with-no-package-constraints", { - release: DEFAULT_RELEASE_TRACK + '@v1', - timeout: 300, + release: DEFAULT_RELEASE_TRACK + '@v1' }); s.cd("myapp"); diff --git a/tools/tool-testing/sandbox.js b/tools/tool-testing/sandbox.js index 86ab0b9f4d..03bd27d317 100644 --- a/tools/tool-testing/sandbox.js +++ b/tools/tool-testing/sandbox.js @@ -83,7 +83,7 @@ export default class Sandbox { this.warehouse = files.pathJoin(this.root, 'tropohouse'); this._makeWarehouse(this.options.warehouse); } - + const meteorScript = process.platform === "win32" ? "meteor.bat" : "meteor"; // Figure out the 'meteor' to run @@ -211,7 +211,7 @@ export default class Sandbox { // multiple calls to createApp with the same template get the same cache? // This is a little tricky because isopack-buildinfo.json uses absolute // paths. - run.waitSecs(options.timeout || 120); + run.waitSecs(120); run.expectExit(0); }); }