test: swap arguments in strictEqual()

This commit also comments the third argument
PR-URL: #21660
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>

Author: sohailrajdev97

test/pummel/test-timers.js

@@ -31,7 +31,7 @@ let interval_count = 0;
 clearTimeout(null);
 clearInterval(null);
 
-assert.strictEqual(true,setTimeoutinstanceofFunction);
+assert.strictEqual(setTimeoutinstanceofFunction,true);
 conststarttime=newDate();
 setTimeout(common.mustCall(function(){
 constendtime=newDate();
@@ -40,7 +40,7 @@ setTimeout(common.mustCall(function() {
 assert.ok(diff>0);
 console.error(`diff: ${diff}`);
 
-assert.strictEqual(true,1000-WINDOW<diff&&diff<1000+WINDOW);
+assert.strictEqual(1000-WINDOW<diff&&diff<1000+WINDOW,true);
 }),1000);
 
 // this timer shouldn't execute
@@ -57,23 +57,23 @@ setInterval(function() {
 
 constt=interval_count*1000;
 
-assert.strictEqual(true,t-WINDOW<diff&&diff<t+WINDOW);
+assert.strictEqual(t-WINDOW<diff&&diff<t+WINDOW,true);
 
-assert.strictEqual(true,interval_count<=3);
+assert.strictEqual(interval_count<=3,true);
 if(interval_count===3)
 clearInterval(this);
 },1000);
 
 
 // Single param:
 setTimeout(function(param){
-assert.strictEqual('test param',param);
+assert.strictEqual(param,'test param');
 },1000,'test param');
 
 letinterval_count2=0;
 setInterval(function(param){
 ++interval_count2;
-assert.strictEqual('test param',param);
+assert.strictEqual(param,'test param');
 
 if(interval_count2===3)
 clearInterval(this);
@@ -82,15 +82,15 @@ setInterval(function(param) {
 
 // Multiple param
 setTimeout(function(param1,param2){
-assert.strictEqual('param1',param1);
-assert.strictEqual('param2',param2);
+assert.strictEqual(param1,'param1');
+assert.strictEqual(param2,'param2');
 },1000,'param1','param2');
 
 letinterval_count3=0;
 setInterval(function(param1,param2){
 ++interval_count3;
-assert.strictEqual('param1',param1);
-assert.strictEqual('param2',param2);
+assert.strictEqual(param1,'param1');
+assert.strictEqual(param2,'param2');
 
 if(interval_count3===3)
 clearInterval(this);
@@ -120,8 +120,9 @@ clearTimeout(y);
 
 
 process.on('exit',function(){
-assert.strictEqual(3,interval_count);
-assert.strictEqual(11,count4);
-assert.strictEqual(0,expectedTimeouts,
-'clearTimeout cleared too many timeouts');
+assert.strictEqual(interval_count,3);
+assert.strictEqual(count4,11);
+
+// Check that the correct number of timers ran.
+assert.strictEqual(expectedTimeouts,0);
 });