test: remove common.isOSXMojave

common.isOSXMojave was added because it was believed that there was a bug in macOS Mojave that allowed unprivileged users to bind to privileged ports. As it turns out, that was a feature not a bug. It is likely to be in all future versions of macOS. Remove isOSXMojave and skip appropriate tests based on isOSX. Refs: https://news.ycombinator.com/item?id=18302380 PR-URL: #25658 Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Beth Griggs <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Gus Caplan <[email protected]> Reviewed-By: Minwoo Jung <[email protected]>
nodejs · May 16, 2019 · 52d321d · 52d321d
1 parent a382932
commit 52d321d
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 7 deletions.
diff --git a/test/common/index.js b/test/common/index.js
@@ -99,8 +99,6 @@ const isOpenBSD = process.platform === 'openbsd';
 const isLinux = process.platform === 'linux';
 const isOSX = process.platform === 'darwin';
 
-const isOSXMojave = isOSX && (os.release().startsWith('18'));
-
 const enoughTestMem = os.totalmem() > 0x70000000; /* 1.75 Gb */
 const cpus = os.cpus();
 const enoughTestCpu = Array.isArray(cpus) &&
@@ -754,7 +752,6 @@ module.exports = {
   isMainThread,
   isOpenBSD,
   isOSX,
-  isOSXMojave,
   isSunOS,
   isWindows,
   localIPv6Hosts,

diff --git a/test/parallel/test-cluster-bind-privileged-port.js b/test/parallel/test-cluster-bind-privileged-port.js
@@ -23,8 +23,8 @@
 const common = require('../common');
 
 // Skip on OS X Mojave. https://github.com/nodejs/node/issues/21679
-if (common.isOSXMojave)
-  common.skip('bypass test for Mojave due to OSX issue');
+if (common.isOSX)
+  common.skip('macOS may allow ordinary processes to use any port');
 
 if (common.isWindows)
   common.skip('not reliable on Windows.');

diff --git a/test/parallel/test-cluster-shared-handle-bind-privileged-port.js b/test/parallel/test-cluster-shared-handle-bind-privileged-port.js
@@ -23,8 +23,8 @@
 const common = require('../common');
 
 // Skip on OS X Mojave. https://github.com/nodejs/node/issues/21679
-if (common.isOSXMojave)
-  common.skip('bypass test for Mojave due to OSX issue');
+if (common.isOSX)
+  common.skip('macOS may allow ordinary processes to use any port');
 
 if (common.isWindows)
   common.skip('not reliable on Windows');