Skip to content

Commit 5a227eb

Browse files
authored
Fix benchmarks (#1897)
1 parent 08e507d commit 5a227eb

File tree

6 files changed

+1638
-43
lines changed

6 files changed

+1638
-43
lines changed

packages/openapi-fetch/biome.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
"extends": ["../../biome.json"],
44
"files": {
55
"include": ["./src/", "./test/"],
6-
"ignore": ["**/test/**/schemas/**"]
6+
"ignore": ["./test/**/schemas/**", "./test/bench/*.min.js"]
77
},
88
"linter": {
99
"rules": {

packages/openapi-fetch/package.json

+1-2
Original file line numberDiff line numberDiff line change
@@ -67,8 +67,7 @@
6767
"version": "pnpm run prepare && pnpm run build"
6868
},
6969
"dependencies": {
70-
"openapi-typescript-helpers": "workspace:^",
71-
"vitest-fetch-mock": "^0.3.0"
70+
"openapi-typescript-helpers": "workspace:^"
7271
},
7372
"devDependencies": {
7473
"axios": "^1.7.7",

packages/openapi-fetch/test/index.bench.js renamed to packages/openapi-fetch/test/bench/index.bench.js

+16-14
Original file line numberDiff line numberDiff line change
@@ -2,25 +2,27 @@ import axios from "axios";
22
import { Fetcher } from "openapi-typescript-fetch";
33
import { createApiFetchClient } from "feature-fetch";
44
import superagent from "superagent";
5-
import { afterAll, beforeAll, bench, describe, vi } from "vitest";
6-
import createFetchMock from "vitest-fetch-mock";
7-
import createClient, { createPathBasedClient } from "../dist/index.js";
8-
import * as openapiTSCodegen from "./fixtures/openapi-typescript-codegen.min.js";
5+
import { afterAll, bench, describe, vi } from "vitest";
6+
import createClient, { createPathBasedClient } from "../../dist/index.js";
7+
import * as openapiTSCodegen from "./openapi-typescript-codegen.min.js";
98

109
const BASE_URL = "https://api.test.local";
1110

12-
const fetchMocker = createFetchMock(vi);
11+
const fetchMock = vi.fn(
12+
() =>
13+
new Promise((resolve) => {
14+
process.nextTick(() => {
15+
resolve(Response.json({}, { status: 200 }));
16+
});
17+
}),
18+
);
19+
vi.stubGlobal("fetch", fetchMock);
20+
21+
afterAll(() => {
22+
vi.unstubAllGlobals();
23+
});
1324

1425
describe("setup", () => {
15-
beforeAll(() => {
16-
// mock global fetch in this benchmark, without any delaly, shared state or resources
17-
fetchMocker.enableMocks();
18-
});
19-
20-
afterAll(() => {
21-
fetchMocker.disableMocks();
22-
});
23-
2426
bench("openapi-fetch", async () => {
2527
createClient({ baseUrl: BASE_URL });
2628
});

0 commit comments

Comments
 (0)