BC-6082 Add Logger to KNL-Module

apps/server/src/modules/class/class.module.ts

@@ -1,8 +1,10 @@
 import { Module } from '@nestjs/common';
+import { LoggerModule } from '@src/core/logger';
 import { ClassService } from './service';
 import { ClassesRepo } from './repo';
 
 @Module({
+	imports: [LoggerModule],
 	providers: [ClassService, ClassesRepo],
 	exports: [ClassService],
 })

apps/server/src/modules/class/service/class.service.spec.ts

@@ -4,6 +4,7 @@ import { InternalServerErrorException } from '@nestjs/common';
 import { Test, TestingModule } from '@nestjs/testing';
 import { EntityId } from '@shared/domain/types';
 import { setupEntities } from '@shared/testing';
+import { LegacyLogger } from '@src/core/logger';
 import { Class } from '../domain';
 import { classFactory } from '../domain/testing';
 import { classEntityFactory } from '../entity/testing';
@@ -24,6 +25,10 @@ describe(ClassService.name, () => {
 					provide: ClassesRepo,
 					useValue: createMock<ClassesRepo>(),
 				},
+				{
+					provide: LegacyLogger,
+					useValue: createMock<LegacyLogger>(),
+				},
 			],
 		}).compile();
 

apps/server/src/modules/class/service/class.service.ts

@@ -1,11 +1,14 @@
 import { Injectable, InternalServerErrorException } from '@nestjs/common';
 import { EntityId } from '@shared/domain/types';
+import { LegacyLogger } from '@src/core/logger';
 import { Class } from '../domain';
 import { ClassesRepo } from '../repo';
 
 @Injectable()
 export class ClassService {
-	constructor(private readonly classesRepo: ClassesRepo) {}
+	constructor(private readonly classesRepo: ClassesRepo, private readonly logger: LegacyLogger) {
+		this.logger.setContext(ClassService.name);
+	}
 
 	public async findClassesForSchool(schoolId: EntityId): Promise<Class[]> {
 		const classes: Class[] = await this.classesRepo.findAllBySchoolId(schoolId);
@@ -19,8 +22,9 @@ export class ClassService {
 		return classes;
 	}
 
-	// FIXME There is no usage of this method
 	public async deleteUserDataFromClasses(userId: EntityId): Promise<number> {
+		this.logger.log({ action: 'Deleting data from Classes for user ', userId });
+
 		if (!userId) {
 			throw new InternalServerErrorException('User id is missing');
 		}
@@ -35,6 +39,7 @@ export class ClassService {
 		});
 
 		await this.classesRepo.updateMany(updatedClasses);
+		this.logger.log({ action: 'Deleted data from Classes for user ', userId });
 
 		return updatedClasses.length;
 	}

apps/server/src/modules/files/service/files.service.spec.ts

@@ -2,6 +2,7 @@ import { ObjectId } from '@mikro-orm/mongodb';
 import { Test, TestingModule } from '@nestjs/testing';
 import { createMock, DeepMocked } from '@golevelup/ts-jest';
 import { setupEntities } from '@shared/testing';
+import { LegacyLogger } from '@src/core/logger';
 import { FilesService } from './files.service';
 import { FilesRepo } from '../repo';
 import { fileEntityFactory, filePermissionEntityFactory } from '../entity/testing';
@@ -20,6 +21,10 @@ describe(FilesService.name, () => {
 					provide: FilesRepo,
 					useValue: createMock<FilesRepo>(),
 				},
+				{
+					provide: LegacyLogger,
+					useValue: createMock<LegacyLogger>(),
+				},
 			],
 		}).compile();
 

apps/server/src/modules/files/service/files.service.ts

@@ -1,17 +1,21 @@
 import { Injectable } from '@nestjs/common';
 import { EntityId } from '@shared/domain/types';
