[bug] 포트폴리오 단일 삭제 문제

[yonghwankim-dev]

상황

포트폴리오 수익 내역(PortfolioGainHistory) 데이터가 존재한 상태에서 포트폴리오 삭제 API 요청(/api/portfolios/:portfolioId)시 버그가 발생합니다.

원인

포트폴리오 삭제 서비스 수행 중 포트폴리오 데이터 삭제전에 포트폴리오 수익 내역 데이터를 삭제하지 않아서 버그가 발생하였습니다.

@Transactional
public void deletePortfolio(Long portfolioId, AuthMember authMember) {
	log.info("포트폴리오 삭제 서비스 요청 : portfolioId={}, authMember={}", portfolioId, authMember);

	Portfolio findPortfolio = findPortfolio(portfolioId);
	validatePortfolioAuthorization(findPortfolio, authMember.getMemberId());

	List<Long> portfolioStockIds = portfolioHoldingRepository.findAllByPortfolio(findPortfolio).stream()
		.map(PortfolioHolding::getId)
		.collect(Collectors.toList());

	int delTradeHistoryCnt = purchaseHistoryRepository.deleteAllByPortfolioHoldingIdIn(portfolioStockIds);
	log.info("매매이력 삭제 개수 : {}", delTradeHistoryCnt);

	int delPortfolioCnt = portfolioHoldingRepository.deleteAllByPortfolioId(findPortfolio.getId());
	log.info("포트폴리오 종목 삭제 개수 : {}", delPortfolioCnt);

	portfolioRepository.deleteById(findPortfolio.getId());
	log.info("포트폴리오 삭제 : delPortfolio={}", findPortfolio);
}

해결방법

portfolioRepository.deleteById() 메소드 호출전에 삭제하고자 하는 포트폴리오의 포트폴리오 수익 내역 데이터를 전부 삭제하는 코드를 구현합니다.

개선사항

테스트 코드에 샘플 데이터로 포트폴리오 수익 내역 데이터를 추가하고 테스트 통과가 될것.