GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — test: unflake debugger and REPL tests · nodejs/node@f2919db

File tree

Expand file treeCollapse file tree
Open diff view settings
Filter options
  • test
    • parallel
      • test-debugger-extract-function-name.mjs
    • sequential
      • test-debug-prompt.js
Expand file treeCollapse file tree
Open diff view settings
Collapse file

‎test/parallel/test-debugger-extract-function-name.mjs‎

Copy file name to clipboardExpand all lines: test/parallel/test-debugger-extract-function-name.mjs
+6-2Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,14 @@ import startCLI from '../common/debugger.js';
77

88
import assert from 'assert';
99

10-
const cli = startCLI([fixtures.path('debugger', 'three-lines.js')]);
10+
const env = {
11+
...process.env,
12+
NODE_INSPECT_RESUME_ON_START: '1',
13+
};
14+
const cli = startCLI(
15+
[fixtures.path('debugger', 'alive.js')], [], { env });
1116

1217
try {
13-
await cli.waitForInitialBreak();
1418
await cli.waitForPrompt();
1519
await cli.command('exec a = function func() {}; a;');
1620
assert.match(cli.output, /\[Function: func\]/);
Collapse file

‎test/sequential/test-debug-prompt.js‎

Copy file name to clipboardExpand all lines: test/sequential/test-debug-prompt.js
+37-9Lines changed: 37 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -2,17 +2,45 @@
22

33
const common = require('../common');
44
common.skipIfInspectorDisabled();
5+
const assert = require('assert');
6+
const fixtures = require('../common/fixtures');
57
const spawn = require('child_process').spawn;
68

7-
const proc = spawn(process.execPath, ['inspect', 'foo']);
9+
const proc = spawn(process.execPath, [
10+
'inspect', fixtures.path('debugger', 'alive.js'),
11+
]);
812
proc.stdout.setEncoding('utf8');
913

10-
let needToSendExit = true;
14+
const TIMEOUT = common.platformTimeout(10_000);
1115
let output = '';
12-
proc.stdout.on('data', (data) => {
13-
output += data;
14-
if (output.includes('debug> ') && needToSendExit) {
15-
proc.stdin.write('.exit\n');
16-
needToSendExit = false;
17-
}
18-
});
16+
let promptSeen = false;
17+
18+
(async () => {
19+
await new Promise((resolve, reject) => {
20+
const timer = setTimeout(() => {
21+
proc.kill();
22+
reject(new Error(`Timed out waiting for the debugger prompt; output: ${output}`));
23+
}, TIMEOUT);
24+
25+
proc.stdout.on('data', (data) => {
26+
output += data;
27+
if (output.includes('debug> ') && !promptSeen) {
28+
promptSeen = true;
29+
proc.stdin.end('.exit\n');
30+
}
31+
});
32+
33+
proc.once('error', reject);
34+
proc.once('close', common.mustCall((code, signal) => {
35+
clearTimeout(timer);
36+
if (!promptSeen) {
37+
reject(new Error(
38+
`Debugger exited before showing the prompt (code ${code}, signal ${signal}); ` +
39+
`output: ${output}`));
40+
return;
41+
}
42+
assert.strictEqual(code, 0);
43+
resolve();
44+
}));
45+
});
46+
})().then(common.mustCall());

0 commit comments

Comments
0 (0)
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.