Merge pull request #1328 from Moadong/feature/#1326-add-patch-promotion-controller-MOA-742

[feature] 홍보 게시글 수정 API 추가

Author: lepitaaar

backend/src/main/java/moadong/club/controller/PromotionArticleController.java

@@ -5,14 +5,17 @@
 import io.swagger.v3.oas.annotations.tags.Tag;
 import lombok.RequiredArgsConstructor;
 import moadong.club.payload.request.PromotionArticleCreateRequest;
+import moadong.club.payload.request.PromotionArticleUpdateRequest;
 import moadong.club.payload.response.PromotionArticleResponse;
 import moadong.club.service.PromotionArticleService;
 import moadong.global.payload.Response;
 import org.springframework.http.ResponseEntity;
 import org.springframework.security.access.prepost.PreAuthorize;
 import org.springframework.validation.annotation.Validated;
 import org.springframework.web.bind.annotation.GetMapping;
+import org.springframework.web.bind.annotation.PathVariable;
 import org.springframework.web.bind.annotation.PostMapping;
+import org.springframework.web.bind.annotation.PutMapping;
 import org.springframework.web.bind.annotation.RequestBody;
 import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.RestController;
@@ -41,4 +44,15 @@ public ResponseEntity<?> createPromotionArticle(
         promotionArticleService.createPromotionArticle(request);
         return Response.ok("홍보 게시글이 생성되었습니다.");
     }
+
+    @PutMapping("/{articleId}")
+    @Operation(summary = "홍보 게시글 수정", description = "기존 홍보 게시글을 수정합니다.")
+    @PreAuthorize("hasRole('DEVELOPER')")
+    @SecurityRequirement(name = "BearerAuth")
+    public ResponseEntity<?> updatePromotionArticle(
+        @PathVariable String articleId,
+        @RequestBody @Validated PromotionArticleUpdateRequest request) {
+        promotionArticleService.updatePromotionArticle(articleId, request);
+        return Response.ok("홍보 게시글이 수정되었습니다.");
+    }
 }

backend/src/main/java/moadong/club/entity/PromotionArticle.java

@@ -4,6 +4,7 @@
 import lombok.Builder;
 import lombok.Getter;
 import lombok.NoArgsConstructor;
+import moadong.club.payload.request.PromotionArticleUpdateRequest;
 import org.springframework.data.annotation.Id;
 import org.springframework.data.mongodb.core.mapping.Document;
 
@@ -38,4 +39,15 @@ public class PromotionArticle {
 
     @Builder.Default
     private Instant createdAt = Instant.now();
+
+    public void update(PromotionArticleUpdateRequest request, String clubName) {
+        this.clubId = request.clubId();
+        this.clubName = clubName;
+        this.title = request.title();
+        this.location = request.location();
+        this.eventStartDate = request.eventStartDate();
+        this.eventEndDate = request.eventEndDate();
+        this.description = request.description();
+        this.images = request.images();
+    }
 }

backend/src/main/java/moadong/club/payload/dto/PromotionArticleDto.java

@@ -1,9 +1,12 @@
 package moadong.club.payload.dto;
 
