feat: 특정 여행 상세 삭제 API 구현 #24

backend/src/main/java/com/staccato/travel/controller/TravelController.java

@@ -7,6 +7,7 @@
 
 import org.springframework.http.ResponseEntity;
 import org.springframework.validation.annotation.Validated;
+import org.springframework.web.bind.annotation.DeleteMapping;
 import org.springframework.web.bind.annotation.GetMapping;
 import org.springframework.web.bind.annotation.PathVariable;
 import org.springframework.web.bind.annotation.PostMapping;
@@ -51,4 +52,12 @@ public ResponseEntity<Void> updateTravel(
         travelService.updateTravel(travelRequest, travelId);
         return ResponseEntity.ok().build();
     }
+
+    @DeleteMapping("/{travelId}")
+    public ResponseEntity<Void> deleteTravel(
+            @PathVariable @Min(value = 1L, message = "여행 식별자는 양수로 이루어져야 합니다.") Long travelId,
+            @MemberId Long memberId) {
+        travelService.deleteTravel(travelId);
+        return ResponseEntity.ok().build();
+    }
 }

backend/src/main/java/com/staccato/travel/service/TravelService.java

@@ -85,4 +85,13 @@ private TravelResponses getTravelDetailResponses(List<TravelMember> travelMember
                 .toList();
         return TravelResponses.from(travels);
     }
+
+    @Transactional
+    public void deleteTravel(Long travelId) {
+        if (!visitRepository.findAllByTravelId(travelId).isEmpty()) {
+            throw new StaccatoException("해당 여행 상세에 방문 기록이 남아있어 삭제할 수 없습니다.");
+        }
+        visitRepository.deleteByTravelId(travelId);
+        travelRepository.deleteById(travelId);
+    }
 }

backend/src/main/java/com/staccato/visit/repository/VisitRepository.java

@@ -3,9 +3,16 @@
 import java.util.List;
 
 import org.springframework.data.jpa.repository.JpaRepository;
+import org.springframework.data.jpa.repository.Modifying;
+import org.springframework.data.jpa.repository.Query;
+import org.springframework.data.repository.query.Param;
 
 import com.staccato.visit.domain.Visit;
 
 public interface VisitRepository extends JpaRepository<Visit, Long> {
     List<Visit> findAllByTravelId(Long travelId);
+
+    @Modifying(clearAutomatically = true)
+    @Query("UPDATE Visit vi SET vi.isDeleted = true WHERE vi.travel.id = :travelId")
+    void deleteByTravelId(@Param("travelId") Long travelId);
 }

backend/src/test/java/com/staccato/travel/controller/TravelIntegrationTest.java

@@ -9,6 +9,7 @@
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.DynamicTest;
+import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.TestFactory;
 import org.junit.jupiter.params.ParameterizedTest;
 import org.junit.jupiter.params.provider.Arguments;
@@ -226,4 +227,21 @@ private TravelRequest createTravelRequest(int year) {
                 LocalDate.of(year, 7, 1),
                 LocalDate.of(year, 7, 10));
     }
+
+    @DisplayName("사용자가 여행 상세 삭제를 요청하면, 여행 상세를 삭제한다.")
+    @Test
+    void deleteTravel() {
+        // given
+        Long travelId = 1L;
+        TravelRequest travelRequest = new TravelRequest("https://example.com/travels/geumohrm.jpg", "2023 여름 휴가", "친구들과 함께한 여름 휴가 여행", LocalDate.of(2023, 7, 1), LocalDate.of(2023, 7, 10));
+        createTravel(travelRequest);
+
+        // when & then
+        RestAssured.given().pathParam("travelId", travelId).log().all()
+                .header(HttpHeaders.AUTHORIZATION, USER_AUTHORIZATION)
+                .contentType(ContentType.JSON)
+                .when().delete("/travels/{travelId}")
+                .then().log().all()
+                .assertThat().statusCode(HttpStatus.OK.value());
+    }
 }

backend/src/test/java/com/staccato/travel/domain/TravelTest.java

