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.

Sign up for GitHub

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

Jump to bottom

test: Member Service 단위 테스트를 작성한다 #87

Merged
merged 4 commits into from
Oct 20, 2023
Merged

test: Member Service 단위 테스트를 작성한다 #87

merged 4 commits into from
Oct 20, 2023

Conversation

soomanbaek
Copy link
Contributor

@soomanbaek soomanbaek commented Oct 20, 2023 •
edited
Loading

  • Member 테스트 코드를 작성한다 #82
  • MemberApi에서 Member를 받아, MemberService에 전달하도록 수정
    • 수정 전: MemberService에서 SecurityContextHolder.getContext().getAuthentication().getPrincipal()를 통해 받아옴
    • 수정 후: MemberApi에서 @AuthenticationPrincipal을 통해 받아오고 이를 MemberService에 전달
스크린샷 2023-10-21 오후 1 50 02

@soomanbaek soomanbaek added ⚙️ refactor 리팩터링 📊 test 테스트 관련 labels Oct 20, 2023
@soomanbaek soomanbaek self-assigned this Oct 20, 2023
@soomanbaek soomanbaek merged commit 4664734 into main Oct 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@soomanbaek soomanbaek

Labels
⚙️ refactor 리팩터링 📊 test 테스트 관련
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@soomanbaek