Feat/52 api page and search

[minjun3021]

What is this PR?

• Github Issue: API 기능 고도화 #52

Key Changes

Test Checklist

[codecov]

Codecov Report

Attention: 32 lines in your changes are missing coverage. Please review.

Comparison is base (786bcee) 84.90% compared to head (c63bdb4) 84.31%.
Report is 2 commits behind head on main.

❗ Current head c63bdb4 differs from pull request most recent head f5b6043. Consider uploading reports for the commit f5b6043 to get more accurate results

Additional details and impacted files

@@             Coverage Diff              @@
##               main     #104      +/-   ##
============================================
- Coverage     84.90%   84.31%   -0.59%     
- Complexity      135      148      +13     
============================================
  Files            41       43       +2     
  Lines           583      676      +93     
  Branches         15       30      +15     
============================================
+ Hits            495      570      +75     
- Misses           78       86       +8     
- Partials         10       20      +10     

Files	Coverage Δ
.../ticketingservice/controller/BookmarkController.kt	100.00% <100.00%> (+13.33%)	⬆️
...up4/ticketingservice/controller/EventController.kt	100.00% <100.00%> (ø)
...p4/ticketingservice/controller/HealthController.kt	100.00% <100.00%> (ø)
...oup4/ticketingservice/controller/UserController.kt	100.00% <100.00%> (ø)
...lin/com/group4/ticketingservice/dto/ResponseDTO.kt	100.00% <100.00%> (+50.00%)	⬆️
...ticketingservice/filter/JwtAuthenticationFilter.kt	90.90% <ø> (-3.04%)	⬇️
...om/group4/ticketingservice/service/EventService.kt	100.00% <100.00%> (ø)
...group4/ticketingservice/service/BookmarkService.kt	92.30% <66.66%> (-7.70%)	⬇️
...up4/ticketingservice/service/ReservationService.kt	70.37% <0.00%> (-2.71%)	⬇️
...cketingservice/controller/ReservationController.kt	89.18% <80.00%> (-10.82%)	⬇️
... and 2 more

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[junha-ahn]

PR이 너무 거대해져서 보기가 힘드네요.

@ParkJeongseop

모든 PR은 200 ~ 300줄 내외가 되고 1~2일 이내에 리뷰를 완료하여 리뷰의 병목을 해소할 수 있었습니다 뱅크샐러드

물런 신규 기능 추가에 200 ~ 300 줄은 무리겠지만, 앞으로는 단위를 조금 더 쪼개거나, 한 PR에 담더라도, 중간 리뷰 요청해주세요. (PR의 목적은 단순 기능 추가 보다는 코드 리뷰에 중점이 있다고 생각합니다)

현재도 리뷰 남기지만 전체 흐름 파악은 불가능하고, 그냥 단순히 한줄한줄 단위로 읽었습니다.

전체적 개선사항

• 추가한 기능에 대한 테스트 코드 추가
• 전체적으로 page, sort 관련 테스트 추가해주세요. (coverage rule 관련해서도 추가 테스트 코드가 필요해보입니다)
• empty list의 경우 { data: [] }를 리턴한다는 테스트 코드를 컨트롤러테스트로 만들어주세요.
• 전체적으로 header에 Setting하는 방식의 자동화가 필요해보입니다. 현재와 같은 방식은 사용불가능해보입니다. (개발자가 직접 기억하고, 필요한 헤더를 세팅해야 한다)

[junha-ahn]

헤더 관련해서 당장 처리가 힘들다면, 일단 해당 Branch는 Page, Sort 관련해서만 진행했으면 좋겠습니다. (feat/52-api-page-and-search)

• 그리고 새로운 브랜치 feat/52-restful-api-headers 뭐 이런 이름으로 만들어서 작업하시는게 좋을것 같습니다.

위에서도 말했지만 현재 작업 내용이 너무 많습니다.

[minjun3021]

pageable sort관련 test가 integrationtest에 필요해보입니다