From 968fc72ad163cd635f6f33ad598048e327a7a791 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 c5050a7d54..f9de345f11 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(); }));