Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BE] fix: GlobalExceptionHandler 에러 메시지 수정 및 raw type 제거 #604

Merged
merged 2 commits into from
Oct 2, 2024

Conversation

takoyakimchi
Copy link
Contributor

이슈

개발 사항

  • 제네릭에서 raw type 제거
  • Exception.class에 대해서 예외 메시지를 포장해서 응답 + ErrorCode 변경

@takoyakimchi takoyakimchi added this to the Sprint6 milestone Oct 2, 2024
@takoyakimchi takoyakimchi self-assigned this Oct 2, 2024
Copy link

github-actions bot commented Oct 2, 2024

Test Results

173 tests  +130   173 ✅ +130   17s ⏱️ +16s
 38 suites + 33     0 💤 ±  0 
 38 files   + 33     0 ❌ ±  0 

Results for commit 5f786b8. ± Comparison against base commit 1d8f222.

This pull request removes 43 and adds 173 tests. Note that renamed tests count towards both.
com.happy.friendogly.presentation.ui.club.detail.ClubDetailViewModelTest ‑ 모임 상세 로드에 실패하면 사용자에게 실패를 알리는 이벤트를 발생시킨다()
com.happy.friendogly.presentation.ui.club.detail.ClubDetailViewModelTest ‑ 모임 상세가 로드된다()
com.happy.friendogly.presentation.ui.club.detail.ClubDetailViewModelTest ‑ 모임 참여에 성공하면 사용자에게 참여 성공을 알리는 이벤트가 발생된다()
com.happy.friendogly.presentation.ui.club.detail.ClubDetailViewModelTest ‑ 모임 참여할 수 없는 경우 사용자에게 참여 실패를 알리는 이벤트가 발생한다()
com.happy.friendogly.presentation.ui.club.list.ClubListViewModelTest ‑ 등록한 주소가 없는 경우 사용자에게 주소 등록을 요구하는 화면을 표시한다()
com.happy.friendogly.presentation.ui.club.list.ClubListViewModelTest ‑ 등록한 주소의 모임 리스트가 로드된다()
com.happy.friendogly.presentation.ui.club.list.ClubListViewModelTest ‑ 모임 리스트 로드에 실패하면 사용자에게 에러 발생을 나타내는 화면을 표시한다()
com.happy.friendogly.presentation.ui.club.list.ClubListViewModelTest ‑ 적합한 모임이 없는 경우 사용자에게 모임이 없다는 것을 나타내는 화면을 표시한다()
com.happy.friendogly.presentation.ui.mypage.MyPageViewModelTest ‑ 내가 방장인 모임 이동 이벤트가 발생한다()
com.happy.friendogly.presentation.ui.mypage.MyPageViewModelTest ‑ 내가 참여한 모임 이동 이벤트가 발생한다()
…
com.happy.friendogly.DataSourceRoutingTest ‑ readOnly 트랜잭션이면 redaer 데이터소스가 바운딩된다.
com.happy.friendogly.DataSourceRoutingTest ‑ 쓰기전용 트랜잭션이면 Writer 데이터소스가 바운딩된다.
com.happy.friendogly.chat.controller.ChatMessageControllerTest ‑ 두 시점 사이의 채팅 메시지를 조회할 수 있다.
com.happy.friendogly.chat.service.ChatCommandServiceTest ‑ 채팅 메시지를 DB에 저장한다.
com.happy.friendogly.chat.service.ChatCommandServiceTest ‑ 채팅 입장 메시지를 DB에 저장한다.
com.happy.friendogly.chat.service.ChatCommandServiceTest ‑ 채팅 퇴장 메시지를 DB에 저장한다.
com.happy.friendogly.chat.service.ChatQueryServiceTest ‑ since 시간과 until 시간 사이의 (포함 X) 채팅 내역을 조회할 수 있다.
com.happy.friendogly.chat.service.ChatQueryServiceTest ‑ 채팅 메시지 시간 범위 조회에서, 시간 범위가 잘못된 경우 예외가 발생한다.
com.happy.friendogly.chat.service.ChatQueryServiceTest ‑ 채팅방에 들어가 있지 않은 Member는 채팅 내역을 조회할 수 없다.
com.happy.friendogly.chat.service.ChatRoomCommandServiceTest ‑ 1대1 채팅방 저장 과정에서, 이미 채팅방이 존재하면 기존 채팅방 ID를 반환한다.
…

Copy link
Contributor

@ehtjsv2 ehtjsv2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

굿굿 좋습니다

Copy link
Contributor

@J-I-H-O J-I-H-O left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://dahye-jeong.gitbook.io/java/java/effective_java/2021-05-19-generic-dont-use-raw-type

요 내용 참고해보면 좋을 것 같아요! 고생 많으셨습니다~~

Copy link
Member

@jimi567 jimi567 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

귀찮은 작업이 였을텐데 고생하셨어요.

@takoyakimchi takoyakimchi merged commit fc80f69 into develop Oct 2, 2024
3 checks passed
@takoyakimchi takoyakimchi deleted the feature/#603 branch October 18, 2024 04:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

GlobalExceptionHandler 예외 메시지 로직 수정, Raw Type을 사용하지 않도록 수정
4 participants