Skip to content

Commit

Permalink
Update acceptance-test-auth.md
Browse files Browse the repository at this point in the history
  • Loading branch information
boorownie authored Jul 11, 2023
1 parent 10bec70 commit 021b15b
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions reviewguide/atdd-spring/acceptance-test-auth.md
Original file line number Diff line number Diff line change
@@ -1,16 +1,16 @@
## 1단계: 토큰을 이용한 로그인

- 로그인 등 인증 기능에 대한 인수 테스트에서 인증 단계를 잘 적용했는가?
- 인증이 필요한 기능의 인수 테스트에서 적절한 흐름으로 인수 테스트를 작성했는가?
## 1단계: 내 정보 조회 기능 구현
- 사용자 기준으로 인수 조건을 잘 도출했는가?
- `/members/me` 요청에 포함된 header를 통해 token을 잘 추출하고 token을 통해 사용자 정보를 잘 추출했는가?
- 위 방식이 아니라면 `@AuthenticationPrincipal`을 잘 활용했는가?
- 테스트에서 인증에 필요한 로직들의 가독성은 좋은지? 재사용성은 좋은지?
- 인증 인수 테스트에 필요한 초기값(회원 정보, 역할 정보 등)은 어떻게 설정했는지?
- 미션 문서 > 힌트에 DataLodaer를 이용하는 방법을 안내중
- Spring MVC 기능(Interceptor, ArgumentResolver 등)을 잘 활용했는지?
- 구현 할 코드 수가 많지 않아서 코멘트를 주려하기 보다 의도에 맞게 인수 조건과 인수 테스트 작성을 잘 했는지 파악하기
- 눈에 띄는 내용만 피드백 후 머지하여 빠르게 다음 단계 진행하도록 가이드

---

## 2단계: 외부 서비스에 의존하는 인수 테스트
## 2단계: 깃헙 로그인 인수 테스트

- 외부 서비스에 대한 의존이 있는 로그인 기능을 잘 구현했는지?
- 해당 기능을 검증할 수 있는 테스트는 외부 서비스에 대한 의존을 어떻게 처리했는지?
Expand Down

0 comments on commit 021b15b

Please sign in to comment.