change skipped tests handling logic

allure-framework · Aug 22, 2023 · 6419ab2 · 6419ab2
1 parent a74ead7
commit 6419ab2
Showing 1 changed file with 9 additions and 19 deletions.
diff --git a/packages/allure-hermione/src/index.ts b/packages/allure-hermione/src/index.ts
@@ -50,7 +50,6 @@ const hostname = os.hostname();
 
 const hermioneAllureReporter = (hermione: Hermione, opts: AllureReportOptions) => {
   const { ALLURE_REPORTER_DEV_MODE } = process.env;
-  const loadedTests: Map<string, Hermione.Test> = new Map();
   const runningTests: Map<string, AllureTest> = new Map();
   const runtime = new AllureRuntime({
     resultsDir: "allure-results",
@@ -239,11 +238,17 @@ const hermioneAllureReporter = (hermione: Hermione, opts: AllureReportOptions) =
     });
   });
   hermione.on(hermione.events.AFTER_TESTS_READ, (collection) => {
-    // cache all the tests to handle skipped tests in future
+    // handle all skipped (pending) tests
     collection.eachTest((test) => {
-      const testId = getTestId(test);
+      if (!test.pending) {
+        return;
+      }
+
+      const currentTest = createAllureTest(test);
 
-      loadedTests.set(testId(), test);
+      currentTest.status = Status.SKIPPED;
+      currentTest.stage = Stage.FINISHED;
+      currentTest.endTest();
     });
   });
   hermione.on(hermione.events.TEST_BEGIN, (test) => {
@@ -290,23 +295,8 @@ const hermioneAllureReporter = (hermione: Hermione, opts: AllureReportOptions) =
     currentTest.calculateHistoryId();
     currentTest.stage = Stage.FINISHED;
     currentTest.endTest(Date.now());
-    loadedTests.delete(testId());
     runningTests.delete(testId());
   });
-  hermione.on(hermione.events.END, () => {
-    // all tests have been finished
-    if (loadedTests.size === 0) {
-      return;
-    }
-
-    loadedTests.forEach((test) => {
-      const currentTest = createAllureTest(test);
-
-      currentTest.status = Status.SKIPPED;
-      currentTest.stage = Stage.FINISHED;
-      currentTest.endTest();
-    });
-  });
 
   // it needs for tests because we need to read runtime writer data redefined in hermione config
   // eslint-disable-next-line