fix: add tests checking error handling

Author: art049

.moon/project.yml

@@ -61,7 +61,7 @@ tasks:
       NODE_NO_WARNINGS: "1"
 
   test:
-    command: "jest --passWithNoTests"
+    command: "jest --passWithNoTests --silent"
     inputs:
       - "@globs(sources)"
       - "@globs(tests)"

packages/benchmark.js-plugin/tests/unit.test.ts

@@ -15,46 +15,72 @@ beforeEach(() => {
   jest.clearAllMocks();
 });
 
+const benchOptions: Benchmark.Options = {
+  maxTime: 0.01,
+};
+
 describe("Benchmark", () => {
   it("simple benchmark", () => {
     mockCore.isInstrumented.mockReturnValue(false);
     const bench = withCodSpeed(
-      new Benchmark("RegExp", function () {
-        /o/.test("Hello World!");
-      })
+      new Benchmark(
+        "RegExp",
+        function () {
+          /o/.test("Hello World!");
+        },
+        benchOptions
+      )
     );
     const onComplete = jest.fn();
     bench.on("complete", onComplete);
-    bench.run({
-      initCount: 0,
-      maxTime: -Infinity,
-    });
+    bench.run();
     expect(onComplete).toHaveBeenCalled();
     expect(mockCore.startInstrumentation).not.toHaveBeenCalled();
     expect(mockCore.stopInstrumentation).not.toHaveBeenCalled();
   });
   it("check core methods are called", () => {
     mockCore.isInstrumented.mockReturnValue(true);
     withCodSpeed(
-      new Benchmark("RegExpSingle", function () {
-        /o/.test("Hello World!");
-      })
+      new Benchmark(
+        "RegExpSingle",
+        function () {
+          /o/.test("Hello World!");
+        },
+        benchOptions
+      )
     ).run();
     expect(mockCore.startInstrumentation).toHaveBeenCalled();
     expect(mockCore.stopInstrumentation).toHaveBeenCalledWith(
       "packages/benchmark.js-plugin/tests/unit.test.ts::RegExpSingle"
     );
   });
+  it("check error handling", async () => {
+    mockCore.isInstrumented.mockReturnValue(true);
+    const bench = withCodSpeed(
+      new Benchmark(
+        "throwing",
+        () => {
+          throw new Error("test");
+        },
+        benchOptions
+      )
+    );
+    expect(() => bench.run()).toThrowError("test");
+  });
   it.each([true, false])(
     "check console output(instrumented=%p) ",
     async (instrumented) => {
       const logSpy = jest.spyOn(console, "log");
       const warnSpy = jest.spyOn(console, "warn");
       mockCore.isInstrumented.mockReturnValue(instrumented);
       withCodSpeed(
-        new Benchmark("RegExpSingle", function () {
-          /o/.test("Hello World!");
-        })
+        new Benchmark(
+          "RegExpSingle",
+          function () {
+            /o/.test("Hello World!");
+          },
+          benchOptions
+        )
       ).run();
       expect({
         log: logSpy.mock.calls,
@@ -68,9 +94,13 @@ describe("Benchmark.Suite", () => {
   it("simple suite", () => {
     mockCore.isInstrumented.mockReturnValue(false);
     const suite = withCodSpeed(new Benchmark.Suite());
-    suite.add("RegExp", function () {
-      /o/.test("Hello World!");
-    });
+    suite.add(
+      "RegExp",
+      function () {
+        /o/.test("Hello World!");
+      },
+      benchOptions
+    );
     const onComplete = jest.fn();
     suite.on("complete", onComplete);
     suite.run({ maxTime: 0.1, initCount: 1 });
@@ -81,9 +111,13 @@ describe("Benchmark.Suite", () => {
   it("check core methods are called", () => {
     mockCore.isInstrumented.mockReturnValue(true);
     withCodSpeed(new Benchmark.Suite())
-      .add("RegExp", function () {
-        /o/.test("Hello World!");
-      })
+      .add(
+        "RegExp",
+        function () {
+          /o/.test("Hello World!");
+        },
+        benchOptions
+      )
       .run();
     expect(mockCore.startInstrumentation).toHaveBeenCalled();
     expect(mockCore.stopInstrumentation).toHaveBeenCalledWith(
@@ -93,12 +127,16 @@ describe("Benchmark.Suite", () => {
   it("check suite name is in the uri", () => {
     mockCore.isInstrumented.mockReturnValue(true);
     withCodSpeed(new Benchmark.Suite("thesuite"))
-      .add("RegExp", function () {
-        /o/.test("Hello World!");
-      })
+      .add(
+        "RegExp",
+        function () {
+          /o/.test("Hello World!");
+        },
+        benchOptions
+      )
       .add(() => {
         /o/.test("Hello World!");
-      })
+      }, benchOptions)
       .run();
     expect(mockCore.stopInstrumentation).toHaveBeenCalledWith(
       "packages/benchmark.js-plugin/tests/unit.test.ts::thesuite::RegExp"
@@ -107,19 +145,33 @@ describe("Benchmark.Suite", () => {
       "packages/benchmark.js-plugin/tests/unit.test.ts::thesuite::unknown_1"
     );
   });
+  it("check error handling", async () => {
+    mockCore.isInstrumented.mockReturnValue(true);
+    const bench = withCodSpeed(new Benchmark.Suite("thesuite")).add(
+      "throwing",
+      () => {
+        throw new Error("test");
+      }
+    );
+    expect(() => bench.run()).toThrowError("test");
+  });
   it.each([true, false])(
     "check console output(instrumented=%p) ",
     async (instrumented) => {
       const logSpy = jest.spyOn(console, "log");
       const warnSpy = jest.spyOn(console, "warn");
       mockCore.isInstrumented.mockReturnValue(instrumented);
       withCodSpeed(new Benchmark.Suite("thesuite"))
-        .add("RegExp", function () {
-          /o/.test("Hello World!");
-        })
+        .add(
+          "RegExp",
+          function () {
+            /o/.test("Hello World!");
+          },
+          benchOptions
+        )
         .add(() => {
           /o/.test("Hello World!");
-        })
+        }, benchOptions)
         .run();
       expect({
         log: logSpy.mock.calls,

packages/tinybench-plugin/tests/unit.test.ts

@@ -59,6 +59,14 @@ describe("Benchmark.Suite", () => {
       "packages/tinybench-plugin/tests/unit.test.ts::RegExp2"
     );
   });
+  it("check error handling", async () => {
+    mockCore.isInstrumented.mockReturnValue(true);
+    const bench = withCodSpeed(new Bench());
+    bench.add("throwing", async () => {
+      throw new Error("test");
+    });
+    await expect(bench.run()).rejects.toThrowError("test");
+  });
   it.each([true, false])(
     "check console output(instrumented=%p) ",
     async (instrumented) => {