feat(backend): Repositories toegevoegd, databank unit-testbaar gemaakt.
This commit is contained in:
		
							parent
							
								
									2657e49ad6
								
							
						
					
					
						commit
						374de3b21a
					
				
					 15 changed files with 1672 additions and 45 deletions
				
			
		
							
								
								
									
										3
									
								
								backend/.env.test
									
										
									
									
									
										Normal file
									
								
							
							
						
						
									
										3
									
								
								backend/.env.test
									
										
									
									
									
										Normal file
									
								
							|  | @ -0,0 +1,3 @@ | ||||||
|  | PORT=3000 | ||||||
|  | DWENGO_DB_UPDATE=true | ||||||
|  | DWENGO_DB_NAME=":memory:" | ||||||
|  | @ -1,2 +0,0 @@ | ||||||
| -- Create the database |  | ||||||
| CREATE DATABASE dwengo; |  | ||||||
|  | @ -11,12 +11,13 @@ | ||||||
|         "format": "prettier --write src/", |         "format": "prettier --write src/", | ||||||
|         "format-check": "prettier --check src/", |         "format-check": "prettier --check src/", | ||||||
|         "lint": "eslint . --fix", |         "lint": "eslint . --fix", | ||||||
|         "test:unit": "vitest --run" |         "test:unit": "vitest" | ||||||
|     }, |     }, | ||||||
|     "dependencies": { |     "dependencies": { | ||||||
|         "@mikro-orm/core": "^6.4.6", |         "@mikro-orm/core": "6.4.6", | ||||||
|         "@mikro-orm/postgresql": "^6.4.6", |         "@mikro-orm/postgresql": "6.4.6", | ||||||
|         "@mikro-orm/reflection": "^6.4.6", |         "@mikro-orm/sqlite": "6.4.6", | ||||||
|  |         "@mikro-orm/reflection": "6.4.6", | ||||||
|         "dotenv": "^16.4.7", |         "dotenv": "^16.4.7", | ||||||
|         "express": "^5.0.1", |         "express": "^5.0.1", | ||||||
|         "uuid": "^11.1.0" |         "uuid": "^11.1.0" | ||||||
|  |  | ||||||
|  | @ -1,5 +1,5 @@ | ||||||
| import express, { Express, Response } from 'express'; | import express, { Express, Response } from 'express'; | ||||||
| import initORM from './orm.js'; | import { initORM } from './orm.js'; | ||||||
| 
 | 
 | ||||||
| const app: Express = express(); | const app: Express = express(); | ||||||
| const port: string | number = process.env.PORT || 3000; | const port: string | number = process.env.PORT || 3000; | ||||||
|  | @ -19,4 +19,4 @@ async function startServer() { | ||||||
|     }); |     }); | ||||||
| } | } | ||||||
| 
 | 
 | ||||||
| startServer(); | await startServer(); | ||||||
|  |  | ||||||
							
								
								
									
										17
									
								
								backend/src/data/dwengo-entity-repository.ts
									
										
									
									
									
										Normal file
									
								
							
							
						
						
									
										17
									
								
								backend/src/data/dwengo-entity-repository.ts
									
										
									
									
									
										Normal file
									
								
							|  | @ -0,0 +1,17 @@ | ||||||
|  | import {EntityRepository, FilterQuery} from "@mikro-orm/core"; | ||||||
|  | 
 | ||||||
|  | export abstract class DwengoEntityRepository<T extends object> extends EntityRepository<T> { | ||||||
|  |     public async save(entity: T) { | ||||||
|  |         let em = this.getEntityManager(); | ||||||
|  |         em.persist(entity); | ||||||
|  |         await em.flush(); | ||||||
|  |     } | ||||||
|  |     public async deleteWhere(query: FilterQuery<T>) { | ||||||
|  |         let toDelete = await this.findOne(query); | ||||||
|  |         let em = this.getEntityManager(); | ||||||
|  |         if (toDelete) { | ||||||
|  |             em.remove(toDelete); | ||||||
|  |             await em.flush(); | ||||||
|  |         } | ||||||
|  |     } | ||||||
|  | } | ||||||
							
								
								
									
										11
									
								
								backend/src/data/users/student-repository.ts
									
										
									
									
									
										Normal file
									
								
							
							
						
						
									
										11
									
								
								backend/src/data/users/student-repository.ts
									
										
									
									
									
										Normal file
									
								
							|  | @ -0,0 +1,11 @@ | ||||||
