From 702b46c80dfe3ec8f67c15f802db9d2d86a40a6c Mon Sep 17 00:00:00 2001 From: isaacs Date: Thu, 15 Mar 2012 14:53:17 -0700 Subject: [PATCH] Fix invalid timer test Previously, setTimeout(fn, 0) would create a new Timer() object, which has a close() method (and is a bit slower). The recent change to more closely emulate browser setTimeout behavior dodges this path, so this assertion is no longer valid. --- test/simple/test-timers-zero-timeout.js | 2 -- 1 file changed, 2 deletions(-) diff --git a/test/simple/test-timers-zero-timeout.js b/test/simple/test-timers-zero-timeout.js index 9f77d7822d3..eb43f183f89 100644 --- a/test/simple/test-timers-zero-timeout.js +++ b/test/simple/test-timers-zero-timeout.js @@ -38,8 +38,6 @@ var assert = require('assert'); process.on('exit', function() { assert.equal(ncalled, 1); - // timer should be already closed - assert.equal(timer.close(), -1); }); })();