@griddo/ax 12.6.1 → 12.6.2

package/src/__tests__/modules/Settings/Social/Social.uc.test.tsx

@@ -1,171 +0,0 @@
-import { Router } from "react-router-dom";
-
-import { appReducer, type IAppState } from "@ax/containers/App/reducer";
-import { socialReducer } from "@ax/containers/Settings";
-import { type ISitesState, sitesInitialState, sitesReducer } from "@ax/containers/Sites/reducer";
-import { parseTheme } from "@ax/helpers";
-import Social from "@ax/modules/Settings/Social";
-import history from "@ax/routes/history";
-import globalTheme from "@ax/themes/theme.json";
-import type { INavItem, ISite, ISocialState } from "@ax/types";
-
-import { configureStore, type Reducer, type UnknownAction } from "@reduxjs/toolkit";
-import axios from "axios";
-import { ThemeProvider } from "styled-components";
-import type { MockedFunction } from "vitest";
-
-import { act, cleanup, fireEvent, render, screen } from "../../../../../config/tests/test-utils";
-
-/**
- * End-to-end test of the Social screen on the real store — view, thunk, api layer and
- * slice, with only axios mocked (see the `## Profile` section of docs/gainup/README.md
- * for why a `redux-mock-store` test does not count as AC coverage).
- *
- * The sibling Social.test.tsx keeps covering the render rules — which networks show up,
- * the dedup between the instance set and the hardcoded one, the button labels — on a
- * mock store. What it cannot see, and what this file is for, is the round trip: that
- * mounting fetches, that saving PUTs the whole map, and that the form goes clean off
- * the server's answer rather than off the click.
- *
- * The mock echoes a PUT's body back as the response, which is what the API does: the
- * screen rehydrates from that answer, so a mock returning something else would test a
- * server that does not exist.
- */
-vi.mock("axios");
-const mockedAxios = axios as MockedFunction<typeof axios>;
-
-const SITE_ID = 42;
-const SOCIALS: ISocialState = {
-	instagram: "https://instagram.com/griddo",
-	twitter: "https://x.com/griddo",
-};
-const EDITED = "https://instagram.com/griddo-cms";
-
-const mockComponent = () => null;
-const NAV_ITEMS: INavItem[] = [
-	{ title: "Social", path: "/settings/social", component: mockComponent },
-	{ title: "Other", path: "/settings/other", component: mockComponent },
-];
-
-const makeStore = () =>
-	configureStore({
-		// Only what this screen reads. `app` and `sites` are still legacy, so they need
-		// the cast GlobalStore uses (docs/RTK-MIGRATION.md §8.3).
-		reducer: {
-			app: appReducer as Reducer<IAppState, UnknownAction>,
-			social: socialReducer,
-			sites: sitesReducer as Reducer<ISitesState, UnknownAction>,
-		},
-		middleware: (getDefault) => getDefault({ serializableCheck: false, immutableCheck: false }),
-		preloadedState: {
-			sites: { ...sitesInitialState, currentSiteInfo: { id: SITE_ID } as ISite },
-		},
-	});
-
-const socialsCalls = (method: string) =>
-	mockedAxios.mock.calls
-		.map(([config]) => config as { url?: string; method?: string; data?: ISocialState })
-		.filter((config) => config?.method === method && config?.url?.endsWith(`/site/${SITE_ID}/socials`));
-
-const renderScreen = async (store: ReturnType<typeof makeStore>) => {
-	await act(async () => {
-		render(
-			<Router history={history}>
-				<ThemeProvider theme={parseTheme(globalTheme)}>
-					<Social navItems={NAV_ITEMS} currentNavItem={NAV_ITEMS[0]} />
-				</ThemeProvider>
-			</Router>,
-			{ store },
-		);
-	});
-};
-
-const button = (name: string) => screen.getByRole("button", { name });
-
-// TextField debounces its onChange by 300ms, so an edit does not reach `socialForm`
-// until that timer fires. Driving the clock instead of sleeping keeps it deterministic —
-// and without it the test asserts on a form that never changed.
-const DEBOUNCE_MS = 300;
-const typeInto = async (currentValue: string, newValue: string) => {
-	fireEvent.change(screen.getByDisplayValue(currentValue), { target: { value: newValue } });
-	await act(async () => {
-		vi.advanceTimersByTime(DEBOUNCE_MS);
-	});
-};
-
-beforeEach(() => {
-	vi.useFakeTimers();
-	mockedAxios.mockImplementation((async (config: { method?: string; data?: ISocialState }) => ({
-		status: 200,
-		data: config?.method === "PUT" ? config.data : SOCIALS,
-		statusText: "OK",
-		headers: {},
-		config: {},
-	})) as never);
-});
-
-afterEach(() => {
-	vi.useRealTimers();
-	vi.restoreAllMocks();
-	cleanup();
-});
-
-describe("[UC-AX-configure-social-networks] the Social screen end to end", () => {
-	it("AC2 · mounting asks the current site for its networks and prefills each one", async () => {
-		const store = makeStore();
-
-		await renderScreen(store);
-
-		expect(socialsCalls("GET")).toHaveLength(1);
-		expect(store.getState().social).toEqual(SOCIALS);
-		expect(screen.getByDisplayValue(SOCIALS.instagram)).toBeTruthy();
-		expect(screen.getByDisplayValue(SOCIALS.twitter)).toBeTruthy();
-	});
-
-	it("AC6 · editing a network enables Save, and saving PUTs the whole map and leaves the form clean", async () => {
-		const store = makeStore();
-		await renderScreen(store);
-
-		// Pristine off what the server answered — there is nothing to save yet.
-		expect(button("Saved").hasAttribute("disabled")).toBe(true);
-
-		await typeInto(SOCIALS.instagram, EDITED);
-
-		expect(button("Save").hasAttribute("disabled")).toBe(false);
-
-		await act(async () => {
-			fireEvent.click(button("Save"));
-		});
-
-		// The whole map, not just the edited field — the endpoint takes it as a unit.
-		expect(socialsCalls("PUT")).toHaveLength(1);
-		expect(socialsCalls("PUT")[0].data).toEqual({ ...SOCIALS, instagram: EDITED });
-		expect(store.getState().social).toEqual({ ...SOCIALS, instagram: EDITED });
-		// Back to pristine, and off the server's answer: this is the assertion that the
-		// `.unwrap()` in the view protects. Without it the dispatch resolves to the
-		// fulfilled action, truthy even on a failed save.
-		expect(button("Saved").hasAttribute("disabled")).toBe(true);
-	});
-
-	it("AC6 · a failed save keeps the form dirty so the changes are not silently dropped", async () => {
-		const store = makeStore();
-		await renderScreen(store);
-
-		await typeInto(SOCIALS.instagram, EDITED);
-
-		mockedAxios.mockImplementation((async () => ({
-			status: 400,
-			data: { message: "Invalid URL" },
-			statusText: "Bad Request",
-			headers: {},
-			config: {},
-		})) as never);
-
-		await act(async () => {
-			fireEvent.click(button("Save"));
-		});
-
-		expect(button("Save").hasAttribute("disabled")).toBe(false);
-		expect(store.getState().social).toEqual(SOCIALS);
-	});
-});

