Refact/#362 chat domain #367

tidavid1 · 2024-04-18T13:03:49Z

🚀 개발 사항

기존 정규화를 통해 분리된 ChatUser 와 ChatRoom Entity를 사용하지 않고, ChatRoom Document를 활용하여 채팅방 정보를 저장하도록 변경했습니다.
채팅방 저장 형식이 변경됨에 따라 채팅 조회 방식을 이에 맞췄습니다.
- ChatRoom Document를 활용함에 따라 마지막 채팅 기록을 채팅 작성시 ChatRoom Document에 저장하도록 리팩토링을 진행했습니다.
- 이제 채팅방 리스트 조회 과정에서 MySQL + MongoDB 쿼리 조회 방식이 아닌 MongoDB 단일 도큐먼트 조회 방식을 사용합니다.
~~채팅방 리스트 조회 API 응답 값이 변경되었습니다.~~ (프론트와 협의되지 않은 부분이어서 이전 응답으로 롤백했습니다. | 24.04.25 추가)
- ~~기존 채팅 상대 유저 정보를 제공하는 응답 값에서 프로젝트 상세 정보를 제공하도록 변경되었습니다.~~
도큐먼트에 대한 테스트를 추가했습니다.
서비스 레이어에 대한 테스트를 추가했습니다.

이슈 번호

특이 사항 🫶

테스트 작성 진행중입니다.
메인 반영시 기존 RDB에 존재하는 채팅 데이터를 MongoDB로 이관하는 작업이 필요합니다. 이부분은 추가적으로 구현할 예정입니다 :D

github-actions · 2024-04-18T13:05:11Z

Test Results

76 tests 76 ✅ 2s ⏱️
11 suites 0 💤
11 files 0 ❌

Results for commit d926a78.

♻️ This comment has been updated with latest results.

JIN-076 · 2024-04-25T11:06:28Z

src/main/java/io/oeid/mogakgo/domain/chat/infrastructure/ChatRepository.java

@@ -28,15 +26,12 @@ public ChatMessage save(ChatMessage chatMessage, String collectionName) {
        return mongoTemplate.save(chatMessage, collectionName);
    }

-    public CursorPaginationResult<ChatDataApiRes> findAllByCollection(String collectionName,
-        CursorPaginationInfoReq pageable) {
+    public List<ChatMessage> findAllByCollection(String collectionName,


CursorPaginationResult 타입이 아닌 List 타입으로 변경하면서 'cursorId로 받은 값부터 가져오게 한다' 라고 이해가 되었는데 맞나요?🤔

맞습니다 👍🏻

JIN-076 · 2024-04-25T16:26:45Z

src/main/java/io/oeid/mogakgo/domain/chat/entity/vo/ChatRoomDetail.java

+    private final LocalDateTime meetStartTime;
+    private final LocalDateTime meetEndTime;
+
+    private ChatRoomDetail(Long projectId, MeetingInfo meetingInfo) {


전반적으로 보았을 때, 채팅방에 대한 상세 정보를 가져올 때 meetingInfo 관련 데이터를 함께 가져오고 있는데 딱히 쓰이지는 않는 것 같습니다. 해당 데이터를 반환하는 이유가 따로 있나요?

채팅방 상세 정보 조회 API 응답에 프로젝트 상세 정보, 위치, 시간이 필요 하여 응답으로 전송해주고 있습니다. -> 채팅방 접속시 위에 보이는 프로젝트 진행 위치 관련 정보가 상세 정보 조회 API로 반환됩니다!

JIN-076

전반적으로 깔끔하게 리팩토링이 잘 된 것 같습니다! 남은 코멘트 확인 부탁드려요~👍🏻

tidavid1 added 16 commits April 15, 2024 11:17

[REFACT] ChatMessageType Deprecated 처리

86cae82

[REFACT] ChatRoom Document 구현

46c50b6

[TEST] ChatRoom Document 테스트 추가

bc49ee9

[REFACT] ChatRoom 서비스 구현

4a2e1d3

[REFACT] ChatMessageType Enum 제거

0c00015

[REFACT] ChatMessage 리펙토링 및 테스트

96e99f7

[REFACT] 채팅 전송시 채팅방 마지막 메시지 갱신 방식으로 리팩토링

9d3ed6e

[TEST] ChatRoom Document 테스트 추가

17a5e58

[REFACT] ChatController 리펙토링

664cf57

[REFACT] ChatService 리펙토링

a1c9d2d

[REFACT] ChatRoomDataRes 리펙토링

606b3ce

[REFACT] 채팅방 생성 로직 오류 수정

e5de7f0

[REMOVE] 사용하지 않는 클래스 제거 및 @Deprecated 처리

85466da

[TEST] Mock 객체 작동 방식 수정

062f2d5

[TEST] ChatService 테스트 추가

d0da978

[REFACT] 사용하지 않는 정적 팩토리 메서드 제거

c3001d1

tidavid1 added the 🔄 Refactor 기존 개발 된 코드의 수정 label Apr 18, 2024

tidavid1 requested a review from JIN-076 April 18, 2024 13:03

tidavid1 self-assigned this Apr 18, 2024

tidavid1 added 2 commits April 19, 2024 12:15

[TEST] ChatWebSocketService 테스트 추가

af2afdb

[REFACT] API 응답 롤백

7cb783d

JIN-076 reviewed Apr 25, 2024

View reviewed changes

JIN-076 approved these changes Apr 25, 2024

View reviewed changes

Merge branch 'develop' into refact/#362-chat_domain

d926a78

tidavid1 merged commit 7576395 into develop Apr 28, 2024
3 checks passed

tidavid1 deleted the refact/#362-chat_domain branch April 28, 2024 15:39

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Refact/#362 chat domain #367

Refact/#362 chat domain #367

tidavid1 commented Apr 18, 2024 •

edited

Loading

github-actions bot commented Apr 18, 2024 •

edited

Loading

JIN-076 Apr 25, 2024 •

edited

Loading

tidavid1 Apr 25, 2024

JIN-076 Apr 25, 2024

tidavid1 Apr 26, 2024

JIN-076 left a comment

Refact/#362 chat domain #367

Refact/#362 chat domain #367

Conversation

tidavid1 commented Apr 18, 2024 • edited Loading

🚀 개발 사항

이슈 번호

특이 사항 🫶

github-actions bot commented Apr 18, 2024 • edited Loading

Test Results

JIN-076 Apr 25, 2024 • edited Loading

Choose a reason for hiding this comment

tidavid1 Apr 25, 2024

Choose a reason for hiding this comment

JIN-076 Apr 25, 2024

Choose a reason for hiding this comment

tidavid1 Apr 26, 2024

Choose a reason for hiding this comment

JIN-076 left a comment

Choose a reason for hiding this comment

tidavid1 commented Apr 18, 2024 •

edited

Loading

github-actions bot commented Apr 18, 2024 •

edited

Loading

JIN-076 Apr 25, 2024 •

edited

Loading