test : changed test format + added playwright support

tests/integration/students_test.ts

@@ -1,18 +1,25 @@
-// #109 - Integration tests for /students endpoints
-// Teste les opérations DB directement avec une vraie base de données
+// #109 - E2E tests for /students endpoints
+// Appelle les handlers Fresh directement avec un vrai contexte + vraie DB
 
 import { assertEquals, assertExists } from "@std/assert";
+import {
+  makeContextWithAffiliation,
+  makeEmployeeContext,
+  makeGetRequest,
+  makeJsonRequest,
+} from "../helpers/handler.ts";
 import {
   seedPromotions,
   seedStudents,
-  testDb,
   truncateAll,
 } from "../helpers/db_integration.ts";
-import { students } from "$root/databases/schema.ts";
-import { eq } from "npm:drizzle-orm@0.45.2";
+import { handler as studentsHandler } from "$apps/students/api/students.ts";
+import { handler as studentHandler } from "$apps/students/api/students/[numEtud].ts";
+
+// --- GET /students ---
 
 Deno.test({
-  name: "integration students: list all students",
+  name: "e2e students: GET /students returns all students as employee",
   async fn() {
     await truncateAll();
     await seedPromotions([{ id: "PEIP1-2024" }]);
@@ -21,15 +28,42 @@ Deno.test({
       { nom: "Martin", prenom: "Alice", idPromo: "PEIP1-2024" },
     ]);
 
-    const rows = await testDb.select().from(students);
-    assertEquals(rows.length, 2);
+    const req = makeGetRequest("/students");
+    const ctx = makeEmployeeContext();
+    const res = await studentsHandler.GET!(req, ctx);
+
+    assertEquals(res.status, 200);
+    const body = await res.json();
+    assertEquals(body.length, 2);
+    assertExists(body.find((s: { nom: string }) => s.nom === "Dupont"));
   },
   sanitizeResources: false,
   sanitizeOps: false,
 });
 
 Deno.test({
-  name: "integration students: filter by idPromo",
+  name: "e2e students: GET /students returns empty array for non-employee",
+  async fn() {
+    await truncateAll();
+    await seedPromotions([{ id: "PEIP1-2024" }]);
+    await seedStudents([
+      { nom: "Dupont", prenom: "Jean", idPromo: "PEIP1-2024" },
+    ]);
+
+    const req = makeGetRequest("/students");
+    const ctx = makeContextWithAffiliation("student");
+    const res = await studentsHandler.GET!(req, ctx);
+
+    assertEquals(res.status, 200);
+    const body = await res.json();
+    assertEquals(body.length, 0);
+  },
+  sanitizeResources: false,
+  sanitizeOps: false,
+});
+
+Deno.test({
+  name: "e2e students: GET /students?idPromo filters by promotion",
   async fn() {
     await truncateAll();
     await seedPromotions([{ id: "PEIP1-2024" }, { id: "PEIP2-2024" }]);
@@ -39,63 +73,140 @@ Deno.test({
       { nom: "Durand", prenom: "Claire", idPromo: "PEIP2-2024" },
     ]);
 
-    const rows = await testDb
-      .select()
-      .from(students)
-      .where(eq(students.idPromo, "PEIP1-2024"));
-    assertEquals(rows.length, 2);
-    assertEquals(rows.every((s) => s.idPromo === "PEIP1-2024"), true);
+    const req = makeGetRequest("/students", { idPromo: "PEIP1-2024" });
+    const ctx = makeEmployeeContext();
+    const res = await studentsHandler.GET!(req, ctx);
+
+    assertEquals(res.status, 200);
+    const body = await res.json();
+    assertEquals(body.length, 2);
+    assertEquals(
+      body.every((s: { idPromo: string }) => s.idPromo === "PEIP1-2024"),
+      true,
+    );
   },
   sanitizeResources: false,
   sanitizeOps: false,
 });
 
