Skip to content

Conversation

@hong-sile
Copy link
Contributor

@hong-sile hong-sile commented Nov 20, 2023

🚀 작업 내용
LMS의 ✅ 학습 테스트 - jdbcTemplaAPI 페이지를 기준으로 실패하는 테스트 환경을 구성했습니다.
추가적으로 테스트 하나가 빠져있더라고요.
QueryigDao의
findCustomersByFirstName에 해당하는 테스트 추가했습니다.

💬 리뷰 중점사항

main 브랜치 -> 정답이 없고, 테스트코드만 있는 브랜치
각 미션별 브랜치(ex: spring-jdbc-1) 를 별도로 파고 해당 브랜치에 정답이 있는 구조는 어떤가요?

디스코드에서 제시한 의견처럼 환경을 구성하시는 건 어떤가요?

close #3

Copy link
Contributor

@boorownie boorownie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

비어있는 코드 잘 반영해주셨습니다 👍
코멘트 남겼으니 확인 부탁해요!

Comment on lines 30 to 31
List<Object[]> splitUpNames = Arrays.asList("John Woo", "Jeff Dean", "Josh Bloch", "Josh Long")
.stream()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이 부분 줄바꿈의 이유가 있으신가요?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아 별다른 이유는 없고, 무의식적으로 한 줄에 점 하나 쓰는 습관이 있어서 바꾼 것 같습니다.

@hong-sile hong-sile changed the title Feature/#3 jdbc template 학습테스트 실패환경 구성 Refactor/#3 jdbc template 학습테스트 실패환경 구성 Dec 21, 2023
@boorownie boorownie merged commit 5b8931b into cho-log:main Dec 23, 2023
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.

jdbcTemplate 학습 테스트 실패환경 구성

2 participants