Skip to content

Commit 6ee3bbd

Browse files
committed
fix: 코드래빗 리뷰사항 적용
* refactor: 채널 생성 로직 중복 제거 * fix: 사용하지 않는 NewsStatus 제거 * fix: MentorMyPageServiceTest 에서 siteUserId 를 mentorId 처럼 사용하는 문제 수정
1 parent 06434d0 commit 6ee3bbd

File tree

3 files changed

+8
-15
lines changed

3 files changed

+8
-15
lines changed

src/main/java/com/example/solidconnection/mentor/service/MentorMyPageService.java

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -70,11 +70,7 @@ public void updateMentorMyPage(long siteUserId, MentorMyPageUpdateRequest reques
7070
}
7171

7272
private void updateChannel(List<ChannelRequest> channelRequests, Mentor mentor) {
73-
int sequence = CHANNEL_SEQUENCE_START_NUMBER;
74-
List<Channel> newChannels = new ArrayList<>();
75-
for (ChannelRequest request : channelRequests) {
76-
newChannels.add(new Channel(sequence++, request.type(), request.url()));
77-
}
73+
List<Channel> newChannels = buildChannels(channelRequests);
7874
mentor.updateChannels(newChannels);
7975
}
8076

@@ -114,11 +110,16 @@ private void validateChannelRegistrationLimit(List<ChannelRequest> channelReques
114110
}
115111

116112
private void createChannels(List<ChannelRequest> channelRequests, Mentor mentor) {
113+
List<Channel> newChannels = buildChannels(channelRequests);
114+
mentor.createChannels(newChannels);
115+
}
116+
117+
private List<Channel> buildChannels(List<ChannelRequest> channelRequests) {
117118
int sequence = CHANNEL_SEQUENCE_START_NUMBER;
118119
List<Channel> newChannels = new ArrayList<>();
119120
for (ChannelRequest request : channelRequests) {
120121
newChannels.add(new Channel(sequence++, request.type(), request.url()));
121122
}
122-
mentor.createChannels(newChannels);
123+
return newChannels;
123124
}
124125
}

src/main/java/com/example/solidconnection/news/domain/NewsStatus.java

Lines changed: 0 additions & 8 deletions
This file was deleted.

src/test/java/com/example/solidconnection/mentor/service/MentorMyPageServiceTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -193,7 +193,7 @@ class 멘토의_마이페이지를_생성한다 {
193193
mentorMyPageService.createMentorMyPage(tempMentorUser.getId(), request);
194194

195195
// then
196-
Mentor createTempMentor = mentorRepository.findById(tempMentorUser.getId()).get();
196+
Mentor createTempMentor = mentorRepository.findBySiteUserId(tempMentorUser.getId()).get();
197197
assertAll(
198198
() -> assertThat(createTempMentor.getIntroduction()).isEqualTo(introduction),
199199
() -> assertThat(createTempMentor.getPassTip()).isEqualTo(passTip),

0 commit comments

Comments
 (0)