@@ -43,7 +43,7 @@ void withinDuration() {
 
     @DisplayName("여행 상세를 수정 시 기존 방문 기록 날짜를 포함하지 않는 경우 수정에 실패한다.")
     @Test
-    void validateDuration(){
+    void validateDuration() {
         // given
         Travel travel = Travel.builder()
                 .title("2023 여름 여행")

backend/src/test/java/com/staccato/travel/service/TravelServiceTest.java

@@ -18,12 +18,16 @@
 import com.staccato.exception.StaccatoException;
 import com.staccato.member.domain.Member;
 import com.staccato.member.repository.MemberRepository;
+import com.staccato.pin.domain.Pin;
+import com.staccato.pin.repository.PinRepository;
 import com.staccato.travel.domain.Travel;
 import com.staccato.travel.domain.TravelMember;
 import com.staccato.travel.repository.TravelMemberRepository;
 import com.staccato.travel.repository.TravelRepository;
 import com.staccato.travel.service.dto.request.TravelRequest;
 import com.staccato.travel.service.dto.response.TravelResponses;
+import com.staccato.visit.domain.Visit;
+import com.staccato.visit.repository.VisitRepository;
 
 class TravelServiceTest extends ServiceSliceTest {
     @Autowired
@@ -34,6 +38,10 @@ class TravelServiceTest extends ServiceSliceTest {
     private TravelMemberRepository travelMemberRepository;
     @Autowired
     private TravelRepository travelRepository;
+    @Autowired
+    private VisitRepository visitRepository;
+    @Autowired
+    private PinRepository pinRepository;
 
     static Stream<Arguments> yearProvider() {
         return Stream.of(
@@ -90,15 +98,8 @@ private static TravelRequest createTravelRequest(int year) {
     @Test
     void updateTravel() {
         // given
-        Long travelId = 1L;
-        Travel originTravel = new Travel(
-                "https://example.com/travels/geumohrm.jpg",
-                "2023 여름 휴가",
-                "친구들과 함께한 여름 휴가 여행",
-                LocalDate.of(2023, 7, 1),
-                LocalDate.of(2023, 7, 10)
-        );
-        travelRepository.save(originTravel);
+        Travel travel = createAndSaveTravel(2023);
+        Long travelId = travel.getId();
         TravelRequest updatedTravel = new TravelRequest(
                 "https://example.com/travels/geumohrm.jpg",
                 "2023 신나는 여름 휴가",
@@ -109,33 +110,66 @@ void updateTravel() {
 
         // when
         travelService.updateTravel(updatedTravel, travelId);
-        Travel travel = travelRepository.findById(travelId).get();
+        Travel foundedTravel = travelRepository.findById(travelId).get();
 
         // then
         assertAll(
-                () -> assertThat(travel.getId()).isEqualTo(travelId),
-                () -> assertThat(travel.getTitle()).isEqualTo(updatedTravel.travelTitle()),
-                () -> assertThat(travel.getDescription()).isEqualTo(updatedTravel.description()),
-                () -> assertThat(travel.getStartAt()).isEqualTo(updatedTravel.startAt()),
-                () -> assertThat(travel.getEndAt()).isEqualTo(updatedTravel.endAt())
+                () -> assertThat(foundedTravel.getId()).isEqualTo(travelId),
+                () -> assertThat(foundedTravel.getTitle()).isEqualTo(updatedTravel.travelTitle()),
+                () -> assertThat(foundedTravel.getDescription()).isEqualTo(updatedTravel.description()),
+                () -> assertThat(foundedTravel.getStartAt()).isEqualTo(updatedTravel.startAt()),
+                () -> assertThat(foundedTravel.getEndAt()).isEqualTo(updatedTravel.endAt())
         );
     }
 
+    private Travel createAndSaveTravel(int year) {
+        Travel travel = new Travel(
+                "https://example.com/travels/geumohrm.jpg",
+                year + " 여름 휴가",
+                "친구들과 함께한 여름 휴가 여행",
+                LocalDate.of(year, 7, 1),
+                LocalDate.of(year, 7, 10)
+        );
+        return travelRepository.save(travel);
+    }
+
     @DisplayName("존재하지 않는 여행 상세를 수정하려 할 경우 예외가 발생한다.")
     @Test
     void failUpdateTravel() {
         // given
-        TravelRequest travelRequest = new TravelRequest(
-                "https://example.com/travels/geumohrm.jpg",
-                "2023 여름 휴가",
-                "친구들과 함께한 여름 휴가 여행",
-                LocalDate.of(2023, 7, 1),
-                LocalDate.of(2023, 7, 10)
-        );
+        TravelRequest travelRequest = createTravelRequest(2023);
 
         // when & then
         assertThatThrownBy(() -> travelService.updateTravel(travelRequest, 1L))
                 .isInstanceOf(StaccatoException.class)
                 .hasMessage("요청하신 여행을 찾을 수 없어요.");
     }
+
+    @DisplayName("여행 식별값을 통해 여행 상세를 삭제한다.")
+    @Test
+    void deleteTravel() {
+        // given
+        Travel travel = createAndSaveTravel(2023);
+
+        // when
+        travelService.deleteTravel(travel.getId());
+
+        // then
+        Travel foundTravel = travelRepository.findById(travel.getId()).get();
+        assertThat(foundTravel.getIsDeleted()).isTrue();
+    }
+
+    @DisplayName("방문기록이 존재하는 여행 상세에 삭제를 시도할 경우 예외가 발생한다.")
+    @Test
+    void failDeleteTravel() {
+        // given
+        Travel travel = createAndSaveTravel(2023);
+        Pin pin = pinRepository.save(Pin.builder().place("Sample Place").address("Sample Address").build());
+        visitRepository.save(Visit.builder().visitedAt(LocalDate.now()).pin(pin).travel(travel).build());
+
+        // when & then
+        assertThatThrownBy(() -> travelService.deleteTravel(travel.getId()))
+                .isInstanceOf(StaccatoException.class)
+                .hasMessage("해당 여행 상세에 방문 기록이 남아있어 삭제할 수 없습니다.");
+    }
 }