Fix test

apps/server/src/lib/test/fixtures.ts

@@ -35,6 +35,7 @@ import {
   bottles,
   bottlesToDistillers,
   changes,
+  collections,
   comments,
   entities,
   externalSites,
@@ -521,6 +522,21 @@ export const Review = async (
   return result;
 };
 
+export const Collection = async (
+  { ...data }: Partial<Omit<dbSchema.NewCollection, "id">> = {},
+  db: DatabaseType = dbConn,
+): Promise<dbSchema.Collection> => {
+  const [result] = await db
+    .insert(collections)
+    .values({
+      name: faker.company.name(),
+      ...(data as Omit<dbSchema.NewCollection, "name">),
+    })
+    .returning();
+  if (!result) throw new Error("Unable to create fixture");
+  return result;
+};
+
 export const AuthToken = async (
   { user }: { user?: dbSchema.User | null } = {},
   db: DatabaseType = dbConn,

apps/server/src/trpc/routes/collectionBottleDelete.test.ts

@@ -21,6 +21,16 @@ test("requires auth", async () => {
 test("delete bottle in default", async ({ fixtures, defaults }) => {
   const bottle = await fixtures.Bottle();
 
+  const collection = await fixtures.Collection({
+    name: "default",
+    createdById: defaults.user.id,
+  });
+
+  await db.insert(collectionBottles).values({
+    bottleId: bottle.id,
+    collectionId: collection.id,
+  });
+
   const caller = createCaller({
     user: defaults.user,
   });
@@ -35,5 +45,5 @@ test("delete bottle in default", async ({ fixtures, defaults }) => {
     .from(collectionBottles)
     .where(eq(collectionBottles.bottleId, bottle.id));
 
-  expect(bottleList.length).toBe(1);
+  expect(bottleList.length).toBe(0);
 });