CLAP-119 Feat: 작업 승인 처리

parkjaehak · parkjaehak · commit ce3b769b1537 · 2025-01-26T10:49:31.000+09:00
diff --git a/src/main/java/clap/server/adapter/inbound/web/dto/task/ApprovalTaskRequest.java b/src/main/java/clap/server/adapter/inbound/web/dto/task/ApprovalTaskRequest.java
@@ -0,0 +1,22 @@
+package clap.server.adapter.inbound.web.dto.task;
+
+import jakarta.validation.constraints.NotNull;
+
+import java.time.LocalDateTime;
+
+public record ApprovalTaskRequest(
+        @NotNull
+        Long taskId,
+        @NotNull
+        Long mainCategoryId,
+        @NotNull
+        Long categoryId,
+        @NotNull
+        Long processorId,
+        @NotNull
+        LocalDateTime dueDate,
+        @NotNull
+        Long labelId
+
+) {
+}
diff --git a/src/main/java/clap/server/adapter/inbound/web/dto/task/ApprovalTaskResponse.java b/src/main/java/clap/server/adapter/inbound/web/dto/task/ApprovalTaskResponse.java
@@ -0,0 +1,14 @@
+package clap.server.adapter.inbound.web.dto.task;
+
+import clap.server.adapter.outbound.persistense.entity.task.constant.LabelType;
+
+import java.time.LocalDateTime;
+
+public record ApprovalTaskResponse(
+        Long taskId,
+        String processorName,
+        String reviewerName,
+        LocalDateTime deadLine,
+        LabelType labelName
+) {
+}
diff --git a/src/main/java/clap/server/adapter/inbound/web/dto/task/FilterTaskListRequest.java b/src/main/java/clap/server/adapter/inbound/web/dto/task/FilterTaskListRequest.java
@@ -32,8 +32,8 @@ public record FilterTaskListRequest(
         @NotNull
         List<TaskStatus> taskStatus,
 
-        @Schema(description = "정렬 기준", implementation = OrderRequest.class)
+        @Schema(description = "정렬 기준", implementation = OrderTaskRequest.class)
         @NotNull
-        OrderRequest orderRequest
+        OrderTaskRequest orderRequest
 ) {
 }
diff --git a/src/main/java/clap/server/adapter/inbound/web/dto/task/OrderTaskRequest.java b/src/main/java/clap/server/adapter/inbound/web/dto/task/OrderTaskRequest.java
@@ -2,7 +2,7 @@
 
 import io.swagger.v3.oas.annotations.media.Schema;
 
