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: #11241
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
  • Loading branch information
cjihrig authored and italoacasas committed Feb 14, 2017
1 parent ccd1163 commit 480d4cc
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 480d4cc

Please sign in to comment.