test: add comments to empty blocks

Author: Trott

test/addons/uv-handle-leak/test.js

@@ -16,7 +16,9 @@ try {
   const { isMainThread } = require('worker_threads');
   if (!isMainThread)
     common.skip('Cannot run test in environment with clean-exit policy');
-} catch {}
+} catch {
+  // Continue regardless of error
+}
 
 binding.leakHandle();
 binding.leakHandle(0);

test/common/inspector-helper.js

@@ -365,8 +365,9 @@ class NodeInstance extends EventEmitter {
       ['--expose-internals'],
       `${scriptContents}\nprocess._rawDebug('started');`, undefined);
     const msg = 'Timed out waiting for process to start';
-    while (await fires(instance.nextStderrString(), msg, TIMEOUT) !==
-             'started') {}
+    while (await fires(instance.nextStderrString(), msg, TIMEOUT) !== 'started') {
+      // empty
+    }
     process._debugProcess(instance._process.pid);
     return instance;
   }

test/common/report.js

@@ -46,7 +46,9 @@ function validateContent(report, fields = []) {
   } catch (err) {
     try {
       err.stack += util.format('\n------\nFailing Report:\n%O', report);
-    } catch {}
+    } catch {
+      // Continue regardless of error
+    }
     throw err;
   }
 }

test/internet/test-dgram-broadcast-multi-process.js

@@ -184,7 +184,9 @@ if (process.argv[2] !== 'child') {
     const buf = messages[i++];
 
     if (!buf) {
-      try { sendSocket.close(); } catch {}
+      try { sendSocket.close(); } catch {
+        // Continue regardless of error
+      }
       return;
     }
 

test/internet/test-dgram-multicast-multi-process.js

@@ -170,7 +170,9 @@ if (process.argv[2] !== 'child') {
     const buf = messages[i++];
 
     if (!buf) {
-      try { sendSocket.close(); } catch {}
+      try { sendSocket.close(); } catch {
+        // Continue regardless of error
+      }
       return;
     }
 

test/internet/test-dgram-multicast-ssm-multi-process.js

@@ -164,7 +164,9 @@ if (process.argv[2] !== 'child') {
     const buf = messages[i++];
 
     if (!buf) {
-      try { sendSocket.close(); } catch {}
+      try { sendSocket.close(); } catch {
+        // Continue regardless of error
+      }
       return;
     }
 

test/internet/test-dgram-multicast-ssmv6-multi-process.js

@@ -164,7 +164,9 @@ if (process.argv[2] !== 'child') {
     const buf = messages[i++];
 
     if (!buf) {
-      try { sendSocket.close(); } catch {}
+      try { sendSocket.close(); } catch {
+        // Continue regardless of error
+      }
       return;
     }
 

test/message/vm_dont_display_runtime_error.js

@@ -30,7 +30,9 @@ try {
     filename: 'test.vm',
     displayErrors: false
   });
-} catch {}
+} catch {
+  // Continue regardless of error
+}
 
 console.error('middle');
 

test/message/vm_dont_display_syntax_error.js

@@ -30,7 +30,9 @@ try {
     filename: 'test.vm',
     displayErrors: false
   });
-} catch {}
+} catch {
+  // Continue regardless of error
+}
 
 console.error('middle');
 

test/node-api/test_fatal/test_threads.js

@@ -8,8 +8,9 @@ const test_fatal = require(`./build/${common.buildType}/test_fatal`);
 // that crashes the process.
 if (process.argv[2] === 'child') {
   test_fatal.TestThread();
-  // Busy loop to allow the work thread to abort.
-  while (true) {}
+  while (true) {
+    // Busy loop to allow the work thread to abort.
+  }
 }
 
 const p = child_process.spawnSync(