Skip to content

Commit 87991de

Browse files
committed
test: 멘토 관련 테스트 코드 수정
1 parent c8f35d1 commit 87991de

5 files changed

Lines changed: 15 additions & 15 deletions

File tree

src/main/java/com/example/solidconnection/mentor/dto/MentoringApplyRequest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44

55
public record MentoringApplyRequest(
66
@NotNull(message = "멘토 id를 입력해주세요.")
7-
Long siteUserId
7+
Long mentorId
88
) {
99

1010
}

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ public class MentoringCommandService {
3131

3232
@Transactional
3333
public MentoringApplyResponse applyMentoring(long siteUserId, MentoringApplyRequest mentoringApplyRequest) {
34-
long mentorSiteUserId = mentoringApplyRequest.siteUserId();
34+
long mentorSiteUserId = mentoringApplyRequest.mentorId();
3535

3636
Mentor mentor = mentorRepository.findBySiteUserId(mentorSiteUserId)
3737
.orElseThrow(() -> new CustomException(MENTOR_NOT_FOUND));

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

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -77,11 +77,11 @@ class 멘토_단일_조회_성공 {
7777
Channel channel2 = channelFixture.채널(2, mentor);
7878

7979
// when
80-
MentorDetailResponse response = mentorQueryService.getMentorDetails(mentor.getId(), siteUser.getId());
80+
MentorDetailResponse response = mentorQueryService.getMentorDetails(mentor.getSiteUserId(), siteUser.getId());
8181

8282
// then
8383
assertAll(
84-
() -> assertThat(response.id()).isEqualTo(mentor.getId()),
84+
() -> assertThat(response.id()).isEqualTo(mentor.getSiteUserId()),
8585
() -> assertThat(response.nickname()).isEqualTo(mentorUser.getNickname()),
8686
() -> assertThat(response.universityName()).isEqualTo(university.getKoreanName()),
8787
() -> assertThat(response.country()).isEqualTo(university.getCountry().getKoreanName()),
@@ -101,8 +101,8 @@ class 멘토_단일_조회_성공 {
101101
mentoringFixture.대기중_멘토링(mentor.getId(), appliedUser.getId());
102102

103103
// when
104-
MentorDetailResponse notAppliedResponse = mentorQueryService.getMentorDetails(mentor.getId(), notAppliedUser.getId());
105-
MentorDetailResponse appliedResponse = mentorQueryService.getMentorDetails(mentor.getId(), appliedUser.getId());
104+
MentorDetailResponse notAppliedResponse = mentorQueryService.getMentorDetails(mentor.getSiteUserId(), notAppliedUser.getId());
105+
MentorDetailResponse appliedResponse = mentorQueryService.getMentorDetails(mentor.getSiteUserId(), appliedUser.getId());
106106

107107
// then
108108
assertAll(
@@ -159,8 +159,8 @@ void setUp() {
159159
// then
160160
Map<Long, MentorPreviewResponse> mentorPreviewMap = response.content().stream()
161161
.collect(Collectors.toMap(MentorPreviewResponse::id, Function.identity()));
162-
MentorPreviewResponse mentor1Response = mentorPreviewMap.get(mentor1.getId());
163-
MentorPreviewResponse mentor2Response = mentorPreviewMap.get(mentor2.getId());
162+
MentorPreviewResponse mentor1Response = mentorPreviewMap.get(mentor1.getSiteUserId());
163+
MentorPreviewResponse mentor2Response = mentorPreviewMap.get(mentor2.getSiteUserId());
164164
assertAll(
165165
() -> assertThat(mentor1Response.nickname()).isEqualTo(mentorUser1.getNickname()),
166166
() -> assertThat(mentor1Response.universityName()).isEqualTo(university1.getKoreanName()),
@@ -225,10 +225,10 @@ void setUp() {
225225
assertAll(
226226
() -> assertThat(asiaFilteredResponse.content()).hasSize(1)
227227
.extracting(MentorPreviewResponse::id)
228-
.containsExactly(asiaMentor.getId()),
228+
.containsExactly(asiaMentor.getSiteUserId()),
229229
() -> assertThat(europeFilteredResponse.content()).hasSize(1)
230230
.extracting(MentorPreviewResponse::id)
231-
.containsExactly(europeMentor.getId())
231+
.containsExactly(europeMentor.getSiteUserId())
232232
);
233233
}
234234

@@ -240,7 +240,7 @@ void setUp() {
240240
// then
241241
assertThat(response.content()).hasSize(2)
242242
.extracting(MentorPreviewResponse::id)
243-
.containsExactlyInAnyOrder(asiaMentor.getId(), europeMentor.getId());
243+
.containsExactlyInAnyOrder(asiaMentor.getSiteUserId(), europeMentor.getSiteUserId());
244244
}
245245
}
246246
}

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ class 멘토링_신청_테스트 {
8383
@Test
8484
void 멘토링을_성공적으로_신청한다() {
8585
// given
86-
MentoringApplyRequest request = new MentoringApplyRequest(mentor1.getId());
86+
MentoringApplyRequest request = new MentoringApplyRequest(mentor1.getSiteUserId());
8787

8888
// when
8989
MentoringApplyResponse response = mentoringCommandService.applyMentoring(menteeUser.getId(), request);
@@ -102,7 +102,7 @@ class 멘토링_신청_테스트 {
102102
void 동일_멘티_멘토끼리는_재신청되지않는다() {
103103
// given
104104
mentoringFixture.대기중_멘토링(mentor1.getId(), menteeUser.getId());
105-
MentoringApplyRequest request = new MentoringApplyRequest(mentor1.getId());
105+
MentoringApplyRequest request = new MentoringApplyRequest(mentor1.getSiteUserId());
106106

107107
// when & then
108108
assertThatThrownBy(() -> mentoringCommandService.applyMentoring(menteeUser.getId(), request))

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -290,8 +290,8 @@ void setUp() {
290290
// then
291291
Map<Long, MatchedMentorResponse> matchMentorMap = response.content().stream()
292292
.collect(Collectors.toMap(MatchedMentorResponse::id, Function.identity()));
293-
MatchedMentorResponse mentor1Response = matchMentorMap.get(mentor1.getId());
294-
MatchedMentorResponse mentor2Response = matchMentorMap.get(mentor2.getId());
293+
MatchedMentorResponse mentor1Response = matchMentorMap.get(mentor1.getSiteUserId());
294+
MatchedMentorResponse mentor2Response = matchMentorMap.get(mentor2.getSiteUserId());
295295
assertAll(
296296
() -> assertThat(mentor1Response.roomId()).isEqualTo(chatRoom1.getId()),
297297
() -> assertThat(mentor1Response.nickname()).isEqualTo(mentorUser1.getNickname()),

0 commit comments

Comments
 (0)