diff --git a/backend/src/main/java/com/festago/auth/dto/command/AdminLoginResult.java b/backend/src/main/java/com/festago/auth/dto/command/AdminLoginResult.java index 1a14cad3f..e835b7de0 100644 --- a/backend/src/main/java/com/festago/auth/dto/command/AdminLoginResult.java +++ b/backend/src/main/java/com/festago/auth/dto/command/AdminLoginResult.java @@ -2,13 +2,6 @@ import com.festago.auth.domain.AuthType; -// TODO Command에서 반환하는 객체의 이름을 어떻게 하면 좋을까 -// 버저닝을 사용하지 않고 AdminLoginResponse라고 한 뒤 버저닝 된 컨트롤러에서 AdminLoginV1Response 객체로 변환하여 사용? -// 혹은 지금과 같이 AdminLoginResult와 같이 Response라는 이름을 빼고 반환할지..? -// Controller에서 필요한 응답은 username과 authType임. -// 그리고 accessToken은 쿠키로 반환하기 때문에 다음과 같이 accessToken이 필드로 있게 되면 필요하지 않은 응답이 나감 -// 클라이언트에서 필요하지 않은 필드는 무시하지만, accessToken과 같은 보안에 관련된 값일때 필요하지 않은 값은 -// 필요가 없다면 보내지 않는게 좋지 않을까? public record AdminLoginResult( String username, AuthType authType, diff --git a/backend/src/main/java/com/festago/entry/presentation/MemberEntranceController.java b/backend/src/main/java/com/festago/entry/presentation/MemberEntranceController.java index 1fb3e7bd1..c77e06c11 100644 --- a/backend/src/main/java/com/festago/entry/presentation/MemberEntranceController.java +++ b/backend/src/main/java/com/festago/entry/presentation/MemberEntranceController.java @@ -19,7 +19,6 @@ public class MemberEntranceController { private final EntryService entryService; - // TODO: URI 다시 생각해보기 @PostMapping("/member-tickets/{memberTicketId}/qr") @Operation(description = "티켓 제시용 QR 코드를 생성한다.", summary = "티켓 제시용 QR 생성") public ResponseEntity createQR(@Member Long memberId, diff --git a/backend/src/main/java/com/festago/school/application/SchoolCommandService.java b/backend/src/main/java/com/festago/school/application/SchoolCommandService.java index afe34c449..f8ca52475 100644 --- a/backend/src/main/java/com/festago/school/application/SchoolCommandService.java +++ b/backend/src/main/java/com/festago/school/application/SchoolCommandService.java @@ -36,9 +36,6 @@ private void validateCreate(SchoolCreateCommand command) { } } - /** - * TODO 학교를 인증한 학생이 있다면, domain을 변경하는 것은 문제가 될 수 있지 않을까? - */ public void updateSchool(Long schoolId, SchoolUpdateCommand command) { School school = schoolRepository.getOrThrow(schoolId); validateUpdate(school, command); diff --git a/backend/src/main/java/com/festago/socialmedia/application/SocialMediaCommandService.java b/backend/src/main/java/com/festago/socialmedia/application/SocialMediaCommandService.java index 30f73a0b0..acfe69be1 100644 --- a/backend/src/main/java/com/festago/socialmedia/application/SocialMediaCommandService.java +++ b/backend/src/main/java/com/festago/socialmedia/application/SocialMediaCommandService.java @@ -37,7 +37,6 @@ private void validateCreate(SocialMediaCreateCommand command) { if (socialMediaRepository.existsByOwnerIdAndOwnerTypeAndMediaType(ownerId, ownerType, socialMediaType)) { throw new BadRequestException(ErrorCode.DUPLICATE_SOCIAL_MEDIA); } - // TODO 추상적인 에러 코드가 필요할지? ex) ErrorCode.SOCIAL_MEDIA_OWNER_NOT_FOUND if (ownerType == OwnerType.ARTIST && !artistRepository.existsById(ownerId)) { throw new NotFoundException(ErrorCode.ARTIST_NOT_FOUND); } diff --git a/backend/src/test/java/com/festago/presentation/MemberTicketControllerTest.java b/backend/src/test/java/com/festago/presentation/MemberTicketControllerTest.java index f3684905a..fa2cf8e8e 100644 --- a/backend/src/test/java/com/festago/presentation/MemberTicketControllerTest.java +++ b/backend/src/test/java/com/festago/presentation/MemberTicketControllerTest.java @@ -31,7 +31,7 @@ import org.springframework.http.MediaType; import org.springframework.test.web.servlet.MockMvc; -// TODO 해당 테스트 코드 개선 또는 컨트롤러 새롭게 설계 필요 +@Deprecated(forRemoval = true) @CustomWebMvcTest @DisplayNameGeneration(ReplaceUnderscores.class) @SuppressWarnings("NonAsciiCharacters") diff --git a/backend/src/test/java/com/festago/school/application/integration/SchoolDeleteServiceIntegrationTest.java b/backend/src/test/java/com/festago/school/application/integration/SchoolDeleteServiceIntegrationTest.java index d19b8a6ab..8b80d1080 100644 --- a/backend/src/test/java/com/festago/school/application/integration/SchoolDeleteServiceIntegrationTest.java +++ b/backend/src/test/java/com/festago/school/application/integration/SchoolDeleteServiceIntegrationTest.java @@ -26,12 +26,6 @@ import org.junit.jupiter.api.Test; import org.springframework.beans.factory.annotation.Autowired; -/** - * TODO - * 서비스 통합 테스트 시 필요한 의존이 너무 많은 것 같다. 차라리 Cucumber를 사용한 통합 테스트로 해당 테스트가 하는 역할을 옮기고, 서비스 통합 테스트는 Stub을 사용한 단위 테스트로 변경하여 - * 서비스가 행하고자 하는 비즈니스 로직을 나타내는 테스트로 하는게 어떨까 deleteSchool() 메서드의 행위는 Validator.validate()를 호출하여, 던져진 예외가 있으면 삭제에 실패한다. 라고 - * 볼 수 있다. - */ @SuppressWarnings("NonAsciiCharacters") @DisplayNameGeneration(DisplayNameGenerator.ReplaceUnderscores.class) class SchoolDeleteServiceIntegrationTest extends ApplicationIntegrationTest { diff --git a/backend/src/test/java/com/festago/school/domain/SchoolTest.java b/backend/src/test/java/com/festago/school/domain/SchoolTest.java index b398cc89f..9ba4b7e1d 100644 --- a/backend/src/test/java/com/festago/school/domain/SchoolTest.java +++ b/backend/src/test/java/com/festago/school/domain/SchoolTest.java @@ -171,8 +171,6 @@ class 생성 { }).isInstanceOf(ValidException.class); } - // TODO 해당 테스트는 생성자 파라미터 순서가 올바른지 검사하는데 의의가 있음 - // 다만 빌더 패턴을 적용하면 해당 테스트의 필요성이 있을까? @Test void 성공() { // given diff --git a/backend/src/test/java/com/festago/socialmedia/application/SocialMediaCommandServiceTest.java b/backend/src/test/java/com/festago/socialmedia/application/SocialMediaCommandServiceTest.java index 8b6c32e5e..8d5b472a7 100644 --- a/backend/src/test/java/com/festago/socialmedia/application/SocialMediaCommandServiceTest.java +++ b/backend/src/test/java/com/festago/socialmedia/application/SocialMediaCommandServiceTest.java @@ -80,7 +80,6 @@ class createSocialMedia { .hasMessage(ErrorCode.DUPLICATE_SOCIAL_MEDIA.getMessage()); } - // TODO 다른 Owner에 대한 테스트를 어떻게 작성할지? ex) MethodSource @Test void 추가하려는_소셜미디어의_owner가_존재하지_않으면_예외() { // when & then diff --git a/backend/src/test/resources/features/query_festival.feature b/backend/src/test/resources/features/query_festival.feature index c64cba2e1..ea4e18ca9 100644 --- a/backend/src/test/resources/features/query_festival.feature +++ b/backend/src/test/resources/features/query_festival.feature @@ -16,7 +16,6 @@ Feature: 사용자가 축제를 조회하는 기능 Given "테코대학교"에서 시작일이 "2023년 07월 12일", 종료일이 "2023년 07월 12일", 이름이 "축제9"인 축제를 생성한다. Given "테코대학교"에서 시작일이 "2023년 07월 05일", 종료일이 "2023년 07월 07일", 이름이 "축제10"인 축제를 생성한다. - # TODO 순서가 제대로 나오는지 확인할 것 원래는 축제7, 축제10, 축제2 순서였음 Scenario: 2023년 7월 6일에 진행중인 축제를 조회한다. Given 현재 시간은 2023년 7월 6일 15시 30분 이다. Given 어드민 계정으로 로그인한다.