Refactor unit tests (#71)

* Update logout routes

* Refactor DELETE tests

backend/app/api/user/logout/route.test.ts

@@ -0,0 +1,31 @@
+import { verifySession } from '@/lib/session';
+import { GET } from './route';
+
+jest.mock('@/lib/session', () => ({
+  verifySession: jest.fn(),
+  deleteSession: jest.fn(),
+}));
+
+const mockVerifySession = verifySession as jest.Mock;
+
+test('Logout succeeds for valid session', async () => {
+  mockVerifySession.mockResolvedValue({
+    isAuth: true,
+    uid: 2,
+  });
+
+  const res = await GET();
+  const data = await res.json();
+
+  expect(data.success).toBeTruthy();
+});
+
+test('Logout fails for invalid session', async () => {
+  mockVerifySession.mockResolvedValue({
+    isAuth: false,
+  });
+  const res = await GET();
+  const data = await res.json();
+
+  expect(data.error).toEqual('User not authenticated');
+});

backend/app/api/user/auth.test.ts → backend/app/api/user/route.test.ts

@@ -1,16 +1,21 @@
+import { verifySession } from '@/lib/session';
 import { prismaMock } from '@/__test__/singleton';
 import { User } from '@prisma/client';
 import { DELETE } from './route';
 
 jest.mock('@/lib/session', () => ({
-  verifySession: jest.fn(() => ({
-    isAuth: true,
-    uid: 2,
-  })),
-  deleteSession: jest.fn(() => {}),
+  verifySession: jest.fn(),
+  deleteSession: jest.fn(),
 }));
 
+const mockVerifySession = verifySession as jest.Mock;
+
 test('Delete succeeds if user is authenticated', async () => {
+  mockVerifySession.mockResolvedValue({
+    isAuth: true,
+    uid: 1,
+  });
+
   const temporaryUser: User = {
     id: 1,
     username: 'test',
@@ -24,3 +29,15 @@ test('Delete succeeds if user is authenticated', async () => {
 
   expect(data.success).toBe(true);
 });
+
+test('Delete fails for an invalid session', async () => {
+  mockVerifySession.mockResolvedValue({
+    isAuth: false,
+  });
+
+  const res = await DELETE();
+  const data = await res.json();
+
+  expect(res.status).toEqual(400);
+  expect(data.error).toBe('Invalid session');
+});