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

메인 브랜치에 개발용 브랜치 반영 #110

Merged
merged 23 commits into from
Feb 22, 2023
Merged

메인 브랜치에 개발용 브랜치 반영 #110

merged 23 commits into from
Feb 22, 2023

Conversation

akalswl14
Copy link
Member

구현 내용

구현 요약

API 수정

  • [DELETE] /user 유저 탈퇴 API : 버그 수정
    • URI의 오타 수정 ( /user/ -> /user )
    • jwtService를 UserService.delete 메소드의 파라미터로 받지 않도록 개선
    • User 엔티티는 다른 엔티티와의 개인 소유 관계가 아니기 때문에, cascade 삭제가 부적합한 것으로 보여, 수동적으로 삭제하도록 개선
    • 해당 유저의 카테고리 데이터를 먼저 삭제하고, 카테고리 삭제시 cascade 삭제로 전시, 작품, 태그가 삭제되도록 함
    • 카테고리 데이터가 삭제된 이후, 유저 데이터를 삭제함
  • 유저 API의 런칭 목적 authentication 처리 로직 교체
    • 기존에는 개발상 편의를 위해 단순히 1L으로 처리하였음
    • 런칭을 위해, 테스트 계정이 아닌 요청은 정상적으로 처리하도록 메소드 적용하여 처리

테스트

  • 통합 테스트 base class 추가

  • 테스트 상 편의를 위해, User 엔티티의 id를 입력하는 생성자 추가

  • 테스트 상 편의를 위해, 테스트용 유저 정적 변수 선언

  • Category Repository Unit Test 추가

  • 유저 기반 카테고리를 삭제하는 CategoryRepository.deleteAllByUser메소드를 검증하는 레포지토리 테스트 추가

  • 카테고리를 삭제하면 전시데이터도 삭제되는지에 대해, CategoryService.delete 메소드를 검증하는 테스트 추가

  • 기존의 User Service Test를 단위 테스트로 처리하도록 변경

  • UserService.updateUserName 메소드의 서비스 단위 테스트 추가

  • 유저 삭제 API의 통합 테스트 추가

  • UserRepository의 단위 테스트 추가

  • UserRepository.delete를 검증하는 레포지토리 단위 테스트 추가

기타

  • 일부 예외의 메세지 오타 수정

관련 이슈

close #108

le2sky and others added 23 commits February 19, 2023 14:12
유저 탈퇴시 정상적으로 데이터 삭제되지 않는 버그 개선 / 구조 개선 / 테스트 코드 추가 / 일부 authentication 처리 로직 개선
@akalswl14 akalswl14 self-assigned this Feb 22, 2023
@akalswl14 akalswl14 merged commit 3a9b06d into main Feb 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

회원 탈퇴시 사용자 데이터 제거
2 participants