Skip to content

[Artist] 아티스트 검색#100

Merged
Creamcheesepie merged 5 commits into
mainfrom
feat/#92
Dec 17, 2025
Merged

[Artist] 아티스트 검색#100
Creamcheesepie merged 5 commits into
mainfrom
feat/#92

Conversation

@ys0221
Copy link
Copy Markdown
Collaborator

@ys0221 ys0221 commented Dec 16, 2025

🔗 관련 이슈

관련된 이슈 번호를 연결해주세요.
Close #이슈번호 를 쓰면 PR merge 시 자동으로 close 됩니다.

🚀 PR 개요

이 PR이 어떤 변경을 포함하고 있는지 간단히 설명해주세요.

아티스트 검색 기능 추가

  1. 아티스트 이름 전체로 검색 -> 해당 아티스트 출력
  2. 키워드 검색 -> 해당 키워드가 포함된 아티스트 목록 출력

📌 변경 사항

주요 변경 내용을 체크리스트 형태로 정리해주세요.

  • 기능 추가
  • 버그 수정
  • 리팩터링
  • 문서 업데이트
  • 테스트 추가/수정

🧪 테스트 방법

변경된 내용을 어떻게 테스트했는지 구체적으로 적어주세요.

포스트맨을 이용하여 테스트했습니다. 밑에 스크린샷 첨부했습니다!

📸 스크린샷 (선택)

UI 변경 또는 시각적으로 확인할 수 있는 변경이 있다면 첨부해주세요.

스크린샷 2025-12-16 오후 4 24 37 -> 키워드 검색 : 한국어 표기(nameKo 에 해당하는 경우) 스크린샷 2025-12-16 오후 4 24 53 -> 키워드 검색 : artistName 에 해당하는 경우 스크린샷 2025-12-16 오후 4 28 44 -> 아티스트 이름으로 검색

⚠️ 참고 사항

리뷰어가 알아야 할 사항이 있다면 자유롭게 작성해주세요.

아직 아티스트 한국어 표기가 정확하지 않아서 한국어 키워드로 검색하는 경우 되는 경우와 안 되는 경우가 있을 수 있습니다...

@ys0221 ys0221 self-assigned this Dec 16, 2025
@ys0221 ys0221 added the Feature label Dec 16, 2025
@github-actions
Copy link
Copy Markdown

Test Results

1 tests  ±0   1 ✅ ±0   0s ⏱️ ±0s
1 suites ±0   0 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit 8e7c68d. ± Comparison against base commit 697d54e.

@github-actions
Copy link
Copy Markdown

⭐ JaCoCo Coverage

Line Coverage: 5.74%

📄 Coverage Details

Overall Line Coverage: 5.71% (122 covered / 2138 lines)

