Skip to content

[REFACTOR] : SearchUsersResponse를 UserInfoResponse에 통합#260

Merged
yooooonshine merged 1 commit intodevelopfrom
feature/256-refactor-user-info-api
Nov 16, 2025
Merged

[REFACTOR] : SearchUsersResponse를 UserInfoResponse에 통합#260
yooooonshine merged 1 commit intodevelopfrom
feature/256-refactor-user-info-api

Conversation

@yooooonshine
Copy link
Contributor

@yooooonshine yooooonshine commented Nov 16, 2025

개요

작업사항

  • SearchUsersResponse를 UserInfoResponse에 통합

Summary by CodeRabbit

Release Notes

  • Refactor
    • 사용자 검색 API의 응답 구조를 통합하고 이미지 URL 처리 로직을 최적화했습니다.

@coderabbitai
Copy link

coderabbitai bot commented Nov 16, 2025

Caution

Review failed

The pull request is closed.

📋 변경 분석

Walkthrough

사용자 검색 API의 응답 형식을 SearchUsersResponse에서 UserInfoResponse로 통일하는 리팩토링입니다. 서비스 계층의 반환 타입 변경, 컨트롤러 엔드포인트 업데이트, 이미지 URL 처리 로직 단순화를 포함하며 테스트 케이스도 함께 조정되었습니다.

Changes

응집군 / 파일 변경 내용
응답 DTO 통일
src/main/java/hanium/modic/backend/domain/user/service/UserService.java, src/main/java/hanium/modic/backend/web/user/controller/UserController.java
메서드 반환 타입을 Page<SearchUsersResponse>에서 Page<UserInfoResponse>로 변경; 서비스 호출 및 변수 타입 일관성 조정
불필요한 DTO 제거
src/main/java/hanium/modic/backend/web/user/dto/response/SearchUsersResponse.java
SearchUsersResponse 레코드 타입 및 팩토리 메서드 of() 제거
테스트 코드 업데이트
src/test/java/hanium/modic/backend/domain/user/service/UserServiceTest.java
테스트 케이스의 결과 타입을 Page<UserInfoResponse>로 업데이트
임포트 정리
src/test/java/hanium/modic/backend/web/user/controller/UserControllerTest.java
사용하지 않는 임포트(PageImpl, PageRequest, SearchUsersResponse) 제거

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12분

  • 주의할 사항:
    • 모든 임포트 제거가 올바르게 수행되었는지 확인 필요
    • 이미지 URL 처리 로직의 단순화가 모든 엣지 케이스를 적절히 처리하는지 검증
    • 테스트 케이스에서 UserInfoResponse 객체 생성 및 필드 검증이 정확한지 확인

Possibly related PRs

Poem

🐰 응답 형식을 한데 모아,
검색도 조회도 한 가지 옷을 입었네.
SearchUsersResponse 는 안녕, UserInfoResponse 환영해!
깔끔한 API, 그 무엇보다 아름답도다. ✨

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/256-refactor-user-info-api

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a529a2d and de1ae2b.

📒 Files selected for processing (5)
  • src/main/java/hanium/modic/backend/domain/user/service/UserService.java (2 hunks)
  • src/main/java/hanium/modic/backend/web/user/controller/UserController.java (1 hunks)
  • src/main/java/hanium/modic/backend/web/user/dto/response/SearchUsersResponse.java (0 hunks)
  • src/test/java/hanium/modic/backend/domain/user/service/UserServiceTest.java (2 hunks)
  • src/test/java/hanium/modic/backend/web/user/controller/UserControllerTest.java (0 hunks)

Tip

📝 Customizable high-level summaries are now available!

You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.

  • Provide custom instructions to shape the summary (bullet lists, tables, contributor stats, etc.).
  • Use high_level_summary_in_walkthrough to move the summary from the description to the walkthrough section.

Example:

"Create a concise high-level summary as a bullet-point list. Then include a Markdown table showing lines added and removed by each contributing author."


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

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

@yooooonshine yooooonshine merged commit 1683eb8 into develop Nov 16, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant