Skip to content
This repository has been archived by the owner on Aug 13, 2022. It is now read-only.

#66 식당 insert 기능 - 2차 리뷰 반영 #71

Merged
merged 9 commits into from
Mar 5, 2022
Merged

Conversation

ypr821
Copy link
Collaborator

@ypr821 ypr821 commented Feb 9, 2022

[#66] 식당 insert 기능 - 2차 리뷰 반영

리뷰 포인트

  • 테스트를 수정하면서 Response DTO를 생성하였는데 맞는 방법인지 궁금합니다.

체크 포인트

  • annotation 설명 주석 달기
  • 코드 컨벤션 지키기
  • 과도한 축약용어 사용하지 않기
  • 객체지향적인 코드 작성

@ypr821 ypr821 requested review from yeoonjae and f-lab-jd February 9, 2022 11:34
@ypr821 ypr821 changed the title Feature/66 식당 insert 기능 - 2차 리뷰 반영 #66 식당 insert 기능 - 2차 리뷰 반영 Feb 11, 2022
Copy link
Collaborator

@f-lab-jd f-lab-jd left a comment

Choose a reason for hiding this comment

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

@ypr821
테스트코드가 테스트 코드가 아닌 소스코드에 영향을 주어서는 안됩니다.
추가된 클래스 없이 TC 를 짤 수 있는 방법은 없을까요?

@ypr821
Copy link
Collaborator Author

ypr821 commented Feb 18, 2022

@ypr821 테스트코드가 테스트 코드가 아닌 소스코드에 영향을 주어서는 안됩니다. 추가된 클래스 없이 TC 를 짤 수 있는 방법은 없을까요?

@f-lab-jd mock을 사용하거나 테스트 내부에 inner 클래스를 사용하면 안되나요??

f-lab-jd
f-lab-jd previously approved these changes Mar 4, 2022
Copy link
Collaborator

@yeoonjae yeoonjae left a comment

Choose a reason for hiding this comment

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

푸름님 고생 많으셨습니다!!!! 👍🏻👍🏻

@ypr821 ypr821 merged commit 5030c00 into develop Mar 5, 2022
@ypr821 ypr821 deleted the feature/66 branch March 5, 2022 13:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants