Skip to content

[MVC - Step2] 러너덕 미션 제출합니다.#92

Open
Deocksoo wants to merge 10 commits intowoowacourse:Deocksoofrom
Deocksoo:step2-refactor
Open

[MVC - Step2] 러너덕 미션 제출합니다.#92
Deocksoo wants to merge 10 commits intowoowacourse:Deocksoofrom
Deocksoo:step2-refactor

Conversation

@Deocksoo
Copy link

@Deocksoo Deocksoo commented Nov 5, 2019

킹뽀대님 안녕하세요, 이번 미션 리뷰이 러너덕입니다 :)
저번 주에 말씀드린 대로 리펙토링한 Controller들의 테스트 케이스를 추가해보았어요.
이전에 피드백 하여 주셨던 부분 중, 'setUp에서 feature로 만들어야 할 부분과 각각의 테스트 메서드 안에서 준비해야 할 부분을 구분하라' 는 부분을 기억하려고 노력했는데, 잘 되었는지 한번 확인 부탁드립니다!
그리고 각각의 메서드가 끝날 때마다 setup 에서 만들어주었던 feature를 모두 초기화하고 싶었는데, 현재 프로젝트에서 임시로 사용중인 in memory db에는 delete 기능이 없고, controller에서도 해당 api를 만들기가 애매한 상황이에요. 그래서 초기화하지 않고 일단 테스트를 진행해보았는데요, 아래와 같은 고민이 있습니다.

  1. 결국 CRUD는 필요한 기능일테니, 테스트를 위해서 DB에 해당 기능을 만드는 것이 좋을까요?
  2. 그리고 만든다면 각각의 Controller 테스트에서 DB를 직접 이용해 초기화하는 것보다는, acceptance 테스트의 역할에 맞게 api를 이용하는 것이 나을것 같다는 생각이 들어요. 그러면 deleteAll하는 api를 만들어야 할텐데, 이런 위험한 메서드를 만들어도 괜찮은건가요..? 아니면 acceptance 테스트지만 DB를 직접 이용해서 초기화하는 것도 괜찮다고 생각하시나요?

늦게까지 신경써 리뷰해주셔서 감사합니다! 피드백 부탁드려요 :)

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.

1 participant