test: add common.platformTimeout() to dgram test
authorRich Trott <rtrott@gmail.com>
Thu, 28 Jan 2016 23:59:35 +0000 (15:59 -0800)
committerMyles Borins <mborins@us.ibm.com>
Wed, 2 Mar 2016 22:01:11 +0000 (14:01 -0800)
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>
test/parallel/test-dgram-send-empty-buffer.js

index a8ab433..6732bab 100644 (file)
@@ -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));