Skip to content

[test/#287] 검색 도메인 테스트 추가 및 테스트 리팩토링 - #419

Open
2ghrms wants to merge 4 commits into
developfrom
test/#287-search-test
Open

[test/#287] 검색 도메인 테스트 추가 및 테스트 리팩토링#419
2ghrms wants to merge 4 commits into
developfrom
test/#287-search-test

Conversation

@2ghrms

@2ghrms 2ghrms commented Aug 5, 2026

Copy link
Copy Markdown
Member

📌 연관 이슈

🌱 PR 요약

  • Search 도메인 옷 검색/추천 API 테스트 및 MeiliSearch 통합 테스트 작성
  • 다른 도메인의 테스트도 점검 후 수정

🛠작업 내용

  • 테스트 인프라 수정: application-test.ymlapp.base-url 등 누락된 프로퍼티를 채워, @SpringBootTest를 사용하는 모든 테스트의 컨텍스트 로딩이 실패하던 문제를 수정했습니다. (DatabaseCleaner@ElementCollection 조인 테이블을 truncate하지 않던 문제도 함께 수정)
  • 프로덕션 버그 수정:
    • SearchRepositoryCustomImpl.findClothesByKeyword의 QueryDSL 프로젝션이 ClothListResponseparentCategory 필드가 추가된 후 갱신되지 않아, 옷 검색 API가 항상 런타임
      예외를 던지던 버그 수정
    • SearchServiceImpl의 "안 입어본 스타일" 추천 로직이 파라미터를 반대로 넘겨 실제로는 "이미 입어본 스타일" 기록을 추천하던 버그 수정
  • Search 도메인 테스트 추가: SearchServiceTest, SearchDocumentServiceTest, SearchControllerTestmeilisearch-it 프로파일 기반 TestContainers 통합
    테스트(SearchMeiliSearchIntegrationTest) 작성
  • 기존 실패 테스트 수정: StorageUtil.toPublicObjectUrl mock 스텁 누락으로 실패하던 ClothServiceTest/HistoryServiceImplTest 6건 수정
  • 커버리지 보강: 테스트가 전혀 없던 ClothAiController/ClothAiServiceImplextractClothInfo, inferHistoryStyle, detectClothes에 대한 테스트 추가
  • 테스트 메서드명/구조 컨벤션 통일 (@Nested 시나리오 분류, 한국어 문장형 메서드명)

❗리뷰어들께

  • SearchMeiliSearchIntegrationTest는 로컬 Docker(Testcontainers)가 필요합니다. CI에는 별도 조치가 필요 없는지 확인 필요 (Docker-in-Docker 지원 여부).
  • application-test.yml 변경으로 SwaggerSecurityIntegrationTest가 더 이상 local 프로파일을 함께 쓰지 않습니다.

2ghrms added 4 commits August 5, 2026 22:26
- meilisearch-it 프로파일 도입으로 실제 Meilisearch(TestContainers)와 Noop 리포지토리를 전환 가능하게 배선
- 검색/추천 서비스, 문서 변환, 컨트롤러 단위 테스트 및 실제 컨테이너 기반 인덱싱·검색·필터·정렬 통합 테스트 추가
@2ghrms 2ghrms self-assigned this Aug 5, 2026
@2ghrms 2ghrms added the ✅ test label Aug 5, 2026
@2ghrms 2ghrms changed the title [Test/#287] 검색 도메인 테스트 추가 및 테스트 리팩토링 [test/#287] 검색 도메인 테스트 추가 및 테스트 리팩토링 Aug 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[test] Search 도메인 단위 테스트 및 MeiliSearch 포함 통합 테스트 작성

1 participant