refactor: 멤버 파트 응답 속성 이름 변경

woowacourse-teams · Oct 18, 2023 · e44bcc0 · e44bcc0
1 parent 10cea5f
commit e44bcc0
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 8 deletions.
diff --git a/backend/src/main/java/shook/shook/song/application/dto/MemberPartResponse.java b/backend/src/main/java/shook/shook/song/application/dto/MemberPartResponse.java
@@ -12,7 +12,7 @@
 public class MemberPartResponse {
 
     @Schema(description = "멤버 파트 id", example = "1")
-    private final Long partId;
+    private final Long id;
 
     @Schema(description = "멤버 파트 시작 초", example = "30")
     private final int start;

diff --git a/backend/src/test/java/shook/shook/song/application/SongServiceTest.java b/backend/src/test/java/shook/shook/song/application/SongServiceTest.java
@@ -136,7 +136,7 @@ void findById_exist_login_member() {
                                              song.getLikeCountSortedKillingParts().get(2).getId())
                 .hasFieldOrPropertyWithValue("rank", 3)
                 .hasFieldOrPropertyWithValue("likeStatus", true),
-            () -> assertThat(response.getCurrentSong().getMemberPart().getPartId()).isNotNull()
+            () -> assertThat(response.getCurrentSong().getMemberPart().getId()).isNotNull()
         );
     }
 
@@ -314,12 +314,12 @@ void firstFindByMember() {
                                      .toList()).usingRecursiveComparison().isEqualTo(List.of(2L, 1L)),
                 () -> assertThat(result.getCurrentSong().getMemberPart()).isNull(),
                 () -> assertThat(result.getPrevSongs().stream()
-                                     .map(songResponse -> songResponse.getMemberPart().getPartId())
+                                     .map(songResponse -> songResponse.getMemberPart().getId())
                                      .toList())
                     .usingRecursiveComparison()
                     .isEqualTo(List.of(4L, 3L)),
                 () -> assertThat(result.getNextSongs().stream()
-                                     .map(songResponse -> songResponse.getMemberPart().getPartId())
+                                     .map(songResponse -> songResponse.getMemberPart().getId())
                                      .toList())
                     .usingRecursiveComparison()
                     .isEqualTo(List.of(2L, 1L))
@@ -389,7 +389,7 @@ void findSongByIdForBeforeSwipe() {
                            .toList()).usingRecursiveComparison().isEqualTo(List.of(2L, 4L, 1L, 5L));
             assertThat(beforeResponses.stream()
                            .map(SongResponse::getMemberPart)
-                           .map(MemberPartResponse::getPartId)
+                           .map(MemberPartResponse::getId)
                            .toList()).usingRecursiveComparison().isEqualTo(List.of(2L, 4L, 1L, 5L));
         }
 
@@ -433,7 +433,7 @@ void findSongByIdForAfterSwipe() {
                            .toList()).usingRecursiveComparison().isEqualTo(List.of(1L, 5L, 3L));
             assertThat(afterResponses.stream()
                            .map(SongResponse::getMemberPart)
-                           .map(MemberPartResponse::getPartId)
+                           .map(MemberPartResponse::getId)
                            .toList()).usingRecursiveComparison().isEqualTo(List.of(1L, 5L, 3L));
         }
     }
@@ -518,7 +518,7 @@ void findSongById() {
                                              song.getLikeCountSortedKillingParts().get(2).getId())
                 .hasFieldOrPropertyWithValue("rank", 3)
                 .hasFieldOrPropertyWithValue("likeStatus", true),
-            () -> assertThat(response.getMemberPart().getPartId()).isNotNull()
+            () -> assertThat(response.getMemberPart().getId()).isNotNull()
         );
     }
 }
diff --git a/backend/src/test/java/shook/shook/song/ui/SongSwipeControllerTest.java b/backend/src/test/java/shook/shook/song/ui/SongSwipeControllerTest.java
@@ -120,7 +120,7 @@ void showSongsBeforeSongWithId() {
         assertThat(response.stream()
                        .map(SongResponse::getId).toList())
             .containsExactly(1L, 4L, 3L);
-        assertThat(response.get(0).getMemberPart().getPartId()).isNotNull();
+        assertThat(response.get(0).getMemberPart().getId()).isNotNull();
         assertThat(response.get(1).getMemberPart()).isNull();
         assertThat(response.get(2).getMemberPart()).isNull();
     }