lookup: add jsdom

README.md

@@ -148,6 +148,7 @@ For syntax, see [lookup.json](./lib/lookup.json), the available attributes are:
 "useGitClone": true          Use a shallow git clone instead of downloading the module
 "ignoreGitHead":             Ignore the gitHead field if it exists and fallback to using github tags
 "yarn":                      Install and test the project using yarn instead of npm
+"timeoutLength":             Number of milliseconds before timeout
 ```
 
 If you want to pass options to npm, eg `--registry`, you can usually define an

lib/common-args.js

@@ -73,8 +73,7 @@ module.exports = function commonArgs(app) {
     .option('timeout', {
       alias: 'o',
       type: 'number',
-      description: 'Set timeout for npm install',
-      default: 1000 * 60 * 10
+      description: 'Set timeout for npm install'
     })
     .option('yarn', {
       alias: 'y',

lib/lookup.js

@@ -162,6 +162,9 @@ function resolve(context) {
       if (rep.yarn) {
         context.module.useYarn = true;
       }
+      if (rep.timeoutLength) {
+        context.module.timeoutLength = rep.timeoutLength;
+      }
       context.module.flaky = context.options.failFlaky
         ? false
         : isMatch(rep.flaky);

lib/lookup.json

@@ -237,6 +237,12 @@
     "maintainers": "jeresig",
     "ignoreGitHead": true
   },
+  "jsdom": {
+    "maintainers": ["domenic", "Zirro"],
+    "timeoutLength": 1200000,
+    "useGitClone": true,
+    "yarn": true
+  },
   "koa": {
     "flaky": "ubuntu",
     "maintainers": "tj"

lib/package-manager/test.js

@@ -106,7 +106,13 @@ async function test(packageManager, context) {
       );
 
       const proc = spawn(bin, args, options);
-      const finish = timeout(context, proc, runScript, 'Test');
+      const finish = timeout(
+        context,
+        proc,
+        runScript,
+        'Test',
+        context.module.timeoutLength
+      );
 
       proc.stdout.on('data', (data) => {
         context.testOutput.append(data);

lib/timeout.js

@@ -3,9 +3,10 @@
 // Default timeout to 10 minutes if not provided
 const kDefaultTimeout = 1000 * 60 * 10;
 
-function timeout(context, proc, next, step) {
+function timeout(context, proc, next, step, moduleConfigTimeout) {
   let hasRun = false;
-  const delay = context.options.timeoutLength || kDefaultTimeout;
+  const delay =
+    context.options.timeoutLength || moduleConfigTimeout || kDefaultTimeout;
   // Third arg === `true` is the way to signal `finish` that this is a timeout.
   // Otherwise it acts like a "regular" callback, i.e. `(err, ret) => {}`.
   // `if (timedOut)` will overwrite `err` & `ret`, so first 2 args are ignored.