fix: 🐛 목표 금액 DELETE 유즈 케이스 잘못된 필터링 제거 (#132)

* fix: target-amount 삭제 요청 시, amount == -1인 경우 필터링 로직 제거

* test: target amount의 값이 -1이어도 is_read가 성공적으로 true로 바뀌어야 한다

pennyway-app-external-api/src/main/java/kr/co/pennyway/api/apis/ledger/service/TargetAmountDeleteService.java

@@ -16,7 +16,6 @@ public class TargetAmountDeleteService {
     @Transactional
     public void execute(Long targetAmountId) {
         TargetAmount targetAmount = targetAmountService.readTargetAmount(targetAmountId)
-                .filter(TargetAmount::isAllocatedAmount)
                 .orElseThrow(() -> new TargetAmountErrorException(TargetAmountErrorCode.NOT_FOUND_TARGET_AMOUNT));
 
         if (!targetAmount.isThatMonth()) {

pennyway-app-external-api/src/test/java/kr/co/pennyway/api/apis/ledger/integration/TargetAmountIntegrationTest.java

@@ -27,6 +27,7 @@
 import java.time.LocalDateTime;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.springframework.security.test.web.servlet.request.SecurityMockMvcRequestPostProcessors.user;
 import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.patch;
 import static org.springframework.test.web.servlet.result.MockMvcResultHandlers.print;
@@ -270,18 +271,21 @@ void deleteTargetAmountNotThatMonth() throws Exception {
         }
 
         @Test
-        @DisplayName("당월 목표 금액 pk에 대한 접근 권한이 있지만, amount가 이미 -1인 경우 404 Not Found 에러 응답을 반환한다.")
-        @Transactional
+        @DisplayName("당월 목표 금액 pk에 대한 접근 권한이 있으며, amount == -1이어도 isRead가 true로 변경된다.")
         void deleteTargetAmountNotFound() throws Exception {
             // given
             User user = userService.createUser(UserFixture.GENERAL_USER.toUser());
             TargetAmount targetAmount = targetAmountService.createTargetAmount(TargetAmount.of(-1, user));
+            Long targetAmountId = targetAmount.getId();
 
             // when
             ResultActions result = performDeleteTargetAmount(targetAmount.getId(), user);
 
             // then
-            result.andDo(print()).andExpect(status().isNotFound());
+            result.andDo(print()).andExpect(status().isOk());
+            TargetAmount deletedTargetAmount = targetAmountService.readTargetAmount(targetAmountId).orElseThrow();
+            assertEquals(-1, deletedTargetAmount.getAmount());
+            assertTrue(deletedTargetAmount.isRead());
         }
 
         @Test