From a541b4978c26b6e9fe9ca10f226cbe19e195c0d3 Mon Sep 17 00:00:00 2001 From: chungjeongsu Date: Tue, 26 May 2026 18:33:47 +0900 Subject: [PATCH 1/2] =?UTF-8?q?[#233]=20fix:=20Transfer=20=EC=97=94?= =?UTF-8?q?=ED=8B=B0=ED=8B=B0=20totalAmount=20=EB=B6=88=EB=B3=80=EC=8B=9D?= =?UTF-8?q?=20=EC=9D=8C=EC=88=98=EB=A7=8C=20=EC=95=88=EB=90=98=EA=B2=8C=20?= =?UTF-8?q?=EC=88=98=EC=A0=95?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../java/org/triple/backend/transfer/domain/Transfer.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/main/java/org/triple/backend/transfer/domain/Transfer.java b/src/main/java/org/triple/backend/transfer/domain/Transfer.java index 7c40616f..d884b9c3 100644 --- a/src/main/java/org/triple/backend/transfer/domain/Transfer.java +++ b/src/main/java/org/triple/backend/transfer/domain/Transfer.java @@ -128,8 +128,8 @@ private String validateText(final String value, final String fieldName) { } private BigDecimal validateTotalAmount(final BigDecimal value) { - if (value.compareTo(BigDecimal.ZERO) <= 0) { - throw new IllegalArgumentException("totalAmount must be greater than zero"); + if (value.compareTo(BigDecimal.ZERO) < 0) { + throw new IllegalArgumentException("totalAmount must be greater than minus"); } return value; } From d415cdc3f0d30b4bd87aab1fe5c95beb6d860b57 Mon Sep 17 00:00:00 2001 From: chungjeongsu Date: Tue, 26 May 2026 18:42:00 +0900 Subject: [PATCH 2/2] =?UTF-8?q?[#233]=20fix:=20Transfer=20=EC=97=94?= =?UTF-8?q?=ED=8B=B0=ED=8B=B0=20totalAmount=20=EB=B6=88=EB=B3=80=EC=8B=9D?= =?UTF-8?q?=20=EC=9D=8C=EC=88=98=EB=A7=8C=20=EC=95=88=EB=90=98=EA=B2=8C=20?= =?UTF-8?q?=EC=88=98=EC=A0=95=EC=97=90=20=EB=94=B0=EB=A5=B8=20=ED=85=8C?= =?UTF-8?q?=EC=8A=A4=ED=8A=B8=20=EC=88=98=EC=A0=95?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../org/triple/backend/transfer/domain/TransferTest.java | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/test/java/org/triple/backend/transfer/domain/TransferTest.java b/src/test/java/org/triple/backend/transfer/domain/TransferTest.java index 4a950b5e..d2ee7a5a 100644 --- a/src/test/java/org/triple/backend/transfer/domain/TransferTest.java +++ b/src/test/java/org/triple/backend/transfer/domain/TransferTest.java @@ -69,12 +69,13 @@ void changeAccountHolder_blank_throws() { } @Test - @DisplayName("changeTotalAmount는 0 이하 금액이면 예외를 던진다") - void changeTotalAmount_zeroOrNegative_throws() { + @DisplayName("changeTotalAmount는 0을 허용하고 음수 금액이면 예외를 던진다") + void changeTotalAmount_allowsZeroAndRejectsNegative() { Transfer transfer = transfer(); - assertThatThrownBy(() -> transfer.changeTotalAmount(BigDecimal.ZERO)) - .isInstanceOf(IllegalArgumentException.class); + transfer.changeTotalAmount(BigDecimal.ZERO); + + assertThat(transfer.getTotalAmount()).isEqualByComparingTo(BigDecimal.ZERO); assertThatThrownBy(() -> transfer.changeTotalAmount(new BigDecimal("-1"))) .isInstanceOf(IllegalArgumentException.class); }