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 italoacasas committed Jan 30, 2017
1 parent af2bea7 commit 33851d1
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/debugger/test-debugger-repl.js
@@ -1,5 +1,5 @@
'use strict';
require('../common');
const common = require('../common');
const 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 33851d1

Please sign in to comment.