Skip to content

Commit 5d9a193

Browse files
authored
Merge pull request #21361 from davelopez/adapt_rate_limiter_tests_vitest
Migrate rate limiter tests from Jest to Vitest
2 parents 4c65e72 + ee48dc3 commit 5d9a193

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

client/src/api/client/rateLimiter.test.ts

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
2+
13
import type { MessageException } from "@/api";
24
import { GalaxyApi } from "@/api/client";
35
import { useServerMock } from "@/api/client/__mocks__";
@@ -7,11 +9,11 @@ import { DEFAULT_CONFIG } from "./rateLimiter";
79
const { server, http } = useServerMock();
810

911
/** Spy to count number of times a 429 response is returned */
10-
const mock429ResponseSpy = jest.fn();
12+
const mock429ResponseSpy = vi.fn();
1113

1214
describe("Rate Limiter Middleware", () => {
13-
let consoleWarnSpy: jest.SpyInstance;
14-
let consoleErrorSpy: jest.SpyInstance;
15+
let consoleWarnSpy: ReturnType<typeof vi.spyOn>;
16+
let consoleErrorSpy: ReturnType<typeof vi.spyOn>;
1517

1618
/** Helper to verify that there is a 429 response without retries */
1719
function ensure429AndNoRetries(response: Response, error: MessageException | undefined) {
@@ -29,8 +31,8 @@ describe("Rate Limiter Middleware", () => {
2931
}
3032

3133
beforeEach(() => {
32-
consoleWarnSpy = jest.spyOn(console, "warn").mockImplementation();
33-
consoleErrorSpy = jest.spyOn(console, "error").mockImplementation();
34+
consoleWarnSpy = vi.spyOn(console, "warn").mockImplementation(() => {});
35+
consoleErrorSpy = vi.spyOn(console, "error").mockImplementation(() => {});
3436
});
3537
afterEach(() => {
3638
consoleWarnSpy.mockRestore();

0 commit comments

Comments
 (0)