|  | import {DwengoEntityRepository} from "../dwengo-entity-repository"; | ||||||
|  | import {Student} from "../../entities/users/student.entity"; | ||||||
|  | 
 | ||||||
|  | export class StudentRepository extends DwengoEntityRepository<Student> { | ||||||
|  |     public findByUsername(username: string): Promise<Student | null> { | ||||||
|  |         return this.findOne({username: username}); | ||||||
|  |     } | ||||||
|  |     public deleteByUsername(username: string): Promise<void> { | ||||||
|  |         return this.deleteWhere({username: username}); | ||||||
|  |     } | ||||||
|  | } | ||||||
|  | @ -2,12 +2,17 @@ import {User} from "./user.entity"; | ||||||
| import {Collection, Entity, ManyToMany} from '@mikro-orm/core'; | import {Collection, Entity, ManyToMany} from '@mikro-orm/core'; | ||||||
| import {Class} from "../classes/class.entity"; | import {Class} from "../classes/class.entity"; | ||||||
| import {Group} from "../assigments/group.entity"; | import {Group} from "../assigments/group.entity"; | ||||||
|  | import {StudentRepository} from "../../data/users/student-repository"; | ||||||
| 
 | 
 | ||||||
| @Entity() | @Entity({repository: () => StudentRepository}) | ||||||
| export class Student extends User { | export class Student extends User { | ||||||
|     @ManyToMany(() => Class) |     @ManyToMany(() => Class) | ||||||
|     classes!: Collection<Class>; |     classes!: Collection<Class>; | ||||||
| 
 | 
 | ||||||
|     @ManyToMany(() => Group) |     @ManyToMany(() => Group) | ||||||
|     groups!: Collection<Group>; |     groups!: Collection<Group>; | ||||||
|  | 
 | ||||||
|  |     constructor(public username: string, public firstName: string, public lastName: string) { | ||||||
|  |         super(); | ||||||
|  |     } | ||||||
| } | } | ||||||
|  |  | ||||||
|  | @ -1,17 +1,35 @@ | ||||||
| import {Options} from '@mikro-orm/core'; | import {Options} from '@mikro-orm/core'; | ||||||
| import {PostgreSqlDriver} from "@mikro-orm/postgresql"; | import {PostgreSqlDriver} from "@mikro-orm/postgresql"; | ||||||
| import {EnvVars, getEnvVar, getNumericEnvVar} from "./util/envvars"; | import {EnvVars, getEnvVar, getNumericEnvVar} from "./util/envvars"; | ||||||
|  | import {SqliteDriver} from "@mikro-orm/sqlite"; | ||||||
| 
 | 
 | ||||||
| const config: Options = { | const entities = ['dist/**/*.entity.js']; | ||||||
|     driver: PostgreSqlDriver, | const entitiesTs = ['src/**/*.entity.ts']; | ||||||
|     host: getEnvVar(EnvVars.DbHost, {required: true}), | function config(testingMode: boolean = false): Options { | ||||||
|     port: getNumericEnvVar(EnvVars.DbPort, {defaultValue: 5432}), |     if (testingMode) { | ||||||
|     dbName: getEnvVar(EnvVars.DbName, {defaultValue: "dwengo"}), |         return { | ||||||
|     user: getEnvVar(EnvVars.DbUsername, {required: true}), |             driver: SqliteDriver, | ||||||
|     password: getEnvVar(EnvVars.DbPassword, {required: true}), |             dbName: getEnvVar(EnvVars.DbName), | ||||||
|     entities: ['dist/**/*.entity.js'], |             entities: entities, | ||||||
|     entitiesTs: ['src/**/*.entity.ts'], |             entitiesTs: entitiesTs, | ||||||
|     debug: true, | 
 | ||||||
| }; |             // Workaround: vitest: `TypeError: Unknown file extension ".ts"` (ERR_UNKNOWN_FILE_EXTENSION)
 | ||||||
|  |             // (see https://mikro-orm.io/docs/guide/project-setup#testing-the-endpoint)
 | ||||||
|  |             dynamicImportProvider: id => import(id) | ||||||
|  |         }; | ||||||
|  |     } else { | ||||||
|  |         return { | ||||||
|  |             driver: PostgreSqlDriver, | ||||||
|  |             host: getEnvVar(EnvVars.DbHost), | ||||||
|  |             port: getNumericEnvVar(EnvVars.DbPort), | ||||||
|  |             dbName: getEnvVar(EnvVars.DbName), | ||||||
|  |             user: getEnvVar(EnvVars.DbUsername), | ||||||
|  |             password: getEnvVar(EnvVars.DbPassword), | ||||||
|  |             entities: entities, | ||||||
|  |             entitiesTs: entitiesTs, | ||||||
|  |             debug: true | ||||||
|  |         }; | ||||||
|  |     } | ||||||
|  | } | ||||||
| 
 | 
 | ||||||
| export default config; | export default config; | ||||||
|  |  | ||||||
|  | @ -1,10 +1,10 @@ | ||||||
| import { MikroORM } from '@mikro-orm/core'; | import {AnyEntity, EntityName, EntityRepository, MikroORM} from '@mikro-orm/core'; | ||||||
| import config from './mikro-orm.config.js'; | import config from './mikro-orm.config.js'; | ||||||
| import {EnvVars, getEnvVar} from "./util/envvars"; | import {EnvVars, getEnvVar} from "./util/envvars"; | ||||||
| 
 | 
 | ||||||
| export default async function initORM() { | let orm: MikroORM | undefined; | ||||||
|     const orm = await MikroORM.init(config); | export async function initORM(testingMode: boolean = false) { | ||||||
| 
 |     orm = await MikroORM.init(config(testingMode)); | ||||||
|     // Update the database scheme if necessary and enabled.
 |     // Update the database scheme if necessary and enabled.
 | ||||||
|     if (getEnvVar(EnvVars.DbUpdate)) { |     if (getEnvVar(EnvVars.DbUpdate)) { | ||||||
|         await orm.schema.updateSchema(); |         await orm.schema.updateSchema(); | ||||||
|  | @ -17,3 +17,10 @@ export default async function initORM() { | ||||||
|         } |         } | ||||||
|     } |     } | ||||||
| } | } | ||||||
|  | 
 | ||||||
|  | export function getRepository<T extends AnyEntity>(entityName: EntityName<T>): EntityRepository<T> { | ||||||
|  |     if (orm === undefined) { | ||||||
|  |         throw new Error("ORM is not initialized yet"); | ||||||
|  |     } | ||||||
|  |     return orm.em.fork().getRepository(entityName); | ||||||
|  | } | ||||||
|  |  | ||||||
|  | @ -1,15 +1,15 @@ | ||||||
| const PREFIX = "DWENGO_"; | const PREFIX = "DWENGO_"; | ||||||
| const DB_PREFIX = PREFIX + "DB_"; | const DB_PREFIX = PREFIX + "DB_"; | ||||||
| 
 | 
 | ||||||
| type EnvVar = {[key: string]: {name: string, required?: boolean, defaultValue?: boolean}}; | type EnvVar = {key: string, required?: boolean, defaultValue?: any} | ||||||
| 
 | 
 | ||||||
| export const EnvVars: EnvVar = { | export const EnvVars: {[key: string]: EnvVar} = { | ||||||
|     DbHost: {key: DB_PREFIX + "HOST", required: true}, |     "DbHost": {key: DB_PREFIX + "HOST", required: true}, | ||||||
|     DbPort: {key: DB_PREFIX + "PORT", defaultValue: 5432}, |     "DbPort": {key: DB_PREFIX + "PORT", defaultValue: 5432}, | ||||||
|     DbName: {key: DB_PREFIX + "NAME", defaultValue: "dwengo"}, |     "DbName": {key: DB_PREFIX + "NAME", defaultValue: "dwengo"}, | ||||||
|     DbUsername: {key: DB_PREFIX + "USERNAME", required: true}, |     "DbUsername": {key: DB_PREFIX + "USERNAME", required: true}, | ||||||
|     DbPassword: {key: DB_PREFIX + "PASSWORD", required: true}, |     "DbPassword": {key: DB_PREFIX + "PASSWORD", required: true}, | ||||||
|     DbUpdate: {key: DB_PREFIX + "UPDATE", defaultValue: false}, |     "DbUpdate": {key: DB_PREFIX + "UPDATE", defaultValue: false}, | ||||||
| } as const; | } as const; | ||||||
| 
 | 
 | ||||||
