fix(allure-jest): missing hooks in some tests

beforeAll/afterAll hooks were missing for tests of sub-suites
allure-framework · Jul 8, 2024 · d189d9a · d189d9a
1 parent 1b3c982
commit d189d9a
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 2 deletions.
diff --git a/packages/allure-jest/src/environmentFactory.ts b/packages/allure-jest/src/environmentFactory.ts
@@ -191,7 +191,6 @@ const createJestEnvironment = <T extends typeof JestEnvironment>(Base: T): T =>
         return;
       }
 
-      const scopeUuid = last(this.runContext.scopes);
       const threadLabel = ALLURE_THREAD_NAME || JEST_WORKER_ID || process.pid.toString();
       const hostLabel = ALLURE_HOST_NAME || hostname;
       const packageLabel = dirname(this.testPath).split(sep).join(".");
@@ -216,7 +215,7 @@ const createJestEnvironment = <T extends typeof JestEnvironment>(Base: T): T =>
             ...getEnvironmentLabels(),
           ],
         },
-        [scopeUuid],
+        this.runContext.scopes,
       );
 
       this.runtime.updateTest(testUuid, (result) => {

diff --git a/packages/allure-jest/test/spec/hooks.test.ts b/packages/allure-jest/test/spec/hooks.test.ts
@@ -111,6 +111,35 @@ it("reports before and after each hooks", async () => {
   );
 });
 
+it("should report beforeAll/afterAll for tests in sub-suites", async () => {
+  const { tests, groups } = await runJestInlineTest({
+    "sample.test.js": `
+      beforeAll(() => {});
+      afterAll(() => {});
+
+      describe("", () => {
+        it("foo", () => {});
+      });
+    `,
+  });
+
+  const [{ uuid: testUuid }] = tests;
+
+  expect(groups).toHaveLength(2);
+  expect(groups).toEqual(
+    expect.arrayContaining([
+      expect.objectContaining({
+        name: "beforeAll",
+        children: [testUuid],
+      }),
+      expect.objectContaining({
+        name: "afterAll",
+        children: [testUuid],
+      }),
+    ]),
+  );
+});
+
 it("reports failed hooks", async () => {
   const { tests, groups } = await runJestInlineTest({
     "sample.test.js": `