[BE] 체크리스트 업데이트 시 지하철 역도 변경되도록 수정한다

backend/bang-ggood/src/main/java/com/bang_ggood/checklist/controller/ChecklistController.java

@@ -77,6 +77,15 @@ public ResponseEntity<Void> updateChecklistById(
         return ResponseEntity.noContent().build();
     }
 
+    @PutMapping("/v1/checklists/{id}")
+    public ResponseEntity<Void> updateChecklistByIdV1(
+            @AuthRequiredPrincipal User user,
+            @PathVariable("id") long id,
+            @Valid @RequestBody ChecklistRequestV1 checklistRequestV1) {
+        checklistManageService.updateChecklistByIdV1(user, id, checklistRequestV1);
+        return ResponseEntity.noContent().build();
+    }
+
     @DeleteMapping("/checklists/{id}")
     public ResponseEntity<Void> deleteChecklistById(@AuthRequiredPrincipal User user, @PathVariable("id") long id) {
         checklistManageService.deleteChecklistById(user, id);

backend/bang-ggood/src/main/java/com/bang_ggood/checklist/service/ChecklistManageService.java

@@ -224,6 +224,20 @@ public void updateChecklistById(User user, Long checklistId, ChecklistRequest ch
         updateChecklistMaintenances(checklistRequest, checklist);
     }
 
+    @Transactional
+    public void updateChecklistByIdV1(User user, Long checklistId, ChecklistRequestV1 checklistRequestV1) {
+        Checklist checklist = checklistService.readChecklist(user, checklistId);
+
+        ChecklistRequest checklistRequest = checklistRequestV1.toChecklistRequest();
+        roomService.updateRoom(checklist.getRoom(), checklistRequest.toRoomEntity());
+        checklistService.updateChecklist(checklist, checklistRequest.toChecklistEntity(checklist.getRoom(), user));
+
+        updateChecklistOptions(checklistRequest, checklist);
+        updateChecklistQuestions(checklistRequest, checklist);
+        updateChecklistMaintenances(checklistRequest, checklist);
+        updateChecklistStations(checklistRequestV1, checklist);
+    }
+
     private void updateChecklistOptions(ChecklistRequest checklistRequest, Checklist checklist) {
         List<ChecklistOption> checklistOptions = checklistRequest.options().stream()
                 .map(option -> new ChecklistOption(checklist, option))
@@ -252,4 +266,10 @@ private void updateChecklistMaintenances(ChecklistRequest checklistRequest, Chec
                         .toList();
         checklistMaintenanceService.updateMaintenances(checklist.getId(), checklistMaintenances);
     }
+
+    private void updateChecklistStations(ChecklistRequestV1 checklistRequestV1, Checklist checklist) {
+        double latitude = checklistRequestV1.geolocation().latitude();
+        double longitude = checklistRequestV1.geolocation().longitude();
+        checklistStationService.updateChecklistStation(checklist, latitude, longitude);
+    }
 }

backend/bang-ggood/src/main/java/com/bang_ggood/station/repository/ChecklistStationRepository.java

@@ -3,8 +3,10 @@
 import com.bang_ggood.checklist.domain.Checklist;
 import com.bang_ggood.station.domain.ChecklistStation;
 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 org.springframework.transaction.annotation.Transactional;
 import java.util.List;
 
 public interface ChecklistStationRepository extends JpaRepository<ChecklistStation, Long> {
@@ -13,4 +15,11 @@ public interface ChecklistStationRepository extends JpaRepository<ChecklistStati
             "where cs.checklist = :checklist " +
             "and cs.deleted = false")
     List<ChecklistStation> findByChecklist(@Param("checklist") Checklist checklist);
+
+    @Transactional
+    @Modifying(flushAutomatically = true, clearAutomatically = true)
+    @Query("UPDATE ChecklistStation cs " +
+            "SET cs.deleted = true " +
+            "WHERE cs.checklist.id = :checklistId")
+    void deleteAllByChecklistId(Long checklistId);
 }

backend/bang-ggood/src/main/java/com/bang_ggood/station/service/ChecklistStationService.java

