Skip to content

Commit c6bd639

Browse files
committed
fix(volunteer-apply): sonar qube 리뷰 반영
1 parent d0511e9 commit c6bd639

File tree

3 files changed

+5
-5
lines changed

3 files changed

+5
-5
lines changed

src/main/java/com/somemore/volunteerapply/dto/VolunteerApplyCreateRequestDto.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66
import com.fasterxml.jackson.databind.annotation.JsonNaming;
77
import com.somemore.volunteerapply.domain.VolunteerApply;
88
import io.swagger.v3.oas.annotations.media.Schema;
9-
import jakarta.validation.constraints.NotBlank;
109
import jakarta.validation.constraints.NotNull;
1110
import java.util.UUID;
1211
import lombok.Builder;

src/test/java/com/somemore/common/fixture/RecruitBoardFixture.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package com.somemore.common.fixture;
22

33
import static com.somemore.common.fixture.LocalDateTimeFixture.createStartDateTime;
4+
import static com.somemore.recruitboard.domain.RecruitStatus.CLOSED;
45
import static com.somemore.recruitboard.domain.RecruitStatus.COMPLETED;
56
import static com.somemore.recruitboard.domain.VolunteerCategory.OTHER;
67

@@ -322,7 +323,7 @@ public static RecruitBoard createCompletedRecruitBoard() {
322323
public static RecruitBoard createCloseRecruitBoard() {
323324
RecruitBoard recruitBoard = createCompletedRecruitBoard(UUID.randomUUID(),
324325
VOLUNTEER_CATEGORY);
325-
setRecruitStatus(recruitBoard, COMPLETED);
326+
setRecruitStatus(recruitBoard, CLOSED);
326327
return recruitBoard;
327328
}
328329

src/test/java/com/somemore/volunteerapply/service/WithdrawVolunteerApplyServiceTest.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,15 +46,15 @@ void withdraw() {
4646
@Test
4747
void withdrawWhenWrongVolunteerId() {
4848
// given
49-
UUID wrongId = UUID.randomUUID();
50-
5149
VolunteerApply apply = createApply(UUID.randomUUID(), APPROVED, false);
5250
volunteerApplyRepository.save(apply);
5351

52+
UUID wrongId = UUID.randomUUID();
53+
Long applyId = apply.getId();
5454
// when
5555
// then
5656
assertThatThrownBy(
57-
() -> withdrawVolunteerApplyService.withdraw(apply.getId(), wrongId)
57+
() -> withdrawVolunteerApplyService.withdraw(applyId, wrongId)
5858
).isInstanceOf(BadRequestException.class)
5959
.hasMessage(UNAUTHORIZED_VOLUNTEER_APPLY.getMessage());
6060
}

0 commit comments

Comments
 (0)