test: add coverage for dgram _createSocketHandle() #11291

Merged
merged 1 commit into from Feb 13, 2017

Conversation

Projects
None yet
3 participants
@cjihrig
Contributor

cjihrig commented Feb 10, 2017

This commit adds code coverage to _createSocketHandle(), which the cluster module uses to create dgram sockets.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines
Affected core subsystem(s)

test

@cjihrig cjihrig added the dgram label Feb 10, 2017

test: add coverage for dgram _createSocketHandle()
This commit adds code coverage to _createSocketHandle(), which
the cluster module uses to create dgram sockets.

PR-URL: #11291
Reviewed-By: James M Snell <jasnell@gmail.com>

@cjihrig cjihrig merged commit a2fac32 into nodejs:master Feb 13, 2017

@cjihrig cjihrig deleted the cjihrig:create-socket-handle branch Feb 13, 2017

italoacasas added a commit to italoacasas/node that referenced this pull request Feb 14, 2017

test: add coverage for dgram _createSocketHandle()
This commit adds code coverage to _createSocketHandle(), which
the cluster module uses to create dgram sockets.

PR-URL: nodejs#11291
Reviewed-By: James M Snell <jasnell@gmail.com>

jasnell added a commit that referenced this pull request Mar 7, 2017

test: add coverage for dgram _createSocketHandle()
This commit adds code coverage to _createSocketHandle(), which
the cluster module uses to create dgram sockets.

PR-URL: #11291
Reviewed-By: James M Snell <jasnell@gmail.com>

jasnell added a commit that referenced this pull request Mar 7, 2017

test: add coverage for dgram _createSocketHandle()
This commit adds code coverage to _createSocketHandle(), which
the cluster module uses to create dgram sockets.

PR-URL: #11291
Reviewed-By: James M Snell <jasnell@gmail.com>

MylesBorins added a commit that referenced this pull request Mar 9, 2017

test: add coverage for dgram _createSocketHandle()
This commit adds code coverage to _createSocketHandle(), which
the cluster module uses to create dgram sockets.

PR-URL: #11291
Reviewed-By: James M Snell <jasnell@gmail.com>

@MylesBorins MylesBorins referenced this pull request Mar 9, 2017

Merged

v6.10.1 proposal #11759

MylesBorins added a commit that referenced this pull request Mar 9, 2017

test: add coverage for dgram _createSocketHandle()
This commit adds code coverage to _createSocketHandle(), which
the cluster module uses to create dgram sockets.

PR-URL: #11291
Reviewed-By: James M Snell <jasnell@gmail.com>

@MylesBorins MylesBorins referenced this pull request Mar 9, 2017

Merged

v4.8.1 proposal #11760

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment