Skip to content
Permalink
Browse files

test: add not-called check to heap-profiler test

Add `common.mustNotCall()` to make sure there aren't any strange
shenanians in the C++ test that would cause the function to execute when
it shouldn't.

PR-URL: #12985
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
  • Loading branch information...
Trott committed May 11, 2017
1 parent b5ae22d commit b7bc09fd60fa89a572f1de78bf78562bd93adbd3
Showing with 1 addition and 1 deletion.
  1. +1 −1 test/addons/heap-profiler/test.js
@@ -5,7 +5,7 @@ const common = require('../../common');
const binding = require(`./build/${common.buildType}/binding`);

// Create an AsyncWrap object.
const timer = setTimeout(common.noop, 1);
const timer = setTimeout(common.mustNotCall(), 1);
timer.unref();

// Stress-test the heap profiler.

0 comments on commit b7bc09f

Please sign in to comment.
You can’t perform that action at this time.