feat: 모임 중복 초대시 예외처리

bo-an-bo · May 25, 2024 · 6c61f3c · 6c61f3c
1 parent a42cd8e
commit 6c61f3c
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/src/group/group.service.ts b/src/group/group.service.ts
@@ -241,6 +241,7 @@ export class GroupService {
   async inviteEditor(senderId: string, groupId: string, receiverId: string) {
     const group = await this.groupRepository.findOne(groupId);
     this.groupValidator.validateGroupOwner(group, senderId);
+    this.groupValidator.validateGroupInvite(group, receiverId);
 
     group.auth.editors.push(receiverId);
     await this.userService.pushEditor(receiverId, groupId);
@@ -251,6 +252,7 @@ export class GroupService {
   async inviteViewer(senderId: string, groupId: string, receiverId: string) {
     const group = await this.groupRepository.findOne(groupId);
     this.groupValidator.validateGroupViewer(group, senderId);
+    this.groupValidator.validateGroupInvite(group, receiverId);
 
     group.auth.viewers.push(receiverId);
     await this.userService.pushViewer(receiverId, groupId);

diff --git a/src/group/group.validator.ts b/src/group/group.validator.ts
@@ -1,4 +1,4 @@
-import { Injectable, NotFoundException, UnauthorizedException } from '@nestjs/common';
+import { ConflictException, Injectable, NotFoundException, UnauthorizedException } from '@nestjs/common';
 
 import { Group } from './entities/group.entity';
 
@@ -28,6 +28,18 @@ export class GroupValidator {
     }
   }
 
+  validateGroupInvite(group: Group, receiverId: string) {
+    this.validateGroup(group);
+
+    if (
+      group.auth.owner === receiverId ||
+      group.auth.editors.includes(receiverId) ||
+      group.auth.viewers.includes(receiverId)
+    ) {
+      throw new ConflictException('이미 초대된 사용자입니다.');
+    }
+  }
+
   private validateGroup(group: Group) {
     if (!group) {
       throw new NotFoundException('모임을 찾을 수 없습니다.');