feat: tests voor backend/controllers/classes.ts geimplementeerd (FALEN NOG DOOR FOUTEN IN CODE)
This commit is contained in:
		
							parent
							
								
									30ec73a88d
								
							
						
					
					
						commit
						136be2d937
					
				
					 1 changed files with 149 additions and 0 deletions
				
			
		
							
								
								
									
										149
									
								
								backend/tests/controllers/classes.test.ts
									
										
									
									
									
										Normal file
									
								
							
							
						
						
									
										149
									
								
								backend/tests/controllers/classes.test.ts
									
										
									
									
									
										Normal file
									
								
							|  | @ -0,0 +1,149 @@ | ||||||
|  | import { setupTestApp } from '../setup-tests.js'; | ||||||
|  | import { describe, it, expect, beforeAll, beforeEach, vi, Mock } from 'vitest'; | ||||||
|  | import { createClassHandler, getClassHandler, getClassStudentsHandler, getTeacherInvitationsHandler } from '../../src/controllers/classes.js'; | ||||||
|  | import { Request, Response } from 'express'; | ||||||
|  | import { getClassTeacherInvitations } from '../../src/services/class.js'; | ||||||
|  | 
 | ||||||
|  | async function fetchClass(id: string) { | ||||||
|  |     const data = await fetch(`localhost:3000/class/${id}`); | ||||||
|  |     return data; | ||||||
|  | } | ||||||
|  | 
 | ||||||
|  | vi.mock('./getClass', () => ({ | ||||||
|  |     getClass: vi.fn(), | ||||||
|  | })); | ||||||
|  | 
 | ||||||
