test(*): do not manually restore mocks

This already happens via a global config now

Author: pawcoding

test/loader/fetch-collection.e2e-spec.ts

@@ -4,7 +4,6 @@ import {
 } from "astro/content/runtime";
 import { randomUUID } from "crypto";
 import {
-  afterEach,
   assert,
   beforeAll,
   beforeEach,
@@ -52,10 +51,6 @@ describe("fetchCollection", () => {
     superuserToken = token;
   });
 
-  afterEach(() => {
-    vi.resetAllMocks();
-  });
-
   test("should fetch entries without errors", async () => {
     await fetchCollection(options, chunkLoadedMock, superuserToken, undefined);
 

test/loader/live-collection-loader.spec.ts

@@ -1,5 +1,5 @@
 import { LiveCollectionError } from "astro/content/runtime";
-import { afterEach, beforeEach, describe, expect, test, vi } from "vitest";
+import { beforeEach, describe, expect, test, vi } from "vitest";
 import { liveCollectionLoader } from "../../src/loader/live-collection-loader";
 import { createLiveLoaderOptions } from "../_mocks/create-live-loader-options";
 import { createPocketbaseEntry } from "../_mocks/create-pocketbase-entry";
@@ -16,10 +16,6 @@ describe("liveCollectionLoader", async () => {
     plem.parseLiveEntry = vi.fn().mockImplementation((e) => e);
   });
 
-  afterEach(() => {
-    vi.resetAllMocks();
-  });
-
   test("should return live data collection with parsed entries", async () => {
     const entry = createPocketbaseEntry();
 

test/loader/live-entry-loader.spec.ts

@@ -1,5 +1,5 @@
 import { LiveCollectionError } from "astro/content/runtime";
-import { afterEach, beforeEach, describe, expect, test, vi } from "vitest";
+import { beforeEach, describe, expect, test, vi } from "vitest";
 import { liveEntryLoader } from "../../src/loader/live-entry-loader";
 import { createLiveLoaderOptions } from "../_mocks/create-live-loader-options";
 import { createPocketbaseEntry } from "../_mocks/create-pocketbase-entry";
@@ -16,10 +16,6 @@ describe("liveEntryLoader", async () => {
     plem.parseLiveEntry = vi.fn().mockImplementation((e) => e);
   });
 
-  afterEach(() => {
-    vi.resetAllMocks();
-  });
-
   test("should return parsed entry", async () => {
     const entry = createPocketbaseEntry();
     fem.fetchEntry = vi.fn().mockResolvedValue(entry);

test/loader/load-entries.spec.ts

@@ -1,5 +1,5 @@
 import type { LoaderContext } from "astro/loaders";
-import { afterEach, beforeEach, describe, expect, test, vi } from "vitest";
+import { beforeEach, describe, expect, test, vi } from "vitest";
 import { loadEntries } from "../../src/loader/load-entries";
 import { createLoaderContext } from "../_mocks/create-loader-context";
 import { createLoaderOptions } from "../_mocks/create-loader-options";
@@ -18,10 +18,6 @@ describe("loadEntries", async () => {
     context = createLoaderContext();
   });
 
-  afterEach(() => {
-    vi.resetAllMocks();
-  });
-
   test("should call fetchCollection and parseEntry for each entry", async () => {
     const entry = createPocketbaseEntry();
 

test/loader/loader.spec.ts

@@ -1,5 +1,5 @@
 import type { LoaderContext } from "astro/loaders";
-import { afterEach, beforeEach, describe, expect, test, vi } from "vitest";
+import { beforeEach, describe, expect, test, vi } from "vitest";
 import packageJson from "../../package.json";
 import { cleanupEntries } from "../../src/loader/cleanup-entries";
 import { handleRealtimeUpdates } from "../../src/loader/handle-realtime-updates";
@@ -29,10 +29,6 @@ describe("loader", async () => {
     );
   });
 
-  afterEach(() => {
-    vi.resetAllMocks();
-  });
-
   test('should not refresh if shouldRefresh returns "skip"', async () => {
     srm.shouldRefresh = vi.fn().mockReturnValue("skip");
 

test/loader/parse-entry.spec.ts

@@ -1,5 +1,5 @@
 import type { LoaderContext } from "astro/loaders";
-import { afterEach, beforeEach, describe, expect, test, vi } from "vitest";
+import { beforeEach, describe, expect, test, vi } from "vitest";
 import { parseEntry } from "../../src/loader/parse-entry";
 import type { PocketBaseEntry } from "../../src/types/pocketbase-entry.type";
 import { createLoaderContext } from "../_mocks/create-loader-context";
@@ -15,10 +15,6 @@ describe("parseEntry", () => {
     entry = createPocketbaseEntry();
   });
 
-  afterEach(() => {
-    vi.resetAllMocks();
-  });
-
   test("should use default ID if no custom ID field is provided", async () => {
     const options = createLoaderOptions();
 

test/schema/generate-schema.e2e-spec.ts

@@ -1,5 +1,5 @@
 import type { ZodObject, ZodSchema } from "astro/zod";
-import { afterEach, assert, beforeAll, describe, expect, it, vi } from "vitest";
+import { assert, beforeAll, describe, expect, it, vi } from "vitest";
 import type { PocketBaseLoaderOptions } from "../../src";
 import { generateSchema } from "../../src/schema/generate-schema";
 import { transformFileUrl } from "../../src/schema/transform-files";
@@ -29,10 +29,6 @@ describe("generateSchema", () => {
     token = superuserToken;
   });
 
-  afterEach(() => {
-    vi.resetAllMocks();
-  });
-
   describe("load and parse schema", () => {
     it("should return basic schema if no schema is available", async () => {
       const result = (await generateSchema(

test/schema/read-local-schema.spec.ts

@@ -1,6 +1,6 @@
 import fs from "fs/promises";
 import path from "path";
-import { beforeEach, describe, expect, test, vi } from "vitest";
+import { describe, expect, test, vi } from "vitest";
 import { readLocalSchema } from "../../src/schema/read-local-schema";
 
 vi.mock("fs/promises");
@@ -21,10 +21,6 @@ describe("readLocalSchema", () => {
     }
   ];
 
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
   test("should return the schema for the specified collection", async () => {
     vi.spyOn(path, "join").mockReturnValue(localSchemaPath);
     vi.spyOn(fs, "readFile").mockResolvedValue(JSON.stringify(mockSchema));

vitest.config.ts

@@ -7,6 +7,7 @@ export default defineConfig({
     coverage: {
       include: ["src/**/*.ts"],
       exclude: ["src/types/**/*.ts", "src/index.ts", "src/pocketbase-loader.ts"]
-    }
+    },
+    restoreMocks: true
   }
 });