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>
33 lines
901 B
JavaScript
33 lines
901 B
JavaScript
'use strict';
|
|
|
|
// Test --cpu-prof without --cpu-prof-interval. Here we just verify that
|
|
// we manage to generate a profile since it's hard to tell whether we
|
|
// can sample our target function with the default sampling rate across
|
|
// different platforms and machine configurations.
|
|
|
|
const common = require('../common');
|
|
const fixtures = require('../common/fixtures');
|
|
common.skipIfInspectorDisabled();
|
|
|
|
const assert = require('assert');
|
|
const { spawnSyncAndExitWithoutError } = require('../common/child_process');
|
|
|
|
const tmpdir = require('../common/tmpdir');
|
|
const {
|
|
getCpuProfiles,
|
|
env,
|
|
} = require('../common/cpu-prof');
|
|
|
|
{
|
|
tmpdir.refresh();
|
|
spawnSyncAndExitWithoutError(process.execPath, [
|
|
'--cpu-prof',
|
|
fixtures.path('workload', 'fibonacci.js'),
|
|
], {
|
|
cwd: tmpdir.path,
|
|
env,
|
|
});
|
|
const profiles = getCpuProfiles(tmpdir.path);
|
|
assert.strictEqual(profiles.length, 1);
|
|
}
|