+import moadong.club.entity.PromotionArticle;
+
 import java.time.Instant;
 import java.util.List;
 
 public record PromotionArticleDto(
+    String id,
     String clubName,
     String clubId,
     String title,
@@ -13,4 +16,17 @@ public record PromotionArticleDto(
     String description,
     List<String> images
 ) {
+    public static PromotionArticleDto from(PromotionArticle article) {
+        return new PromotionArticleDto(
+            article.getId(),
+            article.getClubName(),
+            article.getClubId(),
+            article.getTitle(),
+            article.getLocation(),
+            article.getEventStartDate(),
+            article.getEventEndDate(),
+            article.getDescription(),
+            article.getImages()
+        );
+    }
 }

backend/src/main/java/moadong/club/payload/request/PromotionArticleUpdateRequest.java

@@ -0,0 +1,19 @@
+package moadong.club.payload.request;
+
+import jakarta.validation.constraints.NotBlank;
+import jakarta.validation.constraints.NotEmpty;
+import jakarta.validation.constraints.NotNull;
+
+import java.time.Instant;
+import java.util.List;
+
+public record PromotionArticleUpdateRequest(
+    @NotBlank String clubId,
+    @NotBlank String title,
+    @NotBlank String location,
+    @NotNull Instant eventStartDate,
+    @NotNull Instant eventEndDate,
+    @NotBlank String description,
+    @NotEmpty List<String> images
+) {
+}

backend/src/main/java/moadong/club/repository/PromotionArticleRepository.java

@@ -1,18 +1,15 @@
 package moadong.club.repository;
 
 import moadong.club.entity.PromotionArticle;
-import moadong.club.payload.dto.PromotionArticleDto;
 import org.springframework.data.mongodb.repository.MongoRepository;
-import org.springframework.data.mongodb.repository.Query;
 import org.springframework.stereotype.Repository;
 
 import java.util.List;
 
 @Repository
 public interface PromotionArticleRepository extends MongoRepository<PromotionArticle, String> {
 
-    @Query(value = "{}", sort = "{ 'createdAt': -1 }")
-    List<PromotionArticleDto> findAllProjectedBy();
+    List<PromotionArticle> findAllByOrderByCreatedAtDesc();
 
     List<PromotionArticle> findByClubIdOrderByCreatedAtDesc(String clubId);
 }

backend/src/main/java/moadong/club/service/PromotionArticleService.java

@@ -5,6 +5,7 @@
 import moadong.club.entity.PromotionArticle;
 import moadong.club.payload.dto.PromotionArticleDto;
 import moadong.club.payload.request.PromotionArticleCreateRequest;
+import moadong.club.payload.request.PromotionArticleUpdateRequest;
 import moadong.club.payload.response.PromotionArticleResponse;
 import moadong.club.repository.ClubRepository;
 import moadong.club.repository.PromotionArticleRepository;
@@ -25,15 +26,16 @@ public class PromotionArticleService {
     private final ClubRepository clubRepository;
 
     public PromotionArticleResponse getPromotionArticles() {
-        List<PromotionArticleDto> articles = promotionArticleRepository.findAllProjectedBy();
+        List<PromotionArticleDto> articles = promotionArticleRepository.findAllByOrderByCreatedAtDesc()
+            .stream()
+            .map(PromotionArticleDto::from)
+            .toList();
         return new PromotionArticleResponse(articles);
     }
 
     @Transactional
     public void createPromotionArticle(PromotionArticleCreateRequest request) {
-        ObjectId clubObjectId = ObjectIdConverter.convertString(request.clubId());
-        Club club = clubRepository.findClubById(clubObjectId)
-            .orElseThrow(() -> new RestApiException(ErrorCode.CLUB_NOT_FOUND));
+        Club club = getClub(request.clubId());
 
         PromotionArticle article = PromotionArticle.builder()
             .clubId(request.clubId())
@@ -48,4 +50,20 @@ public void createPromotionArticle(PromotionArticleCreateRequest request) {
 
         promotionArticleRepository.save(article);
     }
+
+    @Transactional
+    public void updatePromotionArticle(String articleId, PromotionArticleUpdateRequest request) {
+        PromotionArticle article = promotionArticleRepository.findById(articleId)
+            .orElseThrow(() -> new RestApiException(ErrorCode.PROMOTION_ARTICLE_NOT_FOUND));
+        Club club = getClub(request.clubId());
+
+        article.update(request, club.getName());
+        promotionArticleRepository.save(article);
+    }
+
+    private Club getClub(String clubId) {
+        ObjectId clubObjectId = ObjectIdConverter.convertString(clubId);
+        return clubRepository.findClubById(clubObjectId)
+            .orElseThrow(() -> new RestApiException(ErrorCode.CLUB_NOT_FOUND));
+    }
 }

backend/src/main/java/moadong/global/exception/ErrorCode.java

@@ -68,7 +68,11 @@ public enum ErrorCode {
     // 901xx: FCM 관련 오류
     FCMTOKEN_NOT_FOUND(HttpStatus.NOT_FOUND, "901-1", "존재하지 않는 토큰입니다."),
     FCMTOKEN_SUBSCRIBE_ERROR(HttpStatus.INTERNAL_SERVER_ERROR, "901-2", "동아리 구독중에 오류가 발생 하였습니다."),
-    FCMMESSAGE_SEND_ERROR(HttpStatus.INTERNAL_SERVER_ERROR, "901-3", "FCM 메시지 전송 중 오류가 발생했습니다.");
+    FCMMESSAGE_SEND_ERROR(HttpStatus.INTERNAL_SERVER_ERROR, "901-3", "FCM 메시지 전송 중 오류가 발생했습니다."),
+
+    // 902xx: 홍보게시판 오류
+    PROMOTION_ARTICLE_NOT_FOUND(HttpStatus.NOT_FOUND, "902-1", "홍보 게시글이 존재하지 않습니다."),
+    ;
 
     private final HttpStatus httpStatus;
     private final String code;

backend/src/test/java/moadong/club/service/PromotionArticleServiceTest.java

@@ -0,0 +1,125 @@
+package moadong.club.service;
+
+import moadong.club.entity.Club;
+import moadong.club.entity.PromotionArticle;
+import moadong.club.payload.request.PromotionArticleUpdateRequest;
+import moadong.club.payload.response.PromotionArticleResponse;
+import moadong.club.repository.ClubRepository;
+import moadong.club.repository.PromotionArticleRepository;
+import moadong.global.exception.ErrorCode;
+import moadong.global.exception.RestApiException;
+import org.bson.types.ObjectId;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
+import org.mockito.InjectMocks;
+import org.mockito.Mock;
+import org.mockito.junit.jupiter.MockitoExtension;
+
+import java.time.Instant;
+import java.util.List;
+import java.util.Optional;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertThrows;
+import static org.mockito.Mockito.never;
+import static org.mockito.Mockito.verify;
+import static org.mockito.Mockito.when;
+
+@ExtendWith(MockitoExtension.class)
+class PromotionArticleServiceTest {
+
+    @Mock
+    private PromotionArticleRepository promotionArticleRepository;
+
+    @Mock
+    private ClubRepository clubRepository;
+
+    @InjectMocks
+    private PromotionArticleService promotionArticleService;
+
+    @Test
+    void 홍보게시글_목록조회시_id를_포함해_반환한다() {
+        PromotionArticle article = PromotionArticle.builder()
+            .id("article-1")
+            .clubId("65f0c1d5a4b1c92f57d51111")
+            .clubName("모아동")
+            .title("봄 모집")
+            .location("서울")
+            .eventStartDate(Instant.parse("2026-03-01T00:00:00Z"))
+            .eventEndDate(Instant.parse("2026-03-31T00:00:00Z"))
+            .description("설명")
+            .images(List.of("image-1"))
+            .build();
+        when(promotionArticleRepository.findAllByOrderByCreatedAtDesc()).thenReturn(List.of(article));
+
+        PromotionArticleResponse response = promotionArticleService.getPromotionArticles();
+
+        assertEquals(1, response.articles().size());
+        assertEquals("article-1", response.articles().get(0).id());
+        assertEquals("모아동", response.articles().get(0).clubName());
+    }
+
+    @Test
+    void 홍보게시글을_수정한다() {
+        String clubId = new ObjectId().toHexString();
+        PromotionArticle article = PromotionArticle.builder()
+            .id("article-1")
+            .clubId("old-club-id")
+            .clubName("이전 동아리")
+            .title("이전 제목")
+            .location("이전 장소")
+            .eventStartDate(Instant.parse("2026-03-01T00:00:00Z"))
+            .eventEndDate(Instant.parse("2026-03-02T00:00:00Z"))
+            .description("이전 설명")
+            .images(List.of("old-image"))
+            .build();
+        PromotionArticleUpdateRequest request = new PromotionArticleUpdateRequest(
+            clubId,
+            "수정 제목",
+            "수정 장소",
+            Instant.parse("2026-04-01T00:00:00Z"),
+            Instant.parse("2026-04-10T00:00:00Z"),
+            "수정 설명",
+            List.of("new-image-1", "new-image-2")
+        );
+        when(promotionArticleRepository.findById("article-1")).thenReturn(Optional.of(article));
+        when(clubRepository.findClubById(new ObjectId(clubId))).thenReturn(Optional.of(Club.builder()
+            .name("수정 동아리")
+            .category("category")
+            .division("division")
+            .userId("user-1")
+            .build()));
+
+        promotionArticleService.updatePromotionArticle("article-1", request);
+
+        assertEquals(clubId, article.getClubId());
+        assertEquals("수정 동아리", article.getClubName());
+        assertEquals("수정 제목", article.getTitle());
+        assertEquals("수정 장소", article.getLocation());
+        assertEquals(Instant.parse("2026-04-01T00:00:00Z"), article.getEventStartDate());
+        assertEquals(Instant.parse("2026-04-10T00:00:00Z"), article.getEventEndDate());
+        assertEquals("수정 설명", article.getDescription());
+        assertEquals(List.of("new-image-1", "new-image-2"), article.getImages());
+        verify(promotionArticleRepository).save(article);
+    }
+
+    @Test
+    void 존재하지_않는_홍보게시글이면_예외를_던진다() {
+        PromotionArticleUpdateRequest request = new PromotionArticleUpdateRequest(
+            new ObjectId().toHexString(),
+            "수정 제목",
+            "수정 장소",
+            Instant.parse("2026-04-01T00:00:00Z"),
+            Instant.parse("2026-04-10T00:00:00Z"),
+            "수정 설명",
+            List.of("new-image")
+        );
+        when(promotionArticleRepository.findById("missing-article")).thenReturn(Optional.empty());
+
+        RestApiException exception = assertThrows(RestApiException.class,
+            () -> promotionArticleService.updatePromotionArticle("missing-article", request));
+
+        assertEquals(ErrorCode.PROMOTION_ARTICLE_NOT_FOUND, exception.getErrorCode());
+        verify(clubRepository, never()).findClubById(org.mockito.ArgumentMatchers.any());
+    }
+}