Skip to content

Commit

Permalink
test: add tests for process.initgroups
Browse files Browse the repository at this point in the history
- test argument validation
- test function throws if provided group is invalid

PR-URL: nodejs#24154
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
James Herrington authored and kiyomizumia committed Nov 15, 2018
1 parent 64dd6e9 commit 56013ce
Showing 1 changed file with 54 additions and 0 deletions.
54 changes: 54 additions & 0 deletions test/parallel/test-process-initgroups.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,54 @@
'use strict';
const common = require('../common');
const assert = require('assert');

if (common.isWindows || !common.isMainThread) {
assert.strictEqual(process.initgroups, undefined);
return;
}

[undefined, null, true, {}, [], () => {}].forEach((val) => {
assert.throws(
() => {
process.initgroups(val);
},
{
code: 'ERR_INVALID_ARG_TYPE',
name: 'TypeError [ERR_INVALID_ARG_TYPE]',
message:
'The "user" argument must be ' +
'one of type number or string. ' +
`Received type ${typeof val}`
}
);
});

[undefined, null, true, {}, [], () => {}].forEach((val) => {
assert.throws(
() => {
process.initgroups('foo', val);
},
{
code: 'ERR_INVALID_ARG_TYPE',
name: 'TypeError [ERR_INVALID_ARG_TYPE]',
message:
'The "extraGroup" argument must be ' +
'one of type number or string. ' +
`Received type ${typeof val}`
}
);
});

assert.throws(
() => {
process.initgroups(
'fhqwhgadshgnsdhjsdbkhsdabkfabkveyb',
'fhqwhgadshgnsdhjsdbkhsdabkfabkveyb'
);
},
{
code: 'ERR_UNKNOWN_CREDENTIAL',
message:
'Group identifier does not exist: fhqwhgadshgnsdhjsdbkhsdabkfabkveyb'
}
);

0 comments on commit 56013ce

Please sign in to comment.