Package Summary (lowest first)
Package Line % Covered Missed
com/back/web7_9_codecrete_be/domain/chats/controller 0.00% 0 0
com/back/web7_9_codecrete_be/domain/location/service 0.00% 0 25
com/back/web7_9_codecrete_be/domain/chats/service 0.00% 0 0
com/back/web7_9_codecrete_be/domain/plans/controller 0.00% 0 33
com/back/web7_9_codecrete_be/domain/location/controller 0.00% 0 4
com/back/web7_9_codecrete_be/global/rsData 0.00% 0 11
com/back/web7_9_codecrete_be/domain/location/entity 0.00% 0 0
com/back/web7_9_codecrete_be/global/scheduler 0.00% 0 3
com/back/web7_9_codecrete_be/domain/auth/entity 0.00% 0 0
com/back/web7_9_codecrete_be/domain/auth/service 0.00% 0 71
com/back/web7_9_codecrete_be/domain/plans/service 0.00% 0 311
com/back/web7_9_codecrete_be/global/initData 0.00% 0 26
com/back/web7_9_codecrete_be/domain/users/controller 0.00% 0 20
com/back/web7_9_codecrete_be/domain/email/entity 0.00% 0 6
com/back/web7_9_codecrete_be/domain/users/service 0.00% 0 59
com/back/web7_9_codecrete_be/domain/auth/controller 0.00% 0 20
com/back/web7_9_codecrete_be/domain/concerts/controller 0.00% 0 22
com/back/web7_9_codecrete_be/domain/artists/controller 0.00% 0 20
com/back/web7_9_codecrete_be/global/aspect 0.00% 0 4
com/back/web7_9_codecrete_be/global/wikidata 0.36% 1 279
com/back/web7_9_codecrete_be/domain/artists/service 0.82% 3 361
com/back/web7_9_codecrete_be/global/musicbrainz 0.99% 1 100
com/back/web7_9_codecrete_be/domain/email/service 1.92% 1 51
com/back/web7_9_codecrete_be/domain/concerts/entity 4.76% 3 60
com/back/web7_9_codecrete_be/domain/concerts/service 6.80% 21 288
com/back/web7_9_codecrete_be/global/rq 12.90% 4 27
com/back/web7_9_codecrete_be/domain/artists/entity 16.22% 6 31
com/back/web7_9_codecrete_be/domain/users/entity 22.86% 8 27
com/back/web7_9_codecrete_be/domain/plans/entity 24.42% 21 65
com/back/web7_9_codecrete_be/global/security 26.36% 29 81
com/back/web7_9_codecrete_be/global/storage 33.33% 1 2
com/back/web7_9_codecrete_be/global/spotify 35.71% 5 9
com/back/web7_9_codecrete_be/domain/chats/entity 100.00% 1 0
com/back/web7_9_codecrete_be/global/doc 100.00% 17 0
Lowest Covered Classes (Top 20)
Class Line % Covered Missed
com.back.web7_9_codecrete_be.domain.plans.service.PlanService 0.00% 0 311
com.back.web7_9_codecrete_be.domain.users.service.UserService 0.00% 0 55
com.back.web7_9_codecrete_be.domain.artists.service.ArtistService 0.00% 0 51
com.back.web7_9_codecrete_be.domain.concerts.service.ConcertService 0.00% 0 48
com.back.web7_9_codecrete_be.domain.auth.service.AuthService 0.00% 0 44
com.back.web7_9_codecrete_be.domain.concerts.entity.Concert 0.00% 0 38
com.back.web7_9_codecrete_be.global.security.JwtTokenProvider 0.00% 0 37
com.back.web7_9_codecrete_be.domain.plans.entity.Schedule 0.00% 0 36
com.back.web7_9_codecrete_be.domain.plans.controller.PlanController 0.00% 0 33
com.back.web7_9_codecrete_be.domain.users.entity.User 0.00% 0 27
com.back.web7_9_codecrete_be.domain.auth.service.TokenService 0.00% 0 27
com.back.web7_9_codecrete_be.domain.artists.entity.Artist 0.00% 0 26
com.back.web7_9_codecrete_be.global.initData.BaseInitData 0.00% 0 26
com.back.web7_9_codecrete_be.domain.users.controller.UserController 0.00% 0 20
com.back.web7_9_codecrete_be.domain.auth.controller.AuthController 0.00% 0 20
com.back.web7_9_codecrete_be.domain.artists.controller.ArtistsController 0.00% 0 20
com.back.web7_9_codecrete_be.domain.plans.entity.Plan 0.00% 0 18
com.back.web7_9_codecrete_be.domain.concerts.controller.ConcertController 0.00% 0 14
com.back.web7_9_codecrete_be.domain.location.service.TmapService 0.00% 0 13
com.back.web7_9_codecrete_be.domain.concerts.service.KopisApiService$TicketPrice 0.00% 0 13
Changed Classes (from this PR)
Source (PR) Class Prefix Line % Covered Missed
src/main/java/com/back/web7_9_codecrete_be/domain/artists/service/ArtistService.java com.back.web7_9_codecrete_be.domain.artists.service.ArtistService 0.00% 0 51
src/main/java/com/back/web7_9_codecrete_be/domain/artists/controller/ArtistsController.java com.back.web7_9_codecrete_be.domain.artists.controller.ArtistsController 0.00% 0 20
src/main/java/com/back/web7_9_codecrete_be/domain/artists/dto/request/CreateRequest.java com.back.web7_9_codecrete_be.domain.artists.dto.request.CreateRequest 0.00% 0 0
src/main/java/com/back/web7_9_codecrete_be/domain/artists/dto/request/SearchRequest.java com.back.web7_9_codecrete_be.domain.artists.dto.request.SearchRequest 0.00% 0 0
src/main/java/com/back/web7_9_codecrete_be/domain/artists/dto/request/UpdateRequest.java com.back.web7_9_codecrete_be.domain.artists.dto.request.UpdateRequest 0.00% 0 0
src/main/java/com/back/web7_9_codecrete_be/domain/artists/dto/response/ArtistDetailResponse.java com.back.web7_9_codecrete_be.domain.artists.dto.response.ArtistDetailResponse 0.00% 0 0
src/main/java/com/back/web7_9_codecrete_be/domain/artists/dto/response/SearchResponse.java com.back.web7_9_codecrete_be.domain.artists.dto.response.SearchResponse 0.00% 0 0
src/main/java/com/back/web7_9_codecrete_be/domain/artists/repository/ArtistRepository.java com.back.web7_9_codecrete_be.domain.artists.repository.ArtistRepository 0.00% 0 0
src/main/java/com/back/web7_9_codecrete_be/global/error/code/ArtistErrorCode.java com.back.web7_9_codecrete_be.global.error.code.ArtistErrorCode 0.00% 0 0
src/main/java/com/back/web7_9_codecrete_be/domain/artists/service/ArtistEnrichService.java com.back.web7_9_codecrete_be.domain.artists.service.ArtistEnrichService 0.81% 1 123
src/main/java/com/back/web7_9_codecrete_be/domain/artists/service/SpotifyService.java com.back.web7_9_codecrete_be.domain.artists.service.SpotifyService 1.08% 2 183
src/main/java/com/back/web7_9_codecrete_be/domain/artists/entity/Artist.java com.back.web7_9_codecrete_be.domain.artists.entity.Artist 13.33% 4 26
src/main/java/com/back/web7_9_codecrete_be/domain/artists/entity/ArtistType.java com.back.web7_9_codecrete_be.domain.artists.entity.ArtistType 100.00% 3 0

🔗 Full HTML report: See artifact jacoco-full-html on this run → https://github.com/prgrms-web-devcourse-final-project/WEB7_9_codecrete_BE/actions/runs/20259948868

Copy link
Copy Markdown
Collaborator

@larama-C larama-C left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

사진을 추가해주셔서 한눈에 알아보기 쉬웠던 것 같습니다! 수고하셨습니다!


public record SearchRequest(
@NotBlank(message = "검색어를 입력해주세요")
String artistName
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@notblank 외에도 다른 유효성 검증 수단을 추가하면 좋을 것 같습니다.
(정작 이 글 쓰는 팀장도 유효성 검증 수단을 빼먹은게... 👀

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

넵 추가하겠습니다!!

Comment on lines +113 to +114
@Transactional
public List<SearchResponse> search(String artistName) {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

조회이기 때문에 @transactional(readOnly = true)로 하면 더 좋을 것 같습니다!

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

넹 추가하겠습니다!!

@Creamcheesepie Creamcheesepie merged commit 26bfec5 into main Dec 17, 2025
2 checks passed
@Creamcheesepie Creamcheesepie deleted the feat/#92 branch December 17, 2025 00:19
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.

[Artist] 아티스트 검색

4 participants