mirror of
https://github.com/nodejs/node.git
synced 2026-10-28 18:25:13 +08:00
Junhwan Choi
d78337f085
The cpu-prof tests hand-rolled a partial version of what test/common/child_process already provides: log the child's stderr when the status is non-zero, then assert the status. That manual version never shows stdout, the signal, or the command that ran, so a CI failure gives less information than the helper would. Replace those blocks with spawnSyncAndExitWithoutError(). Every existing expectation is preserved -- status 0 is still required, and signal null is now checked as well, matching the helper defaults. Only success-path blocks whose assertions map one-to-one onto the helper defaults are converted. Refs: https://github.com/nodejs/node/pull/65351 Refs: https://github.com/nodejs/node/pull/65552 Assisted-by: claude:opus-5 Signed-off-by: Junhwan Choi <devjunsday@gmail.com> PR-URL: https://github.com/nodejs/node/pull/65694 Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
41 lines
993 B
JavaScript
41 lines
993 B
JavaScript
'use strict';
|
|
|
|
// This tests that relative --cpu-prof-dir works.
|
|
|
|
const common = require('../common');
|
|
const fixtures = require('../common/fixtures');
|
|
common.skipIfInspectorDisabled();
|
|
|
|
const assert = require('assert');
|
|
const fs = require('fs');
|
|
const { spawnSyncAndExitWithoutError } = require('../common/child_process');
|
|
|
|
const tmpdir = require('../common/tmpdir');
|
|
const {
|
|
getCpuProfiles,
|
|
kCpuProfInterval,
|
|
env,
|
|
verifyFrames,
|
|
} = require('../common/cpu-prof');
|
|
|
|
// relative --cpu-prof-dir
|
|
{
|
|
tmpdir.refresh();
|
|
const dir = tmpdir.resolve('prof');
|
|
const { child: output } = spawnSyncAndExitWithoutError(process.execPath, [
|
|
'--cpu-prof',
|
|
'--cpu-prof-interval',
|
|
kCpuProfInterval,
|
|
'--cpu-prof-dir',
|
|
dir,
|
|
fixtures.path('workload', 'fibonacci.js'),
|
|
], {
|
|
cwd: tmpdir.path,
|
|
env,
|
|
});
|
|
assert(fs.existsSync(dir));
|
|
const profiles = getCpuProfiles(dir);
|
|
assert.strictEqual(profiles.length, 1);
|
|
verifyFrames(output, profiles[0], 'fibonacci.js');
|
|
}
|