SoftDelete 시 Update 체크 방식 다들 어떻게 생각하시나요?!

[taewookimm]

SoftDelete 할 때 Update 체크 방식에 대해 의문점이 들어 Discussion을 열었습니다!

체크 방식

1. 프로덕션에서도 성공적으로 Update 되었는지 체크 할지? (실제 조회)
2. 테스트 코드 작성 후 케이스들만 잘 보존 할지?
3. assert로 처리를 할지?

Description

위 3가지 방식들을 고민해 봤는데 프로덕션에서 SoftDelete를 할 때마다 Board 조회가 굳이 일어나야 할까? 한 번의 조회라도 조회는 곧 서버 부하로 이어질 수 있는데 그만큼 민감한 부분인가? 의문이 들었습니다.

그래서 서비스단 코드에서 매번 실제 조회하기보단 assert로 처리하거나 테스트 코드에 작성을 하는 것이 좋지 않을까 생각했습니다!

하지만 사람마다 보는 관점은 다르고, 팀원들의 의견을 듣고 싶습니다!

위 방식과 다른 방식이 존재할 수도 있을 것 같습니다!

여러분들의 생각은 어떠신가요?!

Additional Notes

Assert란?

• 정의: assert는 코드 실행 중 지정한 조건이 무조건 참이어야 한다는 가정을 검증하는 구문입니다.
• 목적: 개발, 테스트 단계에서 주로 사용되며, 프로덕션 환경에서는 기본적으로 실행되지 않는 구문입니다.
• 사용법: -ea (enable assertions) 옵션을 사용하여 활성화할 수 있습니다.
  • 명령어 사용 시: java -ea -jar test.jar
  • IntelliJ 사용 시: Edit Configuration > Modify Options > Add VM Options를 클릭하면 VM Options를 입력할 수 있는 창이 생성됩니다. 해당 입력창에 -ea 옵션을 추가합니다.
• 장점: 서비스 메서드 내에 예상치 못한 상태나 누락될 수 있는 테스트 케이스를 정의하여, 조건이 충족되지 않을 경우 즉시 오류를 감지할 수 있습니다.
• 단점: 서비스 메서드 내에 assert 구문이 포함되면 코드 가독성이 떨어질 수 있고, 이미 작성된 테스트 코드와 중복 검증되는 부분이 있을 수 있습니다.

예시

// board status 가 REMOVED 상태가 아닐 때 DEFAULT 에러 메시지 처리
assert boardRepository.findById(...).orElseThrow(...).getStatus() == BoardStatus.REMOVED : DEFAULT.message();