Skip to content

[TEST] Token 기능 테스트#223

Merged
chanwonlee merged 7 commits into
developfrom
test/208-jwt-token-test
Aug 7, 2025
Merged

[TEST] Token 기능 테스트#223
chanwonlee merged 7 commits into
developfrom
test/208-jwt-token-test

Conversation

@chanwonlee

Copy link
Copy Markdown
Contributor

관련 Issue (필수)

주요 변경 사항 (필수)

  • TokenValidator 테스트 코드 작성
  • TokenExtractor 테스트 코드 작성
  • TokenGenerator 테스트 코드 작성

리뷰어 참고 사항

image 해당 테스트로 Token 관련 클래스 테스트 커버리지 100% 달성하였습니다!

추가 정보

없음

PR 작성 체크리스트 (필수)

  • 제목이 Issue와 동일함을 확인했습니다.
  • 리뷰어를 지정했습니다.
  • 프로젝트를 연결했습니다.

@EndlessMilkyway EndlessMilkyway left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

확인 완료했습니다. 고생하셨습니다!

@Jimin730 Jimin730 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

토큰 관련 테스트 코드 확인했습니다 👍
아직 jwt 관련해서 테스트 코드를 제대로 작성해본적은 없어서 따로 공부할 때 찬원님 코드 참고해봐야겠네요! 고생하셨습니다 !! 👍 👍

@Yunsung-Jo Yunsung-Jo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

고생하셨습니다! 👍

@friox friox left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

깔끔하게 잘 구성하신것같습니다!
고생하셨습니다~ 👍👍

Comment thread src/test/java/com/sudo/railo/auth/security/jwt/TokenGeneratorTest.java Outdated

@Ogu1208 Ogu1208 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

확인했습니다! 전체적으로
//given //when //then
주석이 있다면 조금 더 문서로서의 역할을 강화할 수 있을 것 같습니다.

고생하셨습니다! 👍

@chanwonlee
chanwonlee force-pushed the test/208-jwt-token-test branch from d7a6716 to fa2fe63 Compare August 7, 2025 01:38
@chanwonlee
chanwonlee merged commit 56e2f6f into develop Aug 7, 2025
3 checks passed
@chanwonlee
chanwonlee deleted the test/208-jwt-token-test branch August 7, 2025 01:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[TEST] Token 기능 테스트

6 participants