엔티티 비즈니스 로직 vs 서비스 비즈니스 로직

강의에서 DDD의 개념을 적용하시면서 엔티티에 비즈니스 로직을 넣으셨었는데요.

아직까진 DDD에 대한 개념이 완벽히 정립되지 않아 별도의 자료로 공부를 하고 있습니다.

공부하는 차원에서 서비스 계층에서의 비즈니스 로직 개발로 코드 리팩토링을 스스로 해보고 강의에 나오지 않았던 상품 서비스 테스트 코드 작성 결과 일단 통과가 나오긴 했습니다만... 잘 한 것인지 몰라서 코드 평가를 좀 받아보고 싶어 질문을 올리게 되었습니다.

테스트 코드 역시 각 계층별로 지켜야 하는 테스트 코드 원칙이 있으나 이 부분도 같이 공부하느라 제대로 적용해보기 아직 어려운 단계라 @SpringBootTest를 사용했습니다. 이 부분은 감안해주세요..!

또 다른 추가 질문이 있습니다. 강의에서 엔티티에 구현한 비즈니스 로직을 서비스 계층에 비즈니스 로직을 개발하는 방식으로 바꾸는 과정에서 엔티티를 직접 이용하는 것보다는 DTO를 만들어서 전달하는게 엔티티에 많은 변화를 가하지않는 유일한 방법일까요?

ItemService 코드

package spring.project.service;

import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.stereotype.Service;
import org.springframework.transaction.annotation.Transactional;
import spring.project.domain.Item;
import spring.project.exception.NotEnoughStockException;
import spring.project.repository.ItemRepository;

import java.util.List;

@Service
@Transactional(readOnly = true)
public class ItemService {

	private final ItemRepository itemRepository;
	@Autowired
	public ItemService(ItemRepository itemRepository) {
		this.itemRepository = itemRepository;
	}

	@Transactional
	public Long saveItem(Item item, int quantity) {
		addStock(item, quantity);
		itemRepository.save(item);
		return item.getId();
	}

	private void addStock(Item item, int quantity) {
		item.setStockQuantity(item.getStockQuantity() + quantity);
		itemRepository.save(item);
	}

	@Transactional
	public void removeStock(Item item, int quantity) {
		int restStock = item.getStockQuantity() - quantity;
		if (restStock < 0) {
			throw new NotEnoughStockException("재고가 부족합니다.");
		}
		item.setStockQuantity(restStock);
		itemRepository.save(item);
	}

	public Item findOne(Long itemId) {
		return itemRepository.findOne(itemId);
	}

	public List<Item> findAll() {
		return itemRepository.findAll();
	}
}

ItemServiceTest 코드

package spring.project.service;

import org.assertj.core.api.Assertions;
import org.junit.jupiter.api.DisplayName;
import org.junit.jupiter.api.Test;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.boot.test.context.SpringBootTest;
import spring.project.domain.Album;
import spring.project.domain.Book;
import spring.project.domain.Item;
import spring.project.repository.ItemRepository;

@SpringBootTest
class ItemServiceTest {

	@Autowired private ItemRepository itemRepository;
	@Autowired private ItemService itemService;

	@Test
	@DisplayName("상품 저장 시 상품 저장과 동시에 재고 수량이 증가해야 한다.")
	void save() {
		// given
		Book book = new Book();
		book.setAuthor("홍길동");

		// when
		Long id = itemService.saveItem(book, 3);

		// then
		Item item = itemRepository.findOne(id);
		Assertions.assertThat(item.getName()).isEqualTo(book.getName());
		Assertions.assertThat(item.getStockQuantity()).isEqualTo(3);
	}

	@Test
	@DisplayName("상품 주문이 들어가면 재고 수량에서 주문 수량만큼 차감이 되어야 한다.")
	void remove() {
		// given
		Album album = new Album();
		album.setName("영원");
		Long id = itemService.saveItem(album, 100);

		// when
		itemService.removeStock(album, 4);

		// then
		Item item = itemRepository.findOne(id);
		Assertions.assertThat(item.getStockQuantity()).isEqualTo(96);
	}
}