feature : account api 복수 형태로 수정

this-is-spear · Jan 17, 2024 · 67d8ce6 · 67d8ce6
1 parent 0af8df2
commit 67d8ce6
Show file tree

Hide file tree

Showing 10 changed files with 119 additions and 33 deletions.
diff --git a/src/docs/asciidoc/index.adoc b/src/docs/asciidoc/index.adoc
@@ -12,6 +12,36 @@ endif::[]
 
 == Banking API
 
+=== 계좌 개설
+
+==== HTTP request
+
+include::{snippets}/create-account/http-request.adoc[]
+
+==== HTTP response
+
+include::{snippets}/create-account/http-response.adoc[]
+
+=== 계좌 조회
+
+==== HTTP request
+
+include::{snippets}/accounts/http-request.adoc[]
+
+==== HTTP response
+
+include::{snippets}/accounts/http-response.adoc[]
+
+=== 이체 대상 확인
+
+==== HTTP request
+
+include::{snippets}/targets/http-request.adoc[]
+
+==== HTTP response
+
+include::{snippets}/targets/http-response.adoc[]
+
 === 기록 조회
 
 ==== HTTP request
@@ -52,16 +82,6 @@ include::{snippets}/transfer/http-request.adoc[]
 
 include::{snippets}/transfer/http-response.adoc[]
 
-=== 이체 대상 확인
-
-==== HTTP request
-
-include::{snippets}/targets/http-request.adoc[]
-
-==== HTTP response
-
-include::{snippets}/targets/http-response.adoc[]
-
 == Member API
 
 === 회원가입

diff --git a/src/main/java/bankingapi/banking/application/AccountApplicationService.java b/src/main/java/bankingapi/banking/application/AccountApplicationService.java
@@ -130,4 +130,9 @@ public List<AccountNumber> findAccounts(String principal) {
 				.map(Account::getAccountNumber)
 				.toList();
 	}
+
+	public AccountNumber createAccount(String username) {
+		final var member = memberService.findByEmail(username);
+		return accountService.createAccount(member.getId()).getAccountNumber();
+	}
 }
diff --git a/src/main/java/bankingapi/banking/domain/AccountService.java b/src/main/java/bankingapi/banking/domain/AccountService.java
@@ -83,6 +83,27 @@ public List<AccountHistory> findAccountHistoriesByFromAccountNumber(Account acco
 		return accountHistoryRepository.findByFromAccountNumber(account.getAccountNumber());
 	}
 
+
+	public List<Account> getAccountByMemberId(Long memberId) {
+		return accountRepository.findByUserId(memberId);
+	}
+
+	public Account createAccount(Long id) {
+		AccountNumber accountNumber;
+
+		do {
+			accountNumber = AccountNumberGenerator.generate();
+		} while (accountRepository.findByAccountNumber(accountNumber).isPresent());
+
+		return accountRepository.save(
+				Account.builder()
+						.accountNumber(accountNumber)
+						.balance(Money.zero())
+						.userId(id)
+						.build()
+		);
+	}
+
 	private Account getAccountByAccountNumberWithOptimisticLock(AccountNumber accountNumber) {
 		return accountRepository.findByAccountNumberWithOptimisticLock(accountNumber).orElseThrow();
 	}
@@ -99,8 +120,4 @@ private void recordCompletionTransferMoney(Account fromAccount, Account toAccoun
 		accountHistoryRepository.save(AccountHistory.recordWithdrawHistory(fromAccount, toAccount, money));
 		accountHistoryRepository.save(AccountHistory.recordDepositHistory(toAccount, fromAccount, money));
 	}
-
-	public List<Account> getAccountByMemberId(Long memberId) {
-		return accountRepository.findByUserId(memberId);
-	}
 }
diff --git a/src/main/java/bankingapi/banking/ui/AccountController.java b/src/main/java/bankingapi/banking/ui/AccountController.java
@@ -22,7 +22,7 @@
 import java.util.List;
 
 @RestController
