test_runner: run afterEach hooks in correct order

This commit updates the test runner afterEach hook so that the current test's afterEach hooks run before any ancestor afterEach hooks. Fixes: #51671 PR-URL: #52239 Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Chemi Atlow <[email protected]>
nodejs · May 3, 2024 · 35588cf · 35588cf
1 parent ea8e72e
commit 35588cf
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 7 deletions.
diff --git a/lib/internal/test_runner/test.js b/lib/internal/test_runner/test.js
@@ -6,6 +6,7 @@ const {
   ArrayPrototypeShift,
   ArrayPrototypeSlice,
   ArrayPrototypeSome,
+  ArrayPrototypeSplice,
   ArrayPrototypeUnshift,
   FunctionPrototype,
   MathMax,
@@ -250,6 +251,7 @@ class Test extends AsyncResource {
         after: [],
         beforeEach: [],
         afterEach: [],
+        ownAfterEachCount: 0,
       };
     } else {
       const nesting = parent.parent === null ? parent.nesting :
@@ -269,6 +271,7 @@ class Test extends AsyncResource {
         after: [],
         beforeEach: ArrayPrototypeSlice(parent.hooks.beforeEach),
         afterEach: ArrayPrototypeSlice(parent.hooks.afterEach),
+        ownAfterEachCount: 0,
       };
     }
 
@@ -502,7 +505,15 @@ class Test extends AsyncResource {
         }
       });
     }
-    ArrayPrototypePush(this.hooks[name], hook);
+    if (name === 'afterEach') {
+      // afterEach hooks for the current test should run in the order that they
+      // are created. However, the current test's afterEach hooks should run
+      // prior to any ancestor afterEach hooks.
+      ArrayPrototypeSplice(this.hooks[name], this.hooks.ownAfterEachCount, 0, hook);
+      this.hooks.ownAfterEachCount++;
+    } else {
+      ArrayPrototypePush(this.hooks[name], hook);
+    }
     return hook;
   }
 

diff --git a/test/fixtures/test-runner/output/hooks-with-no-global-test.js b/test/fixtures/test-runner/output/hooks-with-no-global-test.js
@@ -26,14 +26,14 @@ after(() => {
     'describe beforeEach',
     'describe nested beforeEach',
     'describe nested it 1',
-    'describe afterEach',
     'describe nested afterEach',
+    'describe afterEach',
 
     'describe beforeEach',
     'describe nested beforeEach',
     'describe nested test 2',
-    'describe afterEach',
     'describe nested afterEach',
+    'describe afterEach',
 
     'describe nested after',
     'describe after',

diff --git a/test/fixtures/test-runner/output/hooks.js b/test/fixtures/test-runner/output/hooks.js
@@ -18,8 +18,8 @@ describe('describe hooks', () => {
       'beforeEach 1', '1', 'afterEach 1',
       'beforeEach 2', '2', 'afterEach 2',
       'before nested',
-      'beforeEach nested 1', '+beforeEach nested 1', 'nested 1', 'afterEach nested 1', '+afterEach nested 1',
-      'beforeEach nested 2', '+beforeEach nested 2', 'nested 2', 'afterEach nested 2', '+afterEach nested 2',
+      'beforeEach nested 1', '+beforeEach nested 1', 'nested 1', '+afterEach nested 1', 'afterEach nested 1',
+      'beforeEach nested 2', '+beforeEach nested 2', 'nested 2', '+afterEach nested 2', 'afterEach nested 2',
       'after nested',
       'after describe hooks',
     ]);
@@ -139,8 +139,8 @@ test('test hooks', async (t) => {
       'beforeEach 2', '2', 'afterEach 2',
       'beforeEach nested',
       'nested before nested',
-      'beforeEach nested 1', 'nested beforeEach nested 1', 'nested1', 'afterEach nested 1', 'nested afterEach nested 1',
-      'beforeEach nested 2', 'nested beforeEach nested 2', 'nested 2', 'afterEach nested 2', 'nested afterEach nested 2',
+      'beforeEach nested 1', 'nested beforeEach nested 1', 'nested1', 'nested afterEach nested 1', 'afterEach nested 1',
+      'beforeEach nested 2', 'nested beforeEach nested 2', 'nested 2', 'nested afterEach nested 2', 'afterEach nested 2',
       'afterEach nested',
       'nested after nested',
       'after test hooks',