🧪 [Unit Test] tournamentController 단위 테스트

[middlefitting]

📌 개요

• TournamentController 단위테스트를 작성합니다.

💻 작업사항

• TournamentController 단위테스트 작성
• @pathvarible 검증 및 단위테스트 편의를 위해 컨트롤러에 Validated 추가

💡Issue 번호

• 🧪 [Test] TournamentController 단위 테스트 #439

[codecov-commenter]

Codecov Report

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

Comparison is base (3193717) 10.25% compared to head (6e64ea1) 70.10%.
Report is 9 commits behind head on dev.

Files	Patch %	Lines
...erver/domain/noti/service/sns/SlackbotService.java	7.04%	66 Missing ⚠️
...com/gg/server/domain/item/service/ItemService.java	12.28%	46 Missing and 4 partials ⚠️
...gg/server/admin/user/service/UserAdminService.java	44.57%	44 Missing and 2 partials ⚠️
...min/tournament/service/TournamentAdminService.java	81.48%	15 Missing and 15 partials ⚠️
...com/gg/server/domain/game/service/GameService.java	72.89%	14 Missing and 15 partials ⚠️
...erver/admin/season/service/SeasonAdminService.java	70.21%	10 Missing and 18 partials ⚠️
...gg/server/admin/game/service/GameAdminService.java	72.36%	11 Missing and 10 partials ⚠️
...ver/domain/megaphone/service/MegaphoneService.java	56.25%	16 Missing and 5 partials ⚠️
...ver/admin/user/controller/UserAdminController.java	51.35%	17 Missing and 1 partial ⚠️
.../gg/server/domain/match/utils/MatchCalculator.java	60.46%	10 Missing and 7 partials ⚠️
... and 81 more

Additional details and impacted files

@@              Coverage Diff              @@
##                dev     #522       +/-   ##
=============================================
+ Coverage     10.25%   70.10%   +59.84%     
- Complexity       78      831      +753     
=============================================
  Files           255      255               
  Lines          3812     3974      +162     
  Branches        357      365        +8     
=============================================
+ Hits            391     2786     +2395     
+ Misses         3410      976     -2434     
- Partials         11      212      +201     

Flag	Coverage Δ
integrationTest	69.70% <70.22%> (?)
unitTest	14.77% <9.07%> (+4.51%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[Kimhan-nah]

slice test annotation으로 만들었던 @WebMvcUnitTest는 exclude filter 커스텀 때문에 사용하지 않은 건가요?
컨틀로러 계층의 단위 테스트를 작성하려고 하는데 동일하게 진행해야되는 건지 아니면 만들어둔 어노테이션만 선언하면 되는 건지 궁금합니당
그리고 파일명 "*ControllerMvcTest"로 통일하나요?

[middlefitting]

slice test annotation으로 만들었던 @WebMvcUnitTest는 exclude filter 커스텀 때문에 사용하지 않은 건가요? 컨틀로러 계층의 단위 테스트를 작성하려고 하는데 동일하게 진행해야되는 건지 아니면 만들어둔 어노테이션만 선언하면 되는 건지 궁금합니당 그리고 파일명 "*ControllerMvcTest"로 통일하나요?

파일명은 통일하면 좋을 것 같은데 회의때 이야기해보면 좋을 것 같습니다.

[SONGS4RI]

확인하였습니다!