test: 닉네임 시작/끝 숫자 허용 스펙에 맞게 NicknameValidatorTest 기대값 수정#391
Conversation
resolved #388 BaseResponseStatus의 INVALID_START/END_OF_NICKNAME 메시지에 숫자가 허용 문자로 명시되어 있어, 실제 구현(정규식)이 아니라 테스트의 기대값이 스펙과 어긋나 있었음 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 본 PR은 닉네임 유효성 검사 로직의 테스트 코드와 실제 스펙 간의 불일치를 해결하기 위한 변경입니다. 프로덕션 코드의 정규식은 유지하되, 숫자로 시작하거나 끝나는 닉네임을 예외로 처리하던 테스트 케이스를 수정하여 명시된 스펙에 부합하도록 조정하였습니다. Highlights
New Features🧠 You can now enable Memory (public preview) to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize the Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counterproductive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. 닉네임 검사 스펙을 보며, 테스트 코드 다시 쓰네. 숫자는 허용되어야 하니, 이제는 모두 통과하네. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
이번 풀리퀘스트는 닉네임 유효성 검증 시 숫자로 시작하거나 끝나는 닉네임을 허용하도록 테스트 케이스를 수정한 변경사항을 포함하고 있습니다. 리뷰 피드백에서는 저장소 스타일 가이드에 따라 테스트 메서드 이름을 영문으로 작성하고 테스트 내에 'given, when, then' 주석을 추가할 것을 지적했습니다. 또한, 숫자가 허용됨에 따라 기존의 '한글 또는 영어로 시작/끝나지 않는 경우'라는 테스트 메서드 이름이 현재 검증 로직과 맞지 않으므로, 이를 반영하여 이름을 수정할 것을 권장하고 있습니다.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
1 participant
#️⃣ Issue Number
📝 요약(Summary)
NicknameValidatorTest에서 숫자로 시작/끝나는 닉네임(0이름,0test,이름0,test0)을 예외 발생 케이스로 잘못 기대하고 있던 테스트를 수정NicknameValidator의 정규식(프로덕션 코드)은 수정하지 않음 —BaseResponseStatus.INVALID_START_OF_NICKNAME/INVALID_END_OF_NICKNAME의 실제 사용자 노출 메시지("첫/마지막 글자는 한글, 영문, 숫자만 사용할 수 있습니다")에 숫자가 명시적으로 허용 문자로 포함되어 있어, 실제 버그는 정규식이 아니라 테스트의 기대값이 스펙과 어긋나 있던 것으로 확인됨💬 공유사항 to 리뷰어
NicknameValidator.java)는 변경하지 않았습니다. 판단 근거는BaseResponseStatus에 이미 정의된 사용자 노출 에러 메시지 텍스트이며, 혹시 그 메시지 자체가 잘못 작성된 것이고 실제로는 숫자 시작/끝을 막아야 하는 게 맞다면 반대 방향(정규식 수정)으로 다시 논의가 필요합니다../gradlew test로 전체 스위트(84개) 통과 확인했습니다.#387(Ratings NPE) 수정이 이미 develop에 반영된 상태에서 검증했고, 이번 수정으로 나머지 4건도 모두 통과해 현재 develop 기준 전체 테스트가 그린 상태입니다.✅ PR Checklist
PR이 다음 요구 사항을 충족하는지 확인하세요.