Feature/56 커뮤니티 게시글 수정 기능

src/main/java/com/somemore/community/domain/CommunityBoard.java

@@ -1,5 +1,6 @@
 package com.somemore.community.domain;
 
+import com.somemore.community.dto.request.CommunityBoardUpdateRequestDto;
 import com.somemore.global.common.BaseEntity;
 import static lombok.AccessLevel.PROTECTED;
 
@@ -52,4 +53,10 @@ public CommunityBoard(UUID writerId, String title, String content, String imgUrl
     public boolean isWriter(UUID writerId) {
         return this.writerId.equals(writerId);
     }
+
+    public void updateWith(CommunityBoardUpdateRequestDto dto, String imgUrl) {
+        this.title = dto.title();
+        this.content = dto.content();
+        this.imgUrl = imgUrl;
+    }
 }

src/main/java/com/somemore/community/dto/request/CommunityBoardUpdateRequestDto.java

@@ -0,0 +1,18 @@
+package com.somemore.community.dto.request;
+
+import com.fasterxml.jackson.databind.PropertyNamingStrategies;
+import com.fasterxml.jackson.databind.annotation.JsonNaming;
+import io.swagger.v3.oas.annotations.media.Schema;
+import jakarta.validation.constraints.NotBlank;
+import lombok.Builder;
+
+@JsonNaming(PropertyNamingStrategies.SnakeCaseStrategy.class)
+@Builder
+public record CommunityBoardUpdateRequestDto(
+        @Schema(description = "커뮤니티 게시글 제목", example = "11/29 OO도서관 봉사 같이 갈 사람 모집합니다.")
+        @NotBlank(message = "게시글 제목은 필수 값입니다.")
+        String title,
+        @Schema(description = "커뮤니티 게시글 내용", example = "저 포함 5명이 같이 가면 좋을 거 같아요")
+        @NotBlank(message = "게시글 내용은 필수 값입니다.")
+        String content
+) {}

src/main/java/com/somemore/community/service/query/CommunityBoardQueryService.java → src/main/java/com/somemore/community/service/CommunityBoardQueryService.java

@@ -1,12 +1,12 @@
-package com.somemore.community.service.query;
+package com.somemore.community.service;
 
 import com.somemore.center.usecase.query.CenterQueryUseCase;
 import com.somemore.community.domain.CommunityBoard;
 import com.somemore.community.dto.response.CommunityBoardGetDetailResponseDto;
 import com.somemore.community.dto.response.CommunityBoardGetResponseDto;
 import com.somemore.community.dto.response.WriterDetailDto;
 import com.somemore.community.repository.CommunityBoardRepository;
-import com.somemore.community.usecase.query.CommunityBoardQueryUseCase;
+import com.somemore.community.usecase.CommunityBoardQueryUseCase;
 import com.somemore.global.exception.BadRequestException;
 import com.somemore.volunteer.dto.response.VolunteerForCommunityResponseDto;
 import com.somemore.volunteer.usecase.query.FindVolunteerIdUseCase;

src/main/java/com/somemore/community/service/command/CreateCommunityBoardService.java → src/main/java/com/somemore/community/service/CreateCommunityBoardService.java

@@ -1,9 +1,9 @@
-package com.somemore.community.service.command;
+package com.somemore.community.service;
 
 import com.somemore.community.domain.CommunityBoard;
 import com.somemore.community.dto.request.CommunityBoardCreateRequestDto;
 import com.somemore.community.repository.CommunityBoardRepository;
-import com.somemore.community.usecase.command.CreateCommunityBoardUseCase;
+import com.somemore.community.usecase.CreateCommunityBoardUseCase;
 import lombok.RequiredArgsConstructor;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;

src/main/java/com/somemore/community/service/command/DeleteCommunityBoardService.java → src/main/java/com/somemore/community/service/DeleteCommunityBoardService.java

@@ -1,8 +1,8 @@
-package com.somemore.community.service.command;
+package com.somemore.community.service;
 
 import com.somemore.community.domain.CommunityBoard;
 import com.somemore.community.repository.CommunityBoardRepository;
-import com.somemore.community.usecase.command.DeleteCommunityBoardUseCase;
+import com.somemore.community.usecase.DeleteCommunityBoardUseCase;
 import com.somemore.global.exception.BadRequestException;
 import lombok.RequiredArgsConstructor;
 import org.springframework.stereotype.Service;

src/main/java/com/somemore/community/service/UpdateCommunityBoardService.java

@@ -0,0 +1,45 @@
+package com.somemore.community.service;
+
+import com.somemore.community.domain.CommunityBoard;
+import com.somemore.community.dto.request.CommunityBoardUpdateRequestDto;
+import com.somemore.community.repository.CommunityBoardRepository;
+import com.somemore.community.usecase.UpdateCommunityBoardUseCase;
+import com.somemore.global.exception.BadRequestException;
+
+import java.util.UUID;
+import lombok.RequiredArgsConstructor;
+import org.springframework.stereotype.Service;
+import org.springframework.transaction.annotation.Transactional;
+
+import static com.somemore.global.exception.ExceptionMessage.*;
+
+
+@RequiredArgsConstructor
+@Transactional
+@Service
+public class UpdateCommunityBoardService implements UpdateCommunityBoardUseCase {
+
+    private final CommunityBoardRepository communityBoardRepository;
+
+    @Override
+    public void updateCommunityBoard(CommunityBoardUpdateRequestDto requestDto, Long communityBoardId, UUID writerId, String imgUrl) {
+        CommunityBoard communityBoard = getCommunityBoardById(communityBoardId);
+        validateWriter(communityBoard, writerId);
+        communityBoard.updateWith(requestDto, imgUrl);
+
+        communityBoardRepository.save(communityBoard);
+    }
+
+    private CommunityBoard getCommunityBoardById(Long id) {
+        return communityBoardRepository.findById(id)
+                .orElseThrow(() -> new BadRequestException(NOT_EXISTS_COMMUNITY_BOARD.getMessage()));
+    }
+
+    private void validateWriter(CommunityBoard communityBoard, UUID writerId) {
+        if (communityBoard.isWriter(writerId)) {
+            return;
+        }
+
+        throw new BadRequestException(UNAUTHORIZED_COMMUNITY_BOARD.getMessage());
+    }
+}

src/main/java/com/somemore/community/usecase/query/CommunityBoardQueryUseCase.java → src/main/java/com/somemore/community/usecase/CommunityBoardQueryUseCase.java

@@ -1,4 +1,4 @@
-package com.somemore.community.usecase.query;
+package com.somemore.community.usecase;
 
 import com.somemore.community.dto.response.CommunityBoardGetDetailResponseDto;
 import com.somemore.community.dto.response.CommunityBoardGetResponseDto;

src/main/java/com/somemore/community/usecase/command/CreateCommunityBoardUseCase.java → src/main/java/com/somemore/community/usecase/CreateCommunityBoardUseCase.java

@@ -1,4 +1,4 @@
-package com.somemore.community.usecase.command;
+package com.somemore.community.usecase;
 
 import com.somemore.community.dto.request.CommunityBoardCreateRequestDto;
 

src/main/java/com/somemore/community/usecase/command/DeleteCommunityBoardUseCase.java → src/main/java/com/somemore/community/usecase/DeleteCommunityBoardUseCase.java

@@ -1,4 +1,4 @@
-package com.somemore.community.usecase.command;
+package com.somemore.community.usecase;
 
 import java.util.UUID;
 

src/main/java/com/somemore/community/usecase/UpdateCommunityBoardUseCase.java

@@ -0,0 +1,13 @@
+package com.somemore.community.usecase;
+
+import com.somemore.community.dto.request.CommunityBoardUpdateRequestDto;
+
+import java.util.UUID;
+
+public interface UpdateCommunityBoardUseCase {
+    void updateCommunityBoard(
+            CommunityBoardUpdateRequestDto requestDto,
+            Long communityBoardId,
+            UUID writerId,
+            String imgUrl);
+}

