test: add comments and rename test for timer robustness

Trott · RafaelGSS · commit 1344bd2d6f38 · 2024-08-05T11:15:45.000-03:00
The name of the test did not make it clear what it was about. (It also used "timer" in the name instead of "timers" like all the other tests.) I also added a comment to be extra clear about the test purpose and a link to the issue that was originally filed about it. PR-URL: #54008 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Chengzhong Wu <legendecas@gmail.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Jake Yuesong Li <jake.yuesong@gmail.com>
diff --git a/test/parallel/test-timers-process-tampering.js b/test/parallel/test-timers-process-tampering.js
@@ -1,3 +1,6 @@
+// Check that setImmediate works even if process is tampered with.
+// This is a regression test for https://github.com/nodejs/node/issues/17681.
+
 'use strict';
 const common = require('../common');
 global.process = {};  // Boom!
