[FEAT/#406] 지도/제휴 조회 API에 카테고리·학생회 필터링 추가

[BAEK0111]

#️⃣연관된 이슈

close #406

📝작업 내용

기존 /map/nearby, /students/usable API는 필터링 없이 전체 결과를 반환했습니다.
프론트 요구사항에 따라 카테고리(storeCategory)와 학생회(adminId) 기준 필터링을 추가하되, 파라미터를 넘기지 않으면 기존과 동일하게 동작하도록 nullable optional로 구현했습니다.

🔎코드 설명(스크린샷(선택))

StoreCategory enum 추가 및 Store 엔티티 연결

• store/entity/enums/StoreCategory.java 신규 추가
• Store 엔티티에 storeCategory 필드 추가 (@Enumerated(EnumType.STRING))

카테고리/학생회 필터링 — 쿼리 레벨 통합 처리

• findActivePartnershipsByStudentId에 storeCategory, adminId 파라미터 추가
• 쿼리에 AND (:storeCategory IS NULL OR s.storeCategory = :storeCategory), AND (:adminId IS NULL OR a.id = :adminId) 조건 추가 → null이면 전체 조회
• GET /map/nearby (STUDENT), GET /students/usable 양쪽에 적용
• 기존에 서비스 레벨에서 하던 adminId 필터링 블록 제거

API 파라미터

• GET /map/nearby: storeCategory, adminId optional RequestParam 추가 (STUDENT 전용)
• GET /students/usable: storeCategory, adminId optional RequestParam 추가

테스트

• MapServiceImplTest: getStores 테스트 4건 추가 및 수정 (adminId 필터링이 레포 레벨임을 verify로 검증)
• StudentServiceImplTest: storeCategory, adminId 필터링 테스트 각 1건 추가

💬고민사항 및 리뷰 요구사항 (Optional)

• storeCategory, adminId 모두 findActivePartnershipsByStudentId 쿼리 한 곳에서 처리합니다. 필터 조건이 늘어날 경우 Querydsl 도입을 고려할 수 있습니다.
• Store.storeCategory 컬럼이 새로 추가되어 마이그레이션 필요합니다. 기존 데이터의 storeCategory는 null 상태로 유지됩니다.

비고 (Optional)

• storeCategory, adminId 모두 미전송 시 기존 API와 동일하게 동작합니다.

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 4efde6be-52b3-4b18-884b-08982810291c

---

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.}

[eeeeeaaan]

p5
all 파라미터 이름이 약간 모호한 것 같아요
요약 조회 목적이라면 previewOnly / isSummary 등으로 필드명을 좀더 구체적으로 바꾸는 것도 좋아보입니다.

[eeeeeaaan]

외부에서 먼저 admin name 으로 map을 시킨 후에 안에서 필터링을 하는게 탐색 시간에 대해 좋아보여요!!

        .filter(p -> p.getAdmin() != null)
        .collect(Collectors.toMap(
                p -> p.getAdmin().getId(),
                p -> p.getAdmin().getName(),
                (existing, replacement) -> existing // 중복 id 방어
        ));

String adminName = adminNameMap.get(paperAdminId);

이런식으로요!!