Integration Test 추가 #64

Open
opened 2018-02-23 23:55:21 +09:00 by Hide_D · 2 comments
Owner

티엠서버에서의 선동 버그와 같이 마이너 서버에서 천통을 2회 할 정도로 이상이 없어보였던 코드도, 실제로는 심각한 버그가 발생할 수 있음을 알 수 있었습니다.

더욱이 해당 패치는 현 HideD서버에도 적용 대상이었으므로, 지금과 같은 대규모 코드 패치가 지속되었을 때 정상 작동 여부를 보장할 수 있는 테스트 절차가 필수적임을 새삼 확인하게 되었습니다.

현 시점에서 기존 코드에 대해서는 Unittest를 수행하기도 어려운 만큼, integration test 절차를 추가해야할 것 같습니다.

티엠서버에서의 선동 버그와 같이 마이너 서버에서 천통을 2회 할 정도로 이상이 없어보였던 코드도, 실제로는 심각한 버그가 발생할 수 있음을 알 수 있었습니다. 더욱이 해당 패치는 현 HideD서버에도 적용 대상이었으므로, 지금과 같은 대규모 코드 패치가 지속되었을 때 정상 작동 여부를 보장할 수 있는 테스트 절차가 필수적임을 새삼 확인하게 되었습니다. 현 시점에서 기존 코드에 대해서는 Unittest를 수행하기도 어려운 만큼, integration test 절차를 추가해야할 것 같습니다.
Author
Owner

2차로 변경

2차로 변경
Author
Owner

우선 unit test 추가 완료.

현재 DB 클래스 구현만으로 mock DB 추가가 가능한지 확인 필요.

우선 unit test 추가 완료. 현재 DB 클래스 구현만으로 mock DB 추가가 가능한지 확인 필요.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
-
Dependencies

No dependencies set.

Reference: devsam/core#64