package/src/__tests__/store/activityLog.test.ts

@@ -1,301 +0,0 @@
-import { logs } from "@ax/api";
-import { activityLogActions, activityLogReducer } from "@ax/containers/ActivityLog";
-import { appActions } from "@ax/containers/App";
-import { appInitialState, appReducer, type IAppState } from "@ax/containers/App/reducer";
-import { sitesActions } from "@ax/containers/Sites";
-import { usersActions } from "@ax/containers/Users";
-import type { AppDispatch } from "@ax/store";
-import type { ILanguage } from "@ax/types";
-
-import type { LogActivityPaginationRequest, LogContentDTO, LogContentTypeDTO } from "@griddo/api-types";
-import { configureStore, type Reducer, type UnknownAction } from "@reduxjs/toolkit";
-import type { MockInstance } from "vitest";
-
-/**
- * Characterisation tests for the `activityLog` area — the audit trail behind `/log`
- * (UC-AX-view-activity-log). The area had **zero** tests, so this file is written against
- * the legacy `actions.tsx` + `reducer.tsx` first and must keep passing against the slice.
- *
- * Thunks are dispatched through the one-line adapters below: createAsyncThunk takes exactly
- * ONE argument, so the calls change shape when the area migrates and the adapters keep every
- * assertion untouched (docs/RTK-MIGRATION.md §5). Same trick as `redirects.test.ts`.
- *
- * On scope: the `goToLogContent` block below is tagged with no AC on purpose. The use case
- * files deep-linking under step 9 as an *extension* whose navigation belongs to the target
- * screens' own use cases, so these are **mechanics** tests, not AC coverage — they exist
- * because the migration rewrites how that 160-line navigation switch is invoked, and it is
- * the one thing in this wave too big to convert unpinned. They cover the branch SHAPES (site
- * vs global, content vs contentType, language resolution), not all seventeen content types.
- */
-
-const PARAMS: LogActivityPaginationRequest = { page: 1, itemsPerPage: 100 };
-
-const LOG_PAGE = {
-	totalItems: 2,
-	items: [
-		{ date: "2026-08-10", items: [{ id: 1 }] },
-		{ date: "2026-08-09", items: [{ id: 2 }] },
-	],
-};
-
-const EN: ILanguage = { id: 1, locale: "en_GB", label: "en", language: "English", isDefault: true };
-const ES: ILanguage = { id: 2, locale: "es_ES", label: "es", language: "Spanish", isDefault: false };
-
-const SITE_ID = 7;
-
-// `ReturnType<…>` rather than naming the state type: it lives in `reducer.tsx` today and in
-// `slice.ts` afterwards, and this file must not have to follow it. The `app` cast is the one
-// GlobalStore uses — RTK types reducers against redux 5's UnknownAction (§8.3).
-type ActivityLogState = ReturnType<typeof activityLogReducer>;
-
-const makeStore = () =>
-	configureStore({
-		reducer: {
-			activityLog: activityLogReducer as unknown as Reducer<ActivityLogState, UnknownAction>,
-			app: appReducer as Reducer<IAppState, UnknownAction>,
-		},
-		middleware: (getDefault) => getDefault({ serializableCheck: false, immutableCheck: false }),
-		// `goToLogContent` resolves the target language against these two.
-		preloadedState: { app: { ...appInitialState, lang: { locale: EN.locale, id: EN.id }, globalLangs: [EN, ES] } },
-	});
-
-type Store = ReturnType<typeof makeStore>;
-const stateOf = (store: Store) => store.getState().activityLog;
-
-// The adapters. One line each; the migration rewrote these and nothing else.
-//
-// Only `goToLogContent` changed shape — three positional arguments into one object, the
-// single-argument rule of docs/RTK-MIGRATION.md §5. The other three already took one argument,
-// and `setListMode` shed the cast a redux 4 action creator needed to enter an RTK store (§8.3).
-const loadByDay = (store: Store, params = PARAMS) => store.dispatch(activityLogActions.getActivityLog(params));
-const loadByUser = (store: Store, params = PARAMS) => store.dispatch(activityLogActions.getActivityLogByUser(params));
-const setMode = (store: Store, mode: string) => store.dispatch(activityLogActions.setListMode(mode));
-const download = (store: Store, data: { format: string[]; dateRange?: string }) =>
-	store.dispatch(activityLogActions.downloadActivityLog(data as never));
-const goTo = (
-	store: Store,
-	contentType: LogContentTypeDTO | null,
-	content: LogContentDTO | null,
-	site: number | null,
-	// `goToLogContent` is declared with the store's real `AppDispatch` (see thunks.ts, it has to
-	// be, or the nineteen legacy actions it dispatches stop type-checking), and this store mounts
-	// two slices instead of all eighteen — so its dispatch gets widened to the real one. The
-	// thunk only ever reads `app`.
-) => (store.dispatch as AppDispatch)(activityLogActions.goToLogContent({ contentType, content, site }));
-
-const ok = (data: unknown) => ({ status: 200, data });
-
-let handleError: MockInstance;
-
-beforeEach(() => {
-	handleError = vi.spyOn(appActions, "handleError").mockReturnValue((() => undefined) as never);
-});
-
-afterEach(() => {
-	vi.restoreAllMocks();
-});
-
-describe("[UC-AX-view-activity-log] loading the log", () => {
-	it("AC1 · starts empty, in the timeline mode", () => {
-		expect(stateOf(makeStore())).toEqual({ activityLog: { totalItems: 0, items: [] }, listMode: "default" });
-	});
-
-	it("AC1 · loading asks the grouped-by-day endpoint and stores the page it gets", async () => {
-		const store = makeStore();
-		const get = vi.spyOn(logs, "getLogActivityTimelineByDay").mockResolvedValue(ok(LOG_PAGE) as never);
-
-		await loadByDay(store);
-
-		expect(get).toHaveBeenCalledWith(PARAMS);
-		expect(stateOf(store).activityLog).toEqual(LOG_PAGE);
-	});
-
-	it("AC4 · the params travel through untouched, so ordering and search reach the endpoint", async () => {
-		const store = makeStore();
-		const get = vi.spyOn(logs, "getLogActivityTimelineByDay").mockResolvedValue(ok(LOG_PAGE) as never);
-		const sorted = { ...PARAMS, order: "ASC" as const, search: "deleted" };
-
-		await loadByDay(store, sorted);
-
-		expect(get).toHaveBeenCalledWith(sorted);
-	});
-
-	it("AC2 · the grouped-by-user mode asks a different endpoint and lands in the same slot", async () => {
-		const store = makeStore();
-		const byUser = vi.spyOn(logs, "getLogActivityTimelineByUser").mockResolvedValue(ok(LOG_PAGE) as never);
-		const byDay = vi.spyOn(logs, "getLogActivityTimelineByDay").mockResolvedValue(ok(LOG_PAGE) as never);
-
-		await loadByUser(store);
-
-		expect(byUser).toHaveBeenCalledWith(PARAMS);
-		expect(byDay).not.toHaveBeenCalled();
-		expect(stateOf(store).activityLog).toEqual(LOG_PAGE);
-	});
-
-	it("AC2 · the list mode is a separate flag the endpoints do not touch", async () => {
-		const store = makeStore();
-		vi.spyOn(logs, "getLogActivityTimelineByUser").mockResolvedValue(ok(LOG_PAGE) as never);
-
-		setMode(store, "user");
-		await loadByUser(store);
-
-		expect(stateOf(store).listMode).toBe("user");
-		expect(stateOf(store).activityLog).toEqual(LOG_PAGE);
-	});
-
-	it("AC1 · a failed load routes through the error channel and leaves the page as it was", async () => {
-		const store = makeStore();
-		vi.spyOn(logs, "getLogActivityTimelineByDay").mockResolvedValue({ status: 500, data: {} } as never);
-
-		await loadByDay(store);
-
-		expect(handleError).toHaveBeenCalled();
-		expect(stateOf(store).activityLog).toEqual({ totalItems: 0, items: [] });
-	});
-});
-
-describe("[UC-AX-view-activity-log] exporting the log", () => {
-	let click: MockInstance;
-	let createObjectURL: MockInstance;
-
-	beforeEach(() => {
-		// jsdom implements neither of these, and clicking a real <a href> would try to navigate.
-		createObjectURL = vi.fn(() => "blob:activity-log") as unknown as MockInstance;
-		Object.assign(window.URL, { createObjectURL, revokeObjectURL: vi.fn() });
-		click = vi.spyOn(HTMLAnchorElement.prototype, "click").mockImplementation(() => undefined);
-	});
-
-	it("AC6 · exporting posts the request and hands the browser a download named after today", async () => {
-		const store = makeStore();
-		const exportLog = vi.spyOn(logs, "logExport").mockResolvedValue(ok("id;event\n1;CREATED") as never);
-		const request = { dateRange: "7", format: ["csv"] };
-
-		await download(store, request);
-
-		expect(exportLog).toHaveBeenCalledWith(request);
-		expect(createObjectURL).toHaveBeenCalled();
-		expect(click).toHaveBeenCalled();
-		// The extension comes from the requested format, the stem from the date.
-		const link = click.mock.instances[0] as HTMLAnchorElement;
-		expect(link.download).toMatch(/^activity-log-\d{4}-\d{2}-\d{2}\.csv$/);
-	});
-
-	it("AC6 · the requested format decides the extension", async () => {
-		const store = makeStore();
-		vi.spyOn(logs, "logExport").mockResolvedValue(ok("") as never);
-
-		await download(store, { dateRange: "30", format: ["xlsx"] });
-
-		const link = click.mock.instances[0] as HTMLAnchorElement;
-		expect(link.download.endsWith(".xlsx")).toBe(true);
-	});
-
-	it("AC6 · a failed export reports the error and downloads nothing", async () => {
-		const store = makeStore();
-		vi.spyOn(logs, "logExport").mockResolvedValue({ status: 500, data: {} } as never);
-
-		await download(store, { dateRange: "7", format: ["csv"] });
-
-		expect(handleError).toHaveBeenCalled();
-		expect(click).not.toHaveBeenCalled();
-	});
-});
-
-describe("goToLogContent — deep-link mechanics", () => {
-	let push: MockInstance;
-	let getSite: MockInstance;
-	let setLanguage: MockInstance;
-
-	const noopThunk = (() => () => Promise.resolve()) as never;
-
-	beforeEach(() => {
-		push = vi.spyOn(appActions, "setHistoryPush").mockReturnValue((() => undefined) as never);
-		getSite = vi.spyOn(sitesActions, "getSite").mockImplementation(noopThunk);
-		setLanguage = vi.spyOn(appActions, "setLanguage");
-	});
-
-	const content = (over: Partial<LogContentDTO>) =>
-		({ id: 9, type: "site", language: EN.id, ...over }) as LogContentDTO;
-
-	it("a global target navigates without loading any site", async () => {
-		const store = makeStore();
-		const getUser = vi.spyOn(usersActions, "getUser").mockImplementation(noopThunk);
-
-		await goTo(store, null, content({ type: "user" }), null);
-
-		expect(getUser).toHaveBeenCalledWith(9);
-		expect(push).toHaveBeenCalledWith("/users/edit");
-		expect(getSite).not.toHaveBeenCalled();
-	});
-
-	it("a site-scoped target loads the site FIRST and only then navigates", async () => {
-		const store = makeStore();
-		const order: string[] = [];
-		getSite.mockImplementation(((id: number) => {
-			order.push(`getSite:${id}`);
-			return () => Promise.resolve();
-		}) as never);
-		push.mockImplementation(((path: string) => {
-			order.push(`push:${path}`);
-			return () => undefined;
-		}) as never);
-
-		await goTo(store, null, content({ type: "site" }), SITE_ID);
-
-		// The editor reads the loaded site, so the order is the behaviour.
-		expect(order).toEqual([`getSite:${SITE_ID}`, "push:/sites/pages"]);
-	});
-
-	it("the same content type falls back to its global route when there is no site", async () => {
-		const store = makeStore();
-
-		await goTo(store, null, content({ type: "file" }), null);
-
-		expect(getSite).not.toHaveBeenCalled();
-		expect(push).toHaveBeenCalledWith("/media-assets/files");
-	});
-
-	it("a site-only content type does nothing at all without a site", async () => {
-		const store = makeStore();
-
-		await goTo(store, null, content({ type: "menu" }), null);
-
-		expect(getSite).not.toHaveBeenCalled();
-		expect(push).not.toHaveBeenCalled();
-	});
-
-	it("switches the editor language when the content is in another one", async () => {
-		const store = makeStore();
-
-		await goTo(store, null, content({ type: "menu", language: ES.id }), SITE_ID);
-
-		expect(setLanguage).toHaveBeenCalledWith({ locale: ES.locale, id: ES.id });
-	});
-
-	it("keeps the current language when the content is already in it", async () => {
-		const store = makeStore();
-
-		await goTo(store, null, content({ type: "menu", language: EN.id }), SITE_ID);
-
-		expect(setLanguage).toHaveBeenCalledWith({ locale: EN.locale, id: EN.id });
-	});
-
-	it("a content TYPE target is a separate branch: a template opens the page editor", async () => {
-		const store = makeStore();
-		const contentType = { id: 42, type: "template", languageId: EN.id } as LogContentTypeDTO;
-
-		await goTo(store, contentType, null, SITE_ID);
-
-		expect(getSite).toHaveBeenCalledWith(SITE_ID);
-		expect(push).toHaveBeenCalledWith("/sites/pages/editor", true);
-	});
-
-	it("does nothing when neither a content nor a content type is given", async () => {
-		const store = makeStore();
-
-		await goTo(store, null, null, SITE_ID);
-
-		expect(getSite).not.toHaveBeenCalled();
-		expect(push).not.toHaveBeenCalled();
-	});
-});

