chore: use sync-request

CodSpeedHQ · Jan 17, 2024 · cdb8d17 · cdb8d17
1 parent af8f5b9
commit cdb8d17
Show file tree

Hide file tree

Showing 7 changed files with 175 additions and 50 deletions.
diff --git a/packages/benchmark.js-plugin/package.json b/packages/benchmark.js-plugin/package.json
@@ -29,7 +29,8 @@
   "dependencies": {
     "@codspeed/core": "workspace:^2.3.1",
     "lodash": "^4.17.10",
-    "stack-trace": "1.0.0-pre2"
+    "stack-trace": "1.0.0-pre2",
+    "sync-request": "^6.1.0"
   },
   "peerDependencies": {
     "benchmark": "^2.1.0"

diff --git a/packages/benchmark.js-plugin/src/index.ts b/packages/benchmark.js-plugin/src/index.ts
@@ -7,6 +7,7 @@ import {
   tryIntrospect,
 } from "@codspeed/core";
 import Benchmark from "benchmark";
+import request from "sync-request";
 import buildSuiteAdd from "./buildSuiteAdd";
 import getCallingFile from "./getCallingFile";
 import { CodSpeedBenchmark } from "./types";
@@ -15,10 +16,12 @@ declare const __VERSION__: string;
 
 tryIntrospect();
 
-async function doSomeWork() {
-  for (let i = 0; i < 1000; i++) {
-    Math.random();
-  }
+function doSomeWorkSync() {
+  const result = request("GET", "https://google.com");
+
+  console.log(`request ended with status code ${result.statusCode}`);
+
+  return result;
 }
 interface WithCodSpeedBenchmark
   extends Omit<
@@ -195,7 +198,7 @@ async function runBenchmarks({
       await bench.options.setup();
     }
 
-    await doSomeWork();
+    doSomeWorkSync();
 
     if (isAsync) {
       await optimizeFunction(benchPayload);
@@ -213,7 +216,7 @@ async function runBenchmarks({
       })();
     }
 
-    await doSomeWork();
+    doSomeWorkSync();
 
     if (typeof bench.options.teardown === "function") {
       await bench.options.teardown();

diff --git a/packages/tinybench-plugin/package.json b/packages/tinybench-plugin/package.json
@@ -25,7 +25,8 @@
   },
   "dependencies": {
     "@codspeed/core": "workspace:^2.3.1",
-    "stack-trace": "1.0.0-pre2"
+    "stack-trace": "1.0.0-pre2",
+    "sync-request": "^6.1.0"
   },
   "peerDependencies": {
     "tinybench": "^2.3.0"

diff --git a/packages/tinybench-plugin/src/index.ts b/packages/tinybench-plugin/src/index.ts
@@ -8,17 +8,20 @@ import {
 } from "@codspeed/core";
 import path from "path";
 import { get as getStackTrace } from "stack-trace";
+import request from "sync-request";
 import { Bench, Task } from "tinybench";
 import { fileURLToPath } from "url";
 
 declare const __VERSION__: string;
 
 tryIntrospect();
 
-async function doSomeWork() {
-  for (let i = 0; i < 1000; i++) {
-    Math.random();
-  }
+function doSomeWorkSync() {
+  const result = request("GET", "https://google.com");
+
+  console.log(`request ended with status code ${result.statusCode}`);
+
+  return result;
 }
 
 type CodSpeedBenchOptions = Task["opts"] & {
@@ -62,7 +65,7 @@ export function withCodSpeed(bench: Bench): Bench {
 
       await task.opts.beforeAll?.call(task);
 
-      await doSomeWork();
+      doSomeWorkSync();
 
       // run optimizations
       await optimizeFunction(async () => {
@@ -71,7 +74,7 @@ export function withCodSpeed(bench: Bench): Bench {
         await task.opts.afterEach?.call(task);
       });
 
-      await doSomeWork();
+      doSomeWorkSync();
 
       // run instrumented benchmark
       await task.opts.beforeEach?.call(task);

diff --git a/packages/vitest-plugin/package.json b/packages/vitest-plugin/package.json
@@ -27,7 +27,8 @@
     "bench": "vitest bench"
   },
   "dependencies": {
-    "@codspeed/core": "workspace:^2.3.1"
+    "@codspeed/core": "workspace:^2.3.1",
+    "sync-request": "^6.1.0"
   },
   "peerDependencies": {
     "vite": "^4.2.0 || ^5.0.0",

diff --git a/packages/vitest-plugin/src/runner.ts b/packages/vitest-plugin/src/runner.ts
@@ -7,15 +7,19 @@ import {
   teardownCore,
 } from "@codspeed/core";
 import path from "path";
+import request from "sync-request";
 import { Benchmark, Suite } from "vitest";
 import { NodeBenchmarkRunner } from "vitest/runners";
 import { getBenchFn } from "vitest/suite";
 
-async function doSomeWork() {
-  for (let i = 0; i < 1000; i++) {
-    Math.random();
-  }
+function doSomeWorkSync() {
+  const result = request("GET", "https://google.com");
+
+  console.log(`request ended with status code ${result.statusCode}`);
+
+  return result;
 }
+
 const currentFileName =
   typeof __filename === "string"
     ? __filename
@@ -53,14 +57,15 @@ async function runBenchmarkSuite(suite: Suite, parentSuiteName?: string) {
     const fn = getBenchFn(benchmark);
 
     await optimizeFunction(fn);
-    await doSomeWork();
+    console.log(`[CodSpeed] ${uri} running`);
+    doSomeWorkSync();
     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 doSomeWork();
+    doSomeWorkSync();
 
     logCodSpeed(`${uri} done`);
   }