OSSForks/node
1
0
Fork 0
mirror of https://github.com/zebrajr/node.git synced 2025-12-06 12:20:27 +01:00
Code Issues Actions 8 Packages Projects Releases Wiki Activity

test: fix test-setproctitle status when ps is not available

Browse Source 
PR-URL: https://github.com/nodejs/node/pull/59523
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
This commit is contained in:
Antoine du Hamel 2025-08-20 20:24:04 +02:00 committed by GitHub
parent a43f984761
commit 499a5c3451
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 3 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
test/parallel/test-setproctitle.js
View File

@ -4,7 +4,7 @@ const common = require('../common');
const { isMainThread } = require('worker_threads');
// FIXME add sunos support
if (common.isSunOS || common.isIBMi) {
if (common.isSunOS || common.isIBMi || common.isWindows) {
common.skip(`Unsupported platform [${process.platform}]`);
}
@ -25,15 +25,10 @@ assert.notStrictEqual(process.title, title);
process.title = title;
assert.strictEqual(process.title, title);
// Test setting the title but do not try to run `ps` on Windows.
if (common.isWindows) {
common.skip('Windows does not have "ps" utility');
}
try {
execSync('command -v ps');
} catch (err) {
if (err.status === 1) {
if (err.status === 1 || err.status === 127) {
common.skip('The "ps" utility is not available');
}
throw err;
@ -53,5 +48,5 @@ exec(cmd, common.mustSucceed((stdout, stderr) => {
title += ` (${path.basename(process.execPath)})`;
// Omitting trailing whitespace and \n
assert.strictEqual(stdout.replace(/\s+$/, '').endsWith(title), true);
assert.ok(stdout.trimEnd().endsWith(title));
}));