Skip to content

Commit 3244a05

Browse files
committed
chore: executeTransfer 메서드 private 설정
1 parent 6056c7d commit 3244a05

File tree

1 file changed

+10
-4
lines changed

1 file changed

+10
-4
lines changed

src/test/java/banking/transfer/service/TransferServiceUnitTest.java

+10-4
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
import banking.fixture.testEntity.AccountFixture;
1010
import banking.fixture.testEntity.UserFixture;
1111
import banking.transfer.dto.request.TransferRequest;
12+
import banking.transfer.dto.response.TransferDetailResponse;
1213
import banking.transfer.entity.Transfer;
1314
import banking.transfer.enums.TransferType;
1415
import banking.transfer.repository.TransferRepository;
@@ -51,13 +52,14 @@ class TransferServiceUnitTest {
5152
ExchangeRateService exchangeRateService;
5253

5354
@Test
54-
@DisplayName("[이체 성공 테스트] executeTransfer 메서드 검증")
55+
@DisplayName("[이체 성공 테스트] executeTransfer 메서드 검증 (transfer 메서드에서 호출)")
5556
void execute_transfer_succeed_test () {
5657
// given
5758
User user = UserFixture.USER_FIXTURE_1.createUser();
5859
Account withdrawalAccount = AccountFixture.ACCOUNT_FIXTURE_KRW_1.createAccount(user);
5960
Account depositAccount = AccountFixture.ACCOUNT_FIXTURE_KRW_2.createAccount(user);
6061

62+
AccountPublicInfoResponse mockWithdrawalAccountPublicInfo = AccountPublicInfoResponse.of(withdrawalAccount);
6163
AccountPublicInfoResponse mockDepositAccountPublicInfo = AccountPublicInfoResponse.of(depositAccount);
6264

6365
TransferRequest transferRequest = TransferRequest.builder()
@@ -68,21 +70,25 @@ void execute_transfer_succeed_test () {
6870
.build();
6971

7072
when(accountService.findAccountWithLock(anyLong(), anyLong(), anyString())).thenReturn(withdrawalAccount);
73+
doNothing().when(accountService).verifyAccountActiveStatus(withdrawalAccount);
74+
doNothing().when(accountService).verifyAccountActiveStatus(transferRequest.depositAccountNumber());
7175
when(accountService.findAccountPublicInfo(anyString())).thenReturn(mockDepositAccountPublicInfo);
7276
when(exchangeRateService.getExchangeRate(any(), any())).thenReturn(BigDecimal.ONE);
7377
when(accountService.findAccountWithLock(anyLong(), any(Transfer.class))).thenReturn(depositAccount);
74-
75-
TransferService.TransferResultHandler<Transfer> resultHandler = (transfer, wAc, dAc) -> transfer;
78+
when(accountService.findAccountPublicInfo(anyLong(), any(Transfer.class))).thenReturn(mockWithdrawalAccountPublicInfo);
7679

7780
// when
78-
Transfer result = transferService.executeTransfer(user.getId(), transferRequest, resultHandler);
81+
TransferDetailResponse result = transferService.transfer(user.getId(), transferRequest);
7982

8083
// then
8184
assertNotNull(result);
8285
verify(accountService, times(1)).findAccountWithLock(anyLong(), anyLong(), anyString());
86+
verify(accountService, times(1)).verifyAccountActiveStatus(withdrawalAccount);
87+
verify(accountService, times(1)).verifyAccountActiveStatus(transferRequest.depositAccountNumber());
8388
verify(accountService, times(1)).findAccountPublicInfo(depositAccount.getAccountNumber());
8489
verify(exchangeRateService, times(1)).getExchangeRate(depositAccount.getCurrency(), withdrawalAccount.getCurrency());
8590
verify(depositAsyncService, times(1)).processDepositAsync(any(Transfer.class));
91+
verify(accountService, times(1)).findAccountPublicInfo(anyLong(), any(Transfer.class));
8692
}
8793

8894
@Test

0 commit comments

Comments
 (0)