+// --- POST /students ---
+
 Deno.test({
-  name: "integration students: create and retrieve by numEtud",
+  name: "e2e students: POST /students creates a student (201)",
   async fn() {
     await truncateAll();
     await seedPromotions([{ id: "INFO3-2024" }]);
 
-    const [created] = await testDb
-      .insert(students)
-      .values({ nom: "Leroy", prenom: "Paul", idPromo: "INFO3-2024" })
-      .returning();
+    const req = makeJsonRequest("/students", "POST", {
+      nom: "Leroy",
+      prenom: "Paul",
+      idPromo: "INFO3-2024",
+    });
+    const ctx = makeEmployeeContext();
+    const res = await studentsHandler.POST!(req, ctx);
 
-    assertExists(created.numEtud);
-
-    const row = await testDb
-      .select()
-      .from(students)
-      .where(eq(students.numEtud, created.numEtud))
-      .then((r) => r[0] ?? null);
-
-    assertExists(row);
-    assertEquals(row.nom, "Leroy");
-    assertEquals(row.idPromo, "INFO3-2024");
+    assertEquals(res.status, 201);
+    const body = await res.json();
+    assertExists(body.numEtud);
+    assertEquals(body.nom, "Leroy");
+    assertEquals(body.idPromo, "INFO3-2024");
   },
   sanitizeResources: false,
   sanitizeOps: false,
 });
 
 Deno.test({
-  name: "integration students: get by numEtud returns null when not found",
+  name: "e2e students: POST /students 403 for non-employee",
   async fn() {
     await truncateAll();
 
-    const row = await testDb
-      .select()
-      .from(students)
-      .where(eq(students.numEtud, 999999))
-      .then((r) => r[0] ?? null);
+    const req = makeJsonRequest("/students", "POST", {
+      nom: "Test",
+      prenom: "User",
+      idPromo: "PEIP1-2024",
+    });
+    const ctx = makeContextWithAffiliation("student");
+    const res = await studentsHandler.POST!(req, ctx);
 
-    assertEquals(row, null);
+    assertEquals(res.status, 403);
   },
   sanitizeResources: false,
   sanitizeOps: false,
 });
 
 Deno.test({
-  name: "integration students: update student fields",
+  name: "e2e students: POST /students 400 when missing required fields",
+  async fn() {
+    await truncateAll();
+
+    const req = makeJsonRequest("/students", "POST", { nom: "Leroy" });
+    const ctx = makeEmployeeContext();
+    const res = await studentsHandler.POST!(req, ctx);
+
+    assertEquals(res.status, 400);
+  },
+  sanitizeResources: false,
+  sanitizeOps: false,
+});
+
+// --- GET /students/:numEtud ---
+
+Deno.test({
+  name: "e2e students: GET /students/:numEtud returns student",
+  async fn() {
+    await truncateAll();
+    await seedPromotions([{ id: "INFO3-2024" }]);
+    const [s] = await seedStudents([
+      { nom: "Bernard", prenom: "Lucie", idPromo: "INFO3-2024" },
+    ]);
+
+    const req = makeGetRequest(`/students/${s.numEtud}`);
+    const ctx = makeEmployeeContext({ numEtud: String(s.numEtud) });
+    const res = await studentHandler.GET!(req, ctx);
+
+    assertEquals(res.status, 200);
+    const body = await res.json();
+    assertEquals(body.numEtud, s.numEtud);
+    assertEquals(body.nom, "Bernard");
+  },
+  sanitizeResources: false,
+  sanitizeOps: false,
+});
+
+Deno.test({
+  name: "e2e students: GET /students/:numEtud 404 when not found",
+  async fn() {
+    await truncateAll();
+
+    const req = makeGetRequest("/students/999999");
+    const ctx = makeEmployeeContext({ numEtud: "999999" });
+    const res = await studentHandler.GET!(req, ctx);
+
+    assertEquals(res.status, 404);
+  },
+  sanitizeResources: false,
+  sanitizeOps: false,
+});
+
+Deno.test({
+  name: "e2e students: GET /students/:numEtud 403 for non-employee",
+  async fn() {
+    await truncateAll();
+
+    const req = makeGetRequest("/students/12345");
+    const ctx = makeContextWithAffiliation("student", { numEtud: "12345" });
+    const res = await studentHandler.GET!(req, ctx);
+
+    assertEquals(res.status, 403);
+  },
+  sanitizeResources: false,
+  sanitizeOps: false,
+});
+
+// --- PUT /students/:numEtud ---
+
+Deno.test({
+  name: "e2e students: PUT /students/:numEtud updates student",
   async fn() {
     await truncateAll();
     await seedPromotions([{ id: "INFO3-2024" }, { id: "INFO4-2024" }]);
@@ -103,21 +214,47 @@ Deno.test({
       { nom: "Petit", prenom: "Hugo", idPromo: "INFO3-2024" },
     ]);
 
-    const [updated] = await testDb
-      .update(students)
-      .set({ nom: "Grand", idPromo: "INFO4-2024" })
-      .where(eq(students.numEtud, s.numEtud))
-      .returning();
+    const req = makeJsonRequest(`/students/${s.numEtud}`, "PUT", {
+      nom: "Grand",
+      prenom: "Hugo",
+      idPromo: "INFO4-2024",
+    });
+    const ctx = makeEmployeeContext({ numEtud: String(s.numEtud) });
+    const res = await studentHandler.PUT!(req, ctx);
 
-    assertEquals(updated.nom, "Grand");
-    assertEquals(updated.idPromo, "INFO4-2024");
+    assertEquals(res.status, 200);
+    const body = await res.json();
+    assertEquals(body.nom, "Grand");
+    assertEquals(body.idPromo, "INFO4-2024");
   },
   sanitizeResources: false,
   sanitizeOps: false,
 });
 
 Deno.test({
-  name: "integration students: delete student",
+  name: "e2e students: PUT /students/:numEtud 404 when not found",
+  async fn() {
+    await truncateAll();
+    await seedPromotions([{ id: "INFO3-2024" }]);
+
+    const req = makeJsonRequest("/students/999999", "PUT", {
+      nom: "Ghost",
+      prenom: "Ghost",
+      idPromo: "INFO3-2024",
+    });
+    const ctx = makeEmployeeContext({ numEtud: "999999" });
+    const res = await studentHandler.PUT!(req, ctx);
+
+    assertEquals(res.status, 404);
+  },
+  sanitizeResources: false,
+  sanitizeOps: false,
+});
+
+// --- DELETE /students/:numEtud ---
+
+Deno.test({
+  name: "e2e students: DELETE /students/:numEtud returns 204",
   async fn() {
     await truncateAll();
     await seedPromotions([{ id: "INFO3-2024" }]);
@@ -125,48 +262,26 @@ Deno.test({
       { nom: "Thomas", prenom: "Eva", idPromo: "INFO3-2024" },
     ]);
 
-    await testDb.delete(students).where(eq(students.numEtud, s.numEtud));
+    const req = makeGetRequest(`/students/${s.numEtud}`);
+    const ctx = makeEmployeeContext({ numEtud: String(s.numEtud) });
+    const res = await studentHandler.DELETE!(req, ctx);
 
-    const row = await testDb
-      .select()
-      .from(students)
-      .where(eq(students.numEtud, s.numEtud))
-      .then((r) => r[0] ?? null);
-
-    assertEquals(row, null);
+    assertEquals(res.status, 204);
   },
   sanitizeResources: false,
   sanitizeOps: false,
 });
 
 Deno.test({
-  name: "integration students: update non-existent student returns empty",
+  name: "e2e students: DELETE /students/:numEtud 404 when not found",
   async fn() {
     await truncateAll();
 
-    const result = await testDb
-      .update(students)
-      .set({ nom: "Ghost" })
-      .where(eq(students.numEtud, 999999))
-      .returning();
+    const req = makeGetRequest("/students/999999");
+    const ctx = makeEmployeeContext({ numEtud: "999999" });
+    const res = await studentHandler.DELETE!(req, ctx);
 
-    assertEquals(result.length, 0);
-  },
-  sanitizeResources: false,
-  sanitizeOps: false,
-});
-
-Deno.test({
-  name: "integration students: delete non-existent student returns empty",
-  async fn() {
-    await truncateAll();
-
-    const result = await testDb
-      .delete(students)
-      .where(eq(students.numEtud, 999999))
-      .returning();
-
-    assertEquals(result.length, 0);
+    assertEquals(res.status, 404);
   },
   sanitizeResources: false,
   sanitizeOps: false,