-@RequestMapping("account")
+@RequestMapping("/accounts")
 @RequiredArgsConstructor
 public class AccountController {
 
@@ -77,10 +77,16 @@ public ResponseEntity<TargetResponses> getTargets(@AuthenticationPrincipal UserD
 	}
 
 	@GetMapping(
-		value = "/targets",
 		produces = MediaType.APPLICATION_JSON_VALUE
 	)
 	public ResponseEntity<List<AccountNumber>> findAccounts(@AuthenticationPrincipal UserDetails principal) {
 		return ResponseEntity.ok(accountApplicationService.findAccounts(principal.getUsername()));
 	}
+
+	@PostMapping(
+			produces = MediaType.APPLICATION_JSON_VALUE
+	)
+	public ResponseEntity<AccountNumber> createAccount(@AuthenticationPrincipal UserDetails principal) {
+		return ResponseEntity.ok(accountApplicationService.createAccount(principal.getUsername()));
+	}
 }
diff --git a/src/main/java/bankingapi/util/config/SecurityConfiguration.java b/src/main/java/bankingapi/util/config/SecurityConfiguration.java
@@ -5,6 +5,7 @@
 import org.springframework.context.annotation.Bean;
 import org.springframework.context.annotation.Configuration;
 import org.springframework.context.annotation.Primary;
+import org.springframework.http.HttpMethod;
 import org.springframework.security.config.annotation.web.builders.HttpSecurity;
 import org.springframework.security.config.annotation.web.configuration.EnableWebSecurity;
 import org.springframework.security.config.annotation.web.configurers.AbstractHttpConfigurer;
@@ -34,7 +35,7 @@ public SecurityFilterChain securityFilterChain(HttpSecurity http) throws Excepti
 				.requestMatchers("/docs/index.html").permitAll()
 				.requestMatchers("/members/register").anonymous()
 				.requestMatchers("/login").anonymous()
-				.requestMatchers("/account/**").authenticated()
+				.requestMatchers("/accounts/**").authenticated()
 				.requestMatchers("/members/**").authenticated()
         );
 

diff --git a/src/main/java/bankingapi/util/config/WebMvcConfiguration.java b/src/main/java/bankingapi/util/config/WebMvcConfiguration.java
@@ -19,7 +19,7 @@ public class WebMvcConfiguration implements WebMvcConfigurer {
 	@Override
 	public void addInterceptors(InterceptorRegistry registry) {
 		PathMatcherInterceptor matcherInterceptor = new PathMatcherInterceptor(idempotentRequestInterceptor,
-			customPathContainer).includePathPattern("/account/**", HttpMethod.POST);
+			customPathContainer).includePathPattern("/accounts/**", HttpMethod.POST);
 		registry.addInterceptor(matcherInterceptor)
 			.addPathPatterns();
 	}