| /** | /** | ||||||
|  |  | ||||||
							
								
								
									
										33
									
								
								backend/tests/data/users.test.ts
									
										
									
									
									
										Normal file
									
								
							
							
						
						
									
										33
									
								
								backend/tests/data/users.test.ts
									
										
									
									
									
										Normal file
									
								
							|  | @ -0,0 +1,33 @@ | ||||||
|  | import {initializeTests} from "../testutils" | ||||||
|  | import {Student} from "../../src/entities/users/student.entity"; | ||||||
|  | import {describe, it, expect, beforeAll} from "vitest"; | ||||||
|  | import {getRepository} from "../../src/orm"; | ||||||
|  | import {StudentRepository} from "../../src/data/users/student-repository"; | ||||||
|  | 
 | ||||||
|  | const username = "teststudent"; | ||||||
|  | const firstName = "John"; | ||||||
|  | const lastName = "Doe"; | ||||||
|  | describe("StudentRepository", () => { | ||||||
|  |     let studentRepository: StudentRepository; | ||||||
|  | 
 | ||||||
|  |     beforeAll(async () => { | ||||||
|  |         await initializeTests() | ||||||
|  |         studentRepository = getRepository(Student) as StudentRepository; | ||||||
|  |     }); | ||||||
|  | 
 | ||||||
|  |     it("should return the queried student after he was added", async () => { | ||||||
|  |         await studentRepository.insert(new Student(username, firstName, lastName)); | ||||||
|  | 
 | ||||||
|  |         let retrievedStudent = await studentRepository.findByUsername(username); | ||||||
|  |         expect(retrievedStudent).toBeTruthy(); | ||||||
|  |         expect(retrievedStudent?.firstName).toBe(firstName); | ||||||
|  |         expect(retrievedStudent?.lastName).toBe(lastName); | ||||||
|  |     }); | ||||||
|  | 
 | ||||||
|  |     it("should no longer return the queried student after he was removed again", async () => { | ||||||
|  |         await studentRepository.deleteByUsername(username); | ||||||
|  | 
 | ||||||
|  |         let retrievedStudent = await studentRepository.findByUsername(username); | ||||||
|  |         expect(retrievedStudent).toBeNull(); | ||||||
|  |     }); | ||||||
|  | }); | ||||||
|  | @ -4,6 +4,6 @@ describe("Sample test", () => { | ||||||
|     it("should sum to 2", () => { |     it("should sum to 2", () => { | ||||||
|         const expected = 2; |         const expected = 2; | ||||||
|         const result = 1 + 1; |         const result = 1 + 1; | ||||||
|         expect(result).toBe(expected); |         expect(result).equals(expected); | ||||||
|     }); |     }); | ||||||
| }) | }) | ||||||
|  |  | ||||||
							
								
								
									
										7
									
								
								backend/tests/testutils.ts
									
										
									
									
									
										Normal file
									
								
							
							
						
						
									
										7
									
								
								backend/tests/testutils.ts
									
										
									
									
									
										Normal file
									
								
							|  | @ -0,0 +1,7 @@ | ||||||
|  | import {initORM} from "../src/orm"; | ||||||
|  | import dotenv from "dotenv"; | ||||||
|  | 
 | ||||||
|  | export async function initializeTests() { | ||||||
|  |     dotenv.config({path: ".env.test"}); | ||||||
|  |     await initORM(true); | ||||||
|  | } | ||||||
							
								
								
									
										8
									
								
								backend/vitest.config.ts
									
										
									
									
									
										Normal file
									
								
							
							
						
						
									
										8
									
								
								backend/vitest.config.ts
									
										
									
									
									
										Normal file
									
								
							|  | @ -0,0 +1,8 @@ | ||||||
|  | import { defineConfig } from 'vitest/config'; | ||||||
|  | 
 | ||||||
|  | export default defineConfig({ | ||||||
|  |     test: { | ||||||
|  |         environment: 'node', | ||||||
|  |         globals: true | ||||||
|  |     } | ||||||
|  | }); | ||||||
							
								
								
									
										1543
									
								
								package-lock.json
									
										
									
										generated
									
									
									
								
							
							
						
						
									
										1543
									
								
								package-lock.json
									
										
									
										generated
									
									
									
								
							
										
											
												File diff suppressed because it is too large
												Load diff
											
										
									
								
							
		Reference in a new issue
	
	 Gerald Schmittinger
						Gerald Schmittinger