Refactor/201 쪽지 상세조회시 읽음처리 기능 추가 (#202)

* refactor: 쪽지 상세조회시 읽음처리 기능 추가

- Note 엔티티에 isRead 업데이트 기능 구현
- 기존 쪽지 상세조회 로직에 isRead 업데이트 기능 추가

* refactor: 읽음처리 메서드의 책임 분리

- 읽음 처리를 위한 유스케이스 추가
- 읽음 처리 유스케이스 구현체 추가
- 테스트 코드 검증및 구현

* fix: 쪽지 읽음처리 테스트 코드 수정

- noteId를 고정값이 아닌 생성된 쪽지의 id를 사용하도록 수정

Author: 7zrv

src/main/java/com/somemore/note/domain/Note.java

@@ -53,4 +53,8 @@ public static Note create(UUID senderId, UUID receiverId, String title, String c
                 .build();
     }
 
+    public void markAsRead() {
+        this.isRead = true;
+    }
+
 }

src/main/java/com/somemore/note/repository/NoteRepository.java

@@ -13,6 +13,7 @@
 
 public interface NoteRepository {
     Note save(Note note);
+    Optional<Note> findById(Long noteId);
     Page<NoteReceiverViewForCenter> findNotesByReceiverIsCenter(UUID centerId, Pageable pageable);
     Page<NoteReceiverViewForVolunteer> findNotesByReceiverIsVolunteer(UUID volunteerId, Pageable pageable);
     Optional<NoteDetailViewForCenter> findNoteDetailViewReceiverIsCenter(Long noteId);

src/main/java/com/somemore/note/repository/NoteRepositoryImpl.java

@@ -42,6 +42,11 @@ public Note save(Note note) {
         return noteJpaRepository.save(note);
     }
 
+    @Override
+    public Optional<Note> findById(Long noteId) {
+        return noteJpaRepository.findById(noteId);
+    }
+
     @Override
     public Page<NoteReceiverViewForCenter> findNotesByReceiverIsCenter(UUID centerId, Pageable pageable) {
 

src/main/java/com/somemore/note/service/NoteMarkAsReadService.java

@@ -0,0 +1,30 @@
+package com.somemore.note.service;
+
+import com.somemore.global.exception.NoSuchElementException;
+import com.somemore.note.domain.Note;
+import com.somemore.note.repository.NoteRepository;
+import com.somemore.note.usecase.NoteMarkAsReadUseCase;
+import lombok.RequiredArgsConstructor;
+import org.springframework.stereotype.Service;
+import org.springframework.transaction.annotation.Transactional;
+
+import static com.somemore.global.exception.ExceptionMessage.NOT_EXISTS_NOTE;
+
+@RequiredArgsConstructor
+@Service
+@Transactional
+public class NoteMarkAsReadService implements NoteMarkAsReadUseCase {
+
+    private final NoteRepository noteRepository;
+
+    @Override
+    public void noteMarkAsRead(Long noteId) {
+        Note note = getNote(noteId);
+        note.markAsRead();
+    }
+
+    private Note getNote(Long noteId) {
+        return noteRepository.findById(noteId)
+                .orElseThrow(() -> new NoSuchElementException(NOT_EXISTS_NOTE));
+    }
+}

src/main/java/com/somemore/note/service/NoteQueryService.java

@@ -6,6 +6,7 @@
 import com.somemore.note.repository.mapper.NoteDetailViewForVolunteer;
 import com.somemore.note.repository.mapper.NoteReceiverViewForCenter;
 import com.somemore.note.repository.mapper.NoteReceiverViewForVolunteer;
+import com.somemore.note.usecase.NoteMarkAsReadUseCase;
 import com.somemore.note.usecase.NoteQueryUseCase;
 import lombok.RequiredArgsConstructor;
 import org.springframework.data.domain.Page;
@@ -22,6 +23,7 @@
 @Transactional(readOnly = true)
 public class NoteQueryService implements NoteQueryUseCase {
 
+    private final NoteMarkAsReadUseCase noteMarkAsReadUseCase;
     private final NoteRepository noteRepository;
 
     @Override
@@ -36,12 +38,16 @@ public Page<NoteReceiverViewForVolunteer> getNotesForVolunteer(UUID volunteerId,
 
     @Override
     public NoteDetailViewForCenter getNoteDetailForCenter(Long noteId) {
+        noteMarkAsReadUseCase.noteMarkAsRead(noteId);
+
         return noteRepository.findNoteDetailViewReceiverIsCenter(noteId)
                 .orElseThrow(() -> new NoSuchElementException(NOT_EXISTS_NOTE));
     }
 
     @Override
     public NoteDetailViewForVolunteer getNoteDetailForVolunteer(Long noteId) {
+        noteMarkAsReadUseCase.noteMarkAsRead(noteId);
+
         return noteRepository.findNoteDetailViewReceiverIsVolunteer(noteId)
                 .orElseThrow(() -> new NoSuchElementException(NOT_EXISTS_NOTE));
     }

src/main/java/com/somemore/note/usecase/NoteMarkAsReadUseCase.java

@@ -0,0 +1,5 @@
+package com.somemore.note.usecase;
+
+public interface NoteMarkAsReadUseCase {
+    void noteMarkAsRead(Long noteId);
+}

src/test/java/com/somemore/note/service/NoteMarkAsReadServiceTest.java

@@ -0,0 +1,52 @@
+package com.somemore.note.service;
+
+import com.somemore.IntegrationTestSupport;
+import com.somemore.global.exception.NoSuchElementException;
+import com.somemore.note.domain.Note;
+import com.somemore.note.repository.NoteRepository;
+import org.junit.jupiter.api.DisplayName;
+import org.junit.jupiter.api.Test;
+import org.springframework.beans.factory.annotation.Autowired;
+
+import java.util.UUID;
+
+import static com.somemore.global.exception.ExceptionMessage.NOT_EXISTS_NOTE;
+import static org.junit.jupiter.api.Assertions.*;
+
+class NoteMarkAsReadServiceTest extends IntegrationTestSupport {
+
+    @Autowired
+    private NoteMarkAsReadService noteMarkAsReadService;
+
+    @Autowired
+    private NoteRepository noteRepository;
+
+    @DisplayName("쪽지를 읽음 처리할 수 있다.")
+    @Test
+    void noteMarkAsRead() {
+        // given
+        Note note = Note.create(UUID.randomUUID(), UUID.randomUUID(), "제목", "내용");
+        noteRepository.save(note);
+
+        // when
+        noteMarkAsReadService.noteMarkAsRead(note.getId());
+
+        // then
+        Note updatedNote = noteRepository.findById(note.getId())
+                .orElseThrow(() -> new NoSuchElementException(NOT_EXISTS_NOTE));
+
+        assertTrue(updatedNote.getIsRead(), "노트가 읽음 상태로 변경되어야 한다.");
+    }
+
+    @DisplayName("존재하지 않는 쪽지에 대한 업데이트 요청은 예외를 반환한다.")
+    @Test
+    void noteMarkAsRead_throwsException_WhenNoteNotFound() {
+        // given
+        Long nonExistentNoteId = 999L;
+
+        // when & then
+        assertThrows(NoSuchElementException.class,
+                () -> noteMarkAsReadService.noteMarkAsRead(nonExistentNoteId),
+                "존재하지 않는 노트에 대해 예외가 발생해야 한다.");
+    }
+}