|  | describe('Class controllers', () => { | ||||||
|  |     let req: Partial<Request>; | ||||||
|  |     let res: Partial<Response>; | ||||||
|  | 
 | ||||||
|  |     let jsonMock: Mock; | ||||||
|  |     let statusMock: Mock; | ||||||
|  | 
 | ||||||
|  |     beforeAll(async () => { | ||||||
|  |         await setupTestApp(); | ||||||
|  |     }); | ||||||
|  |    | ||||||
|  |     beforeEach(async () =>  { | ||||||
|  |         jsonMock = vi.fn(); | ||||||
|  |         statusMock = vi.fn().mockReturnThis(); | ||||||
|  | 
 | ||||||
|  |         res = { | ||||||
|  |             json: jsonMock, | ||||||
|  |             status: statusMock, | ||||||
|  |         }; | ||||||
|  |     }); | ||||||
|  | 
 | ||||||
|  |     it('should return 404 and error if class is not found', async () => { | ||||||
|  |         req = { | ||||||
|  |             params: { id: 'doesnotexist'}, | ||||||
|  |         } | ||||||
|  | 
 | ||||||
|  |         await getClassHandler(req as Request, res as Response); | ||||||
|  |      | ||||||
|  |         expect(statusMock).toHaveBeenCalledWith(404); | ||||||
|  |         expect(jsonMock).toHaveBeenCalledWith({ error: 'Class not found' }); | ||||||
|  |     }); | ||||||
|  | 
 | ||||||
|  |     it('should return 200 if class is not found', async () => { | ||||||
|  |         req = { | ||||||
|  |             params: { id: 'id01'}, | ||||||
|  |         } | ||||||
|  | 
 | ||||||
|  |         await getClassHandler(req as Request, res as Response); | ||||||
|  |      | ||||||
|  |         // status can either not be called or called with code 200
 | ||||||
|  |         expect( | ||||||
|  |             statusMock.mock.calls.length === 0 || statusMock.mock.calls.some(([arg]) => arg === 200) | ||||||
|  |         ).toBe(true); | ||||||
|  |     }); | ||||||
|  | 
 | ||||||
|  |     it('should return 201 for creating a new class', async () => { | ||||||
|  |         req = { | ||||||
|  |             body: { displayName: 'coolenieuweklas' }, | ||||||
|  |         }; | ||||||
|  |          | ||||||
|  |         await createClassHandler(req as Request, res as Response); | ||||||
|  |          | ||||||
|  |         expect(statusMock).toHaveBeenCalledWith(201); | ||||||
|  |         // TODO: return json should be a classDTO and not named (fixed in #130)
 | ||||||
|  |         //expect(jsonMock).toHaveBeenCalledWith();
 | ||||||
|  |          | ||||||
|  |         // TODO: check if class is actually added to db
 | ||||||
|  |     }); | ||||||
|  |     it.todo('return json should be a classDTO and not named (fixed in #130)') | ||||||
|  |     it.todo('check if class is actually added to db'); | ||||||
|  | 
 | ||||||
|  |     it('should return 400 for trying to create a class without name', async () => { | ||||||
|  |         req = { | ||||||
|  |             body: {}, | ||||||
|  |         }; | ||||||
|  | 
 | ||||||
|  |         await createClassHandler(req as Request, res as Response); | ||||||
|  | 
 | ||||||
|  |         expect(statusMock).toHaveBeenCalledWith(400); | ||||||
|  |         expect(jsonMock).toHaveBeenCalledWith({ error: 'Missing one or more required fields: displayName' }); | ||||||
|  |     }); | ||||||
|  | 
 | ||||||
|  |     it('should return a list of students when calling getClassStudentsHandler', async () => { | ||||||
|  |         req = { | ||||||
|  |             params: { id: 'id01' }, | ||||||
|  |             query: {}, | ||||||
|  |         }; | ||||||
|  | 
 | ||||||
|  |         await getClassStudentsHandler(req as Request, res as Response); | ||||||
|  | 
 | ||||||
|  |         expect(jsonMock).toHaveBeenCalledWith({ students: [ | ||||||
|  |             'DireStraits', | ||||||
|  |             'Nirvana', | ||||||
|  |             'Noordkaap', | ||||||
|  |             'PinkFloyd', | ||||||
|  |             'SmashingPumpkins', | ||||||
|  |             'TheDoors', | ||||||
|  |             'Tool' | ||||||
|  |         ]}); | ||||||
|  |     }); | ||||||
|  | 
 | ||||||
|  |     it('should return 404 not found when calling getClassStudentsHandler on a non-existent class', async () => { | ||||||
|  |         req = { | ||||||
|  |             params: { id: 'doesnotexist' }, | ||||||
|  |             query: {}, | ||||||
|  |         }; | ||||||
|  | 
 | ||||||
|  |         await getClassStudentsHandler(req as Request, res as Response); | ||||||
|  |          | ||||||
|  |         // will fail until code is fixed
 | ||||||
|  |         expect(statusMock).toHaveBeenCalledWith(404); | ||||||
|  |         expect(jsonMock).toHaveBeenCalledWith({ error: 'Class not found' }); | ||||||
|  |     }); | ||||||
|  | 
 | ||||||
|  |     it('should return 200 and a list of teacher-invitations', async () => { | ||||||
|  |         req = { | ||||||
|  |             params: { id: 'id01' }, | ||||||
|  |             query: {}, | ||||||
|  |         }; | ||||||
|  | 
 | ||||||
|  |         await getTeacherInvitationsHandler(req as Request, res as Response); | ||||||
|  | 
 | ||||||
|  |         expect(jsonMock).toHaveBeenCalledWith({"invitations": [ | ||||||
|  |             { | ||||||
|  |             "class": "id01", | ||||||
|  |             "receiver": "LimpBizkit", | ||||||
|  |             "sender": "FooFighters", | ||||||
|  |             } | ||||||
|  |         ]}); | ||||||
|  |     }); | ||||||
|  | 
 | ||||||
|  |     it('should return 404 not found when calling teacher-invitations on a non-existent class', async () => { | ||||||
|  |         req = { | ||||||
|  |             params: { id: 'doesnotexist' }, | ||||||
|  |             query: {}, | ||||||
|  |         }; | ||||||
|  | 
 | ||||||
|  |         await getTeacherInvitationsHandler(req as Request, res as Response); | ||||||
|  | 
 | ||||||
|  |         // will fail until code is fixed
 | ||||||
|  |         expect(statusMock).toHaveBeenCalledWith(404); | ||||||
|  |         expect(jsonMock).toHaveBeenCalledWith({ error: 'Class not found' }); | ||||||
|  |     }); | ||||||
|  | }) | ||||||
		Reference in a new issue
	
	 Adriaan Jacquet
						Adriaan Jacquet