Skip to content

[BE-Review] 2주차 코드리뷰 요청#65

Open
ksundong wants to merge 55 commits intoBE-review-phase2from
dev-BE-phase2
Open

[BE-Review] 2주차 코드리뷰 요청#65
ksundong wants to merge 55 commits intoBE-review-phase2from
dev-BE-phase2

Conversation

@ksundong
Copy link
Member

설명

서비스 레이어는 어떻게 테스트하는게 좋을지 고민입니다.

리뷰해주셔서 감사합니다!

REST Docs도 다음주차에 다시 추가하는걸 고려해봐야겠네요!

ksundong and others added 30 commits June 16, 2020 13:03
굳이 api document가 버전관리 될 필요는 없어서
jar packaging 할 때에만 추가되도록 변경
inputs.dir이 원하는대로 동작하지 않아서 생기는 문제 해결
배달의 민족 REST Doc 참고를 해야할 것 같아요.
User에 추가된 정보가 들어갔는데, 이를 테스트 코드에 추가하지 않아 오류 발생
테스트 코드를 수정하여 해결하였음.
Fix: dev 브랜치 머지 후 코드가 정상 동작하지 않는 오류 수정
[Hotfix] [Deploy] 테스트를 통과하지 못하는 현상 수정
Alex가 사용해 보신 적이 있을까요?
없다면 한 번 사용해 볼만한 것 같아서 추가하였습니다.
hibernate validation에서 검증하지 못하는 경우가 있어서 스프링 부트 버전을 낮췄습니다.
그 과정에서 gradle developmentOnly 설정을 추가해주었습니다.
ksundong added 25 commits June 19, 2020 16:18
BE-#49 이슈 목록 조회 API 개발 및 선택한 이슈 open close API 개발
default 데이터를 넣는 data.sql 추가
initialization-mode를 embedded로 설정
기본 데이터 추가로 인하여 테스트 코드 수정
@ksundong ksundong added the be-review be Review요청 label Jun 22, 2020
@ksundong ksundong requested a review from honux77 June 22, 2020 04:48
@ksundong ksundong self-assigned this Jun 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

be-review be Review요청

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants