Browse Source

test: add common.platformTimeout() to dgram test

Use common.platformTimeout() to mitigate potential for CI to timeout
on slower devices.

Ref: https://github.com/nodejs/node/issues/4937
PR-URL: https://github.com/nodejs/node/pull/4938
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Myles Borins <mborins@us.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
process-exit-stdio-flushing
Rich Trott 9 years ago
parent
commit
d3195615c0
  1. 15
      test/parallel/test-dgram-send-empty-buffer.js

15
test/parallel/test-dgram-send-empty-buffer.js

@ -1,16 +1,13 @@
'use strict';
var common = require('../common');
var dgram = require('dgram');
var client, timer, buf;
const common = require('../common');
const dgram = require('dgram');
if (process.platform === 'darwin') {
console.log('1..0 # Skipped: because of 17894467 Apple bug');
return;
}
client = dgram.createSocket('udp4');
const client = dgram.createSocket('udp4');
client.bind(common.PORT);
@ -19,9 +16,9 @@ client.on('message', function(buffer, bytes) {
client.close();
});
buf = new Buffer(0);
const buf = new Buffer(0);
client.send(buf, 0, 0, common.PORT, '127.0.0.1', function(err, len) { });
timer = setTimeout(function() {
const timer = setTimeout(function() {
throw new Error('Timeout');
}, 200);
}, common.platformTimeout(200));

Loading…
Cancel
Save