+12








fcd7c979df
Co-authored-by: Artur <145776561+arturict@users.noreply.github.com> Co-authored-by: arturict <arturict@users.noreply.github.com> Co-authored-by: Thomas <thomas@ajnart.fr> Co-authored-by: Afonso Jorge Ramos <afonsojorgeramos@gmail.com> Co-authored-by: Danura30082 <danura30082+github@gmail.com> Co-authored-by: Kushie <122986913+kushiemoon-dev@users.noreply.github.com> Co-authored-by: ajnart <thomas@ajnart.fr> Co-authored-by: ajnart <thomas@ajnart.dev> Co-authored-by: chenjun <chenjun@foxmail.com> Co-authored-by: Wikibear <wikibear@users.noreply.github.com> Co-authored-by: Bergasha <bergasha@gmail.com> Co-authored-by: leafyelin <43306196+leafyelin@users.noreply.github.com> Co-authored-by: Lucas Araujo <lucas.lra@gmail.com> Co-authored-by: GreenMonito <86085570+GreenMonito@users.noreply.github.com> Co-authored-by: Nerdy-Technician <roffo@nerdy-technician.social> Co-authored-by: Jun <34545896+jcstrong@users.noreply.github.com> Co-authored-by: Edmond Abraham <54119952+gitedmond@users.noreply.github.com> Co-authored-by: thrawnx <tight.bulb9384@fastmail.com> Co-authored-by: Chris <58918656+nordada@users.noreply.github.com> Co-authored-by: yi111 <153097222+Yi-111-a@users.noreply.github.com> Co-authored-by: Yanis <naamaneyanis@gmail.com> Co-authored-by: Wikibear <me@sej.de> Co-authored-by: alessiottavo <93441886+alessiottavo@users.noreply.github.com>
144 lines
5.2 KiB
TypeScript
144 lines
5.2 KiB
TypeScript
import { beforeEach, describe, expect, it, vi } from "vitest";
|
|
import { NextRequest, NextResponse } from "next/server";
|
|
|
|
import { localeCookieKey } from "@homarr/definitions";
|
|
|
|
const mocks = vi.hoisted(() => ({
|
|
getOnboardingStep: vi.fn(),
|
|
getDefaultLocale: vi.fn(),
|
|
createI18nMiddleware: vi.fn(),
|
|
}));
|
|
|
|
vi.mock("@homarr/db/proxy-reader", () => ({
|
|
getDefaultLocaleForProxyAsync: mocks.getDefaultLocale,
|
|
getOnboardingStepForProxyAsync: mocks.getOnboardingStep,
|
|
}));
|
|
|
|
vi.mock("@homarr/translation/middleware", () => ({
|
|
createI18nMiddleware: mocks.createI18nMiddleware,
|
|
}));
|
|
|
|
const loadProxy = async () => {
|
|
vi.resetModules();
|
|
return import("./proxy");
|
|
};
|
|
|
|
const createRequest = (pathname: string, locale?: string) => {
|
|
const request = new NextRequest(`http://localhost${pathname}`);
|
|
if (locale) request.cookies.set(localeCookieKey, locale);
|
|
return request;
|
|
};
|
|
|
|
describe("locale and onboarding proxy", () => {
|
|
beforeEach(() => {
|
|
vi.clearAllMocks();
|
|
mocks.getOnboardingStep.mockResolvedValue("finish");
|
|
mocks.getDefaultLocale.mockResolvedValue("en");
|
|
mocks.createI18nMiddleware.mockImplementation((defaultLocale: string) => (request: NextRequest) => {
|
|
request.headers.set("x-test-default-locale", defaultLocale);
|
|
return NextResponse.next();
|
|
});
|
|
});
|
|
|
|
it("redirects unfinished onboarding to /init", async () => {
|
|
mocks.getOnboardingStep.mockResolvedValue("integrations");
|
|
const { proxy } = await loadProxy();
|
|
|
|
const response = await proxy(createRequest("/en/dashboard"));
|
|
|
|
expect(response.status).toBe(307);
|
|
expect(response.headers.get("location")).toBe("http://localhost/init");
|
|
expect(mocks.getOnboardingStep).toHaveBeenCalledOnce();
|
|
});
|
|
|
|
it("allows /init while onboarding is unfinished", async () => {
|
|
mocks.getOnboardingStep.mockResolvedValue("start");
|
|
const { proxy } = await loadProxy();
|
|
|
|
const response = await proxy(createRequest("/en/init"));
|
|
|
|
expect(response.status).toBe(200);
|
|
expect(mocks.getOnboardingStep).not.toHaveBeenCalled();
|
|
expect(mocks.getDefaultLocale).toHaveBeenCalledOnce();
|
|
});
|
|
|
|
it("allows administrator sign-in while onboarding is unfinished", async () => {
|
|
mocks.getOnboardingStep.mockResolvedValue("setup");
|
|
const { proxy } = await loadProxy();
|
|
|
|
const response = await proxy(createRequest("/en/auth/login"));
|
|
|
|
expect(response.status).toBe(200);
|
|
expect(mocks.getOnboardingStep).not.toHaveBeenCalled();
|
|
});
|
|
|
|
it("does not treat nested routes ending in init or auth/login as onboarding access routes", async () => {
|
|
mocks.getOnboardingStep.mockResolvedValue("setup");
|
|
const { proxy } = await loadProxy();
|
|
|
|
const initResponse = await proxy(createRequest("/en/boards/init"));
|
|
const loginResponse = await proxy(createRequest("/en/settings/auth/login"));
|
|
|
|
expect(initResponse.headers.get("location")).toBe("http://localhost/init");
|
|
expect(loginResponse.headers.get("location")).toBe("http://localhost/init");
|
|
});
|
|
|
|
it("uses the configured default locale for a missing or invalid locale cookie", async () => {
|
|
mocks.getDefaultLocale.mockResolvedValue("de");
|
|
const { proxy } = await loadProxy();
|
|
|
|
await proxy(createRequest("/en/dashboard", "not-a-locale"));
|
|
|
|
expect(mocks.getDefaultLocale).toHaveBeenCalledOnce();
|
|
expect(mocks.createI18nMiddleware).toHaveBeenCalledWith("de");
|
|
});
|
|
|
|
it("keeps a valid locale cookie and does not query the configured default", async () => {
|
|
const { proxy } = await loadProxy();
|
|
|
|
const response = await proxy(createRequest("/fr/dashboard", "fr"));
|
|
|
|
expect(response.status).toBe(200);
|
|
expect(mocks.getDefaultLocale).not.toHaveBeenCalled();
|
|
expect(mocks.createI18nMiddleware).toHaveBeenCalledWith("en");
|
|
});
|
|
|
|
it("deduplicates concurrent cold-start database reads", async () => {
|
|
let finishOnboarding!: (step: string) => void;
|
|
let finishLocale!: (locale: string) => void;
|
|
mocks.getOnboardingStep.mockImplementation(
|
|
() =>
|
|
new Promise<string>((resolve) => {
|
|
finishOnboarding = resolve;
|
|
}),
|
|
);
|
|
mocks.getDefaultLocale.mockImplementation(
|
|
() =>
|
|
new Promise<string>((resolve) => {
|
|
finishLocale = resolve;
|
|
}),
|
|
);
|
|
const { proxy } = await loadProxy();
|
|
const responses = Array.from({ length: 50 }, () => proxy(createRequest("/en/dashboard", "not-a-locale")));
|
|
|
|
await vi.waitFor(() => expect(mocks.getOnboardingStep).toHaveBeenCalledOnce());
|
|
finishOnboarding("finish");
|
|
await vi.waitFor(() => expect(mocks.getDefaultLocale).toHaveBeenCalledOnce());
|
|
finishLocale("de");
|
|
|
|
await expect(Promise.all(responses)).resolves.toHaveLength(50);
|
|
expect(mocks.getOnboardingStep).toHaveBeenCalledOnce();
|
|
expect(mocks.getDefaultLocale).toHaveBeenCalledOnce();
|
|
});
|
|
|
|
it("clears failed in-flight reads so the next request can recover", async () => {
|
|
mocks.getOnboardingStep.mockRejectedValueOnce(new Error("temporary database failure")).mockResolvedValue("finish");
|
|
const { proxy } = await loadProxy();
|
|
|
|
await expect(proxy(createRequest("/en/dashboard", "en"))).rejects.toThrow("temporary database failure");
|
|
await expect(proxy(createRequest("/en/dashboard", "en"))).resolves.toMatchObject({ status: 200 });
|
|
|
|
expect(mocks.getOnboardingStep).toHaveBeenCalledTimes(2);
|
|
});
|
|
});
|