GeezerVison

The World Wide Web, restored to 800 pixels of glory.

GeezerVison is showing github.com — test: getgroups() may contain duplicate GIDs · nodejs/node@a1cb699

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

test/parallel
- test-process-getgroups.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

`‎test/parallel/test-process-getgroups.js‎`

Copy file name to clipboard

+12-6Lines changed: 12 additions & 6 deletions

Original file line number	Diff line number	Diff line change
`@@ -1,21 +1,23 @@`
`1`	`1`	`'use strict';`
`2`	`2`	`const common = require('../common');`
`3`		`-const assert = require('assert');`
`4`		`-const exec = require('child_process').exec;`
	`3`	`+`
	`4`	+// Check `id -G` and `process.getgroups()` return same groups.
`5`	`5`
`6`	`6`	`if (common.isOSX) {`
`7`	`7`	common.skip('Output of `id -G` is unreliable on Darwin.');
`8`	`8`	`return;`
`9`	`9`	`}`
	`10`	`+const assert = require('assert');`
	`11`	`+const exec = require('child_process').exec;`
`10`	`12`
`11`	`13`	`if (typeof process.getgroups === 'function') {`
`12`		`- const groups = process.getgroups();`
	`14`	`+ const groups = unique(process.getgroups());`
`13`	`15`	`assert(Array.isArray(groups));`
`14`	`16`	`assert(groups.length > 0);`
`15`	`17`	`exec('id -G', function(err, stdout) {`
`16`		`- if (err) throw err;`
`17`		`- const real_groups = stdout.match(/\d+/g).map(Number);`
`18`		`- assert.strictEqual(groups.length, real_groups.length);`
	`18`	`+ assert.ifError(err);`
	`19`	`+ const real_groups = unique(stdout.match(/\d+/g).map(Number));`
	`20`	`+ assert.deepStrictEqual(groups, real_groups);`
`19`	`21`	`check(groups, real_groups);`
`20`	`22`	`check(real_groups, groups);`
`21`	`23`	`});`
`@@ -24,3 +26,7 @@ if (typeof process.getgroups === 'function') {`
`24`	`26`	`function check(a, b) {`
`25`	`27`	`for (let i = 0; i < a.length; ++i) assert.notStrictEqual(b.indexOf(a[i]), -1);`
`26`	`28`	`}`
	`29`	`+`
	`30`	`+function unique(groups) {`
	`31`	`+ return [...new Set(groups)].sort();`
	`32`	`+}`

0 commit comments

Comments

0 (0)

You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.