fix: add ON DELETE CASCADE to user_id reference in carts table and update secure cookie setting based on environment

Author: diegotc86

src/db/migrations/initial.sql

@@ -36,7 +36,7 @@ CREATE TABLE IF NOT EXISTS products (
 CREATE TABLE IF NOT EXISTS carts (
     id SERIAL PRIMARY KEY,
     session_cart_id UUID UNIQUE DEFAULT gen_random_uuid(),
-    user_id INTEGER REFERENCES users(id),
+    user_id INTEGER REFERENCES users(id) ON DELETE CASCADE,
     created_at TIMESTAMP DEFAULT CURRENT_TIMESTAMP,
     updated_at TIMESTAMP DEFAULT CURRENT_TIMESTAMP
 );

src/e2e/demo.signin.spec.ts

@@ -1,6 +1,38 @@
 import { test, expect } from "@playwright/test";
 
+import { hashPassword } from "@/lib/security";
+import type { CreateUserDTO } from "@/models/user.model";
+import {
+  createUser,
+  deleteUser,
+  getUserByEmail,
+} from "@/repositories/user.repository";
+
 test.describe("Visitante inicio sesion", () => {
+  let testUserId: number;
+
+  test.beforeAll(async () => {
+    const testUser: CreateUserDTO = {
+      email: "[email protected]",
+      name: null,
+      password: await hashPassword("letmein"),
+      isGuest: false,
+    };
+
+    const existingUser = await getUserByEmail(testUser.email);
+
+    if (existingUser) {
+      await deleteUser(existingUser.id);
+    }
+
+    const user = await createUser(testUser);
+    testUserId = user.id;
+  });
+
+  test.afterAll(async () => {
+    await deleteUser(testUserId);
+  });
+
   test("test", async ({ page }) => {
     await page.goto("http://localhost:5173/");
     await page.getByTestId("login").click();

src/session.server.ts

@@ -27,7 +27,7 @@ const { getSession, commitSession, destroySession } =
       path: "/",
       sameSite: "lax",
       secrets: ["s3cret1"],
-      secure: true,
+      secure: process.env.NODE_ENV === "production", // true en producción, false en desarrollo
     },
   });