Skip to content

[FEAT] 일반 구성원 조회/등록 개선 및 프로젝트 페이징 수정#600

Merged
kangkings merged 4 commits into
devfrom
feat/#597-to-single-filter
Jul 21, 2026
Merged

[FEAT] 일반 구성원 조회/등록 개선 및 프로젝트 페이징 수정#600
kangkings merged 4 commits into
devfrom
feat/#597-to-single-filter

Conversation

@kangkings

@kangkings kangkings commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

#️⃣연관된 이슈

close #597
close #598
close #599

📝 작업 내용

  • 일반 구성원 목록 조회 필터를 단일 필터 방식으로 변경
  • 일반 구성원 추가 시 기존 삭제 데이터 재사용 로직을 덮어쓰기 방식으로 변경
  • 프로젝트 목록 조회 시 category 필터가 count 쿼리에도 적용되도록 수정
  • 프로젝트 목록 조회 페이징 메타 데이터 회귀 테스트 보강

🙏 리뷰 요구사항 (선택)

작은 수정 건이 많아 작업 편의를 위해 여러 이슈를 한 번에 작업했습니다. 양해 부탁드립니다!

대신 작업 내용은 커밋 단위로 깔끔하게 나눠두었습니다!

검증 내용:

  • ./gradlew test --tests org.ject.support.domain.project.repository.ProjectQueryRepositoryTest -x jacocoTestCoverageVerification
  • 로컬 API GET /projects?category=SEMESTER_1&page=0&size=6&sort=sorted 응답 확인
  • 로컬 API GET /projects?category=SEMESTER_1&page=1&size=6&sort=sorted 응답 확인

Summary by CodeRabbit

  • 변경 사항

    • 구성원 학기 검색 필터를 다중 선택에서 단일 선택 방식으로 변경했습니다.
    • 프로젝트 카테고리별 조회 시 전체 개수와 페이지 정보가 필터 결과와 일치하도록 개선했습니다.
    • 기존 구성원 복구 후 프로필 정보가 최신 요청 값으로 갱신됩니다.
    • 구성원 검색 시 팀과 활동 상태 조건 검증을 강화했습니다.
  • 테스트

    • 단일 필터 검색, 구성원 프로필 갱신, 페이지네이션 정보를 검증하는 테스트를 보강했습니다.

@coderabbitai

coderabbitai Bot commented Jul 21, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

회원 학기 검색 필터를 다중 값에서 단일 값으로 변경하고, 검증·조회 쿼리·테스트를 갱신했다. 삭제 회원 복구와 프로필 갱신을 분리했으며, 프로젝트 카테고리 조회의 count 조건과 페이지네이션 테스트를 보완했다.

Changes

회원 검색 필터 단일화

Layer / File(s) Summary
검색 조건과 조회 검증·쿼리 변경
src/main/java/org/ject/support/admin/member/dto/request/MemberSemesterSearchCondition.java, src/main/java/org/ject/support/admin/member/service/AdminMemberUseCase.java, src/main/java/org/ject/support/domain/member/repository/MemberActivityQueryRepositoryImpl.java
회원 검색 필터를 단일 값으로 변경하고 생성·조회 검증 및 eq 기반 검색·카운트 조건을 적용했다.
단일 필터 요청과 조회 테스트 갱신
src/test/java/org/ject/support/admin/member/controller/AdminMemberSemesterControllerTest.java, src/test/java/org/ject/support/admin/member/service/*Test.java, src/test/java/org/ject/support/domain/member/repository/MemberActivityRepositoryTest.java
요청 파라미터, 조건 객체 생성, 팀·상태 예외 및 저장소 검색 테스트를 단일 값 방식으로 갱신했다.

기존 회원 복구 흐름

Layer / File(s) Summary
복구와 프로필 갱신 흐름
src/main/java/org/ject/support/admin/member/service/AdminMemberService.java, src/main/java/org/ject/support/domain/member/entity/Member.java, src/test/java/org/ject/support/admin/member/service/AdminMemberServiceTest.java
삭제 회원 복구와 프로필 갱신을 별도 동작으로 분리하고, 기존 회원 테스트에서 프로필 덮어쓰기를 검증했다.

프로젝트 페이지네이션 count 보정

Layer / File(s) Summary
카테고리 count와 페이지 결과 검증
src/main/java/org/ject/support/domain/project/repository/ProjectQueryRepositoryImpl.java, src/test/java/org/ject/support/domain/project/repository/ProjectQueryRepositoryTest.java
카테고리 조건을 count 쿼리에 추가하고 전체 요소 수, 페이지 수, 다음 페이지 여부와 범위 초과 페이지를 검증했다.

Estimated code review effort: 3 (Moderate) | ~25 minutes

Poem

토끼가 당근처럼 필터를 골라
하나씩 쏙쏙 조건을 담았네
지운 회원은 다시 폴짝 살아나고
프로필도 새 옷을 입었네
프로젝트 페이지 숫자도 가지런해
깡충, 테스트까지 반짝! 🐇

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning [#597, #598]은 충족되지만 [#599]의 팀원 조회 ACTIVE/COMPLETED 필터 추가는 변경 내역에서 확인되지 않습니다. 프로젝트별 팀원 조회 쿼리에 ACTIVE 또는 COMPLETED 상태만 포함되도록 필터를 추가하세요.
Docstring Coverage ⚠️ Warning Docstring coverage is 26.92% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (3 passed)
Check name Status Explanation
Out of Scope Changes check ✅ Passed 변경된 파일들은 모두 일반 구성원 조회/등록과 프로젝트 페이징 개선 범위에 맞아 보이며 명백한 무관 변경은 없습니다.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 일반 구성원 조회/등록 개선과 프로젝트 페이징 수정이라는 핵심 변경을 정확히 요약합니다.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/#597-to-single-filter

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.

@kangkings
kangkings requested a review from 023-dev July 21, 2026 02:37
@kangkings kangkings self-assigned this Jul 21, 2026
@kangkings kangkings added the 👨🏻‍💻backend 백엔드 작업 label Jul 21, 2026
@github-actions

Copy link
Copy Markdown

Test Results

417 tests  +1   417 ✅ +1   13s ⏱️ ±0s
 77 suites ±0     0 💤 ±0 
 77 files   ±0     0 ❌ ±0 

Results for commit f2bf597. ± Comparison against base commit 0a9ba05.

@github-actions

Copy link
Copy Markdown

📊테스트 커버리지

Overall Project 85.34% 🍏
Files changed 100% 🍏

File Coverage
AdminMemberService.java 100% 🍏
AdminMemberUseCase.java 100% 🍏
Member.java 100% 🍏
ProjectQueryRepositoryImpl.java 100% 🍏
MemberActivityQueryRepositoryImpl.java 99.55% 🍏

@kangkings
kangkings merged commit bfccbc9 into dev Jul 21, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

👨🏻‍💻backend 백엔드 작업

Projects

Status: Done

2 participants