-public record OrderRequest(
+public record OrderTaskRequest(
         @Schema(description = "정렬 기준", example = "REQUESTED_AT")
         String sortBy,
 
diff --git a/src/main/java/clap/server/adapter/inbound/web/task/FindTaskController.java b/src/main/java/clap/server/adapter/inbound/web/task/FindTaskController.java
@@ -27,7 +27,7 @@ public class FindTaskController {
     private final FindTaskListUsecase taskListUsecase;
 
     @Operation(summary = "사용자 요청 작업 목록 조회")
-    @Secured({"ROLE_USER"})
+    @Secured({"ROLE_USER", "ROLE_MANAGER"})
     @GetMapping("/requests")
     public ResponseEntity<Page<FilterTaskListResponse>> findTasksRequestedByUser(
             @RequestParam(defaultValue = "0") int page,
diff --git a/src/main/java/clap/server/adapter/inbound/web/task/ManagementTaskController.java b/src/main/java/clap/server/adapter/inbound/web/task/ManagementTaskController.java
@@ -1,10 +1,8 @@
 package clap.server.adapter.inbound.web.task;
 
 import clap.server.adapter.inbound.security.SecurityUserDetails;
-import clap.server.adapter.inbound.web.dto.task.CreateTaskRequest;
-import clap.server.adapter.inbound.web.dto.task.CreateTaskResponse;
-import clap.server.adapter.inbound.web.dto.task.UpdateTaskRequest;
-import clap.server.adapter.inbound.web.dto.task.UpdateTaskResponse;
+import clap.server.adapter.inbound.web.dto.task.*;
+import clap.server.application.port.inbound.task.ApprovalTaskUsecase;
 import clap.server.application.port.inbound.task.CreateTaskUsecase;
 import clap.server.application.port.inbound.task.UpdateTaskUsecase;
 import clap.server.common.annotation.architecture.WebAdapter;
@@ -13,6 +11,7 @@
 import jakarta.validation.Valid;
 import lombok.RequiredArgsConstructor;
 import org.springframework.http.ResponseEntity;
+import org.springframework.security.access.annotation.Secured;
 import org.springframework.security.core.annotation.AuthenticationPrincipal;
 import org.springframework.web.bind.annotation.*;
 
@@ -26,6 +25,7 @@ public class ManagementTaskController {
 
     private final CreateTaskUsecase createTaskUsecase;
     private final UpdateTaskUsecase updateTaskUsecase;
+    private final ApprovalTaskUsecase approvalTaskUsecase;
 
     @Operation(summary = "작업 요청 생성")
     @PostMapping
@@ -42,4 +42,13 @@ public ResponseEntity<UpdateTaskResponse> updateTask(
             @AuthenticationPrincipal SecurityUserDetails userInfo){
         return ResponseEntity.ok(updateTaskUsecase.updateTask(userInfo.getUserId(), updateTaskRequest));
     }
+
+    @Operation(summary = "작업 승인")
+    @Secured({"ROLE_MANAGER"})
+    @PostMapping("/approval")
+    public ResponseEntity<ApprovalTaskResponse> approvalTask(
+            @RequestBody @Valid ApprovalTaskRequest approvalTaskRequest,
+            @AuthenticationPrincipal SecurityUserDetails userInfo){
+        return ResponseEntity.ok(approvalTaskUsecase.approvalTaskByReviewer(userInfo.getUserId(), approvalTaskRequest));
+    }
 }
diff --git a/src/main/java/clap/server/adapter/outbound/persistense/MemberPersistenceAdapter.java b/src/main/java/clap/server/adapter/outbound/persistense/MemberPersistenceAdapter.java
@@ -85,6 +85,5 @@ public List<Task> findTasksByMemberIdAndStatus(Long memberId, List<TaskStatus> t
                 .map(taskPersistenceMapper::toDomain)
                 .collect(Collectors.toList());
     }
-
 }
 
diff --git a/src/main/java/clap/server/adapter/outbound/persistense/repository/LabelPersistenceAdapter.java b/src/main/java/clap/server/adapter/outbound/persistense/repository/LabelPersistenceAdapter.java
@@ -0,0 +1,26 @@
+package clap.server.adapter.outbound.persistense.repository;
+
+import clap.server.adapter.outbound.persistense.entity.task.LabelEntity;
+import clap.server.adapter.outbound.persistense.entity.task.TaskEntity;
+import clap.server.adapter.outbound.persistense.mapper.LabelPersistenceMapper;
+import clap.server.adapter.outbound.persistense.repository.task.LabelRepository;
+import clap.server.application.port.outbound.task.LoadLabelPort;
+import clap.server.common.annotation.architecture.PersistenceAdapter;
+import clap.server.domain.model.task.Label;
+import lombok.RequiredArgsConstructor;
+
+import java.util.Optional;
+
+@PersistenceAdapter
+@RequiredArgsConstructor
+public class LabelPersistenceAdapter implements LoadLabelPort {
+
+    private final LabelRepository labelRepository;
+    private final LabelPersistenceMapper labelPersistenceMapper;
+
+    @Override
+    public Optional<Label> findById(Long id) {
+        Optional<LabelEntity> labelEntity = labelRepository.findById(id);
+        return labelEntity.map(labelPersistenceMapper::toDomain);
+    }
+}
diff --git a/src/main/java/clap/server/application/Task/ApprovalTaskService.java b/src/main/java/clap/server/application/Task/ApprovalTaskService.java
@@ -0,0 +1,49 @@
+package clap.server.application.Task;
+
+import clap.server.adapter.inbound.web.dto.task.ApprovalTaskRequest;
+import clap.server.adapter.inbound.web.dto.task.ApprovalTaskResponse;
+import clap.server.application.mapper.TaskMapper;
+import clap.server.application.port.inbound.domain.CategoryService;
+import clap.server.application.port.inbound.domain.LabelService;
+import clap.server.application.port.inbound.domain.MemberService;
+import clap.server.application.port.inbound.domain.TaskService;
+import clap.server.application.port.inbound.task.ApprovalTaskUsecase;
+import clap.server.application.port.outbound.task.CommandTaskPort;
+import clap.server.common.annotation.architecture.ApplicationService;
+import clap.server.domain.model.member.Member;
+import clap.server.domain.model.task.Category;
+import clap.server.domain.model.task.Label;
+import clap.server.domain.model.task.Task;
+import clap.server.exception.ApplicationException;
+import clap.server.exception.code.MemberErrorCode;
+import clap.server.exception.code.TaskErrorCode;
+import lombok.RequiredArgsConstructor;
+import org.springframework.transaction.annotation.Transactional;
+
+@ApplicationService
+@RequiredArgsConstructor
+public class ApprovalTaskService implements ApprovalTaskUsecase {
+
+    private final MemberService memberService;
+    private final TaskService taskService;
+    private final CategoryService categoryService;
+    private final LabelService labelService;
+    private final CommandTaskPort commandTaskPort;
+
+    @Override
+    @Transactional
+    public ApprovalTaskResponse approvalTaskByReviewer(Long reviewerId, ApprovalTaskRequest approvalTaskRequest) {
+        Member reviewer = memberService.findActiveMember(reviewerId);
+        if (!reviewer.isReviewer()) {
+            throw new ApplicationException(MemberErrorCode.NOT_A_REVIEWER);
+        }
+        Task task = taskService.findById(approvalTaskRequest.taskId());
+        Member processor = memberService.findById(approvalTaskRequest.processorId());
+        Category mainCategory = categoryService.findById(approvalTaskRequest.mainCategoryId());
+        Category category = categoryService.findById(approvalTaskRequest.categoryId());
+        Label label = labelService.findById(approvalTaskRequest.labelId());
+
+        task.approveTask(reviewer, processor, approvalTaskRequest.dueDate(),mainCategory, category, label);
+        return TaskMapper.toApprovalTaskResponse(commandTaskPort.save(task));
+    }
+}
diff --git a/src/main/java/clap/server/application/Task/CreateTaskService.java b/src/main/java/clap/server/application/Task/CreateTaskService.java
@@ -3,7 +3,6 @@
 import clap.server.adapter.inbound.web.dto.task.CreateTaskRequest;
 import clap.server.adapter.inbound.web.dto.task.CreateTaskResponse;
 
-import clap.server.adapter.outbound.persistense.entity.notification.constant.NotificationType;
 import clap.server.application.mapper.TaskMapper;
 import clap.server.application.port.inbound.domain.CategoryService;
 import clap.server.application.port.inbound.domain.MemberService;
@@ -13,13 +12,11 @@
 
 import clap.server.common.annotation.architecture.ApplicationService;
 import clap.server.domain.model.member.Member;
-import clap.server.domain.model.notification.Notification;
 import clap.server.domain.model.task.Attachment;
 import clap.server.domain.model.task.Category;
 import clap.server.domain.model.task.Task;
 import lombok.RequiredArgsConstructor;
 
-import org.springframework.context.ApplicationEventPublisher;
 import org.springframework.transaction.annotation.Transactional;
 
 import java.util.List;
diff --git a/src/main/java/clap/server/application/Task/UpdateTaskService.java b/src/main/java/clap/server/application/Task/UpdateTaskService.java
@@ -2,6 +2,7 @@
 
 import clap.server.adapter.inbound.web.dto.task.UpdateTaskRequest;
 import clap.server.adapter.inbound.web.dto.task.UpdateTaskResponse;
+import clap.server.adapter.outbound.persistense.entity.task.constant.TaskStatus;
 import clap.server.application.mapper.AttachmentMapper;
 import clap.server.application.mapper.TaskMapper;
 import clap.server.application.port.inbound.domain.CategoryService;
@@ -17,6 +18,8 @@
 import clap.server.domain.model.task.Category;
 import clap.server.domain.model.task.Task;
 
+import clap.server.exception.ApplicationException;
+import clap.server.exception.code.TaskErrorCode;
 import lombok.RequiredArgsConstructor;
 import lombok.extern.slf4j.Slf4j;
 import org.springframework.transaction.annotation.Transactional;
@@ -43,7 +46,9 @@ public UpdateTaskResponse updateTask(Long requesterId, UpdateTaskRequest updateT
         memberService.findActiveMember(requesterId);
         Category category = categoryService.findById(updateTaskRequest.categoryId());
         Task task = taskService.findById(updateTaskRequest.taskId());
-        //TODO: 작업이 요청 상태인 경우만 업데이트 가능
+        if(task.getTaskStatus() != TaskStatus.REQUESTED){
+            throw new ApplicationException(TaskErrorCode.TASK_STATUS_MISMATCH);
+        }
         task.updateTask(category, updateTaskRequest.title(), updateTaskRequest.description());
         Task updatedTask = commandTaskPort.save(task);
 
diff --git a/src/main/java/clap/server/application/mapper/TaskMapper.java b/src/main/java/clap/server/application/mapper/TaskMapper.java
@@ -77,4 +77,14 @@ public static FindTaskDetailsResponse toFindTaskDetailResponse(Task task, List<A
                 attachmentResponses
         );
     }
+
+    public static ApprovalTaskResponse toApprovalTaskResponse(Task approvedTask) {
+        return new ApprovalTaskResponse(
+                approvedTask.getTaskId(),
+                approvedTask.getProcessor().getNickname(),
+                approvedTask.getReviewer().getNickname(),
+                approvedTask.getDueDate(),
+                approvedTask.getLabel().getLabelType()
+        );
+    }
 }
diff --git a/src/main/java/clap/server/application/port/inbound/domain/LabelService.java b/src/main/java/clap/server/application/port/inbound/domain/LabelService.java
@@ -0,0 +1,20 @@
+package clap.server.application.port.inbound.domain;
+
+import clap.server.application.port.outbound.task.LoadLabelPort;
+import clap.server.domain.model.task.Label;
+import clap.server.exception.ApplicationException;
+import clap.server.exception.code.TaskErrorCode;
+import lombok.RequiredArgsConstructor;
+import org.springframework.stereotype.Service;
+
+@Service
+@RequiredArgsConstructor
+public class LabelService {
+
+    private final LoadLabelPort loadLabelPort;
+
+    public Label findById(Long labelId) {
+        return loadLabelPort.findById(labelId).orElseThrow(
+                ()-> new ApplicationException(TaskErrorCode.LABEL_NOT_FOUND));
+    }
+}
diff --git a/src/main/java/clap/server/application/port/inbound/task/ApprovalTaskUsecase.java b/src/main/java/clap/server/application/port/inbound/task/ApprovalTaskUsecase.java
@@ -0,0 +1,8 @@
+package clap.server.application.port.inbound.task;
+
+import clap.server.adapter.inbound.web.dto.task.ApprovalTaskRequest;
+import clap.server.adapter.inbound.web.dto.task.ApprovalTaskResponse;
+
+public interface ApprovalTaskUsecase {
+    ApprovalTaskResponse approvalTaskByReviewer(Long userId, ApprovalTaskRequest approvalTaskRequest);
+}
diff --git a/src/main/java/clap/server/application/port/outbound/task/LoadLabelPort.java b/src/main/java/clap/server/application/port/outbound/task/LoadLabelPort.java
@@ -0,0 +1,10 @@
+package clap.server.application.port.outbound.task;
+
+import clap.server.domain.model.task.Label;
+
+import java.util.Optional;
+
+public interface LoadLabelPort {
+
+    Optional<Label> findById(Long id);
+}
diff --git a/src/main/java/clap/server/domain/model/member/Member.java b/src/main/java/clap/server/domain/model/member/Member.java
@@ -49,4 +49,7 @@ public String getNickname() {
         return memberInfo != null ? memberInfo.getNickname() : null;
     }
 
+    public boolean isReviewer() {
+        return this.memberInfo != null && this.memberInfo.isReviewer();
+    }
 }
diff --git a/src/main/java/clap/server/domain/model/task/Category.java b/src/main/java/clap/server/domain/model/task/Category.java
@@ -13,31 +13,35 @@
 @SuperBuilder
 @NoArgsConstructor(access = AccessLevel.PROTECTED)
 public class Category extends BaseTime {
-   private Long categoryId;
-   private Member admin;
-   private Category mainCategory;
-   private String code; 
-   private String name; 
-   private boolean isDeleted;
-   private String descriptionExample;
-   private LocalDateTime createdAt; 
-   private LocalDateTime updatedAt;
+    private Long categoryId;
+    private Member admin;
+    private Category mainCategory;
+    private String code;
+    private String name;
+    private boolean isDeleted;
+    private String descriptionExample;
+    private LocalDateTime createdAt;
+    private LocalDateTime updatedAt;
 
-   public static Category createMainCategory(Member admin, String code, String name) {
-      return Category.builder()
-              .admin(admin)
-              .code(code)
-              .name(name)
-              .build();
-   }
+    public static Category createMainCategory(Member admin, String code, String name) {
+        return Category.builder()
+                .admin(admin)
+                .code(code)
+                .name(name)
+                .build();
+    }
 
-   public static Category createSubCategory(Member admin, Category mainCategory, String code, String name) {
-      return Category.builder()
-              .mainCategory(mainCategory)
-              .admin(admin)
-              .code(code)
-              .name(name)
-              .build();
-   }
- }
+    public static Category createSubCategory(Member admin, Category mainCategory, String code, String name) {
+        return Category.builder()
+                .mainCategory(mainCategory)
+                .admin(admin)
+                .code(code)
+                .name(name)
+                .build();
+    }
+
+    public void updateMainCategory(Category mainCategory){
+        this.mainCategory = mainCategory;
+    }
+}
 
diff --git a/src/main/java/clap/server/domain/model/task/Task.java b/src/main/java/clap/server/domain/model/task/Task.java
diff --git a/src/main/java/clap/server/exception/code/MemberErrorCode.java b/src/main/java/clap/server/exception/code/MemberErrorCode.java
diff --git a/src/main/java/clap/server/exception/code/TaskErrorCode.java b/src/main/java/clap/server/exception/code/TaskErrorCode.java
diff --git a/src/test/java/clap/server/application/service/task/FindTaskListServiceTest.java b/src/test/java/clap/server/application/service/task/FindTaskListServiceTest.java

Original file line number	Diff line number	Diff line change
`@@ -85,6 +85,5 @@ public List<Task> findTasksByMemberIdAndStatus(Long memberId, List<TaskStatus> t`
`85`	`85`	`.map(taskPersistenceMapper::toDomain)`
`86`	`86`	`.collect(Collectors.toList());`
`87`	`87`	`}`
`88`		`-`
`89`	`88`	`}`
`90`	`89`