unfinished tests now use jests computed failure status

Author: formaceft-93

packages/allure-jest/src/environmentFactory.ts

@@ -236,16 +236,16 @@ const createJestEnvironment = <T extends typeof JestEnvironment>(Base: T): T =>
       const testUuid = this.runContext.executables.pop();
 
       if (testUuid) {
-        const { details } = this.#statusAndDetails(test.errors);
+        const { status, details } = this.#statusAndDetails(test.errors);
         let tr: TestResult | undefined;
         this.runtime.updateTest(testUuid, (result) => {
           tr = result;
         });
-        // hook failure, finish as skipped
+        // hook failure, finish with the status reported by Jest
         if (tr?.status === undefined && tr?.stage === Stage.RUNNING) {
           this.runtime.updateTest(testUuid, (result) => {
             result.stage = Stage.FINISHED;
-            result.status = Status.SKIPPED;
+            result.status = status;
             result.statusDetails = {
               ...result.statusDetails,
               ...details,

packages/allure-jest/test/spec/hooks.test.ts

@@ -194,7 +194,7 @@ it("should report failed beforeAll hooks", async () => {
     expect.arrayContaining([
       expect.objectContaining({
         name: "test 1",
-        status: Status.SKIPPED,
+        status: Status.BROKEN,
         stage: Stage.FINISHED,
         statusDetails: expect.objectContaining({
           message: "foo",
@@ -204,7 +204,7 @@ it("should report failed beforeAll hooks", async () => {
 
       expect.objectContaining({
         name: "test 2",
-        status: Status.SKIPPED,
+        status: Status.BROKEN,
         stage: Stage.FINISHED,
         statusDetails: expect.objectContaining({
           message: "foo",
@@ -252,7 +252,7 @@ it("should report failed beforeEach hooks", async () => {
   expect(testResult).toEqual(
     expect.objectContaining({
       name: "sample test",
-      status: Status.SKIPPED,
+      status: Status.BROKEN,
       stage: Stage.FINISHED,
       statusDetails: expect.objectContaining({
         message: "foo",
@@ -283,6 +283,50 @@ it("should report failed beforeEach hooks", async () => {
   );
 });
 
+it("should report tests as failed on failed assertions in before hooks", async () => {
+  const { tests } = await runJestInlineTest({
+    "sample.test.js": `
+      describe("beforeAll assertion", () => {
+        beforeAll(() => {
+          expect(1).toStrictEqual(2);
+        });
+
+        it("test 1", () => {});
+        it("test 2", () => {});
+      });
+
+      describe("beforeEach assertion", () => {
+        beforeEach(() => {
+          expect(1).toStrictEqual(2);
+        });
+
+        it("test 3", () => {});
+      });
+    `,
+  });
+
+  expect(tests).toHaveLength(3);
+  expect(tests).toEqual(
+    expect.arrayContaining([
+      expect.objectContaining({
+        name: "test 1",
+        status: Status.FAILED,
+        stage: Stage.FINISHED,
+      }),
+      expect.objectContaining({
+        name: "test 2",
+        status: Status.FAILED,
+        stage: Stage.FINISHED,
+      }),
+      expect.objectContaining({
+        name: "test 3",
+        status: Status.FAILED,
+        stage: Stage.FINISHED,
+      }),
+    ]),
+  );
+});
+
 it("should report failed afterEach hooks", async () => {
   const { tests, groups } = await runJestInlineTest({
     "sample.test.js": `