Junit 如何在mockito中测试内部循环
Hi i have methodJunit 如何在mockito中测试内部循环,junit,mockito,Junit,Mockito,Hi i have methodinsertOrUpdateProductsToDB（Product Product）用于使用Broadleaf的catalogService在数据库中执行插入操作，catalog Service正在数据库中执行所有保存操作。我的方法应使用restClient产品作为参数。在传递restClient产品后，我们将使用ProductConversion类将此产品转换为Broadleafproduct。在产品转换中，仅对将rest产品转换为Broadleafprodu
insertOrUpdateProductsToDB（Product Product）catalogServiceverify(mainProduct).getAdditionalSkus().add(sku);
verify(mainProduct).setProductOptions(productOptionList);
insertOrUpdateProductsToDB（产品产品）productOption=catalogService.saveProductOption（产品选项）此处productOption为空，请告诉我们如何在循环内测试循环，同样的情况也会发生
for (Sku skuWithProductOptions : productConversion.createSkuWithProductOptions(product, mainProduct,productOptionList)) {
catalogService.saveSku(skuWithProductOptions);
}
请检查我的测试用例是否正确
要测试的类和插入器更新产品数据库（产品产品）
方法
import com.admin.exception.AdminGenericException;
import com.admin.exception.AdminRestException;
import com.admin.util.helper.ProductConversion;
import com.admin.wrapper.getproducts.req.ObjectFactory;
import com.admin.wrapper.getproducts.resp.Product;
import com.admin.wrapper.getproducts.resp.Response;
import com.mycompany.rest.service.client.RestClientUtil;
import com.mycompany.util.constants.ApplicationConstants;
@Service
public class GetProductsServiceImpl {
private static final Logger logger = Logger.getLogger(GetProductsServiceImpl.class);
@Resource(name = "blCatalogService")
protected CatalogService catalogService;
public void setCatalogService(CatalogService catalogService) {
this.catalogService = catalogService;
}
protected RestClientUtil restClientUtil;
public void setRestClientUtil(RestClientUtil restClientUtil) {
this.restClientUtil = restClientUtil;
}
@Value("#{configProperties['salePriceRate']}")
private long salePriceRate;
public void setRetailPriceRate(long retailPriceRate) {
this.retailPriceRate = retailPriceRate;
}
@Value("#{configProperties['retailPriceRate']}")
private long retailPriceRate;
public void setSalePriceRate(long salePriceRate) {
this.salePriceRate = salePriceRate;
}
//Insertion/Update DB logic
public String insertOrUpdateProductsToDB(Product product) {
logger.debug("Start of : insertOrUpdateProductsToDB()");
try {
List<String> category = new ArrayList<String> (Arrays.asList(ApplicationConstants.CATEGORY));
ProductConversion productConversion = new ProductConversion();
List<ProductOption> productOptionList = new ArrayList<ProductOption>();
if (category.contains(product.getCategory().toUpperCase())) {
org.broadleafcommerce.core.catalog.domain.Product mainProduct=catalogService.createProduct(new ProductType("org.broadleafcommerce.core.catalog.domain.Product", "Normal Product"));
mainProduct = productConversion.createProduct(mainProduct,product);
Sku sku=catalogService.createSku();
mainProduct.setDefaultSku(sku);
mainProduct = productConversion.addSkuToProduct(mainProduct, product, salePriceRate,retailPriceRate);
for (ProductOption productOption : productConversion.createProductOptions(product, mainProduct)) {
productOption.setAllowedValues(productConversion.createProductOptionValues(product,productOption));
productOption = catalogService.saveProductOption(productOption);
productOptionList.add(productOption);
}
sku = catalogService.saveSku(mainProduct.getDefaultSku());
mainProduct.getAdditionalSkus().add(sku);
mainProduct.setProductOptions(productOptionList);
mainProduct = catalogService.saveProduct(mainProduct);
for (Sku skuWithProductOptions : productConversion.createSkuWithProductOptions(product, mainProduct,productOptionList)) {
catalogService.saveSku(skuWithProductOptions);
}
}
logger.debug("End of : insertOrUpdateProductsToDB()");
return "Product inserted into DB successfully";
}
catch (Exception e) {
logger.error("Error:", e);
return "Insertion of product into DB Failed ";
}
}
//Insertion service for DB
public String insertProductsIntoDB(){
logger.debug("Start of : insertProductsIntoDB()");
int insertionCount=0;
try{
com.admin.wrapper.getproducts.resp.Response resp = getAvailableProductsFromPBS();
for (Product product : resp.getProducts().getProduct()) {
if(catalogService.findProductById(Long.parseLong(product.getId()))==null){
String str=insertOrUpdateProductsToDB(product);
if(str.equalsIgnoreCase("Product inserted into DB successfully")){
insertionCount=insertionCount+1;
}
}
}
logger.debug(insertionCount+" Products inserted into DB successfully");
logger.debug("End of : insertProductsIntoDB()");
return insertionCount+" Products inserted into DB successfully";
}catch (AdminRestException e) {
logger.error("Error:", e);
return e.getMessage();
}
}
}
我有几句话可能无法回答你最初的问题。但我希望他们能引导您更好地重构这段代码。此外，您展示的代码示例还不足以为您指出确切的问题；这是测试方法中的一个NPE，所以应该不难追踪
话虽如此，我想提出一点
- 测试代码是精心编制的，在我看来，这段代码过度使用了Mockito。总的来说，这段代码看起来太复杂了，无论如何都无法正确测试。我不认为它是按照TDD原则编写的（TDD在测试和设计应用程序时非常方便）
- 您可能希望在一个方法中遵循不超过10行代码的通用准则，这通常有助于分离关注点并确定更简单的代码/意图。如果设计正确（不泄露概念或变量），这些更简单的代码可以更容易地更改和测试。例如，您可能希望提取一个保存单个
产品的方法，并仅测试该产品
- 更引人注目的是，这段代码似乎有点程序化（即使是在对象内部）。并没有用商业术语真正解释意图（好的，这是关于在DB中保存内容的，但是出于什么原因有这么多逻辑，这个原因应该出现在方法名称中）
- test和Mockito很奇怪，代码不应该在集合上迭代以进行存根验证
for (Product pbsProduct : pbsResponse.getProducts().getProduct()) {
when(catalogService.createProduct(new ProductType("org.broadleafcommerce.core.catalog.domain.Product","Normal Product"))).thenReturn(mainProduct);
when(catalogService.createSku()).thenReturn(sku);
when(catalogService.saveProductOption(productOption)).thenReturn(productOption);
when(catalogService.saveSku(sku)).thenReturn(sku);
when(catalogService.saveProduct(mainProduct)).thenReturn(mainProduct);
when(catalogService.saveSku(sku)).thenReturn(sku);
getProductsServiceImpl.insertOrUpdateProductsToDB(pbsProduct);
verify(mainProduct,times(2)).setDefaultSku(sku);
verify(mainProduct).getAdditionalSkus().add(sku);
verify(mainProduct).setProductOptions(productOptionList);
break;
}
- 在伪代码中，我将首先尝试使用给定的提取保存逻辑，当/时，然后使用BBDD关键字（它们有助于澄清在哪个场景和上下文中需要测试什么）。将fixture和断言保持在最低限度，您更愿意处理多个测试方法，而不是多个复杂的测试方法
@Test
public void ensure_product_is_saved_in_the_catalog() {
// given
Product a_simple_product = ProductBuilder.simpleProduct().build();
when(catalogService.doSomething(....))).thenReturn(mainProduct);
// when
productsService.saveProduct(product);
// then
verify(catalogService).doSomethingElseWith(mainProduct);
}
如果产品数据断言与测试场景相关，那么编写一个实际测试数据的测试（使用JUnit断言、AssertJ等）。不要嘲笑产品
并逐步进行每个测试，然后在需要时重构以保持代码的可管理性（如有必要，在另一个类中提取一个方法，等等）
- 你绝对应该读下面的书，它们帮助了很多程序员获得了更好的代码或代码。当然，这份清单并非详尽无遗
希望能有所帮助。实际上，要测试的方法不是我写的，而是别人写的，这就是为什么我必须保持它的真实性。仅供参考，在保存产品之前，必须先保存sku，然后才能将产品保存到db中。因此，可能是这个原因，此方法是这样编写的。但问题是，当我进入productOption保存的循环时，我得到的productOption为null，返回方法为catalogService。saveProductOption（productOption）；我无法证实为什么会这样。
for (Product pbsProduct : pbsResponse.getProducts().getProduct()) {
when(catalogService.createProduct(new ProductType("org.broadleafcommerce.core.catalog.domain.Product","Normal Product"))).thenReturn(mainProduct);
when(catalogService.createSku()).thenReturn(sku);
when(catalogService.saveProductOption(productOption)).thenReturn(productOption);
when(catalogService.saveSku(sku)).thenReturn(sku);
when(catalogService.saveProduct(mainProduct)).thenReturn(mainProduct);
when(catalogService.saveSku(sku)).thenReturn(sku);
getProductsServiceImpl.insertOrUpdateProductsToDB(pbsProduct);
verify(mainProduct,times(2)).setDefaultSku(sku);
verify(mainProduct).getAdditionalSkus().add(sku);
verify(mainProduct).setProductOptions(productOptionList);
break;
}
@Test
public void ensure_product_is_saved_in_the_catalog() {
// given
Product a_simple_product = ProductBuilder.simpleProduct().build();
when(catalogService.doSomething(....))).thenReturn(mainProduct);
// when
productsService.saveProduct(product);
// then
verify(catalogService).doSomethingElseWith(mainProduct);
}