Java 单元测试时的NPE_Java_Unit Testing_Mockito

Java 单元测试时的NPE

java unit-testing

Java 单元测试时的NPE,java,unit-testing,mockito,Java,Unit Testing,Mockito,我正在尝试为名为getBestSellers（）的方法编写单元测试这是： package bookstore.scraper.book.scrapingtypeservice; import bookstore.scraper.enums.Bookstore; import bookstore.scraper.book.Book; import bookstore.scraper.fetcher.empik.EmpikFetchingBookService; import bookstore

我正在尝试为名为

getBestSellers（）

的方法编写单元测试

这是：

package bookstore.scraper.book.scrapingtypeservice;

import bookstore.scraper.enums.Bookstore;
import bookstore.scraper.book.Book;
import bookstore.scraper.fetcher.empik.EmpikFetchingBookService;
import bookstore.scraper.fetcher.merlin.MerlinFetchingBookService;
import bookstore.scraper.urlproperties.EmpikUrlProperties;
import bookstore.scraper.urlproperties.MerlinUrlProperties;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.stereotype.Service;

import java.util.EnumMap;
import java.util.List;
import java.util.Map;

import static bookstore.scraper.utilities.JSoupConnector.connect;

@Service
public class BestSellersService {

    private final EmpikUrlProperties empikUrlProperties;
    private final MerlinUrlProperties merlinUrlProperties;
    private final EmpikFetchingBookService empikBookService;
    private final MerlinFetchingBookService merlinBookService;

    @Autowired
    public BestSellersService(EmpikFetchingBookService empikBookService, MerlinFetchingBookService merlinBookService, EmpikUrlProperties empikUrlProperties, MerlinUrlProperties merlinUrlProperties) {
        this.empikBookService = empikBookService;
        this.merlinBookService = merlinBookService;
        this.empikUrlProperties = empikUrlProperties;
        this.merlinUrlProperties = merlinUrlProperties;
    }

    public Map<Bookstore, List<Book>> getBestSellers() {
        Map<Bookstore, List<Book>> bookstoreWithBestSellers = new EnumMap<>(Bookstore.class);

        bookstoreWithBestSellers.put(Bookstore.EMPIK, empikBookService
                .get5BestSellersEmpik(connect(empikUrlProperties.getEmpik().getBestSellers())));
        bookstoreWithBestSellers.put(Bookstore.MERLIN, merlinBookService
                .get5BestSellersMerlin(connect(merlinUrlProperties.getMerlin().getBestSellers())));

        return bookstoreWithBestSellers;
    }
}

我已经调试过了，我已经看到了

empikUrlProperties.getEmpik().getBestSellers()))

给了我NPE。所以我设置了如下行为：

when(empikUrlProperties.getEmpik().getBestSellers()).thenReturn(anyString());
when(merlinUrlProperties.getMerlin().getBestSellers()).thenReturn(anyString());

现在它给了我NPE和stactrace：

ava.lang.NullPointerException
    at bookstore.scraper.book.scrapingtypeservice.BestSellersServiceTest.getBestSellers(BestSellersServiceTest.java:48)
    at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
    at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
.
.
.

连接

测试方法中使用的方法：

@UtilityClass
public class JSoupConnector {

    public static Document connect(String url) {
        try {
            return Jsoup.connect(url).get();
        } catch (IOException e) {
            throw new IllegalArgumentException("Cannot connect to" + url);
        }
    }
}

属性类（与merlin相同）

我做错了什么？为什么当我在设置模拟时放置

any（）

时，它首先不起作用：

when(empikUrlProperties.getEmpik().getBestSellers()).thenReturn(anyString());

您已经模拟了

empikUrlProperties

，这很好，但是您没有告诉mock在调用

getEmpik（）

时该做什么。因此，该方法调用将在测试中返回null，在生产代码中也会返回null（在生成这一行之前）——因此这就是调用

getBestSellers（）

时出现NPE的原因

因此，设置模型，类似于：

@Mock
private EmpikUrlProperties empikUrlProperties;
@Mock
private EmpikUrlProperties.Empik empikMock;

when(empikUrlProperties.getEmpik()).thenReturn(empikMock);
when(empikMock.getBestSellers()).thenReturn(anyString());

设置模拟时：

when(empikUrlProperties.getEmpik().getBestSellers()).thenReturn(anyString());

您已经模拟了

empikUrlProperties

，这很好，但是您没有告诉mock在调用

getEmpik（）

时该做什么。因此，该方法调用将在测试中返回null，在生产代码中也会返回null（在生成这一行之前）——因此这就是调用

getBestSellers（）

时出现NPE的原因

因此，设置模型，类似于：

@Mock
private EmpikUrlProperties empikUrlProperties;
@Mock
private EmpikUrlProperties.Empik empikMock;

when(empikUrlProperties.getEmpik()).thenReturn(empikMock);
when(empikMock.getBestSellers()).thenReturn(anyString());

有很多问题，但最主要的是你误解了模拟、参数求值和

any（）

的工作原理

你正在使用

when(empikBookService.get5BestSellersEmpik(any())).thenReturn(empikBestsellers);

这告诉mock empikBookService，无论何时调用其

get5BestSellersEmpik

方法，它都应该返回

empikBestsellers

，无论传递给该方法的参数是什么

在执行测试时，实际代码作为参数传递的是什么？它传递由返回的值

connect(empikUrlProperties.getEmpik().getBestSellers())

关键部分是首先计算这个表达式，然后将其结果作为参数传递给

get5BestSellersEmpik（）

方法

就像你做的那样

System.out.println(a + b)

a+b

首先进行评估。如果结果为42，则值42将传递给println（），println将打印42

所以，为了让你的测试不会失败

 connect(empikUrlProperties.getEmpik().getBestSellers())

必须成功评估。它的结果并不重要，因为您已经将mock配置为接受任何参数。但这无关紧要

你想做什么

when(empikUrlProperties.getEmpik().getBestSellers()).thenReturn(anyString());

那没有任何意义

首先，因为

empikUrlProperties.getEmpik（）

将返回null，因为

empikUrlProperties

是一个模拟，而模拟默认返回null。因此，

null.getBestSellers（）

将导致NullPointerException

其次，因为告诉mock它应该返回任何字符串是没有意义的。如果您不关心它应该返回的字符串，那么您可以自己选择一个字符串，并让它返回该字符串

anyString（）

实际上返回null，所以您告诉它返回null

所以你需要解决这个问题。始终考虑您的代码在做什么，而不是尝试应用一个配方

最后，您的测试还调用了

connect（…）

，这是一种staic方法，您没有（也不能）模拟它。此方法也将被调用。它尝试连接到一个实际的URL。因此，如果在该URL的测试期间没有任何响应，那么这也不会起作用。此

connect（）

方法实际上应该是服务依赖项的一部分，并且应该模拟此依赖项。

存在许多问题，但主要问题是您误解了模拟、参数求值和

any（）