test: improve pummel/test-timers.js

* use Date.now() instead of new Date() because only the timestamp is ever used, so we don't need the full Date object * use separate start times recorded for the two different test cases * improve assertion messages PR-URL: #35175 Reviewed-By: Daijiro Wachi <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Shingo Inoue <[email protected]>

Author: Trott

test/pummel/test-timers.js

@@ -26,10 +26,11 @@ const assert = require('assert');
 constWINDOW=200;// Why does this need to be so big?
 
 
-conststarttime=newDate();
 {
+conststarttime=Date.now();
+
 setTimeout(common.mustCall(function(){
-constendtime=newDate();
+constendtime=Date.now();
 
 constdiff=endtime-starttime;
 assert.ok(diff>0);
@@ -46,21 +47,23 @@ const starttime = new Date();
 }
 
 {
+conststarttime=Date.now();
+
 letinterval_count=0;
 
 setInterval(common.mustCall(function(){
 interval_count+=1;
-constendtime=newDate();
+constendtime=Date.now();
 
 constdiff=endtime-starttime;
 assert.ok(diff>0);
 console.error(`diff: ${diff}`);
 
 constt=interval_count*1000;
 
-assert.strictEqual(t-WINDOW<diff&&diff<t+WINDOW,true);
+assert.ok(t-WINDOW<diff&&diff<t+WINDOW,`t: ${t}`);
 
-assert.strictEqual(interval_count<=3,true);
+assert.ok(interval_count<=3,`interval_count: ${interval_count}`);
 if(interval_count===3)
 clearInterval(this);
 },3),1000);