chore: remove calls to mongoMeasurement - DO NOT MERGE

CodSpeedHQ · Jan 17, 2024 · 382f368 · 382f368
1 parent 4662ad7
commit 382f368
Show file tree

Hide file tree

Showing 5 changed files with 29 additions and 30 deletions.
diff --git a/packages/benchmark.js-plugin/src/index.ts b/packages/benchmark.js-plugin/src/index.ts
@@ -195,22 +195,22 @@ async function runBenchmarks({
 
     if (isAsync) {
       await optimizeFunction(benchPayload);
-      await mongoMeasurement.start(uri);
+      // await mongoMeasurement.start(uri);
       await (async function __codspeed_root_frame__() {
         Measurement.startInstrumentation();
         await benchPayload();
         Measurement.stopInstrumentation(uri);
       })();
-      await mongoMeasurement.stop(uri);
+      // await mongoMeasurement.stop(uri);
     } else {
       optimizeFunctionSync(benchPayload);
-      await mongoMeasurement.start(uri);
+      // await mongoMeasurement.start(uri);
       (function __codspeed_root_frame__() {
         Measurement.startInstrumentation();
         benchPayload();
         Measurement.stopInstrumentation(uri);
       })();
-      await mongoMeasurement.stop(uri);
+      // await mongoMeasurement.stop(uri);
     }
 
     if (typeof bench.options.teardown === "function") {

diff --git a/packages/tinybench-plugin/src/index.ts b/packages/tinybench-plugin/src/index.ts
@@ -69,13 +69,13 @@ export function withCodSpeed(bench: Bench): Bench {
       // run instrumented benchmark
       await task.opts.beforeEach?.call(task);
 
-      await mongoMeasurement.start(uri);
+      // await mongoMeasurement.start(uri);
       await (async function __codspeed_root_frame__() {
         Measurement.startInstrumentation();
         await task.fn();
         Measurement.stopInstrumentation(uri);
       })();
-      await mongoMeasurement.stop(uri);
+      // await mongoMeasurement.stop(uri);
 
       await task.opts.afterEach?.call(task);
 

diff --git a/packages/tinybench-plugin/tests/index.integ.test.ts b/packages/tinybench-plugin/tests/index.integ.test.ts
@@ -29,8 +29,8 @@ describe("Benchmark.Suite", () => {
     await bench.run();
 
     expect(onComplete).toHaveBeenCalled();
-    expect(mockCore.mongoMeasurement.start).not.toHaveBeenCalled();
-    expect(mockCore.mongoMeasurement.stop).not.toHaveBeenCalled();
+    // expect(mockCore.mongoMeasurement.start).not.toHaveBeenCalled();
+    // expect(mockCore.mongoMeasurement.stop).not.toHaveBeenCalled();
     expect(mockCore.Measurement.startInstrumentation).not.toHaveBeenCalled();
     expect(mockCore.Measurement.stopInstrumentation).not.toHaveBeenCalled();
   });
@@ -42,10 +42,10 @@ describe("Benchmark.Suite", () => {
       })
       .run();
 
-    expect(mockCore.mongoMeasurement.start).toHaveBeenCalledWith(
-      "packages/tinybench-plugin/tests/index.integ.test.ts::RegExp"
-    );
-    expect(mockCore.mongoMeasurement.stop).toHaveBeenCalledTimes(1);
+    // expect(mockCore.mongoMeasurement.start).toHaveBeenCalledWith(
+    //   "packages/tinybench-plugin/tests/index.integ.test.ts::RegExp"
+    // );
+    // expect(mockCore.mongoMeasurement.stop).toHaveBeenCalledTimes(1);
     expect(mockCore.Measurement.startInstrumentation).toHaveBeenCalled();
     expect(mockCore.Measurement.stopInstrumentation).toHaveBeenCalledWith(
       "packages/tinybench-plugin/tests/index.integ.test.ts::RegExp"
@@ -62,13 +62,13 @@ describe("Benchmark.Suite", () => {
       })
       .run();
 
-    expect(mockCore.mongoMeasurement.start).toHaveBeenCalledWith(
-      "packages/tinybench-plugin/tests/index.integ.test.ts::RegExp"
-    );
-    expect(mockCore.mongoMeasurement.start).toHaveBeenCalledWith(
-      "packages/tinybench-plugin/tests/index.integ.test.ts::RegExp2"
-    );
-    expect(mockCore.mongoMeasurement.stop).toHaveBeenCalledTimes(2);
+    // expect(mockCore.mongoMeasurement.start).toHaveBeenCalledWith(
+    //   "packages/tinybench-plugin/tests/index.integ.test.ts::RegExp"
+    // );
+    // expect(mockCore.mongoMeasurement.start).toHaveBeenCalledWith(
+    //   "packages/tinybench-plugin/tests/index.integ.test.ts::RegExp2"
+    // );
+    // expect(mockCore.mongoMeasurement.stop).toHaveBeenCalledTimes(2);
     expect(mockCore.Measurement.stopInstrumentation).toHaveBeenCalledWith(
       "packages/tinybench-plugin/tests/index.integ.test.ts::RegExp"
     );

diff --git a/packages/vitest-plugin/src/__tests__/runner.test.ts b/packages/vitest-plugin/src/__tests__/runner.test.ts
@@ -48,13 +48,13 @@ describe("CodSpeedRunner", () => {
     );
 
     // run
-    expect(coreMocks.mongoMeasurement.start).toHaveBeenCalledWith(
-      "packages/vitest-plugin/src/__tests__/runner.test.ts::test bench"
-    );
+    // expect(coreMocks.mongoMeasurement.start).toHaveBeenCalledWith(
+    //   "packages/vitest-plugin/src/__tests__/runner.test.ts::test bench"
+    // );
     expect(coreMocks.Measurement.startInstrumentation).toHaveBeenCalledTimes(1);
     expect(benchFn).toHaveBeenCalledTimes(2);
     expect(coreMocks.Measurement.stopInstrumentation).toHaveBeenCalledTimes(1);
-    expect(coreMocks.mongoMeasurement.stop).toHaveBeenCalledTimes(1);
+    // expect(coreMocks.mongoMeasurement.stop).toHaveBeenCalledTimes(1);
     expect(console.log).toHaveBeenCalledWith(
       "[CodSpeed] packages/vitest-plugin/src/__tests__/runner.test.ts::test bench done"
     );
@@ -99,13 +99,13 @@ describe("CodSpeedRunner", () => {
     );
 
     // run
-    expect(coreMocks.mongoMeasurement.start).toHaveBeenCalledWith(
-      "packages/vitest-plugin/src/__tests__/runner.test.ts::nested suite::test bench"
-    );
+    // expect(coreMocks.mongoMeasurement.start).toHaveBeenCalledWith(
+    //   "packages/vitest-plugin/src/__tests__/runner.test.ts::nested suite::test bench"
+    // );
     expect(coreMocks.Measurement.startInstrumentation).toHaveBeenCalledTimes(1);
     expect(benchFn).toHaveBeenCalledTimes(2);
     expect(coreMocks.Measurement.stopInstrumentation).toHaveBeenCalledTimes(1);
-    expect(coreMocks.mongoMeasurement.stop).toHaveBeenCalledTimes(1);
+    // expect(coreMocks.mongoMeasurement.stop).toHaveBeenCalledTimes(1);
     expect(console.log).toHaveBeenCalledWith(
       "[CodSpeed] packages/vitest-plugin/src/__tests__/runner.test.ts::nested suite::test bench done"
     );

diff --git a/packages/vitest-plugin/src/runner.ts b/packages/vitest-plugin/src/runner.ts
@@ -2,7 +2,6 @@ import {
   getGitDir,
   logDebug,
   Measurement,
-  mongoMeasurement,
   optimizeFunction,
   setupCore,
   teardownCore,
@@ -49,14 +48,14 @@ async function runBenchmarkSuite(suite: Suite, parentSuiteName?: string) {
     const fn = getBenchFn(benchmark);
 
     await optimizeFunction(fn);
-    await mongoMeasurement.start(uri);
+    // await mongoMeasurement.start(uri);
     await (async function __codspeed_root_frame__() {
       Measurement.startInstrumentation();
       // @ts-expect-error we do not need to bind the function to an instance of tinybench's Bench
       await fn();
       Measurement.stopInstrumentation(uri);
     })();
-    await mongoMeasurement.stop(uri);
+    // await mongoMeasurement.stop(uri);
 
     logCodSpeed(`${uri} done`);
   }