src/main/java/com/somemore/global/exception/ExceptionMessage.java

@@ -10,7 +10,7 @@ public enum ExceptionMessage {
 
     NOT_EXISTS_CENTER("존재하지 않는 기관 ID 입니다."),
     NOT_EXISTS_COMMUNITY_BOARD("존재하지 않는 게시글 입니다."),
-    UNAUTHORIZED_COMMUNITY_BOARD("게시글 삭제 권한이 없습니다."),
+    UNAUTHORIZED_COMMUNITY_BOARD("해당 게시글에 권한이 없습니다."),
     NOT_EXISTS_LOCATION("존재하지 않는 위치 ID 입니다."),
     NOT_EXISTS_RECRUIT_BOARD("존재하지 않는 봉사 모집글 ID 입니다."),
     UNAUTHORIZED_RECRUIT_BOARD("자신이 작성한 봉사 모집글이 아닙니다."),

src/test/java/com/somemore/community/service/query/CommunityBoardQueryServiceTest.java → src/test/java/com/somemore/community/service/CommunityBoardQueryServiceTest.java

@@ -1,4 +1,4 @@
-package com.somemore.community.service.query;
+package com.somemore.community.service;
 
 import com.somemore.IntegrationTestSupport;
 import com.somemore.auth.oauth.OAuthProvider;
@@ -9,8 +9,8 @@
 import com.somemore.community.dto.response.CommunityBoardGetDetailResponseDto;
 import com.somemore.community.dto.response.CommunityBoardGetResponseDto;
 import com.somemore.community.repository.CommunityBoardRepository;
-import com.somemore.community.usecase.command.CreateCommunityBoardUseCase;
-import com.somemore.community.usecase.command.DeleteCommunityBoardUseCase;
+import com.somemore.community.usecase.CreateCommunityBoardUseCase;
+import com.somemore.community.usecase.DeleteCommunityBoardUseCase;
 import com.somemore.global.exception.BadRequestException;
 import com.somemore.global.exception.ExceptionMessage;
 import com.somemore.volunteer.domain.Volunteer;

src/test/java/com/somemore/community/service/command/CreateCommunityBoardServiceTest.java → src/test/java/com/somemore/community/service/CreateCommunityBoardServiceTest.java

@@ -1,4 +1,4 @@
-package com.somemore.community.service.command;
+package com.somemore.community.service;
 
 import static org.assertj.core.api.Assertions.assertThat;
 

src/test/java/com/somemore/community/service/command/DeleteCommunityBoardServiceTest.java → src/test/java/com/somemore/community/service/DeleteCommunityBoardServiceTest.java

@@ -1,13 +1,13 @@
-package com.somemore.community.service.command;
+package com.somemore.community.service;
 
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatExceptionOfType;
 
 import com.somemore.IntegrationTestSupport;
 import com.somemore.community.dto.request.CommunityBoardCreateRequestDto;
 import com.somemore.community.repository.CommunityBoardRepository;
-import com.somemore.community.usecase.command.CreateCommunityBoardUseCase;
-import com.somemore.community.usecase.query.CommunityBoardQueryUseCase;
+import com.somemore.community.usecase.CreateCommunityBoardUseCase;
+import com.somemore.community.usecase.CommunityBoardQueryUseCase;
 import com.somemore.global.exception.BadRequestException;
 import com.somemore.global.exception.ExceptionMessage;
 import org.assertj.core.api.ThrowableAssert;

src/test/java/com/somemore/community/service/UpdateCommunityBoardServiceTest.java

@@ -0,0 +1,98 @@
+package com.somemore.community.service;
+
+import static org.assertj.core.api.Assertions.assertThat;
+import static org.assertj.core.api.Assertions.assertThatExceptionOfType;
+
+import com.somemore.IntegrationTestSupport;
+import com.somemore.community.domain.CommunityBoard;
+import com.somemore.community.dto.request.CommunityBoardCreateRequestDto;
+import com.somemore.community.dto.request.CommunityBoardUpdateRequestDto;
+import com.somemore.community.repository.CommunityBoardRepository;
+import com.somemore.community.usecase.CreateCommunityBoardUseCase;
+import com.somemore.global.exception.BadRequestException;
+import com.somemore.global.exception.ExceptionMessage;
+import org.assertj.core.api.ThrowableAssert;
+import org.junit.jupiter.api.AfterEach;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.DisplayName;
+import org.junit.jupiter.api.Test;
+import org.springframework.beans.factory.annotation.Autowired;
+
+import java.util.Optional;
+import java.util.UUID;
+
+class UpdateCommunityBoardServiceTest extends IntegrationTestSupport {
+
+    @Autowired
+    private CreateCommunityBoardUseCase createCommunityBoardUseCase;
+    @Autowired
+    private CommunityBoardRepository communityBoardRepository;
+    @Autowired
+    private UpdateCommunityBoardService updateCommunityBoardService;
+
+    private UUID writerId;
+    private Long communityId;
+    private String imgUrl;
+
+    @BeforeEach
+    void setUp() {
+        CommunityBoardCreateRequestDto dto = CommunityBoardCreateRequestDto.builder()
+                .title("커뮤니티 테스트 제목")
+                .content("커뮤니티 테스트 내용")
+                .build();
+
+        writerId = UUID.randomUUID();
+        imgUrl = "https://image.test.url/123";
+
+        communityId = createCommunityBoardUseCase.createCommunityBoard(dto, writerId, imgUrl);
+    }
+
+    @AfterEach
+    void tearDown() {
+        communityBoardRepository.deleteAllInBatch();
+    }
+
+    @DisplayName("커뮤니티 게시글을 수정한다.")
+    @Test
+    void updateCommunityBoard() {
+
+        //given
+        CommunityBoardUpdateRequestDto dto = CommunityBoardUpdateRequestDto.builder()
+                .title("수정된 커뮤니티 테스트 제목")
+                .content("수정된 커뮤니티 테스트 내용")
+                .build();
+
+        String newImgUrl = "https://image.test.url/567";
+
+        //when
+        updateCommunityBoardService.updateCommunityBoard(dto, communityId, writerId, newImgUrl);
+
+        //then
+        Optional<CommunityBoard> updatedCommunityBoard = communityBoardRepository.findById(communityId);
+        assertThat(updatedCommunityBoard).isNotNull();
+        assertThat(updatedCommunityBoard.get().getId()).isEqualTo(communityId);
+        assertThat(updatedCommunityBoard.get().getWriterId()).isEqualTo(writerId);
+        assertThat(updatedCommunityBoard.get().getTitle()).isEqualTo("수정된 커뮤니티 테스트 제목");
+        assertThat(updatedCommunityBoard.get().getContent()).isEqualTo("수정된 커뮤니티 테스트 내용");
+        assertThat(updatedCommunityBoard.get().getImgUrl()).isEqualTo("https://image.test.url/567");
+    }
+
+    @DisplayName("작성자가 아닌 id로 커뮤니티 게시글을 수정하고자 할 때 예외를 던진다.")
+    @Test
+    void updateCommunityBoardNotWriterId() {
+
+        //given
+        CommunityBoardUpdateRequestDto dto = CommunityBoardUpdateRequestDto.builder()
+                .title("수정된 커뮤니티 테스트 제목")
+                .content("수정된 커뮤니티 테스트 내용")
+                .build();
+
+        //when
+        ThrowableAssert.ThrowingCallable callable = () -> updateCommunityBoardService.updateCommunityBoard(dto, communityId, UUID.randomUUID(), null);
+
+        //then
+        assertThatExceptionOfType(BadRequestException.class)
+                .isThrownBy(callable)
+                .withMessage(ExceptionMessage.UNAUTHORIZED_COMMUNITY_BOARD.getMessage());
+    }
+}