feat: 소셜 로그인 사용자는 비밀번호 변경을 할 수 없다.

[whqtker]

관련 이슈

• resolves: feat: OAuth 로그인 사용자는 비밀변경 탭에 접근할 수 없다. #494

작업 내용

소셜 로그인 사용자가 비밀번호 변경을 시도하면 예외가 발생하도록 작성했습니다.

특이 사항

리뷰 요구사항 (선택)

[coderabbitai]

Walkthrough

1. 에러 코드 추가: ErrorCode 열거형에 OAUTH_USER_CANNOT_CHANGE_PASSWORD(HttpStatus.BAD_REQUEST.value(), "소셜 로그인 사용자는 비밀번호를 변경할 수 없습니다.") 상수를 추가했습니다.
2. 비즈니스 로직 변경: MyPageService의 updatePassword()에 인증 타입이 이메일 기반이 아닐 경우 OAUTH_USER_CANNOT_CHANGE_PASSWORD 예외를 던지도록 가드문을 추가했습니다.
3. 테스트 추가: MyPageServiceTest에 매개변수화된 테스트를 추가해 KAKAO 및 APPLE 같은 소셜 로그인 사용자가 비밀번호 변경을 시도할 때 해당 예외가 발생하는지 검증합니다.
4. 리소스 업데이트: src/main/resources/secret 파일의 서브모듈 커밋 해시가 갱신되었습니다.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 분

• 검토 시 주의할 파일/영역:
  • src/main/java/com/example/solidconnection/common/exception/ErrorCode.java — 새 상수의 메시지와 위치, 기존 메시지 관례 일치 여부 확인.
  • src/main/java/com/example/solidconnection/siteuser/service/MyPageService.java — AuthType 판별 로직의 조건 순서와 예외 발생 시점(비밀번호 검증 전)이 의도와 일치하는지 확인.
  • src/test/java/com/example/solidconnection/siteuser/service/MyPageServiceTest.java — 매개변수화된 테스트가 모든 대상 AuthType을 커버하는지와 예외 타입/메시지 일치성 검증.
  • src/main/resources/secret — 서브모듈 해시 변경이 의도된 업데이트인지 확인.

Suggested reviewers

• wibaek
• lsy1307

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	제목 'feat: 소셜 로그인 사용자는 비밀번호 변경을 할 수 없다.'는 변경사항의 핵심을 명확하게 요약하고 있습니다.
Description check	✅ Passed	PR 설명은 필수 섹션(관련 이슈, 작업 내용)을 포함하고 있으며, 변경 내용을 기능 단위로 설명하고 있습니다.
Linked Issues check	✅ Passed	OAuth 사용자 비밀번호 변경 불가 로직이 구현되었으며, 에러 코드 추가, 서비스 로직 수정, 테스트 작성이 이루어졌습니다.
Out of Scope Changes check	✅ Passed	모든 변경사항이 #494 이슈(OAuth 사용자 비밀번호 변경 제한)와 관련되어 있으며, 불필요한 변경은 발견되지 않습니다.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2b1bd8a and 3862441.

📒 Files selected for processing (4)

• src/main/java/com/example/solidconnection/common/exception/ErrorCode.java (1 hunks)
• src/main/java/com/example/solidconnection/siteuser/service/MyPageService.java (3 hunks)
• src/main/resources/secret (1 hunks)
• src/test/java/com/example/solidconnection/siteuser/service/MyPageServiceTest.java (3 hunks)

✅ Files skipped from review due to trivial changes (1)

• src/main/resources/secret

🚧 Files skipped from review as they are similar to previous changes (2)

• src/main/java/com/example/solidconnection/siteuser/service/MyPageService.java
• src/test/java/com/example/solidconnection/siteuser/service/MyPageServiceTest.java

🧰 Additional context used

🧠 Learnings (2)

📓 Common learnings

Learnt from: nayonsoso
Repo: solid-connection/solid-connect-server PR: 448
File: src/main/java/com/example/solidconnection/siteuser/domain/SiteUser.java:118-121
Timestamp: 2025-08-12T21:43:45.760Z
Learning: SiteUser 도메인의 updatePassword 메서드에서는 파라미터 이름을 newEncodedPassword로 하거나 Javadoc을 추가해서 인코딩된 비밀번호가 들어와야 한다는 것을 명시해야 합니다.

📚 Learning: 2025-08-12T21:43:45.760Z

Learnt from: nayonsoso
Repo: solid-connection/solid-connect-server PR: 448
File: src/main/java/com/example/solidconnection/siteuser/domain/SiteUser.java:118-121
Timestamp: 2025-08-12T21:43:45.760Z
Learning: SiteUser 도메인의 updatePassword 메서드에서는 파라미터 이름을 newEncodedPassword로 하거나 Javadoc을 추가해서 인코딩된 비밀번호가 들어와야 한다는 것을 명시해야 합니다.

Applied to files:

• src/main/java/com/example/solidconnection/common/exception/ErrorCode.java

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: build

🔇 Additional comments (1)

src/main/java/com/example/solidconnection/common/exception/ErrorCode.java (1)

70-70: 좋습니다! 에러 코드가 올바르게 정의되었습니다.

새로운 에러 코드가 적절하게 추가되었습니다:

1. 배치 위치: 인증 섹션 내에서 다른 비밀번호 관련 에러들과 함께 논리적으로 배치되었습니다.
2. 상태 코드: BAD_REQUEST 사용은 기존의 비밀번호 검증 에러들(PASSWORD_MISMATCH, PASSWORD_NOT_CHANGED 등)과 일관성을 유지합니다.
3. 메시지: 사용자 친화적이고 명확하게 작성되었습니다.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Hexeong]

LGTM~!

[Gyuhyeok99]

사소한 리뷰니 approve했습니다~

[Gyuhyeok99]

예외가 발생한다로 가는 게 저희 컨벤션에 맞겠네요!

[whqtker]

반영했습니다 !

1f4ccc8