Skip to content

[FEAT] Feat/auth test #68

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

Merged
merged 5 commits into from
Nov 23, 2023
Merged

[FEAT] Feat/auth test #68

merged 5 commits into from
Nov 23, 2023

Conversation

jinjoo-lab
Copy link
Collaborator

@jinjoo-lab jinjoo-lab commented Nov 23, 2023

추가/수정한 기능 설명

  1. AuthService의 saveMember(회원가입) 테스트를 Kakao , Apple 사용자 분리하여 구현
  2. 등록된 회원에 대해 로그인 후 토큰이 발급되는가 테스트

특이사항

check list

  • 모든 단위 테스트를 돌려보고 기존에 작동하던 테스트에 영향이 없는 것을 확인했나요?
  • 추가/수정사항을 설명했나요?

@jinjoo-lab jinjoo-lab requested a review from ohksj77 November 23, 2023 10:50
Copy link
Member

@ohksj77 ohksj77 left a comment

Choose a reason for hiding this comment

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

코멘트 읽어줘!

// when
AfterLoginResponse response = authService.saveMember(kakaoRequest);
// then
assertThat(response.getStatus().equals(AuthStatus.SIGNIN));
Copy link
Member

Choose a reason for hiding this comment

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

assertThat() 안의 값이 true인지 false인지는 테스트 통과에 영향을 안줘..


@DatabaseTest
@DisplayName("AuthService의 ")
public class AuthServiceTest {
Copy link
Member

Choose a reason for hiding this comment

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

서비스 테스트는 LoginTest를 상속 받으면 돼! 그리고 DatabaseTest 어노테이션은 LoginTest에 있기 때문에 필요 없음!

@ohksj77 ohksj77 added the test 테스트 수행 label Nov 23, 2023
@jinjoo-lab jinjoo-lab merged commit e560788 into master Nov 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test 테스트 수행
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants