test_runner: do not invoke after hook when test is empty

PR-URL: #51389 Fixes: #51371 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
nodejs · Feb 15, 2024 · e0789fb · e0789fb
1 parent 76582e4
commit e0789fb
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/lib/internal/test_runner/test.js b/lib/internal/test_runner/test.js
@@ -587,7 +587,9 @@ class Test extends AsyncResource {
 
     const { args, ctx } = this.getRunArgs();
     const after = async () => {
-      if (this.hooks.after.length > 0) {
+      // If its a root test then check for global after hook else check for parent after hook
+      const check = this.parent ? this.parent.hooks.after.length > 0 : this.hooks.after.length > 0;
+      if (check) {
         await this.runHook('after', { __proto__: null, args, ctx });
       }
     };

diff --git a/test/parallel/test-runner-skip-after-hook.js b/test/parallel/test-runner-skip-after-hook.js
@@ -0,0 +1,10 @@
+'use strict';
+// Refs: https://github.com/nodejs/node/issues/51371
+const common = require('../common');
+const { test } = require('node:test');
+
+test('test', async (t) => {
+  t.after(common.mustNotCall(() => {
+    t.fail('should not run');
+  }));
+});