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

User 도메인 Layer 별 Dto 수정 테스트코드 작성 #121

Merged
merged 6 commits into from
Sep 8, 2024

Conversation

rivkode
Copy link
Member

@rivkode rivkode commented Sep 8, 2024

관련 Issue

  • 이슈 번호를 기재해주세요

변경 사항

  • 기입 예: 회원가입 기능 구현
  • 기입 예: 프로필 수정 기능 구현
  • 기입 예: 로그인 체크 처리가 안 되는 버그를 수정

Todo List

  • 기입 예: 예외 처리를 추가적으로 해줘야 합니다.
  • 기입 예: 아직 포스트맨 테스트를 해보지 않았습니다.

Check List

  • 포스트맨으로 체크해 보았나요?

트러블 슈팅

  • 기입 예 - 1. ~~ 발생한 예외 이름 또는 뭐 하다가 예외 발생했다. 기록

발생한 예외

// 예외

예외가 발생한 코드

// 코드

원인 분석

이러이러한 것이 원인이었다!

수정한 코드

// 수정한 코드

- findByUserToken()
- existsByEmail()
- findByEmail()
- findAllByUserToken()
- findAllByUserId()
- updateMyInfo()
- nullExceptionUpdateMyInfo()
- search()
- findAllByUserToken()
- findAllByUserId()
- facade 메서드 생성
- domain layer 용 dto 생성
- 사용하지 않는 클래스 제거
@rivkode rivkode self-assigned this Sep 8, 2024
Copy link

github-actions bot commented Sep 8, 2024

Test Results

20 tests   20 ✅  7s ⏱️
 5 suites   0 💤
 5 files     0 ❌

Results for commit d82f37d.

@rivkode rivkode merged commit f8656bc into develop Sep 8, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant