test: enable no-empty ESLint rule

PR-URL: #41831
Backport-PR-URL: #42160
Refs: https://eslint.org/docs/rules/no-empty
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>

Author: Trott

test/.eslintrc.yaml

@@ -5,10 +5,11 @@ env:
   es6: true
 
 rules:
+  multiline-comment-style: ["error", "separate-lines"]
+  no-empty: error
   no-var: error
   prefer-const: error
   symbol-description: off
-  multiline-comment-style: ["error", "separate-lines"]
 
   no-restricted-syntax:
     # Config copied from .eslintrc.js

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,7 @@ 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');
     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');