From 00fd81740cf33e33c03040ee39d8f2cd6a425959 Mon Sep 17 00:00:00 2001 From: John Finnerty Date: Thu, 10 Sep 2026 08:49:41 +1200 Subject: [PATCH] test: make timeouts configurable Signed-off-by: John Finnerty --- test/common.js | 7 ++++--- test/plugin/frame-test.js | 2 +- test/plugin/inspect-test.js | 2 +- test/plugin/stack-test.js | 2 +- test/plugin/usage-test.js | 2 +- test/plugin/workqueue-test.js | 2 +- 6 files changed, 9 insertions(+), 8 deletions(-) diff --git a/test/common.js b/test/common.js index 68fb78ca..827580da 100644 --- a/test/common.js +++ b/test/common.js @@ -12,6 +12,8 @@ exports.projectDir = path.join(__dirname, '..'); exports.core = path.join(os.tmpdir(), 'core'); exports.promptDelay = 200; +exports.testTimeout = parseInt(process.env.TEST_TIMEOUT) || 40000; +exports.testTimeoutLong = parseInt(process.env.TEST_TIMEOUT_LONG) || 60000; function llnodeDebug() { // Node v4.x does not support rest @@ -170,7 +172,6 @@ SessionOutput.prototype.linesUntil = function linesUntil(regexp, callback) { function Session(options) { EventEmitter.call(this); - const timeout = parseInt(process.env.TEST_TIMEOUT) || 40000; const lldbBin = process.env.TEST_LLDB_BINARY || 'lldb'; const env = Object.assign({}, process.env); @@ -205,8 +206,8 @@ function Session(options) { this.lldb.stdin.write(`target create "${options.executable}"` + ` --core "${options.core}"\n`); } - this.stdout = new SessionOutput(this, this.lldb.stdout, timeout); - this.stderr = new SessionOutput(this, this.lldb.stderr, timeout); + this.stdout = new SessionOutput(this, this.lldb.stdout, exports.testTimeout); + this.stderr = new SessionOutput(this, this.lldb.stderr, exports.testTimeout); // Map these methods to stdout for compatibility with legacy tests. this.wait = SessionOutput.prototype.wait.bind(this.stdout); diff --git a/test/plugin/frame-test.js b/test/plugin/frame-test.js index 8b2a3453..d0a20844 100644 --- a/test/plugin/frame-test.js +++ b/test/plugin/frame-test.js @@ -62,7 +62,7 @@ async function testFrameList(t, sess, frameNumber, sourceCode, cb) { } tape('v8 stack', async (t) => { - t.timeoutAfter(30000); + t.timeoutAfter(common.testTimeout); const sess = common.Session.create('frame-scenario.js'); sess.waitBreak = promisify(sess.waitBreak); diff --git a/test/plugin/inspect-test.js b/test/plugin/inspect-test.js index 4510c948..f25cc491 100644 --- a/test/plugin/inspect-test.js +++ b/test/plugin/inspect-test.js @@ -678,7 +678,7 @@ function verifyInvalidExpr(t, sess) { } tape('v8 inspect', (t) => { - t.timeoutAfter(30000); + t.timeoutAfter(common.testTimeout); const sess = common.Session.create('inspect-scenario.js'); diff --git a/test/plugin/stack-test.js b/test/plugin/stack-test.js index 248ac79e..e87539f2 100644 --- a/test/plugin/stack-test.js +++ b/test/plugin/stack-test.js @@ -5,7 +5,7 @@ const tape = require('tape'); const common = require('../common'); tape('v8 stack', (t) => { - t.timeoutAfter(45000); + t.timeoutAfter(common.testTimeoutLong); const sess = common.Session.create('stack-scenario.js'); sess.waitBreak(() => { diff --git a/test/plugin/usage-test.js b/test/plugin/usage-test.js index 7f908ddf..ec8c803c 100644 --- a/test/plugin/usage-test.js +++ b/test/plugin/usage-test.js @@ -7,7 +7,7 @@ function containsLine(lines, re) { } tape('usage messages', (t) => { - t.timeoutAfter(30000); + t.timeoutAfter(common.testTimeout); const sess = common.Session.create('inspect-scenario.js'); diff --git a/test/plugin/workqueue-test.js b/test/plugin/workqueue-test.js index 7b71933e..3edd2162 100644 --- a/test/plugin/workqueue-test.js +++ b/test/plugin/workqueue-test.js @@ -26,7 +26,7 @@ function testWorkqueueCommands(t, sess) { } tape('v8 workqueue commands', (t) => { - t.timeoutAfter(60000); + t.timeoutAfter(common.testTimeoutLong); const sess = common.Session.create('workqueue-scenario.js'); sess.timeoutAfter