Skip to content

Commit

Permalink
test: add coverage to dgram receive error case
Browse files Browse the repository at this point in the history
This commit adds coverage for the case where a dgram socket
handle receives a message, but nread < 0, indicating an error.

PR-URL: nodejs#11241
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
  • Loading branch information
cjihrig authored and krydos committed Feb 25, 2017
1 parent 283321d commit 2a95c63
Showing 1 changed file with 16 additions and 0 deletions.
16 changes: 16 additions & 0 deletions test/parallel/test-dgram-recv-error.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
'use strict';
const common = require('../common');
const assert = require('assert');
const dgram = require('dgram');
const s = dgram.createSocket('udp4');

s.on('error', common.mustCall((err) => {
s.close();

// Don't check the full error message, as the errno is not important here.
assert(/^Error: recvmsg/.test(err));
assert.strictEqual(err.syscall, 'recvmsg');
}));

s.on('message', common.mustNotCall('no message should be received.'));
s.bind(common.mustCall(() => s._handle.onmessage(-1, s._handle, null, null)));

0 comments on commit 2a95c63

Please sign in to comment.