Change skipped tests handling logic

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

packages/allure-hermione/test/fixtures/only.js

@@ -0,0 +1,5 @@
+it("first", () => {});
+
+it("second", () => {});
+
+it.only("third", () => {});

packages/allure-hermione/test/spec/only.test.ts

@@ -0,0 +1,18 @@
+import { expect } from "chai";
+import Hermione from "hermione";
+import { describe, it } from "mocha";
+import { getTestResultByName } from "../runner";
+import { HermioneAllure } from "../types";
+
+describe("only", () => {
+  it("reports only one spec", async () => {
+    const hermione = new Hermione("./test/.hermione.conf.js") as HermioneAllure;
+
+    await hermione.run(["./test/fixtures/only.js"], {});
+
+    const results = hermione.allure.writer.results;
+
+    expect(results).length(1);
+    expect(results[0].name).eq("third");
+  });
+});