Skip to content

Commit

Permalink
test: fix process.title expectation
Browse files Browse the repository at this point in the history
`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
#10456

PR-URL: #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>
  • Loading branch information
thefourtheye authored and MylesBorins committed Mar 9, 2017
1 parent ae338da commit 443dd50
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/debugger/test-debugger-repl.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
'use strict';
require('../common');
var common = require('../common');
var repl = require('./helper-debugger-repl.js');

repl.startDebugger('breakpoints.js');
Expand Down Expand Up @@ -57,7 +57,7 @@ addTest('c', [

// Execute
addTest('exec process.title', [
/node/
common.isFreeBSD || common.isOSX || common.isLinux ? /node/ : ''
]);

// Execute
Expand Down

0 comments on commit 443dd50

Please sign in to comment.