Refactor/user auth type

[minjee2758]

PR 생성 시 아래 항목을 채워주세요.

제목 예시: feat : Pull request template 작성

(작성 후 이 안내 문구는 삭제해주세요)

---

작업 내용

• 어떤 기능(또는 수정 사항)을 구현했는지 간략하게 설명해주세요.
• 예) "회원가입 API에 이메일 중복 검사 기능 추가"

---

변경 사항

• 구현한 주요 로직, 클래스, 메서드 등을 bullet 형식으로 기술해주세요.
• 예)
  • UserService.createUser() 메서드 추가
  • @Email 유효성 검증 적용

---

트러블 슈팅

• 구현 중 마주한 문제와 해결 방법을 기술해주세요.
• 예)
  • 문제: @Transactional이 적용되지 않음
  • 해결: 메서드 호출 방식 변경 (this. → AopProxyUtils. 사용)

---

해결해야 할 문제

• 기능은 동작하지만 리팩토링이나 논의가 필요한 부분을 적어주세요.
• 예)D
  • UserController에서 비즈니스 로직 일부 처리 → 서비스로 이전 고려 필요

---

참고 사항

• 기타 공유하고 싶은 정보나 참고한 문서(링크 등)가 있다면 작성해주세요.

---

코드 리뷰 전 확인 체크리스트

• 불필요한 콘솔 로그, 주석 제거
• 커밋 메시지 컨벤션 준수 (type : )
• 기능 정상 동작 확인

Summary by CodeRabbit

• 버그 수정
  • 특정 사용자에 대한 인증 유형 목록이 항상 비어 반환되던 문제를 수정하여, 실제 보유한 인증 유형이 정상적으로 조회됩니다.
  • 계정 설정, 권한 검증, 로그인 옵션 등 인증 유형에 의존하는 화면/기능에서 정보가 정확히 표시되고 동작 일관성이 향상되었습니다.
  • 잘못된 빈 결과로 인한 예외적 흐름과 혼선을 제거해 사용자 경험을 개선했습니다.

[coderabbitai]

Tip

🔌 Remote MCP (Model Context Protocol) integration is now available!

Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats.

✨ Finishing Touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch refactor/user-authType

---

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

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

• Review comments: Directly reply to a review comment made by CodeRabbit. Example:
  • I pushed a fix in commit <commit_id>, please review it.
  • Open a follow-up GitHub issue for this discussion.
• Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query.
• PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
  • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
  • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

CodeRabbit Commands (Invoked using PR/Issue comments)

Type @coderabbitai help to get the list of available commands.

Other keywords and placeholders

• Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
• Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
• Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

• You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
• Please see the configuration documentation for more information.
• If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Status, Documentation and Community

• Visit our Status Page to check the current availability of CodeRabbit.
• Visit our Documentation for detailed information on how to use CodeRabbit.
• Join our Discord Community to get help, request features, and share feedback.
• Follow us on X/Twitter for updates and announcements.

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (2)

src/main/java/org/ezcode/codetest/infrastructure/persistence/repository/user/UserAuthTypeRepositoryImpl.java (2)

31-33: 반환 리스트의 변경 가능성(immutability) 일관화 제안

현재 getUserAuthType(User)는 stream().toList()로 읽기 전용 리스트를 반환하지만, 변경된 getUserAuthTypesByUser(User)는 JPA가 반환한 리스트(가변) 그대로를 돌려줍니다. 두 메서드의 반환 리스트 변경 가능성이 달라 혼선을 줄 수 있습니다. 일관성을 위해 읽기 전용으로 감싸는 것을 권장합니다.

아래처럼 한 줄만 변경하면 됩니다(Null 반환 가능성이 없다면 안전합니다. 만약 null 가능성이 있다면 코멘트로 알려주세요. List.of()로 대체 가능하도록 제안 드리겠습니다).

-    return userAuthTypeJpaRepository.findUserAuthTypeByUser(user);
+    return List.copyOf(userAuthTypeJpaRepository.findUserAuthTypeByUser(user));

---

26-27: 중복 AuthType 방지 필요 시 .distinct() 적용 검토

동일 유저가 동일 AuthType을 중복으로 가질 가능성을 데이터 제약으로 막지 않는다면, List<AuthType> 결과에서 중복을 제거하는 편이 안전합니다. 성능 측면에서는 JPA Projection으로 AuthType만 select 하는 방법도 고려해주세요.

-        return userAuthTypeJpaRepository.findUserAuthTypeByUser(user).stream()
-            .map(UserAuthType::getAuthType).toList();
+        return userAuthTypeJpaRepository.findUserAuthTypeByUser(user).stream()
+            .map(UserAuthType::getAuthType)
+            .distinct()
+            .toList();

참고: Projection 적용 시에는 UserAuthTypeJpaRepository에 @Query 또는 인터페이스 기반 프로젝션을 추가하는 방식이 적합합니다.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

💡 Knowledge Base configuration:

• MCP integration is disabled by default for public repositories
• Jira integration is disabled by default for public repositories
• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 0f78457 and a43b809.

📒 Files selected for processing (1)

• src/main/java/org/ezcode/codetest/infrastructure/persistence/repository/user/UserAuthTypeRepositoryImpl.java (1 hunks)

⏰ 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