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.

Sign up for GitHub

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

Jump to bottom

[FEAT/#46] 번호 인증 예외 케이스 세분화 및 테스트 코드 이슈 해결 #47

Merged
merged 18 commits into from
Feb 1, 2025
Merged

[FEAT/#46] 번호 인증 예외 케이스 세분화 및 테스트 코드 이슈 해결 #47

merged 18 commits into from
Feb 1, 2025

Conversation

sung-silver
Copy link
Contributor

@sung-silver sung-silver commented Feb 1, 2025 •
edited
Loading

Related Issue 🚀

Work Description ✏️

  • 번호 인증에 대한 예외를 세분화하였습니다
    • 기존: 인증코드, 번호 인증 유형, 전화번호 3개 중 하나라도 일치하지 않으면 404 not found 예외를 전달
    • 변경: 번호 인증 유형과 전화번호로 생성된 인증 내역이 없는 경우에 404 not found 예외를 전달하고, 인증 내역은 존재하나 인증코드가 일치하지 않는 경우에 대해서는 400 예외를 반환
  • jwt pem 파일로 인해 테스트 코드가 돌아가지 않는 문제를 해결했습니다
    • 로컬에서 동작하는 것까지 확인하여 ci yml을 수정해 github actions에서도 테스트 코드를 통과해야지만 동작하도록 스크립트를 변경해주었습니다
    • 관련 값을 노션에도 업데이트 해두었습니다! 로컬에서 테스트를 돌려보고 싶다면 test.env 파일을 생성해주세요

PR Point 📸

  • 어떤 리뷰든 환영 합니다~

@sung-silver
[FEAT] PhoneVerificationRetriever의 find 메서드 구현 변경
3d75ed3 
번호 인증 이력이 존재하지 않는 경우와 인증 코드가 유효하지 않은 경우에 대한 예외 세분화를 위해 변경
@sung-silver
[FEAT] 전화번호 인증코드 불일치에 대한 예외 추가
8b2f4f1
@sung-silver
[FEAT] VerifyVerificationResult에서 isVerified 필드 제거
60388bf
@sung-silver
[FEAT] 번호 인증 예외 세분화(인증 이력이 존재하지 않는 경우, 인증 코드가 유효하지 않은 경우)
896950b
@sung-silver
[FEAT] AuthResponse에서 VerifyResult from 메서드 isVerified 필드 제거
49b11f0
@sung-silver
[TEST] 번호 인증 예외 세분화에 따른 test 코드 수정
8db1449
@sung-silver
[TEST] 테스트 실패 해결
2ce395c
@sung-silver
[CHORE] ci에 test.env 파일 생성 명령어 추가
c357bfa
@sung-silver
[CHORE] Dockerfile에서 -x test 옵션 제거
53ef2f3
@sung-silver
[CHORE] jwt pem 파일 복사 로직 ci에 추가
7db51f2
@sung-silver
[CHORE] yml 정리 및 노션에 관련 값 공유
07f8419
@sung-silver sung-silver added the 🎁 feature 새로운 기능을 개발하거나 추가, 변경할 경우(spring boot 내의 기능 코드) label Feb 1, 2025
@sung-silver sung-silver self-assigned this Feb 1, 2025
@height Height
Copy link

height bot commented Feb 1, 2025

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

hyunw9
hyunw9 approved these changes Feb 1, 2025
View reviewed changes
Copy link
Contributor

@hyunw9 hyunw9 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 !

상용서버에서 빌드시 진행하는 테스트는 한번 돌아가는걸 봐야할거같네용

Dockerfile Outdated
@@ -10,7 +10,7 @@ WORKDIR /app-build
COPY . /app-build

# create .jar
RUN echo "Build with PROFILE=${PROFILE}" && ./gradlew build -x test -Pprofile=${PROFILE} --no-daemon
RUN echo "Build with PROFILE=${PROFILE}" && ./gradlew build -Pprofile=${PROFILE} --no-daemon
Copy link
Contributor

Choose a reason for hiding this comment

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

이 부분 현재는 상용 빌드시 테스트를 진행하지 않고 있는걸로 아는데, 진행해도 문제 없겠지요 ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

이거 지금 수정했는데 수정된 ci, cd, dockerfile 3가지 확인해봐주시면 좋을 것 같아요!!

@sung-silver sung-silver merged commit ff90113 into dev Feb 1, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hyunw9 hyunw9 hyunw9 approved these changes

@yummygyudon yummygyudon Awaiting requested review from yummygyudon

Assignees

@sung-silver sung-silver

Labels
🎁 feature 새로운 기능을 개발하거나 추가, 변경할 경우(spring boot 내의 기능 코드) size/L
Projects
SOPT makers 인증서버 분리
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[FEAT] 번호 인증 예외 케이스 세분화
2 participants
@sung-silver @hyunw9