package/src/__tests__/store/analytics.test.ts

@@ -1,134 +0,0 @@
-import { analytics } from "@ax/api";
-import { analyticsActions, analyticsReducer } from "@ax/containers/Analytics";
-import { appActions } from "@ax/containers/App";
-import type { IAnalytics } from "@ax/types";
-
-import { configureStore } from "@reduxjs/toolkit";
-import type { MockInstance } from "vitest";
-
-/**
- * Characterisation tests for the `analytics` area, written against the store and
- * the exported action creators before the area became a slice — see the sibling
- * domains.test.ts for why that order is the point.
- *
- * Two behaviours here are load-bearing and easy to lose in a rewrite:
- * `setAnalytics` MERGES onto the state (the legacy reducer spread the payload, and
- * `Sites/actions.tsx` dispatches it with whatever the site's metrics endpoint
- * returned), and the global save fires its three requests in PARALLEL. Both
- * assertions below are the ones that came over from the legacy run untouched.
- *
- * What did have to change are the CALLS, and that is the second finding of this
- * migration: a createAsyncThunk takes exactly ONE argument, so the two-parameter
- * writes now take an object (`{ scriptCode, siteId }`), and the optional `siteId`
- * became explicit (`null` = global, which the API spells "global"). Every call site
- * of a multi-argument thunk pays this — see docs/RTK-MIGRATION.md §5.
- */
-
-const ANALYTICS: IAnalytics = {
-	scriptCode: "<script>ga</script>",
-	siteScriptCodeExists: true,
-	dimensions: [{ id: 1, name: "campus", type: "onlyDimensions", values: [] } as never],
-	groups: [],
-};
-
-const SITE_ID = 7;
-
-const makeStore = () => configureStore({ reducer: { analytics: analyticsReducer } });
-type Store = ReturnType<typeof makeStore>;
-const stateOf = (store: Store) => (store.getState() as { analytics: IAnalytics }).analytics;
-
-const ok = { status: 200, data: {} };
-
-describe("[UC-AX-configure-seo-analytics] analytics state", () => {
-	let store: Store;
-	let handleError: MockInstance;
-
-	beforeEach(() => {
-		store = makeStore();
-		handleError = vi.spyOn(appActions, "handleError").mockReturnValue((() => undefined) as never);
-	});
-
-	afterEach(() => {
-		vi.restoreAllMocks();
-	});
-
-	it("AC2 · starts with an empty script and no dimensions", () => {
-		expect(stateOf(store)).toEqual({ scriptCode: "", siteScriptCodeExists: false, dimensions: [], groups: [] });
-	});
-
-	it("AC2 · setAnalytics merges onto the state instead of replacing it", () => {
-		store.dispatch(analyticsActions.setAnalytics(ANALYTICS));
-
-		// A partial payload — which is what the site metrics endpoint can return —
-		// must leave the untouched fields alone.
-		store.dispatch(analyticsActions.setAnalytics({ scriptCode: "<script>gtm</script>" } as IAnalytics));
-
-		expect(stateOf(store)).toEqual({ ...ANALYTICS, scriptCode: "<script>gtm</script>" });
-	});
-
-	it("AC2 · loading the global analytics puts them in the store", async () => {
-		vi.spyOn(analytics, "getAnalytics").mockResolvedValue({ status: 200, data: ANALYTICS } as never);
-
-		await store.dispatch(analyticsActions.getAnalytics(null));
-
-		expect(analytics.getAnalytics).toHaveBeenCalledWith("global");
-		expect(stateOf(store)).toEqual(ANALYTICS);
-	});
-
-	it("AC2 · a site load asks for that site, not for global", async () => {
-		vi.spyOn(analytics, "getAnalytics").mockResolvedValue({ status: 200, data: ANALYTICS } as never);
-
-		await store.dispatch(analyticsActions.getAnalytics(SITE_ID));
-
-		expect(analytics.getAnalytics).toHaveBeenCalledWith(SITE_ID);
-	});
-
-	it("AC2 · saving the global analytics fires exactly the three writes and reloads", async () => {
-		const code = vi.spyOn(analytics, "updateScriptCode").mockResolvedValue(ok as never);
-		const dimensions = vi.spyOn(analytics, "createDimensions").mockResolvedValue(ok as never);
-		const groups = vi.spyOn(analytics, "createDimensionsGroups").mockResolvedValue(ok as never);
-		const reload = vi.spyOn(analytics, "getAnalytics").mockResolvedValue({ status: 200, data: ANALYTICS } as never);
-
-		await store.dispatch(analyticsActions.updateAnalytics({ analytics: ANALYTICS, siteId: null }));
-
-		expect(code).toHaveBeenCalledWith("global", { scriptCode: ANALYTICS.scriptCode });
-		expect(dimensions).toHaveBeenCalledWith("global", { dimensions: ANALYTICS.dimensions });
-		expect(groups).toHaveBeenCalledWith("global", { groups: ANALYTICS.groups });
-		expect(reload).toHaveBeenCalled();
-	});
-
-	it("AC2 · one failing write among the three reports the count and skips the reload", async () => {
-		const failure = { status: 400, data: { message: ["bad dimension", "bad group"] } };
-		vi.spyOn(analytics, "updateScriptCode").mockResolvedValue(ok as never);
-		vi.spyOn(analytics, "createDimensions").mockResolvedValue(failure as never);
-		vi.spyOn(analytics, "createDimensionsGroups").mockResolvedValue(ok as never);
-		const reload = vi.spyOn(analytics, "getAnalytics").mockResolvedValue({ status: 200, data: ANALYTICS } as never);
-
-		await store.dispatch(analyticsActions.updateAnalytics({ analytics: ANALYTICS, siteId: null }));
-
-		expect(handleError).toHaveBeenCalledWith(failure, true, "The action failed due to 2 errors.");
-		expect(reload).not.toHaveBeenCalled();
-	});
-
-	it("AC5 · a site only writes its script code, and reloads that site", async () => {
-		const code = vi.spyOn(analytics, "updateScriptCode").mockResolvedValue(ok as never);
-		const dimensions = vi.spyOn(analytics, "createDimensions").mockResolvedValue(ok as never);
-		const reload = vi.spyOn(analytics, "getAnalytics").mockResolvedValue({ status: 200, data: ANALYTICS } as never);
-
-		await store.dispatch(analyticsActions.updateScriptCode({ scriptCode: "<script>site</script>", siteId: SITE_ID }));
-
-		expect(code).toHaveBeenCalledWith(SITE_ID, { scriptCode: "<script>site</script>" });
-		expect(dimensions).not.toHaveBeenCalled();
-		expect(reload).toHaveBeenCalledWith(SITE_ID);
-	});
-
-	it("AC5 · resetting a site to the global defaults deletes its script code and reloads", async () => {
-		const remove = vi.spyOn(analytics, "deleteScriptCode").mockResolvedValue(ok as never);
-		const reload = vi.spyOn(analytics, "getAnalytics").mockResolvedValue({ status: 200, data: ANALYTICS } as never);
-
-		await store.dispatch(analyticsActions.deleteScriptCode(SITE_ID));
-
-		expect(remove).toHaveBeenCalledWith(SITE_ID);
-		expect(reload).toHaveBeenCalledWith(SITE_ID);
-	});
-});