Coffee-Street / strada

api server
MIT License
2 stars 2 forks source link

orderingdetail entity to id #80

Closed whdqhdqkr closed 2 years ago

whdqhdqkr commented 2 years ago

What this PR does / why we need it:

Which issue/s this PR fixes

How Has This Been Tested?

Checklist:

ken1336 commented 2 years ago

테스트 격리가 안된것 같습니다. Ordering 도메인 테스트 후에 insert된 데이터를 지우는 부분이 없네요.

스크린샷 2021-12-19 오후 8 28 01

bread 도메인 테스트에서는 케이스별로 테스트를 진행 후 @AfterEach를 사용하여 dummy 데이터를 삭제하여 케이스를 격리 시킵니다. 그런데 Ordering 도메인을 테스트하면서 insert된 bread는 OrderingDetail와 OneToOne 연관관계를 가지기 때문에 delete하는 과정에서 ConstraintViolationException이 발생하는것 같습니다.

로컬에서 테스트할때 문제가 없었던 이유는, Ordering 도메인을 테스트 하기 전 bread 도메인을 먼저 테스트해서 그런것 같습니다. 테스트 순서가 로컬과 리모트에서 다르게 동작하는것 같습니다.

whdqhdqkr commented 2 years ago

@ken1336 전에 고생하시던 그 이슈를 저도 만난거였네요 ㅠㅠ 이런 경우 따로 Integration Test를 독립된 환경에서 실행시켜야 할거 같은데... 그 때 혹시 어떤 방법을 쓰신건지 알 수 있을까요?