Merge pull request #148 from prgrms-aibe-devcourse/fix/145-pot-delete-plant-item

fix: 화분 삭제 시 수확 완료 PlantItem 보존되도록 수정

Author: noeyoseel

src/main/java/com/Rootin/domain/garden/service/PotService.java

@@ -228,7 +228,7 @@ public PotResponse updatePot(Long potId, Long userId, PotUpdateRequest request)
      * 2. 화분에 속한 TIL 기반으로 작성된 AI 분석 결과(AiResult) 조회 및 삭제 처리
      *    (AiResult 삭제 시 cascade = CascadeType.ALL, orphanRemoval = true 설정에 의해 ai_result_til 중간 테이블 레코드도 자동 제거됨)
      * 3. 화분에 속한 모든 TIL 포스트 삭제 처리
-     * 4. 화분에 심겨진 모든 식물(PlantItem) 데이터 삭제 처리
+     * 4. 화분에 심겨진 활성 식물(isHarvested=false) 삭제 처리 — 수확 완료 항목은 도감 기록 보존을 위해 제외
      * 5. 화분(Pot) 데이터 삭제
      */
     @Transactional
@@ -253,8 +253,8 @@ public void deletePot(Long potId, Long userId) {
             tilRepository.deleteAll(tils);
         }
 
-        // 3. 해당 화분에 연결된 모든 식물 아이템(PlantItem) 삭제 처리
-        List<PlantItem> plantItems = plantItemRepository.findByPotIdOrderByIdAsc(potId);
+        // 3. 해당 화분에 연결된 활성 식물 아이템(isHarvested=false/null)만 삭제 — 수확 완료 항목은 도감 보존
+        List<PlantItem> plantItems = plantItemRepository.findActivePlantItemsByPotId(potId);
         if (!plantItems.isEmpty()) {
             plantItemRepository.deleteAll(plantItems);
         }

src/test/java/com/Rootin/domain/garden/service/PotServiceTest.java

@@ -456,6 +456,46 @@ void deletePotSuccess() {
         assertThat(wateringLogRepository.findById(wateringLog.getId())).isPresent();
     }
 
+    @Test
+    @DisplayName("화분 삭제 시 수확 완료(isHarvested=true)된 PlantItem은 삭제되지 않고 도감 기록이 보존된다")
+    void deletePotPreservesHarvestedPlantItems() {
+        // given
+        User user = User.builder()
+                .email("harvestpreserve@rootin.com")
+                .nickname("수확보존자")
+                .build();
+        userRepository.save(user);
+
+        PotResponse createdPot = potService.createPot(
+                user.getId(),
+                PotCreateRequest.builder().title("도감보존 화분").build()
+        );
+        Long potId = createdPot.getId();
+
+        Plant defaultPlant = plantRepository
+                .findFirstByNameAndGradeAndGrowthStage("기본 씨앗", Grade.COMMON, GrowthStage.SEED)
+                .orElseThrow();
+
+        // 수확 완료된 PlantItem 직접 저장 (도감 기록)
+        PlantItem harvestedItem = PlantItem.builder()
+                .userId(user.getId())
+                .potId(potId)
+                .plantId(defaultPlant.getId())
+                .isHarvested(true)
+                .harvestedAt(java.time.LocalDateTime.now().minusDays(1))
+                .harvestedLevel(2)
+                .harvestedStageIndex(3)
+                .build();
+        PlantItem saved = plantItemRepository.save(harvestedItem);
+
+        // when
+        potService.deletePot(potId, user.getId());
+
+        // then — 화분은 삭제되고, 수확 완료 PlantItem(도감 기록)은 보존
+        assertThat(potRepository.findById(potId)).isEmpty();
+        assertThat(plantItemRepository.findById(saved.getId())).isPresent();
+    }
+
     @Test
     @DisplayName("타인의 화분을 삭제하려고 시도하면 FORBIDDEN 예외가 발생한다")
     void deletePotForbidden() {