Skip to content

test: add more test cases of server.listen option #11778

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
joyeecheung wants to merge 5 commits into from
Closed

test: add more test cases of server.listen option #11778

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
7b65fa6
test: add more test cases of server.listen option
joyeecheung Mar 10, 2017
5ca2e5e
check generated handle doesn't have fd -1
joyeecheung Mar 10, 2017
180e207
separate path and handle tests, clean up sockets
joyeecheung Mar 10, 2017
2b7354c
avoid pipe conflict
joyeecheung Mar 10, 2017
445a68b
don't test fd on windows
joyeecheung Mar 10, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
avoid pipe conflict
  • Loading branch information
@joyeecheung
joyeecheung committed Mar 10, 2017
commit 2b7354cf95743b67bebe64b8147311ae28b90f01
3 changes: 2 additions & 1 deletion test/parallel/test-net-server-listen-handle.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,9 @@ function closePipeServer(handle) {

let counter = 0;

// Avoid conflict with listen-path
function randomPipePath() {
return common.PIPE + '-listen-pipe-' + (counter++);
return common.PIPE + '-listen-handle-' + (counter++);
}

function randomHandle(type) {
Expand Down
20 changes: 14 additions & 6 deletions test/parallel/test-net-server-listen-path.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,40 +2,48 @@

const common = require('../common');
const net = require('net');
function close() { this.close(); }

common.refreshTmpDir();

function closeServer() {
return common.mustCall(function() {
this.close();
});
}

let counter = 0;

// Avoid conflict with listen-handle
function randomPipePath() {
return common.PIPE + '-listen-pipe-' + (counter++);
return common.PIPE + '-listen-path-' + (counter++);
}

// Test listen(path)
{
const handlePath = randomPipePath();
net.createServer()
.listen(handlePath)
.on('listening', common.mustCall(close));
.on('listening', closeServer());
}

// Test listen({path})
{
const handlePath = randomPipePath();
net.createServer()
.listen({path: handlePath})
.on('listening', common.mustCall(close));
.on('listening', closeServer());
}

// Test listen(path, cb)
{
const handlePath = randomPipePath();
net.createServer()
.listen(handlePath, common.mustCall(close));
.listen(handlePath, closeServer());
}

// Test listen(path, cb)
{
const handlePath = randomPipePath();
net.createServer()
.listen({path: handlePath}, common.mustCall(close));
.listen({path: handlePath}, closeServer());
}