diff --git a/lib/executor.js b/lib/executor.js index 5198d7f..84c0605 100644 --- a/lib/executor.js +++ b/lib/executor.js @@ -115,7 +115,9 @@ function setPort(app, instructions) { process.env.npm_package_config_port, app.get('port'), 3000 - ], isFinite); + ], function(p) { + return p != null; + }); if (port !== undefined) { var portType = typeof port; diff --git a/test/executor.test.js b/test/executor.test.js index 997c253..30da00d 100644 --- a/test/executor.test.js +++ b/test/executor.test.js @@ -417,10 +417,11 @@ describe('executor', function() { assert.equal(app.get('port'), 3000); }); - it('should ignore non-numeric port values in ENV', function() { - process.env.PORT = '123invalid'; + it('should respect named pipes port values in ENV', function() { + var NAMED_PORT = '\\.\\pipe\\test'; + process.env.PORT = NAMED_PORT; boot.execute(app, someInstructions({ config: { port: 3000 } })); - assert.equal(app.get('port'), 3000); + assert.equal(app.get('port'), NAMED_PORT); }); });