Skip to content

Commit 33851d1

Browse files
thefourtheyeitaloacasas
authored andcommitted
test: fix process.title expectation
`process.title` would work properly only in FreeBSD, OSX, and Linux as per test/parallel/test-setproctitle.js. This patch makes sure that the test expects an empty string in other platforms. This patch helps fix the SmartOS failures in https://ci.nodejs.org/job/node-test-commit/6962/ for nodejs#10456 PR-URL: nodejs#10597 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Evan Lucas <evanlucas@me.com>
1 parent af2bea7 commit 33851d1

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

test/debugger/test-debugger-repl.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
'use strict';
2-
require('../common');
2+
const common = require('../common');
33
const repl = require('./helper-debugger-repl.js');
44

55
repl.startDebugger('breakpoints.js');
@@ -57,7 +57,7 @@ addTest('c', [
5757

5858
// Execute
5959
addTest('exec process.title', [
60-
/node/
60+
common.isFreeBSD || common.isOSX || common.isLinux ? /node/ : ''
6161
]);
6262

6363
// Execute

0 commit comments

Comments
 (0)