woowacourse-teams / 2024-corea

코드리뷰 매칭 플랫폼, CoReA
https://code-review-area.com/
13 stars 7 forks source link

[BE] 참여한 전체 방들 조회할 때, 실패 원인 같이 보내는 기능 구현(#625) #629

Closed github-actions[bot] closed 4 days ago

github-actions[bot] commented 6 days ago

📌 관련 이슈

✨ PR 세부 내용

기존엔 하나의 방을 조회할 때만 매칭 실패 여부를 확인 후 매칭 실패 원인에 대한 메시지를 전달했지만, 지금은 참여한 방 전체 조회 시에도 매칭 실패한 방들이 있다면 매칭 실패 원인에 대한 메시지를 같이 보내도록 수정하였습니다.

{
    "rooms": [
        {
            "id": 1,
            ....
            "roomStatus": "FAIL",
            "message": "방의 최소 참여 인원보다 참가자가 부족하여 매칭이 진행되지 않았습니다."
        },
        {
            "id": 2,
            ....
            "roomStatus": "FAIL",
            "message": "방의 최소 참여 인원보다 참가자가 부족하여 매칭이 진행되지 않았습니다."
        }
    ],
    "isLastPage": true,
    "pageNumber": 0
}

+ 테스트쪽도 조금 같이 손봤습니다. 앞으로 엣지 케이스나 놓친 부분이 있다면 계속해서 추가로 테스트 작성할 것 같아요~

github-actions[bot] commented 6 days ago

Test Results

 57 files   57 suites   8s :stopwatch: 168 tests 162 :white_check_mark: 6 :zzz: 0 :x: 175 runs  169 :white_check_mark: 6 :zzz: 0 :x:

Results for commit b1b155e1.

:recycle: This comment has been updated with latest results.

jcoding-play commented 4 days ago

이와 별개로, 저희가 리팩토링을 진행하는 만큼 드는 욕심으론 Response 들을 별개로 끊어내는 작업을 하고 싶다는 욕심도 드네요.

👍 좋아여~! 이 부분 프론트와 같이 상의해보면서 정하면 좋겠네요~! 😀