@@ -19,6 +19,21 @@ public class ChecklistStationService {
 
     @Transactional
     public void createChecklistStations(Checklist checklist, double latitude, double longitude) {
+        saveChecklistStations(checklist, latitude, longitude);
+    }
+
+    @Transactional(readOnly = true)
+    public List<ChecklistStation> readChecklistStationsByChecklist(Checklist checklist) {
+        return checklistStationRepository.findByChecklist(checklist);
+    }
+
+    @Transactional
+    public void updateChecklistStation(Checklist checklist, double latitude, double longitude) {
+        checklistStationRepository.deleteAllByChecklistId(checklist.getId());
+        saveChecklistStations(checklist, latitude, longitude);
+    }
+
+    private void saveChecklistStations(Checklist checklist, double latitude, double longitude) {
         List<SubwayStationResponse> responses = subwayStationService.readNearestStation(latitude, longitude)
                 .getStations();
         List<ChecklistStation> checklistStations = new ArrayList<>();
@@ -32,9 +47,4 @@ public void createChecklistStations(Checklist checklist, double latitude, double
 
         checklistStationRepository.saveAll(checklistStations);
     }
-
-    @Transactional(readOnly = true)
-    public List<ChecklistStation> readChecklistStationsByChecklist(Checklist checklist) {
-        return checklistStationRepository.findByChecklist(checklist);
-    }
 }

backend/bang-ggood/src/test/java/com/bang_ggood/checklist/ChecklistFixture.java

@@ -157,6 +157,15 @@ public static ChecklistRequest CHECKLIST_UPDATE_REQUEST() {
         );
     }
 
+    public static ChecklistRequestV1 CHECKLIST_UPDATE_REQUEST_V1() {
+        return new ChecklistRequestV1(
+                RoomFixture.ROOM_UPDATE_REQUEST(), List.of(1, 2, 3, 4),
+                List.of(QUESTION_1_CREATE_REQUEST(), QUESTION_2_CREATE_REQUEST(),
+                        QUESTION_3_CREATE_REQUEST(), QUESTION_5_UPDATE_REQUEST()),
+                ChecklistStationRequest.of(37.5, 127.1)
+        );
+    }
+
     public static ChecklistRequest CHECKLIST_UPDATE_REQUEST_NO_ROOM_NAME() {
         return new ChecklistRequest(
                 RoomFixture.ROOM_CREATE_REQUEST_NO_ROOM_NAME(),

backend/bang-ggood/src/test/java/com/bang_ggood/checklist/controller/ChecklistE2ETest.java

@@ -154,6 +154,21 @@ void updateChecklist() {
                 .statusCode(204);
     }
 
+    @DisplayName("체크리스트 수정 v1 성공")
+    @Test
+    void updateChecklistV1() {
+        long checklistId = checklistManageService.createChecklistV1(this.getAuthenticatedUser(),
+                ChecklistFixture.CHECKLIST_CREATE_REQUEST_V1());
+
+        RestAssured.given().log().all()
+                .contentType(ContentType.JSON)
+                .headers(this.headers)
+                .body(ChecklistFixture.CHECKLIST_UPDATE_REQUEST_V1())
+                .when().put("/v1/checklists/" + checklistId)
+                .then().log().all()
+                .statusCode(204);
+    }
+
     @DisplayName("체크리스트 수정 실패: 방 이름을 넣지 않은 경우")
     @Test
     void updateChecklist_noRoomName_exception() {

backend/bang-ggood/src/test/java/com/bang_ggood/station/repository/ChecklistRepositoryTest.java → backend/bang-ggood/src/test/java/com/bang_ggood/station/repository/ChecklistStationRepositoryTest.java

@@ -19,7 +19,7 @@
 
 import static org.assertj.core.api.Assertions.assertThat;
 
-public class ChecklistRepositoryTest extends IntegrationTestSupport {
+public class ChecklistStationRepositoryTest extends IntegrationTestSupport {
 
     @Autowired
     RoomRepository roomRepository;
@@ -30,11 +30,12 @@ public class ChecklistRepositoryTest extends IntegrationTestSupport {
     @Autowired
     UserRepository userRepository;
 
+    @Autowired
+    ChecklistStationRepository checklistStationRepository;
+
     Room room;
     User user;
     Checklist checklist;
-    @Autowired
-    ChecklistStationRepository checklistStationRepository;
 
     @BeforeEach
     void setUp() {
@@ -63,4 +64,19 @@ void findByChecklistId_noData_exception() {
         assertThat(checklistStationRepository.findByChecklist(checklist))
                 .isEmpty();
     }
+
+    @DisplayName("체크리스트 아이디를 통한 삭제 성공")
+    @Test
+    void deleteAllByChecklistId_noData_exception() {
+        // given
+        ChecklistStation checklistStation1 = new ChecklistStation(checklist, "잠실", "2호선", 5);
+        ChecklistStation checklistStation2 = new ChecklistStation(checklist, "잠실", "8호선", 6);
+        checklistStationRepository.saveAll(List.of(checklistStation1, checklistStation2));
+
+        // when
+        checklistStationRepository.deleteAllByChecklistId(checklist.getId());
+
+        // then
+        assertThat(checklistStationRepository.findByChecklist(checklist)).isEmpty();
+    }
 }