diff --git a/src/test/java/bankingapi/acceptance/BankingAcceptanceTest.java b/src/test/java/bankingapi/acceptance/BankingAcceptanceTest.java
@@ -326,7 +326,7 @@ void withdraw_concurrency_10times() throws Exception {
 		transferParams.put("amount", transferMoney);
 
 		return mockMvc.perform(
-			post("/account/{accountNumber}/transfer", fromAccountNumber)
+			post("/accounts/{accountNumber}/transfer", fromAccountNumber)
 				.with(user(username).password(password).roles("MEMBER"))
 				.contentType(MediaType.APPLICATION_JSON)
 				.header(IDEMPOTENT_KEY, UUID.randomUUID().toString())
@@ -340,7 +340,7 @@ void withdraw_concurrency_10times() throws Exception {
 		depositParams.put("amount", depositMoney);
 
 		return mockMvc.perform(
-			post("/account/{accountNumber}/deposit", accountNumber)
+			post("/accounts/{accountNumber}/deposit", accountNumber)
 				.with(user(username).password(password).roles("MEMBER"))
 				.contentType(MediaType.APPLICATION_JSON)
 				.header(IDEMPOTENT_KEY, UUID.randomUUID().toString())
@@ -354,7 +354,7 @@ void withdraw_concurrency_10times() throws Exception {
 		depositParams.put("amount", depositMoney);
 
 		return mockMvc.perform(
-			post("/account/{accountNumber}/withdraw", accountNumber)
+			post("/accounts/{accountNumber}/withdraw", accountNumber)
 				.with(user(username).password(password).roles("MEMBER"))
 				.contentType(MediaType.APPLICATION_JSON)
 				.header(IDEMPOTENT_KEY, UUID.randomUUID().toString())
@@ -364,7 +364,7 @@ void withdraw_concurrency_10times() throws Exception {
 
 	private ResultActions 계좌_조회_요청(String accountNumber, String username, String password) throws Exception {
 		return mockMvc.perform(
-			get("/account/{accountNumber}/history", accountNumber)
+			get("/accounts/{accountNumber}/history", accountNumber)
 				.with(user(username).password(password).roles("MEMBER"))
 				.accept(MediaType.APPLICATION_JSON)
 		);

diff --git a/src/test/java/bankingapi/acceptance/IdempotencyAcceptanceTest.java b/src/test/java/bankingapi/acceptance/IdempotencyAcceptanceTest.java
@@ -24,7 +24,7 @@ void deposit_and_withdraw_5000() throws Exception {
 		depositParams.put("amount", 입금할_돈);
 
 		var 계좌_입금 = mockMvc.perform(
-			post("/account/{accountNumber}/deposit", 나의계좌)
+			post("/accounts/{accountNumber}/deposit", 나의계좌)
 				.with(user(이메일).password(비밀번호).roles("MEMBER"))
 				.contentType(MediaType.APPLICATION_JSON)
 				.content(objectMapper.writeValueAsString(depositParams))

diff --git a/src/test/java/bankingapi/documentation/AccountDocumentation.java b/src/test/java/bankingapi/documentation/AccountDocumentation.java
@@ -12,6 +12,7 @@
 
 import java.util.UUID;
 
+import bankingapi.banking.domain.AccountNumber;
 import org.junit.jupiter.api.Test;
 import org.springframework.http.MediaType;
 
@@ -27,9 +28,8 @@ void getHistory() throws Exception {
 		when(accountApplicationService.getHistory(USER.getUsername(), 계좌_번호))
 			.thenReturn(계좌_내역);
 		mockMvc.perform(
-				get("/account/{accountNumber}/history", 계좌_번호)
+				get("/accounts/{accountNumber}/history", 계좌_번호)
 					.with(csrf())
-					.with(user(USER.getUsername()).roles("MEMBER"))
 			).andExpect(status().isOk())
 			.andDo(document(
 				"history",
@@ -44,12 +44,11 @@ void getHistory() throws Exception {
 	void deposit() throws Exception {
 		doNothing().when(accountApplicationService).deposit(USER.getUsername(), 계좌_번호, 이만원);
 		mockMvc.perform(
-				post("/account/{accountNumber}/deposit", 계좌_번호)
+				post("/accounts/{accountNumber}/deposit", 계좌_번호)
 					.contentType(MediaType.APPLICATION_JSON)
 					.content(objectMapper.writeValueAsString(이만원))
 					.header(IDEMPOTENT_KEY, UUID.randomUUID().toString())
 					.with(csrf())
-					.with(user(USER.getUsername()).roles("MEMBER"))
 			).andExpect(status().isOk())
 			.andDo(document(
 				"deposit",
@@ -64,12 +63,11 @@ void deposit() throws Exception {
 	void withdraw() throws Exception {
 		doNothing().when(accountApplicationService).withdraw(USER.getUsername(), 계좌_번호, 이만원);
 		mockMvc.perform(
-				post("/account/{accountNumber}/withdraw", 계좌_번호)
+				post("/accounts/{accountNumber}/withdraw", 계좌_번호)
 					.contentType(MediaType.APPLICATION_JSON)
 					.content(objectMapper.writeValueAsString(이만원))
 					.header(IDEMPOTENT_KEY, UUID.randomUUID().toString())
 					.with(csrf())
-					.with(user(USER.getUsername()).roles("MEMBER"))
 			).andExpect(status().isOk())
 			.andDo(document(
 				"withdraw",
@@ -86,12 +84,11 @@ void transfer() throws Exception {
 		doNothing().when(accountApplicationService).transfer(USER.getUsername(), 계좌_번호, command);
 
 		mockMvc.perform(
-				post("/account/{accountNumber}/transfer", 계좌_번호)
+				post("/accounts/{accountNumber}/transfer", 계좌_번호)
 					.contentType(MediaType.APPLICATION_JSON)
 					.content(objectMapper.writeValueAsString(command))
 					.header(IDEMPOTENT_KEY, UUID.randomUUID().toString())
 					.with(csrf())
-					.with(user(USER.getUsername()).roles("MEMBER"))
 			).andExpect(status().isOk())
 			.andDo(document(
 				"transfer",
@@ -106,8 +103,8 @@ void transfer() throws Exception {
 	void getTargets() throws Exception {
 		when(accountApplicationService.getTargets(USER.getUsername())).thenReturn(타겟목록);
 		mockMvc.perform(
-				get("/account/transfer/targets")
-					.with(user(USER.getUsername()).roles("MEMBER"))
+				get("/accounts/transfer/targets")
+					.with(csrf())
 			).andExpect(status().isOk())
 			.andDo(print())
 			.andDo(document(
@@ -116,4 +113,36 @@ void getTargets() throws Exception {
 				getDocumentResponse()
 			));
 	}
+
+	@Test
+	@WithMockMember
+	void findAccounts() throws Exception {
+		when(accountApplicationService.findAccounts(USER.getUsername())).thenReturn(계좌목록);
+		mockMvc.perform(
+						get("/accounts")
+								.with(csrf())
+				).andExpect(status().isOk())
+				.andDo(print())
+				.andDo(document(
+						"accounts",
+						getDocumentRequest(),
+						getDocumentResponse()
+				));
+	}
+
+	@Test
+	@WithMockMember
+	void createAccount() throws Exception {
+		when(accountApplicationService.createAccount(USER.getUsername())).thenReturn(new AccountNumber(계좌_번호));
+		mockMvc.perform(
+						post("/accounts")
+								.with(csrf())
+				).andExpect(status().isOk())
+				.andDo(print())
+				.andDo(document(
+						"create-account",
+						getDocumentRequest(),
+						getDocumentResponse()
+				));
+	}
 }
diff --git a/src/test/java/bankingapi/fixture/DocumentationFixture.java b/src/test/java/bankingapi/fixture/DocumentationFixture.java
@@ -3,8 +3,10 @@
 import java.time.LocalDateTime;
 import java.util.List;
 
+import bankingapi.banking.domain.Account;
 import bankingapi.banking.domain.AccountNumber;
 import bankingapi.banking.domain.HistoryType;
+import bankingapi.banking.domain.Money;
 import bankingapi.banking.dto.HistoryResponse;
 import bankingapi.banking.dto.HistoryResponses;
 import bankingapi.banking.dto.TargetResponse;
@@ -34,4 +36,10 @@ public class DocumentationFixture {
 		new TargetResponse("name2", "member2@email.com", AccountNumberGenerator.generate()),
 		new TargetResponse("name3", "member3@email.com", AccountNumberGenerator.generate())
 	));
+
+	public static final List<AccountNumber> 계좌목록 =List.of(
+			AccountNumberGenerator.generate(),
+			AccountNumberGenerator.generate(),
+			AccountNumberGenerator.generate()
+	);
 }