test_runner: call {before,after}Each() on suites

Prior to this commit, beforeEach() and afterEach() hooks were
not called on test suites (describe()). This commit addresses that.

Fixes: #45028
PR-URL: #45161
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: James M Snell <[email protected]>

Author: cjihrig

lib/internal/test_runner/test.js

@@ -731,13 +731,24 @@ class Suite extends Test {
       }
 
       const hookArgs = this.getRunArgs();
+
+      if (this.parent?.hooks.beforeEach.length > 0) {
+        await this.parent[kRunHook]('beforeEach', hookArgs);
+      }
+
       await this[kRunHook]('before', hookArgs);
+
       const stopPromise = stopTest(this.timeout, this.signal);
       const subtests = this.skipped || this.error ? [] : this.subtests;
       const promise = SafePromiseAll(subtests, (subtests) => subtests.start());
 
       await SafePromiseRace([promise, stopPromise]);
       await this[kRunHook]('after', hookArgs);
+
+      if (this.parent?.hooks.afterEach.length > 0) {
+        await this.parent[kRunHook]('afterEach', hookArgs);
+      }
+
       this.pass();
     } catch (err) {
       if (isTestFailureError(err)) {

test/message/test_runner_hooks.js

@@ -15,10 +15,12 @@ describe('describe hooks', () => {
       'before describe hooks',
       'beforeEach 1', '1', 'afterEach 1',
       'beforeEach 2', '2', 'afterEach 2',
+      'beforeEach nested',
       'before nested',
       'beforeEach nested 1', 'nested 1', 'afterEach nested 1',
       'beforeEach nested 2', 'nested 2', 'afterEach nested 2',
       'after nested',
+      'afterEach nested',
       'after describe hooks',
     ]);
   });

test/message/test_runner_test_name_pattern.js

@@ -34,8 +34,8 @@ test('top level test enabled', common.mustCall(async (t) => {
 
 describe('top level describe enabled', () => {
   before(common.mustCall());
-  beforeEach(common.mustCall(2));
-  afterEach(common.mustCall(2));
+  beforeEach(common.mustCall(4));
+  afterEach(common.mustCall(4));
   after(common.mustCall());
 
   it('nested it disabled', common.mustNotCall());