fix: IssueResponse 수정

se-cau · May 26, 2024 · abd9cca · abd9cca
1 parent 88a5c81
commit abd9cca
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 7 deletions.
diff --git a/src/main/java/se/issuetrackingsystem/issue/controller/IssueController.java b/src/main/java/se/issuetrackingsystem/issue/controller/IssueController.java
@@ -25,7 +25,7 @@ public class IssueController {
 
     @PostMapping
     public void issueCreate(@RequestBody IssueRequest issueRequest, @RequestParam("projectId") Long projectid){
-        this.issueService.create(projectid,issueRequest.getTitle(),issueRequest.getDescription(),issueRequest.getUserid());
+        this.issueService.create(projectid,issueRequest.getTitle(),issueRequest.getDescription(),issueRequest.getUserid(),issueRequest.getPriority());
     }
 
     @GetMapping
@@ -54,13 +54,13 @@ public void issueDelete(@RequestParam("issueId") Long issueid){
     @PatchMapping
     public void issueModify(@RequestBody IssueRequest issueRequest,@RequestParam("issueId") Long issueid){
         Issue issue = this.issueService.getIssue(issueid);
-        this.issueService.modify(issue,issueRequest.getDescription());
+        this.issueService.modify(issue,issueRequest.getDescription(),issueRequest.getPriority());
     }
 
     @PostMapping("/assignees")
     public void issueSetAssignee(@RequestBody IssueRequest issueRequest,@RequestParam("issueId") Long issueid){
         Issue issue = this.issueService.getIssue(issueid);
-        this.issueService.setAssignee(issue,issueRequest.getAssigneeid());
+        this.issueService.setAssignee(issue,issueRequest.getUserid());
     }
 
     @GetMapping("/{status}")

diff --git a/src/main/java/se/issuetrackingsystem/issue/dto/IssueRequest.java b/src/main/java/se/issuetrackingsystem/issue/dto/IssueRequest.java
@@ -12,7 +12,7 @@ public class IssueRequest {
 
     private String description;
 
-    private Long assigneeid;
+    private Issue.Priority priority;
 
     private Issue.Status status;
 

diff --git a/src/main/java/se/issuetrackingsystem/issue/service/IssueService.java b/src/main/java/se/issuetrackingsystem/issue/service/IssueService.java
@@ -31,12 +31,13 @@ public class IssueService {
     private final ProjectRepository projectRepository;
     private final ProjectContributorRepository projectContributorRepository;
 
-    public Issue create(Long projectid, String title, String description, Long reporterid){
+    public Issue create(Long projectid, String title, String description, Long reporterid, Issue.Priority priority){
         Issue issue = new Issue();
         Project project = this.projectRepository.findById(projectid).orElseThrow(()->new CustomException(ErrorCode.PROJECT_NOT_FOUND));
         issue.setProject(project);
         issue.setTitle(title);
         issue.setDescription(description);
+        issue.setPriority(priority);
         issue.setReporter(userRepository.findById(reporterid).orElseThrow(() -> new CustomException(ErrorCode.USER_NOT_FOUND)));
         issue.setCreated_at(LocalDateTime.now());
         this.issueRepository.save(issue);
@@ -68,9 +69,10 @@ public List<Issue> getList(Long projectid, Issue.Status status){
         return issues;
     }
 
-    public void modify(Issue issue,String description){
+    public void modify(Issue issue, String description, Issue.Priority priority){
         issue.setDescription(description);
         issue.setUpdated_at(LocalDateTime.now());
+        issue.setPriority(priority);
         this.issueRepository.save(issue);
     }
 

diff --git a/src/test/java/se/issuetrackingsystem/IssueTrackingSystemApplicationTests.java b/src/test/java/se/issuetrackingsystem/IssueTrackingSystemApplicationTests.java
@@ -50,7 +50,7 @@ class IssueTrackingSystemApplicationTests {
 
 	@Test
 	public void testIssueCreate(){
-		Issue issue1 = this.issueService.create(27L,"Hello JUnit","Hello Spring", 28L);
+		Issue issue1 = this.issueService.create(27L,"Hello JUnit","Hello Spring", 28L, Issue.Priority.MAJOR);
 
 		Issue issue2 = this.issueRepository.findById(issue1.getId()).orElseThrow(()->new CustomException(ErrorCode.ISSUE_NOT_FOUND));