From 2c056a40c7d82ddf2b915c4ec4e181bf2008ba24 Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Sat, 29 Oct 2016 15:39:02 -0700 Subject: [PATCH] test: remove timer in test-dgram-send-empty-array The timer is not necessary. The test will timeout via the test harness if the test fails. This should resolve spurious CI failures. PR-URL: https://github.com/nodejs/node/pull/9361 Reviewed-By: Anna Henningsen Reviewed-By: Santiago Gimeno Reviewed-By: Colin Ihrig Reviewed-By: James M Snell Reviewed-By: Evan Lucas Reviewed-By: Luigi Pinca --- test/parallel/test-dgram-send-empty-array.js | 5 ----- 1 file changed, 5 deletions(-) diff --git a/test/parallel/test-dgram-send-empty-array.js b/test/parallel/test-dgram-send-empty-array.js index c5050a7d548265..f9de345f112254 100644 --- a/test/parallel/test-dgram-send-empty-array.js +++ b/test/parallel/test-dgram-send-empty-array.js @@ -11,14 +11,9 @@ if (common.isOSX) { const client = dgram.createSocket('udp4'); -const timer = setTimeout(function() { - throw new Error('Timeout'); -}, common.platformTimeout(200)); - client.on('message', common.mustCall(function onMessage(buf, info) { const expected = Buffer.alloc(0); assert.ok(buf.equals(expected), 'message was received correctly'); - clearTimeout(timer); client.close(); }));