인프런 커뮤니티 질문&답변
작성한 질문수
테스트 코드에 대해 질문 있습니다.
해결된 질문
작성
·
139
0
private String createNextProductNumber() {
String latestProductNumber = productRepository.findLatestProductNumber();
if (latestProductNumber == null) {
return "001";
}
int latestProductNumberInt = Integer.parseInt(latestProductNumber);
int nextProductNumberInt = latestProductNumberInt + 1;
return String.format("%03d", nextProductNumberInt);
}
Integer.parseInt(latestProductNumber); 이 부분 처럼 값을 숫자로 바꿀때 latestProductNumber가 숫자인지 아닌지에 대한 검증은 안해도 되는건지 궁금합니다.
public OrderResponse createOrder(OrderCreateRequest request, LocalDateTime registeredDateTime) {
List<String> productNumbers = request.getProductNumbers();
List<Product> products = findProductsBy(productNumbers);
deductStockQuantities(products);
Order order = Order.create(products, registeredDateTime);
Order savedOrder = orderRepository.save(order);
return OrderResponse.of(savedOrder);
}
...
private static List<String> extractStockProductNumbers(List<Product> products) {
return products.stream()
.filter(product -> ProductType.containsStockType(product.getType()))
.map(Product::getProductNumber)
.collect(Collectors.toList());
}
private Map<String, Stock> createStockMapBy(List<String> stockProductNumbers) {
List<Stock> stocks = stockRepository.findAllByProductNumberIn(stockProductNumbers);
return stocks.stream()
.collect(Collectors.toMap(Stock::getProductNumber, s -> s));
}
private static Map<String, Long> createCountingMapBy(List<String> stockProductNumbers) {
return stockProductNumbers.stream()
.collect(Collectors.groupingBy(p -> p, Collectors.counting()));
}
위 상황처럼 createOrder()메서드가 아래 private 메서드를 모두 호출 하므로 createOrder() 메서드만 테스트하고 나머지 메서드는 테스트를 안 해도 되는건가요?
질문의 요지는 private이냐 아니야가 아니라 특정 메서드에서 다른 메서드를 모두 호출한다면 특정 메서드만 테스트를 해도 되는지 아니면 각각의 메서드도 테스트를 해야하는지 궁금합니다.
답변 1
0
안녕하세요, 감바스 님!
1. Integer.parseInt(latestProductNumber); 이 부분 처럼 값을 숫자로 바꿀때 latestProductNumber가 숫자인지 아닌지에 대한 검증은 안해도 되는건지 궁금합니다.
테스트는 항상, 필요하다고 생각한다면 작성하면 되는데요.
위 케이스의 경우, 이미 "상품의 고유번호가 숫자다"라는 정책이 기반이 되고 있으므로 테스트를 진행하지 않아도 될 것 같아요.
이미 ProductRepository를 통해 조회한 상품은, 그 이전 단계인 상품 저장 시점에 이미 "상품의 고유번호가 숫자다" 라는 정책에 의해 검증된 상품 번호만 저장되었을 것이기 때문입니다.
2. 위 상황처럼 createOrder()메서드가 아래 private 메서드를 모두 호출 하므로 createOrder() 메서드만 테스트하고 나머지 메서드는 테스트를 안 해도 되는건가요?
질문의 요지는 private이냐 아니야가 아니라 특정 메서드에서 다른 메서드를 모두 호출한다면 특정 메서드만 테스트를 해도 되는지 아니면 각각의 메서드도 테스트를 해야하는지 궁금합니다.
위 경우는 애초에 private 메서드라 public 메서드를 통하지 않고는 테스트 할 방법이 없습니다.
질문주신 것처럼 만약 public 메서드가 다른 public 메서드를 호출하는 상황이라면, 각각 테스트를 작성하는 것이 좋겠네요 ㅎㅎ
도움이 되셨기를 바랍니다.
감사합니다 🙂