[BE] 페어룸 삭제 기능 구현 #753
Merged
[BE] 페어룸 삭제 기능 구현 #753
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
기존 페어룸 조회 기능에 삭제된 페어룸을 검증하는 로직도 추가함.
Closed
reddevilmidzy
requested changes
Oct 11, 2024
| @@ -115,6 +116,7 @@ void createReferenceLink(final String url, String accessCodeText, String categor | |||
| .post("/api/" + accessCodeText + "/reference-link"); | |||
| } | |||
|
|
|||
| @Disabled | |||
There was a problem hiding this comment.
저거 실패하는 테스트 id가 하드코딩돼서 문제가 된 거 였어요
커밋
여기서 디스애이블 지우면 ci 터질 거 같으니 요 커밋참고해서 수정해주세요
| @@ -74,4 +74,11 @@ ResponseEntity<List<PairRoomMemberResponse>> getPairRooms( | |||
| @ApiResponse(responseCode = "200", description = "페어룸 존재 여부", content = @Content(mediaType = MediaType.APPLICATION_JSON_VALUE, | |||
| schema = @Schema(implementation = PairRoomExistResponse.class))) | |||
| ResponseEntity<PairRoomExistResponse> pairRoomExists(String accessCode); | |||
|
|
|||
Comment on lines
15
to
16
| COMPLETED, | ||
| DELETE; |
There was a problem hiding this comment.
mysql 컬럼이 현재 enum으로 되어 있어서 변경해줘야 할 거 같은데 그건 제가 하겠습니다
There was a problem hiding this comment.
아 생각해보니까 delete 보나 deleted가 더 좋을 거 같아요..!
reddevilmidzy
approved these changes
Oct 11, 2024
koust6u
approved these changes
Oct 11, 2024
| @@ -91,6 +92,21 @@ void throw_exception_when_find_not_exist_access_code() { | |||
| .isExactlyInstanceOf(PairRoomNotFoundException.class); | |||
| } | |||
|
|
|||
| @Test | |||
| @Transactional | |||
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
연관된 이슈
구현한 기능
상세 설명
논리 삭제 방식으로 페어룸 삭제 API를 구현하였습니다.
추가로 전체 테스트를 돌리면
레퍼런스 링크를 삭제하는 요청테스트가 실패하는 이슈가 있습니다. 해당 테스트를 단일, 혹은 컨트롤러 테스트를 동시에 실행하면 문제가 없지만레퍼런스 링크 서비스 테스트+ 실패하는 해당 테스트 + 다른 컨트롤러 테스트 이렇게 세가지를 함께 실행하면 테스트가 실패하는것으로 보입니다.에러 로그는 "카테고리 링크가 존재하지 않다"고 나오는데 아마 테스트 초기화 과정에서 실행 순서에 의해 꼬인거 같습니다. 지금 몸상태가 좋지 않아 빠르게 해결이 힘들거 같아 우선 비활성화 후 PR 보냅니다.