Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 9f15bc4

Browse filesBrowse files
jnord99MylesBorins
authored andcommitted
test: skip failing tests for osx mojave
Refs: #21679 PR-URL: #23550 Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: George Adams <george.adams@uk.ibm.com> Reviewed-By: Gus Caplan <me@gus.host> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
1 parent aba1ff2 commit 9f15bc4
Copy full SHA for 9f15bc4

File tree

Expand file treeCollapse file tree

4 files changed

+37
-1
lines changed
Open diff view settings
Filter options
Expand file treeCollapse file tree

4 files changed

+37
-1
lines changed
Open diff view settings
Collapse file

‎test/common/index.js‎

Copy file name to clipboardExpand all lines: test/common/index.js
+2-1Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,8 @@ exports.isSunOS = process.platform === 'sunos';
4747
exports.isFreeBSD = process.platform === 'freebsd';
4848
exports.isOpenBSD = process.platform === 'openbsd';
4949
exports.isLinux = process.platform === 'linux';
50-
exports.isOSX = process.platform === 'darwin';
50+
const isOSX = exports.isOSX = process.platform === 'darwin';
51+
exports.isOSXMojave = isOSX && (os.release().startsWith('18'));
5152

5253
exports.enoughTestMem = os.totalmem() > 0x70000000; /* 1.75 Gb */
5354
const cpus = os.cpus();
Collapse file
+25Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
'use strict';
2+
const common = require('../common');
3+
4+
// This test should fail on macOS (10.14) due to an issue with privileged ports.
5+
6+
const assert = require('assert');
7+
const cluster = require('cluster');
8+
const net = require('net');
9+
10+
if (!common.isOSXMojave)
11+
assert.fail('Code should fail only on macOS Mojave.');
12+
13+
14+
if (cluster.isMaster) {
15+
cluster.fork().on('exit', common.mustCall((exitCode) => {
16+
assert.strictEqual(exitCode, 0);
17+
}));
18+
} else {
19+
const s = net.createServer(common.mustNotCall());
20+
s.listen(42, common.mustNotCall('listen should have failed'));
21+
s.on('error', common.mustCall((err) => {
22+
assert.strictEqual(err.code, 'EACCES');
23+
process.disconnect();
24+
}));
25+
}
Collapse file

‎test/parallel/test-cluster-bind-privileged-port.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-cluster-bind-privileged-port.js
+5Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,11 @@
2121

2222
'use strict';
2323
const common = require('../common');
24+
25+
// Skip on OS X Mojave. https://github.com/nodejs/node/issues/21679
26+
if (common.isOSXMojave)
27+
common.skip('bypass test for Mojave due to OSX issue');
28+
2429
if (common.isWindows)
2530
common.skip('not reliable on Windows.');
2631

Collapse file

‎test/parallel/test-cluster-shared-handle-bind-privileged-port.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-cluster-shared-handle-bind-privileged-port.js
+5Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,11 @@
2121

2222
'use strict';
2323
const common = require('../common');
24+
25+
// Skip on OS X Mojave. https://github.com/nodejs/node/issues/21679
26+
if (common.isOSXMojave)
27+
common.skip('bypass test for Mojave due to OSX issue');
28+
2429
if (common.isWindows)
2530
common.skip('not reliable on Windows');
2631

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.