test(api-member): Rest API 로그인 테스트 코드를 추가한다. #59
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
bulmandu
added
✅ TEST
sinbom
requested changes
Mar 15, 2023
| import static org.springframework.test.web.servlet.result.MockMvcResultHandlers.print; | ||
| import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status; | ||
|
|
||
| @Nested |
There was a problem hiding this comment.
해당 애노테이션은 중첩 구조의 테스트 메소드를 위한 애노테이션입니다.
There was a problem hiding this comment.
이 케이스에선 중첩을 의미하는 @nested가 굳이 필요가 없군요!
| @BeforeAll | ||
| static void registerUser(@Autowired UserRepository userRepository, | ||
| @Autowired PasswordEncoder passwordEncoder){ | ||
| userRepository.save(createUser(passwordEncoder)); | ||
| } |
There was a problem hiding this comment.
테스트를 위한 User 엔티티를 @BeforeAll 메소드에서 영속화를 하고 있네요! 모든 테스트 클래스의 모든 테스트를 수행하고 다른 테스트 클래스의 테스트를 수행할 때, 동일한 테스트 애플리케이션 컨텍스트에 속한 테스트들의 결과에 영향을 미치지 않으려면 테스트를 위해 영속화한 데이터를 지워주어야합니다. @AfterAll 메소드를 통해 지워주는 작업을 진행해주세요
There was a problem hiding this comment.
테스트 코드 경험이 없어서 이런부분 까지 생각을 못했네요 이건 JPA를 잘 사용하지 못한다는 의미이기도 하네요 허허허😅
다른 테스트 코드에 영향이 가지않으려면 이 컨텍스트의 모든 테스트들이 끝나고 사용했던 데이터를 모두 삭제해줘야 하는걸 깨달았습니다.
- 불필요한 @nested 제거 - 영속화 데이터 삭제
bulmandu
force-pushed
the
test/rest-api-login
branch
from
8e81d1a to
ce841f9
Compare
sinbom
approved these changes
Mar 18, 2023
sinbom
added a commit
that referenced
this pull request
Apr 5, 2023
* fix(oauth2): oauth2 custom 구현체에서 발생한 에러를 Oauth2 에러 핸들러에서 처리할 수 있도록 수정 (#50) - CustomOauth2TokenCustomizer, RedisOauth2AuthorizationService 내부에서 발생한 예외는 AuthenticationFailureHandler에서 처리할 수 없는 예외이므로 Oauth2AuthenticationException으로 래핑해서 예외를 던지도록 수정 * feat(api-member): restAPI 로그인 성공, 실패 핸들러 추가, 폼로그인 redirectUrl 부분 수정 (#52) * feat(api-member): restAPI 로그인 성공, 실패 핸들러 추가, 폼로그인 redirectUrl 부분 수정 * chore(api-member): PR 리뷰 내용 적용 * feat(oauth2): oauth2 client authorize 성공/실패의 리다이렉트 url을 설정하는 파라미터 추가 (#58) * feat(oauth2): oauth2 client authorize 성공/실패의 리다이렉트 url을 설정하는 파라미터 추가 - oauth2 client authorize endpoint 요청시 리다이렉트 url 파라미터를 세션에 저장하는 필터 추가 - 세션에 저장된 리다이렉트 url로 리다이렉트 시키는 oauth2 client success/failure handler 추가 * feat: 리다이렉트 허용 origin을 application.yml에서 관리하도록 수정 * fix(cors): 개발 환경 cors 허용 origin 목록 오타 수정 (#62) * test(api-member): Rest API 로그인 테스트 코드를 추가한다. (#59) * test(api-member): Rest API 로그인 테스트 코드를 추가한다. * test(api-member): pr리뷰 반영 - 불필요한 @nested 제거 - 영속화 데이터 삭제 * docs(oauth2): oauth2 authorization server 문서 수정 (#65) - 각 endpoint 마다 발생할 수 있는 에러 케이스에 대한 응답과 설명 추가 --------- Co-authored-by: minsu koo <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
💌 설명
Rest API 로그인 e2e 테스트 코드를 아래의 3가지 케이스로 추가했습니다.
📎 관련 이슈
#53
💡 논의해볼 사항
📝 참고자료