Ty/scrum 180 backend integration tests (#155)

Co-authored-by: Austin-X <[email protected]>
Co-authored-by: minhhaitran08 <[email protected]>
Co-authored-by: kevin-lann <[email protected]>
Co-authored-by: Masahisa Sekita <[email protected]>
Co-authored-by: Dmitriy Prokopchuk <[email protected]>
Co-authored-by: Austin-X <[email protected]>
Co-authored-by: dawangk <[email protected]>
Co-authored-by: kevin-lann <[email protected]>
Co-authored-by: MasahisaSekita <[email protected]>
Co-authored-by: minhhaitran08 <[email protected]>

Author: 7 people

course-matrix/backend/__tests__/integration-tests/timetableGenerate.test.ts

@@ -1,6 +1,7 @@
-import request from "supertest";
+import { afterAll, describe, expect, it, jest } from "@jest/globals";
 import { Request, Response } from "express";
-import { jest, describe, it, expect, afterAll } from "@jest/globals";
+import request from "supertest";
+
 import app, { server } from "../../src/index";
 
 jest.mock("@ai-sdk/openai", () => ({
@@ -167,16 +168,18 @@ describe("Authentication API", () => {
 
   describe("POST /auth/login", () => {
     it("should return 200 and a token for valid credentials", async () => {
-      const response = await request(app)
-        .post("/auth/login")
-        .send({ email: "validUser", password: "validPassword" });
+      const response = await request(app).post("/auth/login").send({
+        email: "validUser",
+        password: "validPassword",
+      });
       expect(response.status).toBe(200);
       expect(response.body).toHaveProperty("token");
     });
     it("should return 401 for invalid credentials", async () => {
-      const response = await request(app)
-        .post("/auth/login")
-        .send({ email: "invalidUser", password: "wrongPassword" });
+      const response = await request(app).post("/auth/login").send({
+        email: "invalidUser",
+        password: "wrongPassword",
+      });
       expect(response.status).toBe(401);
       expect(response.body).toHaveProperty("error", "Invalid credentials");
     });
@@ -237,7 +240,9 @@ describe("Authentication API", () => {
     it("should return 200 and a success message when email is provided", async () => {
       const response = await request(app)
         .post("/auth/request-password-reset")
-        .send({ email: "[email protected]" });
+        .send({
+          email: "[email protected]",
+        });
       expect(response.status).toBe(200);
       expect(response.body).toHaveProperty(
         "message",
@@ -256,9 +261,10 @@ describe("Authentication API", () => {
 
   describe("POST /auth/reset-password", () => {
     it("should return 200 and a success message when password and token are provided", async () => {
-      const response = await request(app)
-        .post("/auth/reset-password")
-        .send({ password: "newPassword123", token: "validToken" });
+      const response = await request(app).post("/auth/reset-password").send({
+        password: "newPassword123",
+        token: "validToken",
+      });
       expect(response.status).toBe(200);
       expect(response.body).toHaveProperty(
         "message",
@@ -267,9 +273,9 @@ describe("Authentication API", () => {
     });
 
     it("should return 400 if password or token is missing", async () => {
-      const response = await request(app)
-        .post("/auth/reset-password")
-        .send({ password: "newPassword123" });
+      const response = await request(app).post("/auth/reset-password").send({
+        password: "newPassword123",
+      });
       expect(response.status).toBe(400);
       expect(response.body).toHaveProperty(
         "error",
@@ -280,9 +286,9 @@ describe("Authentication API", () => {
 
   describe("DELETE /auth/accountDelete", () => {
     it("should return 200 and a success message when userId is provided", async () => {
-      const response = await request(app)
-        .delete("/auth/accountDelete")
-        .send({ userId: "12345" });
+      const response = await request(app).delete("/auth/accountDelete").send({
+        userId: "12345",
+      });
       expect(response.status).toBe(200);
       expect(response.body).toHaveProperty(
         "message",

course-matrix/backend/__tests__/unit-tests/auth.test.ts

@@ -0,0 +1,60 @@
+import { describe, expect, it, jest, test } from "@jest/globals";
+
+import { supabase } from "../../src/db/setupDb";
+import getOfferings from "../../src/services/getOfferings";
+
+jest.mock("../../src/db/setupDb", () => ({
+  supabase: {
+    schema: jest.fn(),
+  },
+}));
+
+type SupabaseQueryResult = Promise<{ data: any; error: any }>;
+
+describe("getOfferings", () => {
+  it("returns offering data for a valid course and semester", async () => {
+    const mockData = [
+      {
+        id: 1,
+        course_id: 123,
+        meeting_section: "L01",
+        offering: "Fall 2025",
+        day: "Mon",
+        start: "10:00",
+        end: "11:00",
+        location: "Room 101",
+        current: 30,
+        max: 40,
+        is_waitlisted: false,
+        delivery_mode: "In-Person",
+        instructor: "Dr. Smith",
+        notes: "",
+        code: "ABC123",
+      },
+    ];
+
+    // Build the method chain mock
+    const eqMock2 = jest.fn<() => SupabaseQueryResult>().mockResolvedValue({
+      data: mockData,
+      error: null,
+    });
+    const eqMock1 = jest.fn(() => ({ eq: eqMock2 }));
+    const selectMock = jest.fn(() => ({ eq: eqMock1 }));
+    const fromMock = jest.fn(() => ({ select: selectMock }));
+    const schemaMock = jest.fn(() => ({ from: fromMock }));
+
+    // Replace supabase.schema with our chain
+    (supabase.schema as jest.Mock).mockImplementation(schemaMock);
+
+    // Act
+    const result = await getOfferings(123, "Fall 2025");
+
+    // Assert
+    expect(schemaMock).toHaveBeenCalledWith("course");
+    expect(fromMock).toHaveBeenCalledWith("offerings");
+    expect(selectMock).toHaveBeenCalled();
+    expect(eqMock1).toHaveBeenCalledWith("course_id", 123);
+    expect(eqMock2).toHaveBeenCalledWith("offering", "Fall 2025");
+    expect(result).toEqual(mockData);
+  });
+});