Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weโ€™ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[#22][TEST] application layer test #27

Merged
merged 7 commits into from
May 17, 2024
Merged

[#22][TEST] application layer test #27

merged 7 commits into from
May 17, 2024

Conversation

Caramel1004
Copy link
Owner

๐Ÿชต ์ž‘์—… ๋ธŒ๋žœ์น˜

test / #22

โœ… ํ…Œ์ŠคํŠธ ๋ชฉ๋ก

  • ์†ก๊ธˆ ๋ฐ ์†ก๊ธˆ ๊ฒฌ์ ์„œ ๊ด€๋ จ(Remittance Service Test)

    • ์†ก๊ธˆ ๊ฒฌ์ ์„œ ๋ฐœํ–‰
    • ์†ก๊ธˆ ์š”์ฒญ
    • ํŠน์ • ์œ ์ €์˜ ๋ชจ๋“  ์†ก๊ธˆ ๊ฑฐ๋ž˜ ์ด๋ ฅ ์กฐํšŒ
  • ํšŒ์› ๊ด€๋ จ(Member Service Test)

    • ํšŒ์› ๊ฐ€์ž…

โ™ป๏ธ ์•ˆ์“ฐ๋Š” import๋ฌธ ์ฝ”๋“œ ์‚ญ์ œ
โ™ป๏ธ  [refactor] ์ฐธ์กฐ ํƒ€์ž… ๋ณ€๊ฒฝ RemittanceQuoteV2DTO -> RemittanceEntityV2
โ™ป๏ธ  [refactor] ๋ถˆํ•„์š”ํ•œ import๋ฌธ ์‚ญ์ œ, ์•ˆ์“ฐ๋Š” ์ „์—ญ๋ณ€์ˆ˜ ์‚ญ์ œ, ์ฝ”๋“œ ๋ธ”๋Ÿญ ์ •๋ฆฌ
โ™ป๏ธ  [refactor] Test data ๋ชจ๋‹ˆํ„ฐ๋ง์„ ์œ„ํ•œ toString๋ฉ”์†Œ๋“œ ์˜ค๋ฒ„๋ผ์ด๋”ฉ
โ™ป๏ธ  [refactor] Test์šฉ ์ฟผ๋ฆฌ๋ฉ”์†Œ๋“œ ์ถ”๊ฐ€
โœ…  [test] ํšŒ์›๊ด€๋ จ application๋‹จ ์„ฑ๊ณต ํ…Œ์ŠคํŠธ ์ฝ”๋“œ ์ถ”๊ฐ€
โœ… [test] ์†ก๊ธˆ ๋ฐ ์†ก๊ธˆ ๊ฒฌ์ ์„œ ๊ด€๋ จ application๋‹จ ์„ฑ๊ณต ํ…Œ์ŠคํŠธ ์ฝ”๋“œ ์ถ”๊ฐ€
@Caramel1004 Caramel1004 merged commit 0cfc2c8 into main May 17, 2024
@Caramel1004 Caramel1004 self-assigned this May 17, 2024
@Caramel1004 Caramel1004 added the โœ… test โœ… ํ…Œ์ŠคํŠธ ์ฝ”๋“œ ์ถ”๊ฐ€ label May 17, 2024
@Caramel1004 Caramel1004 linked an issue May 17, 2024 that may be closed by this pull request
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
โœ… test โœ… ํ…Œ์ŠคํŠธ ์ฝ”๋“œ ์ถ”๊ฐ€
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[#22][TEST]application layer test
1 participant