From 893632ec07995a0ff76952e652e57cd1e16cfeaa Mon Sep 17 00:00:00 2001 From: Howard Hellyer Date: Tue, 14 Feb 2017 14:14:05 +0000 Subject: [PATCH] test: run test-setproctitle where supported Setting the process title has been enabled in libuv on AIX and z/OS. The latest level of libuv skips only skips testing of uv_set_process_title when __sun is #defined. This change simplifies the skip test so the test is only skipped when common.isSunOS is true to match libuv. Skip running the `ps` part of the test on Windows. PR-URL: https://github.com/nodejs/node/pull/11416 Reviewed-By: Colin Ihrig Reviewed-By: James M Snell Reviewed-By: Gibson Fahnestock Reviewed-By: Michael Dawson --- test/parallel/test-setproctitle.js | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/test/parallel/test-setproctitle.js b/test/parallel/test-setproctitle.js index 9dd603bfdc..a971d604b3 100644 --- a/test/parallel/test-setproctitle.js +++ b/test/parallel/test-setproctitle.js @@ -3,7 +3,7 @@ const common = require('../common'); // FIXME add sunos support -if (!(common.isFreeBSD || common.isOSX || common.isLinux)) { +if (common.isSunOS) { console.log(`1..0 # Skipped: Unsupported platform [${process.platform}]`); return; } @@ -21,6 +21,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) + return; + exec(`ps -p ${process.pid} -o args=`, function callback(error, stdout, stderr) { assert.ifError(error); assert.strictEqual(stderr, '');