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

[#78, #83] Mapper & InAdapter Test #88

Merged
merged 10 commits into from
May 20, 2024
Merged

[#78, #83] Mapper & InAdapter Test #88

merged 10 commits into from
May 20, 2024

Conversation

jalju0804
Copy link
Contributor

@jalju0804 jalju0804 commented May 19, 2024

close #78
close #83

Test에 대해서 Port에 값이 잘 들어가는 지는 ArgumentCaptor를 통해서 확인했습니다.
InAdapter에 대해서 vaild가 잘 작동하는 지는 validator로 확인했습니다.
Mapper에 대해서 기존의 Stream으로 사용하던 것들을 하나의 interface로 통합하고 default를 통해 로직을 추가했습니다.

@donggni0712 donggni0712 merged commit 64e3979 into main May 20, 2024
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Refactor, Test] Mapper 구현 및 Mapper 테스트 코드 작성 [Test] InAdapter Test
2 participants