test: add comments and rename test for timer robustness

Trott · web-flow · commit 82cf0f2f3c9e · 2024-07-25T17:33:38.000Z
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!
