N21-2287 Fix partial course sync start for admins

apps/server/src/modules/group/domain/group.spec.ts

@@ -1,11 +1,10 @@
+import { ObjectId } from '@mikro-orm/mongodb';
 import { RoleReference, UserDO } from '@shared/domain/domainobject';
 import { groupFactory, roleFactory, userDoFactory } from '@shared/testing';
-
-import { ObjectId } from '@mikro-orm/mongodb';
 import { Group } from './group';
 import { GroupUser } from './group-user';
 
-describe('Group (Domain Object)', () => {
+describe(Group.name, () => {
 	describe('removeUser', () => {
 		describe('when the user is in the group', () => {
 			const setup = () => {
@@ -193,4 +192,117 @@ describe('Group (Domain Object)', () => {
 			});
 		});
 	});
+
+	describe('removeUser', () => {
+		describe('when the user is a member of the group', () => {
+			const setup = () => {
+				const userId = new ObjectId().toHexString();
+				const roleId = new ObjectId().toHexString();
+				const groupUser = new GroupUser({
+					userId,
+					roleId,
+				});
+				const group = groupFactory.build({
+					users: [groupUser],
+				});
+
+				return {
+					group,
+					userId,
+				};
+			};
+
+			it('should return true', () => {
+				const { group, userId } = setup();
+
+				const result = group.isMember(userId);
+
+				expect(result).toEqual(true);
+			});
+		});
+
+		describe('when the user is a member of the group and has the requested role', () => {
+			const setup = () => {
+				const userId = new ObjectId().toHexString();
+				const roleId = new ObjectId().toHexString();
+				const groupUser = new GroupUser({
+					userId,
+					roleId,
+				});
+				const group = groupFactory.build({
+					users: [groupUser],
+				});
+
+				return {
+					group,
+					userId,
+					roleId,
+				};
+			};
+
+			it('should return true', () => {
+				const { group, userId, roleId } = setup();
+
+				const result = group.isMember(userId, roleId);
+
+				expect(result).toEqual(true);
+			});
+		});
+
+		describe('when the user is a member of the group, but has a different role', () => {
+			const setup = () => {
+				const userId = new ObjectId().toHexString();
+				const roleId = new ObjectId().toHexString();
+				const groupUser = new GroupUser({
+					userId,
+					roleId: new ObjectId().toHexString(),
+				});
+				const group = groupFactory.build({
+					users: [groupUser],
+				});
+
+				return {
+					group,
+					userId,
+					roleId,
+				};
+			};
+
+			it('should return false', () => {
+				const { group, userId, roleId } = setup();
+
+				const result = group.isMember(userId, roleId);
+
+				expect(result).toEqual(false);
+			});
+		});
+
+		describe('when the user is not a member of the group', () => {
+			const setup = () => {
+				const userId = new ObjectId().toHexString();
+				const roleId = new ObjectId().toHexString();
+				const groupUser = new GroupUser({
+					userId: new ObjectId().toHexString(),
+					roleId: new ObjectId().toHexString(),
+				});
+				const group = groupFactory.build({
+					users: [groupUser],
+				});
+
+				return {
+					group,
+					userId,
+					roleId,
+				};
+			};
+
+			it('should return false', () => {
+				const { group, userId } = setup();
+
+				const result = group.isMember(userId);
+
+				expect(result).toEqual(false);
+			});
+		});
+	});
 });

apps/server/src/modules/group/domain/group.ts

@@ -50,17 +50,25 @@ export class Group extends DomainObject<GroupProps> {
 		return this.props.validPeriod;
 	}
 
-	removeUser(user: UserDO): void {
+	public removeUser(user: UserDO): void {
 		this.props.users = this.props.users.filter((groupUser: GroupUser): boolean => groupUser.userId !== user.id);
 	}
 
-	isEmpty(): boolean {
+	public isEmpty(): boolean {
 		return this.props.users.length === 0;
 	}
 
-	addUser(user: GroupUser): void {
+	public addUser(user: GroupUser): void {
 		if (!this.users.find((u: GroupUser): boolean => u.userId === user.userId)) {
 			this.users.push(user);
 		}
 	}
+
+	public isMember(userId: EntityId, roleId?: EntityId): boolean {
+		const isMember: boolean = this.users.some(
+			(groupUser: GroupUser) => groupUser.userId === userId && (roleId ? groupUser.roleId === roleId : true)
+		);
+
+		return isMember;
+	}
 }

apps/server/src/modules/learnroom/domain/do/course.spec.ts

@@ -0,0 +1,53 @@
+import { ObjectId } from '@mikro-orm/mongodb';
+import { courseFactory } from '../../testing';
+import { Course } from './course';
+
+describe(Course.name, () => {
+	describe('isTeacher', () => {
+		describe('when the user is a teacher in the course', () => {
+			const setup = () => {
+				const userId = new ObjectId().toHexString();
+				const course = courseFactory.build({
+					teacherIds: [userId],
+				});
+
+				return {
+					course,
+					userId,
+				};
+			};
+
+			it('should return true', () => {
+				const { course, userId } = setup();
+
+				const result = course.isTeacher(userId);
+
+				expect(result).toEqual(true);
+			});
+		});
+
+		describe('when the user is not a teacher in the course', () => {
+			const setup = () => {
+				const userId = new ObjectId().toHexString();
+				const course = courseFactory.build({
+					teacherIds: [new ObjectId().toHexString()],
+					studentIds: [userId],
+					substitutionTeacherIds: [userId],
+				});
+
+				return {
+					course,
+					userId,
+				};
+			};
+
+			it('should return false', () => {
+				const { course, userId } = setup();
+
+				const result = course.isTeacher(userId);
+
+				expect(result).toEqual(false);
+			});
+		});
+	});
+});

apps/server/src/modules/learnroom/domain/do/course.ts

@@ -106,4 +106,10 @@ export class Course extends DomainObject<CourseProps> {
 	get excludeFromSync(): SyncAttribute[] | undefined {
 		return this.props.excludeFromSync;
 	}
+
+	public isTeacher(userId: EntityId): boolean {
+		const isMember: boolean = this.teachers.some((teacherId: EntityId) => teacherId === userId);
+
+		return isMember;
+	}
 }