[#40] feat: ChatContext 수정

Author: pythonstrup

src/main/java/com/nexters/teamace/chat/domain/ChatContext.java

@@ -4,9 +4,7 @@
 import static com.nexters.teamace.common.exception.ValidationErrorMessage.CHAT_ROOM_ID_POSITIVE;
 import static com.nexters.teamace.common.exception.ValidationErrorMessage.PREVIOUS_CHATS_NOT_NULL;
 
-import java.util.List;
-
-public record ChatContext(Long chatRoomId, List<Chat> previousChats) {
+public record ChatContext(Long chatRoomId, Chats previousChats) {
     public ChatContext {
         if (chatRoomId == null) {
             throw new IllegalArgumentException(CHAT_ROOM_ID_NOT_NULL);

src/main/java/com/nexters/teamace/chat/domain/ChatRoom.java

@@ -39,6 +39,6 @@ public ChatContext toChatContext() {
             throw new IllegalStateException(
                     "ChatRoom ID must not be null when creating ChatContext");
         }
-        return new ChatContext(this.id, this.chats.getChats());
+        return new ChatContext(this.id, this.chats);
     }
 }

src/test/java/com/nexters/teamace/chat/domain/ChatContextTest.java

@@ -27,7 +27,7 @@ class Context_with_valid_chatRoomId_and_empty_chats {
             void it_creates_chat_context() {
                 // Given
                 final Long chatRoomId = 1L;
-                final List<Chat> previousChats = List.of();
+                final Chats previousChats = new Chats();
 
                 // When
                 final ChatContext chatContext = new ChatContext(chatRoomId, previousChats);
@@ -50,15 +50,15 @@ void it_creates_chat_context() {
                 final Long chatRoomId = 5L;
                 final Chat chat1 = Chat.create(chatRoomId, MessageType.USER, "Hello");
                 final Chat chat2 = Chat.create(chatRoomId, MessageType.SYSTEM, "Hi there");
-                final List<Chat> previousChats = List.of(chat1, chat2);
+                final Chats previousChats = new Chats(List.of(chat1, chat2));
 
                 // When
                 final ChatContext chatContext = new ChatContext(chatRoomId, previousChats);
 
                 // Then
                 then(chatContext.chatRoomId()).isEqualTo(chatRoomId);
                 then(chatContext.previousChats()).isEqualTo(previousChats);
-                then(chatContext.previousChats()).hasSize(2);
+                then(chatContext.previousChats().size()).isEqualTo(2);
             }
         }
     }
@@ -76,7 +76,7 @@ class Context_when_chatRoomId_is_null {
             void it_throws_IllegalArgumentException() {
                 // Given
                 final Long chatRoomId = null;
-                final List<Chat> previousChats = List.of();
+                final Chats previousChats = new Chats();
 
                 // When & Then
                 thenThrownBy(() -> new ChatContext(chatRoomId, previousChats))
@@ -94,7 +94,7 @@ class Context_when_chatRoomId_is_zero {
             void it_throws_IllegalArgumentException() {
                 // Given
                 final Long chatRoomId = 0L;
-                final List<Chat> previousChats = List.of();
+                final Chats previousChats = new Chats();
 
                 // When & Then
                 thenThrownBy(() -> new ChatContext(chatRoomId, previousChats))
@@ -112,7 +112,7 @@ class Context_when_chatRoomId_is_negative {
             void it_throws_IllegalArgumentException() {
                 // Given
                 final Long chatRoomId = -1L;
-                final List<Chat> previousChats = List.of();
+                final Chats previousChats = new Chats();
 
                 // When & Then
                 thenThrownBy(() -> new ChatContext(chatRoomId, previousChats))
@@ -130,7 +130,7 @@ class Context_when_chatRoomId_is_one {
             void it_creates_chat_context() {
                 // Given
                 final Long chatRoomId = 1L;
-                final List<Chat> previousChats = List.of();
+                final Chats previousChats = new Chats();
 
                 // When
                 final ChatContext chatContext = new ChatContext(chatRoomId, previousChats);
@@ -154,7 +154,7 @@ class Context_when_previousChats_is_null {
             void it_throws_IllegalArgumentException() {
                 // Given
                 final Long chatRoomId = 1L;
-                final List<Chat> previousChats = null;
+                final Chats previousChats = null;
 
                 // When & Then
                 thenThrownBy(() -> new ChatContext(chatRoomId, previousChats))

src/test/java/com/nexters/teamace/chat/domain/ChatRoomTest.java

@@ -143,7 +143,7 @@ void it_returns_ChatContext_with_all_chats() {
 
                 // Then
                 then(chatContext).extracting("chatRoomId").isEqualTo(chatRoom.getId());
-                then(chatContext.previousChats()).hasSize(2);
+                then(chatContext.previousChats().size()).isEqualTo(2);
                 then(chatContext.previousChats())
                         .extracting("type", "message")
                         .containsExactly(

src/test/java/com/nexters/teamace/chat/domain/ChatsTest.java

@@ -48,7 +48,7 @@ void it_creates_chat_list_with_provided_chats() {
                 // Then
                 then(chats.size()).isEqualTo(2);
                 then(chats.isEmpty()).isFalse();
-                then(chats.getChats()).containsExactly(chat1, chat2);
+                then(chats).containsExactly(chat1, chat2);
             }
         }
 
@@ -99,27 +99,6 @@ void it_adds_chat_to_end_of_list() {
         }
     }
 
-    @Nested
-    @DisplayName("채팅 목록 조회")
-    class Describe_getChats {
-
-        @Test
-        @DisplayName("수정 불가능한 채팅 목록을 반환한다")
-        void it_returns_unmodifiable_chat_list() {
-            // Given
-            final Chat chat = Chat.create(1L, MessageType.USER, "Hello");
-            final Chats chats = new Chats(List.of(chat));
-
-            // When
-            final List<Chat> result = chats.getChats();
-
-            // Then
-            then(result).containsExactly(chat);
-            thenThrownBy(() -> result.add(Chat.create(2L, MessageType.SYSTEM, "Test")))
-                    .isInstanceOf(UnsupportedOperationException.class);
-        }
-    }
-
     @Nested
     @DisplayName("첫 번째 채팅 조회")
     class Describe_getFirst {