search

f-lab-edu / nestjs-user-api

연습용
MIT License
1 stars 0 forks source link

[구현] ADD: test 도입 #12

Closed yujin-min closed 3 months ago

yujin-min commented 3 months ago

8 test 도입 구현

badge 추가

유저 서비스 유닛 테스트 추가

sonarcloud[bot] commented 3 months ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

yujin-min commented 3 months ago

모킹이 나을지 데이터베이스를 하나 사용하는게 나을지 모르겠어서 모킹으로 진행했습니다. 유저 서비스를 테스트하는데 계좌가 문제가 되는 상황이라 현재 구조가 유닛 테스트에 적합하지 않은 구조라는 생각이 들었습니다. 전반적으로 네스트에서 테스트를 이렇게 작성하는게 맞는지 하는 의문이 있었습니다 😢

f-lab-namu commented 3 months ago

모킹이 나을지 데이터베이스를 하나 사용하는게 나을지 모르겠어서 모킹으로 진행했습니다.

좋다고 생각합니다.

유저 서비스를 테스트하는데 계좌가 문제가 되는 상황이라 현재 구조가 유닛 테스트에 적합하지 않은 구조라는 생각이 들었습니다.

테스트를 해보면 작성한 코드의 디자인이 좋은지/나쁜지 판단할 수 있습니다. 테스트가 어렵다면 좀 더 테스트가 수월하도록 리펙토링을 통해 코드 디자인을 개선해야 합니다. 또한 테스트 하기 용이하다는 말은 SOLID 원칙 등 여러가지가 잘 지켜졌을때 가능해 집니다.

전반적으로 네스트에서 테스트를 이렇게 작성하는게 맞는지 하는 의문이 있었습니다

네, 이번 PR 에서 보았을때 괜찮았습니다. Test 가 nest.js native, pure test, controller test 등 여러 상황에서 다른 형태를 가지게 될텐데요, 현재 테스트 작성한 코드는 적절하다고 보입니다.