diff --git a/test/disabled/test-setuidgid.js b/test/disabled/test-setuidgid.js deleted file mode 100644 index ff674f869039d2..00000000000000 --- a/test/disabled/test-setuidgid.js +++ /dev/null @@ -1,23 +0,0 @@ -'use strict'; -// Requires special privileges -var common = require('../common'); -var assert = require('assert'); - -var oldgid = process.getgid(); -process.setgid('nobody'); -var newgid = process.getgid(); -assert.notEqual(newgid, oldgid, 'gids expected to be different'); - -var olduid = process.getuid(); -process.setuid('nobody'); -var newuid = process.getuid(); -assert.notEqual(newuid, olduid, 'uids expected to be different'); - -try { - process.setuid('nobody1234'); -} catch (e) { - assert.strictEqual(e.message, - 'failed to resolve group', - 'unexpected error message' - ); -} diff --git a/test/parallel/test-process-setuid-setgid.js b/test/parallel/test-process-setuid-setgid.js new file mode 100644 index 00000000000000..080bc54a8fba1b --- /dev/null +++ b/test/parallel/test-process-setuid-setgid.js @@ -0,0 +1,48 @@ +'use strict'; + +const common = require('../common'); + +const assert = require('assert'); + +if (common.isWindows) { + // uid/gid functions are POSIX only + assert.strictEqual(process.getuid, undefined); + assert.strictEqual(process.setuid, undefined); + assert.strictEqual(process.getgid, undefined); + assert.strictEqual(process.setgid, undefined); + return; +} + +assert.throws(() => { + process.setuid('fhqwhgadshgnsdhjsdbkhsdabkfabkveybvf'); +}, /^Error: setuid user id does not exist$/); + +// If we're not running as super user... +if (process.getuid() !== 0) { + assert.doesNotThrow(() => { + process.getgid(); + process.getuid(); + }); + + assert.throws( + () => { process.setgid('nobody'); }, + /^Error: (EPERM, .+|setgid group id does not exist)$/ + ); + + assert.throws( + () => { process.setuid('nobody'); }, + /^Error: EPERM, / + ); + return; +} + +// If we are running as super user... +const oldgid = process.getgid(); +process.setgid('nobody'); +const newgid = process.getgid(); +assert.notStrictEqual(newgid, oldgid); + +const olduid = process.getuid(); +process.setuid('nobody'); +const newuid = process.getuid(); +assert.notStrictEqual(newuid, olduid);