+import { LegacyLogger } from '@src/core/logger';
 import { FileEntity } from '../entity';
 import { FilesRepo } from '../repo';
 
 @Injectable()
 export class FilesService {
-	constructor(private readonly repo: FilesRepo) {}
+	constructor(private readonly repo: FilesRepo, private readonly logger: LegacyLogger) {
+		this.logger.setContext(FilesService.name);
+	}
 
 	async findFilesAccessibleByUser(userId: EntityId): Promise<FileEntity[]> {
 		return this.repo.findByPermissionRefId(userId);
 	}
 
 	async removeUserPermissionsToAnyFiles(userId: EntityId): Promise<number> {
+		this.logger.log({ action: 'Deleting Permissions To Any Files for user ', userId });
 		const entities = await this.repo.findByPermissionRefId(userId);
 
 		if (entities.length === 0) {
@@ -22,6 +26,8 @@ export class FilesService {
 
 		await this.repo.save(entities);
 
+		this.logger.log({ action: 'Deleted Permissions To Any Files for user ', userId });
+
 		return entities.length;
 	}
 
@@ -30,6 +36,7 @@ export class FilesService {
 	}
 
 	async markFilesOwnedByUserForDeletion(userId: EntityId): Promise<number> {
+		this.logger.log({ action: 'Marking Files For Deletion Owned By user ', userId });
 		const entities = await this.repo.findByOwnerUserId(userId);
 
 		if (entities.length === 0) {
@@ -40,6 +47,8 @@ export class FilesService {
 
 		await this.repo.save(entities);
 
+		this.logger.log({ action: 'Marked Files For Deletion Owned By user ', userId });
+
 		return entities.length;
 	}
 }

apps/server/src/modules/learnroom/service/course.service.spec.ts

@@ -4,6 +4,7 @@
 import { CourseRepo, UserRepo } from '@shared/repo';
 import { courseFactory, setupEntities, userFactory } from '@shared/testing';
 import { CourseService } from './course.service';
+import { LegacyLogger } from '@src/core/logger';
 
 describe('CourseService', () => {
 	let module: TestingModule;
@@ -24,6 +25,10 @@
 					provide: CourseRepo,
 					useValue: createMock<CourseRepo>(),
 				},
+				{
+					provide: LegacyLogger,
+					useValue: createMock<LegacyLogger>(),
+				},
 			],
 		}).compile();
 		courseRepo = module.get(CourseRepo);

apps/server/src/modules/learnroom/service/course.service.ts

@@ -2,10 +2,13 @@ import { Injectable } from '@nestjs/common';
 import { Course } from '@shared/domain/entity';
 import { Counted, EntityId } from '@shared/domain/types';
 import { CourseRepo } from '@shared/repo';
+import { LegacyLogger } from '@src/core/logger';
 
 @Injectable()
 export class CourseService {
-	constructor(private readonly repo: CourseRepo) {}
+	constructor(private readonly repo: CourseRepo, private readonly logger: LegacyLogger) {
+		this.logger.setContext(CourseService.name);
+	}
 
 	async findById(courseId: EntityId): Promise<Course> {
 		return this.repo.findById(courseId);
@@ -18,11 +21,13 @@ export class CourseService {
 	}
 
 	public async deleteUserDataFromCourse(userId: EntityId): Promise<number> {
+		this.logger.log({ action: 'Deleting data from Courses for user ', userId });
 		const [courses, count] = await this.repo.findAllByUserId(userId);
 
 		courses.forEach((course: Course) => course.removeUser(userId));
 
 		await this.repo.save(courses);
+		this.logger.log({ action: 'Deleting data from Courses for user ', userId });
 
 		return count;
 	}

apps/server/src/modules/learnroom/service/coursegroup.service.spec.ts

@@ -2,6 +2,7 @@ import { createMock, DeepMocked } from '@golevelup/ts-jest';
 import { Test, TestingModule } from '@nestjs/testing';
 import { CourseGroupRepo, UserRepo } from '@shared/repo';
 import { courseGroupFactory, setupEntities, userFactory } from '@shared/testing';
+import { LegacyLogger } from '@src/core/logger';
 import { CourseGroupService } from './coursegroup.service';
 
 describe('CourseGroupService', () => {
@@ -23,6 +24,10 @@ describe('CourseGroupService', () => {
 					provide: CourseGroupRepo,
 					useValue: createMock<CourseGroupRepo>(),
 				},
+				{
+					provide: LegacyLogger,
+					useValue: createMock<LegacyLogger>(),
+				},
 			],
 		}).compile();
 		courseGroupRepo = module.get(CourseGroupRepo);

apps/server/src/modules/learnroom/service/coursegroup.service.ts

@@ -2,10 +2,13 @@ import { Injectable } from '@nestjs/common';
 import { CourseGroup } from '@shared/domain/entity';
 import { Counted, EntityId } from '@shared/domain/types';
 import { CourseGroupRepo } from '@shared/repo';
+import { LegacyLogger } from '@src/core/logger';
 
 @Injectable()
 export class CourseGroupService {
-	constructor(private readonly repo: CourseGroupRepo) {}
+	constructor(private readonly repo: CourseGroupRepo, private readonly logger: LegacyLogger) {
+		this.logger.setContext(CourseGroupService.name);
+	}
 
 	public async findAllCourseGroupsByUserId(userId: EntityId): Promise<Counted<CourseGroup[]>> {
 		const [courseGroups, count] = await this.repo.findByUserId(userId);
@@ -14,11 +17,13 @@ export class CourseGroupService {
 	}
 
 	public async deleteUserDataFromCourseGroup(userId: EntityId): Promise<number> {
+		this.logger.log({ action: 'Deleting data from CourseGroup for user ', userId });
 		const [courseGroups, count] = await this.repo.findByUserId(userId);
 
 		courseGroups.forEach((courseGroup) => courseGroup.removeStudent(userId));
 
 		await this.repo.save(courseGroups);
+		this.logger.log({ action: 'Deleted data from CourseGroup for user ', userId });
 
 		return count;
 	}

apps/server/src/modules/learnroom/service/dashboard.service.spec.ts

@@ -4,6 +4,7 @@ import { DashboardElementRepo, IDashboardRepo, UserRepo } from '@shared/repo';
 import { setupEntities, userFactory } from '@shared/testing';
 import { LearnroomMetadata, LearnroomTypes } from '@shared/domain/types';
 import { DashboardEntity, GridElement } from '@shared/domain/entity';
+import { LegacyLogger } from '@src/core/logger';
 import { DashboardService } from '.';
 
 const learnroomMock = (id: string, name: string) => {
@@ -44,6 +45,10 @@ describe(DashboardService.name, () => {
 					provide: DashboardElementRepo,
 					useValue: createMock<DashboardElementRepo>(),
 				},
+				{
+					provide: LegacyLogger,
+					useValue: createMock<LegacyLogger>(),
+				},
 			],
 		}).compile();
 		dashboardService = module.get(DashboardService);

apps/server/src/modules/learnroom/service/dashboard.service.ts

@@ -1,18 +1,24 @@
 import { Inject, Injectable } from '@nestjs/common';
 import { EntityId } from '@shared/domain/types';
 import { IDashboardRepo, DashboardElementRepo } from '@shared/repo';
+import { LegacyLogger } from '@src/core/logger';
 
 @Injectable()
 export class DashboardService {
 	constructor(
 		@Inject('DASHBOARD_REPO') private readonly dashboardRepo: IDashboardRepo,
-		private readonly dashboardElementRepo: DashboardElementRepo
-	) {}
+		private readonly dashboardElementRepo: DashboardElementRepo,
+		private readonly logger: LegacyLogger
+	) {
+		this.logger.setContext(DashboardService.name);
+	}
 
 	async deleteDashboardByUserId(userId: EntityId): Promise<number> {
+		this.logger.log({ action: 'Deleting dasboard for userId - ', userId });
 		const usersDashboard = await this.dashboardRepo.getUsersDashboard(userId);
 		await this.dashboardElementRepo.deleteByDashboardId(usersDashboard.id);
 		const result = await this.dashboardRepo.deleteDashboardByUserId(userId);
+		this.logger.log({ action: 'Deleted dasboard for userId - ', userId });
 
 		return result;
 	}

apps/server/src/modules/lesson/service/lesson.service.spec.ts

@@ -4,6 +4,7 @@ import { FilesStorageClientAdapterService } from '@modules/files-storage-client'
 import { Test, TestingModule } from '@nestjs/testing';
 import { ComponentProperties, ComponentType } from '@shared/domain/entity';
 import { lessonFactory, setupEntities } from '@shared/testing';
+import { LegacyLogger } from '@src/core/logger';
 import { LessonRepo } from '../repository';
 import { LessonService } from './lesson.service';
 
@@ -26,6 +27,10 @@ describe('LessonService', () => {
 					provide: FilesStorageClientAdapterService,
 					useValue: createMock<FilesStorageClientAdapterService>(),
 				},
+				{
+					provide: LegacyLogger,
+					useValue: createMock<LegacyLogger>(),
+				},
 			],
 		}).compile();
 		lessonService = module.get(LessonService);

apps/server/src/modules/lesson/service/lesson.service.ts

@@ -3,14 +3,18 @@ import { Injectable } from '@nestjs/common';
 import { ComponentProperties, LessonEntity } from '@shared/domain/entity';
 import { Counted, EntityId } from '@shared/domain/types';
 import { AuthorizationLoaderService } from '@src/modules/authorization';
+import { LegacyLogger } from '@src/core/logger';
 import { LessonRepo } from '../repository';
 
 @Injectable()
 export class LessonService implements AuthorizationLoaderService {
 	constructor(
 		private readonly lessonRepo: LessonRepo,
-		private readonly filesStorageClientAdapterService: FilesStorageClientAdapterService
-	) {}
+		private readonly filesStorageClientAdapterService: FilesStorageClientAdapterService,
+		private readonly logger: LegacyLogger
+	) {
+		this.logger.setContext(LessonService.name);
+	}
 
 	async deleteLesson(lesson: LessonEntity): Promise<void> {
 		await this.filesStorageClientAdapterService.deleteFilesOfParent(lesson.id);
@@ -33,6 +37,7 @@ export class LessonService implements AuthorizationLoaderService {
 	}
 
 	async deleteUserDataFromLessons(userId: EntityId): Promise<number> {
+		this.logger.log({ action: 'Deleting User Data From Lesson for user ', userId });
 		const lessons = await this.lessonRepo.findByUserId(userId);
 
 		const updatedLessons = lessons.map((lesson: LessonEntity) => {
@@ -47,6 +52,8 @@ export class LessonService implements AuthorizationLoaderService {
 
 		await this.lessonRepo.save(updatedLessons);
 
+		this.logger.log({ action: 'Deleted User Data From Lesson for user ', userId });
+
 		return updatedLessons.length;
 	}
 }

apps/server/src/modules/pseudonym/service/pseudonym.service.spec.ts

@@ -7,6 +7,7 @@ import { IFindOptions } from '@shared/domain/interface';
 
 import { LtiToolDO, Page, Pseudonym, UserDO } from '@shared/domain/domainobject';
 import { externalToolFactory, ltiToolDOFactory, pseudonymFactory, userDoFactory } from '@shared/testing/factory';
+import { LegacyLogger } from '@src/core/logger';
 import { PseudonymSearchQuery } from '../domain';
 import { ExternalToolPseudonymRepo, PseudonymsRepo } from '../repo';
 import { PseudonymService } from './pseudonym.service';
@@ -30,6 +31,10 @@ describe('PseudonymService', () => {
 					provide: ExternalToolPseudonymRepo,
 					useValue: createMock<ExternalToolPseudonymRepo>(),
 				},
+				{
+					provide: LegacyLogger,
+					useValue: createMock<LegacyLogger>(),
+				},
 			],
 		}).compile();
 

apps/server/src/modules/pseudonym/service/pseudonym.service.ts

@@ -5,15 +5,19 @@ import { Injectable, InternalServerErrorException } from '@nestjs/common';
 import { LtiToolDO, Page, Pseudonym, UserDO } from '@shared/domain/domainobject';
 import { IFindOptions } from '@shared/domain/interface';
 import { v4 as uuidv4 } from 'uuid';
+import { LegacyLogger } from '@src/core/logger';
 import { PseudonymSearchQuery } from '../domain';
 import { ExternalToolPseudonymRepo, PseudonymsRepo } from '../repo';
 
 @Injectable()
 export class PseudonymService {
 	constructor(
 		private readonly pseudonymRepo: PseudonymsRepo,
-		private readonly externalToolPseudonymRepo: ExternalToolPseudonymRepo
-	) {}
+		private readonly externalToolPseudonymRepo: ExternalToolPseudonymRepo,
+		private readonly logger: LegacyLogger
+	) {
+		this.logger.setContext(PseudonymService.name);
+	}
 
 	public async findByUserAndToolOrThrow(user: UserDO, tool: ExternalTool | LtiToolDO): Promise<Pseudonym> {
 		if (!user.id || !tool.id) {
@@ -73,6 +77,7 @@ export class PseudonymService {
 	}
 
 	public async deleteByUserId(userId: string): Promise<number> {
+		this.logger.log({ action: 'Deleting Pseudonyms for user ', userId });
 		if (!userId) {
 			throw new InternalServerErrorException('User id is missing');
 		}
@@ -81,6 +86,7 @@ export class PseudonymService {
 			this.deletePseudonymsByUserId(userId),
 			this.deleteExternalToolPseudonymsByUserId(userId),
 		]);
+		this.logger.log({ action: 'Deleted Pseudonyms for user ', userId });
 
 		return deletedPseudonyms + deletedExternalToolPseudonyms;
 	}