Skip to content

Commit

Permalink
[BE] 매칭 로직 원복 및 수정(#624) (#627)
Browse files Browse the repository at this point in the history
* refactor: 도메인 수정

* feat: 리뷰어, 리뷰이 조회 API 기능 구현

* refactor: 중복된 기능 코드 제거

* docs: 메서드 시그니쳐 통일

* feat: 시연용 데이터 추가

* refactor: 패키지 이동으로 인한 오류 수정

* feat: 시연용 데이터 추가

* fix: REMOTE_ORIGIN 그냥 변수로 변경

* feat: 데이터 추가

* feat: 서브모듈 반영

* feat: response 생성 때 reviewer, reviewee 분리

* feat: application 설정 변경

* feat: 데모 데이터 함수로 분리

* fix: 누락된 saveAll 추가

* fix: 데이터 정합성 수정

* fix: roomId 상수 변경

* feat: 피드백 키워드 뒤 .제거

* refactor: 3차 데모데이 데이터 변경

* feat: room 4에 대한 케이스도 추가

* feat: room 4 매칭 추가

* fix: 응답 내 프로필 링크로 변경

* 최신 브랜치 병합

* feat: submodule 업데이트

* refactor: 서브모듈 변경

* feat: submodule 업데이트

* refactor: 매칭 로직 원복

* fix: 매칭 로직 오류 수정

* refactor: 변수명 직관적으로 수정

* refactor: 리뷰어를 제외하고 matchingSize 만큼 추가 매칭하도록 수정

* refactor: 리뷰어가 matchingSize 만큼 리뷰이와 매칭되도록 수정

* feat: submodule 업데이트

* refactor: 피드백 반영

* refactor: 변수명 수정

---------

Co-authored-by: hjk0761 <[email protected]>
Co-authored-by: youngsu5582 <[email protected]>
Co-authored-by: HyunJoong Kim <[email protected]>
  • Loading branch information
4 people committed Oct 21, 2024
1 parent 21e360f commit 0d84d56
Showing 1 changed file with 6 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -100,6 +100,8 @@ void matchPairsWithReviewer() {

List<Pair> pairs = matchingStrategy.matchPairs(participations, room.getMatchingSize());

long reviewerSize = participations.stream().filter(Participation::isReviewer).count();

for (Participation participation : participations) {
long reviewerCount = pairs.stream()
.filter(pair -> pair.getDeliver().getGithubUserId().equals(participation.getMemberGithubId()))
Expand All @@ -108,12 +110,14 @@ void matchPairsWithReviewer() {
.filter(pair -> pair.getReceiver().getGithubUserId().equals(participation.getMemberGithubId()))
.count();
if (participation.getMemberRole().isReviewer()) {
assertThat(reviewerCount).isLessThanOrEqualTo(participation.getMatchingSize());
//MemberRole.REVIEWER 는 matchingSize 만큼 리뷰하고, 이무에게도 리뷰받지 않는다.
assertThat(reviewerCount).isEqualTo(participation.getMatchingSize());
assertThat(revieweeCount).isZero();
} else {
//MemberRole.BOTH 는 최소 roomMatchingSize, 최대 matchingSize 만큼 리뷰하고, 최소 roomMatchingSize, 최대 matchingSize + MemberRole.REVIEWER 수 만큼 리뷰를 받는다.
assertThat(reviewerCount).isLessThanOrEqualTo(participation.getMatchingSize());
assertThat(reviewerCount).isGreaterThanOrEqualTo(room.getMatchingSize());
assertThat(revieweeCount).isLessThanOrEqualTo(participation.getMatchingSize());
assertThat(revieweeCount).isLessThanOrEqualTo(participation.getMatchingSize() + reviewerSize);
assertThat(revieweeCount).isGreaterThanOrEqualTo(room.getMatchingSize());
}
}
Expand Down

0 comments on commit 0d84d56

Please sign in to comment.