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

[feat] User, Auth 도메인의 테스트코드를 작성한다. #45

Merged
merged 5 commits into from
Nov 4, 2024

Conversation

juuuunny
Copy link
Contributor

@juuuunny juuuunny commented Nov 4, 2024

✅ PR 유형

어떤 변경 사항이 있었나요?

  • 새로운 기능 추가
  • 버그 수정
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

  • 유저 회원가입, 정보 조회, 닉네임 중복체크 테스트 코드 작성
  • 토큰 재발행 테스트 코드 작성
스크린샷 2024-11-04 오후 2 56 37 스크린샷 2024-11-04 오후 2 56 50

생각해보면 당연한 것이긴 한데 쿠키를 이용한 api는 당연히 실제 브라우저의 쿠키를 이용하는 게 당연한 거 같아요.
스웨거에서 jwt 토큰 같은 거 필요하면 당연히 개발자도구 브라우저에 있는 쿠키에서 빼서 쓰는게 맞고,
토큰 재발행 이런 거에서도 쿠키의 리프레시 토큰 빼서 쓰고 성공하면 쿠키에 어세스토큰, 리프레시토큰 업데이트 자동으로 되는 것처럼요.

스크린샷 2024-11-04 오후 2 59 07 이렇게 브라우저 쿠키에 토큰이 있을 경우 잘 되고, 스크린샷 2024-11-04 오후 3 00 32 없으면 이렇게 에러메시지가 뜹니다.

파라미터 쿠키 있는 거에는 아무 값이나 넣어도 되는데 사실상 클라이언트측에서 이 api는 토큰이 필요한 api구나라는 것을 알게해주기 위해 이렇게 하였습니다. 이게 없으면 머가 필요한지 클라이언트측에서 알 수가 없을 거 같아서!
(그래서 클라이언트측에 미리 이렇게 있으면 토큰이 필요한 api이기때문에 쿠키에 토큰값을 직접 넣어달라고 말해주는 게 나을 것 같아요! )


✏️ 관련 이슈

본인이 작업한 내용이 어떤 Issue Number와 관련이 있는지만 작성해주세요

ex)


🎸 기타 사항 or 추가 코멘트

@juuuunny juuuunny self-assigned this Nov 4, 2024
@juuuunny juuuunny added 🚀 feat 새로운 기능 추가 / 일부 코드 추가 / 일부 코드 수정 (리팩토링과 구분) / 디자인 요소 수정 😆 JUNHYEONG 준형 Issue or PR labels Nov 4, 2024
Copy link
Member

@bbbang105 bbbang105 left a comment

Choose a reason for hiding this comment

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

아하 글쿤요 언급해주는 건 좋을 것 같습니다!!
테스트 코드 짜느라 고생하셨습니다 👍🏻 LGTM!

@juuuunny juuuunny merged commit c1fed80 into develop Nov 4, 2024
1 check passed
@juuuunny juuuunny deleted the feature/#43/user-test branch November 4, 2024 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 feat 새로운 기능 추가 / 일부 코드 추가 / 일부 코드 수정 (리팩토링과 구분) / 디자인 요소 수정 😆 JUNHYEONG 준형 Issue or PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat] : User, Auth 도메인의 테스트코드를 작성한다.
2 participants