Skip to content

Commit

Permalink
chore: TODO 주석 제거
Browse files Browse the repository at this point in the history
  • Loading branch information
seokjin8678 committed May 27, 2024
1 parent dd1051a commit dc2f515
Show file tree
Hide file tree
Showing 9 changed files with 1 addition and 23 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -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,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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<EntryCodeResponse> createQR(@Member Long memberId,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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")
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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 {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -171,8 +171,6 @@ class 생성 {
}).isInstanceOf(ValidException.class);
}

// TODO 해당 테스트는 생성자 파라미터 순서가 올바른지 검사하는데 의의가 있음
// 다만 빌더 패턴을 적용하면 해당 테스트의 필요성이 있을까?
@Test
void 성공() {
// given
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,6 @@ class createSocialMedia {
.hasMessage(ErrorCode.DUPLICATE_SOCIAL_MEDIA.getMessage());
}

// TODO 다른 Owner에 대한 테스트를 어떻게 작성할지? ex) MethodSource
@Test
void하려는_소셜미디어의_owner가_존재하지_않으면_예외() {
// when & then
Expand Down
1 change: 0 additions & 1 deletion backend/src/test/resources/features/query_festival.feature
Original file line number Diff line number Diff line change
Expand Up @@ -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 어드민 계정으로 로그인한다.
Expand Down

0 comments on commit dc2f515

Please sign in to comment.