unify mock db seeding approach

Author: imaitland

app/mocks/browser.ts

@@ -3,8 +3,10 @@ import { handlers } from "./evolver";
 import { seedDatabase } from "./db-data";
 
 // Initialize the test database with seed data
+// Since the DB mock is a singleton, we can just call this once
 if (process.env.NODE_ENV === "test") {
   seedDatabase();
 }
 
+// see app/entry.client.ts for the worker.start() call
 export const worker = setupWorker(...handlers);

app/mocks/db-data.ts

@@ -18,6 +18,7 @@ export const db = factory({
 });
 
 // Seed the database with initial data
+
 export function seedDatabase() {
   // Clear any existing data
   drop(db);
@@ -30,6 +31,3 @@ export function seedDatabase() {
     name: TEST_DEVICE_NAME,
   });
 }
-
-// Initialize the database
-seedDatabase();

app/mocks/server.ts

@@ -1,6 +1,13 @@
 import { setupServer } from "msw/node";
 import { handlers } from "./evolver";
 
+import { seedDatabase } from "./db-data";
+
+// Initialize the test database with seed data
+// Since the DB mock is a singleton, we can just call this once
+if (process.env.NODE_ENV === "test") {
+  seedDatabase();
+}
 // Note: not included here because it's not a HTTP mock - but to be aware of:
 // the DB is mocked using the prisma mock, see app/utils/db.server.ts
 export const server = setupServer(...handlers);