fix: status and statusDetails logic

e2e/src/programmatic/grouping/client/auth/LoginScreen.test.ts

@@ -7,7 +7,7 @@
  */
 
 import LoginHelper from '../../../../utils/LoginHelper';
-import {allure} from "../../../../../../src/api";
+import {$Tag, $Epic, $Feature, $Story, allure} from 'jest-allure2-reporter/api';
 
 $Tag('client');
 $Epic('Authentication');
@@ -38,6 +38,7 @@ describe('Login screen', () => {
     });
 
     it('should show error on short or invalid password format', () => {
+      allure.status('failed', { message: 'The password is too short' });
       // ...
     });
   });

e2e/src/programmatic/grouping/server/controllers/__snapshots__/forgotPassword.test.ts.snap

@@ -0,0 +1,8 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`POST /forgot-password should return 401 if user is not found 1`] = `
+{
+  "code": 401,
+  "seed": 0.06426173461501827,
+}
+`;

e2e/src/programmatic/grouping/server/controllers/forgotPassword.test.ts

@@ -1,10 +1,12 @@
+import { $Tag, $Epic, $Feature, $Story } from 'jest-allure2-reporter/api';
+
 $Tag('server');
 $Epic('Authentication');
 $Feature('Restore account');
 describe('POST /forgot-password', () => {
   $Story('Validation');
   it('should return 401 if user is not found', () => {
-    // ...
+    expect({ code: 401, seed: Math.random() }).toMatchSnapshot();
   });
 
   $Story('Happy path');

package.json

@@ -116,7 +116,7 @@
   "dependencies": {
     "@noomorph/allure-js-commons": "^2.3.0",
     "ci-info": "^3.8.0",
-    "jest-metadata": "^1.3.1",
+    "jest-metadata": "^1.4.1",
     "lodash.snakecase": "^4.1.1",
     "node-fetch": "^2.6.7",
     "pkg-up": "^3.1.0",

src/environment/listener.ts

@@ -1,9 +1,4 @@
 import type { Circus } from '@jest/types';
-import type {
-  AllureTestCaseMetadata,
-  AllureTestStepMetadata,
-  Status,
-} from 'jest-allure2-reporter';
 import type {
   EnvironmentListenerFn,
   TestEnvironmentCircusEvent,
@@ -33,7 +28,7 @@ const listener: EnvironmentListenerFn = (context) => {
     .on('add_hook', addSourceLocation)
     .on('add_hook', addHookType)
     .on('add_test', addSourceLocation)
-    .on('test_start', executableStart)
+    .on('test_start', testStart)
     .on('test_todo', testSkip)
     .on('test_skip', testSkip)
     .on('test_done', testDone)
@@ -111,80 +106,85 @@ function addSourceCode({ event }: TestEnvironmentCircusEvent) {
 }
 
 // eslint-disable-next-line no-empty-pattern
-function executableStart({}: TestEnvironmentCircusEvent) {
-  const metadata: AllureTestStepMetadata = {
-    start: Date.now(),
+function executableStart(
+  _event: TestEnvironmentCircusEvent<
+    Circus.Event & { name: 'hook_start' | 'test_fn_start' }
+  >,
+) {
+  realm.runtimeContext.getCurrentMetadata().assign({
     stage: 'running',
-  };
-
-  realm.runtimeContext.getCurrentMetadata().assign(metadata);
+    start: Date.now(),
+  });
 }
 
 function executableFailure({
-  event,
+  event: { error },
 }: TestEnvironmentCircusEvent<
   Circus.Event & { name: 'test_fn_failure' | 'hook_failure' }
 >) {
-  const metadata: AllureTestStepMetadata = {
-    stop: Date.now(),
+  realm.runtimeContext.getCurrentMetadata().assign({
+    status: error && isMatcherError(error) ? 'failed' : 'broken',
+    statusDetails: error?.message
+      ? {
+          message: error.message,
+          trace: error.stack,
+        }
+      : undefined,
     stage: 'interrupted',
-    status: 'failed',
-  };
-
-  if (event.error) {
-    const message = event.error.message ?? `${event.error}`;
-    const trace = event.error.stack;
-
-    metadata.statusDetails = { message, trace };
-  }
-
-  realm.runtimeContext.getCurrentMetadata().assign(metadata);
-}
-
-// eslint-disable-next-line no-empty-pattern
-function executableSuccess({}: TestEnvironmentCircusEvent) {
-  const metadata: AllureTestStepMetadata = {
     stop: Date.now(),
-    stage: 'finished',
-    status: 'passed',
-  };
+  });
+}
 
-  realm.runtimeContext.getCurrentMetadata().assign(metadata);
+function executableSuccess(
+  _event: TestEnvironmentCircusEvent<
+    Circus.Event & { name: 'test_fn_success' | 'hook_success' }
+  >,
+) {
+  realm.runtimeContext
+    .getCurrentMetadata()
+    .defaults({
+      status: 'passed',
+    })
+    .assign({
+      stage: 'finished',
+      stop: Date.now(),
+    });
 }
 
-function testSkip() {
-  const metadata: AllureTestCaseMetadata = {
-    stop: Date.now(),
-    stage: 'pending',
-    status: 'skipped',
-  };
+function testStart(
+  _event: TestEnvironmentCircusEvent<Circus.Event & { name: 'test_start' }>,
+) {
+  realm.runtimeContext.getCurrentMetadata().set('start', Date.now());
+}
 
-  realm.runtimeContext.getCurrentMetadata().assign(metadata);
+function testSkip(
+  _event: TestEnvironmentCircusEvent<
+    Circus.Event & { name: 'test_skip' | 'test_todo' }
+  >,
+) {
+  realm.runtimeContext.getCurrentMetadata().set('stop', Date.now());
 }
 
 function testDone({
   event,
 }: TestEnvironmentCircusEvent<Circus.Event & { name: 'test_done' }>) {
+  const current = realm.runtimeContext.getCurrentMetadata();
   const hasErrors = event.test.errors.length > 0;
-  const errorStatus: Status = event.test.errors.some((errors) => {
-    return Array.isArray(errors)
-      ? errors.some(isMatcherError)
-      : isMatcherError(errors);
-  })
-    ? 'failed'
-    : 'broken';
-
-  const metadata: AllureTestCaseMetadata = {
-    stop: Date.now(),
-    stage: hasErrors ? 'interrupted' : 'finished',
-    status: hasErrors ? errorStatus : 'passed',
-  };
+  if (hasErrors) {
+    const isFailed = event.test.errors.some((errors) => {
+      return Array.isArray(errors)
+        ? errors.some(isMatcherError)
+        : isMatcherError(errors);
+    });
+
+    current.defaults({ status: isFailed ? 'failed' : 'broken' });
+  }
 
-  realm.runtimeContext.getCurrentMetadata().assign(metadata);
+  current.set('stop', Date.now());
 }
 
 function isMatcherError(error: any) {
-  return Boolean(error?.matcherResult);
+  return error && 'matcherResult' in error;
 }
 
 export default listener;

src/metadata/MetadataSquasher.ts

@@ -6,7 +6,7 @@ import type {
   AllureTestStepMetadata,
 } from 'jest-allure2-reporter';
 
-import { getStart, getStop } from './utils';
+import { getStage, getStart, getStatus, getStop } from './utils';
 import { PREFIX } from './constants';
 import {
   mergeTestCaseMetadata,
@@ -45,7 +45,8 @@ export class MetadataSquasher {
 
     return {
       ...result,
-
+      ...getStatus(metadata),
+      stage: getStage(metadata),
       start: getStart(metadata),
       stop: getStop(metadata),
       steps: [...befores, ...steps, ...afters],

src/metadata/constants.ts

@@ -1,8 +1,10 @@
 export const PREFIX = 'allure2' as const;
 
 export const START = [PREFIX, 'start'] as const;
-
 export const STOP = [PREFIX, 'stop'] as const;
+export const STAGE = [PREFIX, 'stage'] as const;
+export const STATUS = [PREFIX, 'status'] as const;
+export const STATUS_DETAILS = [PREFIX, 'statusDetails'] as const;
 
 export const CURRENT_STEP = [PREFIX, 'currentStep'] as const;
 export const DESCRIPTION = [PREFIX, 'description'] as const;

src/metadata/proxies/AllureMetadataProxy.ts

@@ -35,6 +35,11 @@ export class AllureMetadataProxy<T = unknown> {
     return this;
   }
 
+  defaults(values: Partial<T>): this {
+    this.$metadata.defaults(this.$localPath(), values);
+    return this;
+  }
+
   protected $localPath(key?: keyof T, ...innerKeys: string[]): string[] {
     const allKeys = key ? [key as string, ...innerKeys] : innerKeys;
     return [PREFIX, ...allKeys];

src/metadata/proxies/AllureTestItemMetadataProxy.ts

@@ -39,7 +39,7 @@ export class AllureTestItemMetadataProxy<
   }
 
   $stopStep(): this {
-    const currentStep = this.$metadata.get(CURRENT_STEP, []) as string[];
+    const currentStep = this.$metadata.get(CURRENT_STEP, [] as string[]);
     this.$metadata.set(CURRENT_STEP, currentStep.slice(0, -2));
     return this;
   }

src/metadata/utils/getStage.ts

@@ -0,0 +1,14 @@
+import type { TestInvocationMetadata } from 'jest-metadata';
+import type { Stage } from 'jest-allure2-reporter';
+
+import { STAGE } from '../constants';
+
+export const getStage = (testInvocation: TestInvocationMetadata) => {
+  for (const invocation of testInvocation.allInvocations()) {
+    if (invocation.get(STAGE) === 'interrupted') {
+      return 'interrupted';
+    }
+  }
+
+  return testInvocation.get(STAGE) as Stage | undefined;
+};

src/metadata/utils/getStatus.ts

@@ -0,0 +1,23 @@
+import type { TestInvocationMetadata } from 'jest-metadata';
+import type { Status, StatusDetails } from 'jest-allure2-reporter';
+
+import { STATUS, STATUS_DETAILS } from '../constants';
+
+export const getStatus = (testInvocation: TestInvocationMetadata) => {
+  let status: Status | undefined;
+  let statusDetails: StatusDetails | undefined;
+
+  const items = [...testInvocation.allInvocations(), testInvocation].reverse();
+  for (const item of items) {
+    status ??= item.get(STATUS) as Status;
+    statusDetails ??= item.get(STATUS_DETAILS) as StatusDetails;
+    if (status && statusDetails) {
+      break;
+    }
+  }
+
+  return {
+    status,
+    statusDetails,
+  };
+};

src/metadata/utils/index.ts

@@ -1,2 +1,4 @@
+export * from './getStage';
 export * from './getStart';
+export * from './getStatus';
 export * from './getStop';

src/options/default-options/testCase.ts

@@ -64,7 +64,8 @@ export const testCase: ResolvedTestCaseCustomizer = {
     testCaseMetadata.start ??
     (testCaseMetadata.stop ?? Date.now()) - (testCase.duration ?? 0),
   stop: ({ testCaseMetadata }) => testCaseMetadata.stop ?? Date.now(),
-  stage: ({ testCase }) => getTestCaseStage(testCase),
+  stage: ({ testCase, testCaseMetadata }) =>
+    testCaseMetadata.stage ?? getTestCaseStage(testCase),
   status: ({ testCase, testCaseMetadata }) =>
     testCaseMetadata.status ?? getTestCaseStatus(testCase),
   statusDetails: ({ testCase, testCaseMetadata }) =>

src/realms/AllureRealm.ts

@@ -16,7 +16,9 @@ export class AllureRealm {
         'config',
       );
       if (!config) {
-        throw new Error('Shared reporter config is not defined');
+        throw new Error(
+          "Cannot receive jest-allure2-reporter's config from the parent process. Have you set up Jest test environment correctly?",
+        );
       }
 
       return config as SharedReporterConfig;