Проблема Mockito: mock вызывает фактический метод

У меня проблема с Mockito, когда в тесте вызывается реальный метод, а не метод издевательства. Искал часами, но не смог найти подходящего ответа.

Вот служба, которую я тестирую:

@Service
public class DwpApiService {

    @Autowired
    private RestTemplate restTemplate = new RestTemplate();

    @Bean
    public RestTemplate restTemplate() {
        return new RestTemplate();
    }

    private GeoService geoService = new GeoService();

    private String baseUri = "XXXX";

    private double cityLatt = 51.5074;
    private double cityLong = -0.1278;

    public List<PersonApiModel> getAllUsersInCityOrWithinDistanceOfCity(String cityName, double distanceInMiles) throws RuntimeException {

        HashMap<Integer, PersonApiModel> usersInCityOrWithinDistance = new HashMap<>();

        List<PersonApiModel> usersInCity = getAllUsersInCity(cityName).getBody();

        for (PersonApiModel person: usersInCity) {
            usersInCityOrWithinDistance.put(person.getId(), person);
        }

        List<PersonApiModel> allUsers = getAllUsers().getBody();

        for (PersonApiModel person : allUsers) {
            boolean withinDistance = geoService.isLocationWithinDistance(distanceInMiles, cityLatt, cityLong, person);
            if (withinDistance && !usersInCityOrWithinDistance.containsKey(person.getId())) {
                usersInCityOrWithinDistance.put(person.getId(), person);
            }
        }

        return new ArrayList<>(usersInCityOrWithinDistance.values());
    }

    public ResponseEntity<List<PersonApiModel>> getAllUsersInCity(String cityName) throws RuntimeException {
        String cap = cityName.substring(0, 1).toUpperCase() + cityName.substring((1));

        if (!cap.equals("London")) {
            throw new IllegalArgumentException("Invalid city name. Please only use the city of London.");
        }

        return restTemplate.exchange(
                baseUri + "city/" + cap + "/users",
                HttpMethod.GET,
                null,
                new ParameterizedTypeReference<List<PersonApiModel>>(){}
        );
    }

    public ResponseEntity<List<PersonApiModel>> getAllUsers() throws RuntimeException {
        return restTemplate.exchange(
                baseUri + "/users",
                HttpMethod.GET,
                null,
                new ParameterizedTypeReference<List<PersonApiModel>>(){}
        );
    }

}

Этот сервис вызывает следующий класс GeoService:

public class GeoService {

    //taken from https://www.movable-type.co.uk/scripts/latlong.html

    public boolean isLocationWithinDistance(double radiusInMiles,
                                            double sourceLat, double sourceLong,
                                            PersonApiModel personApiModel) throws IllegalArgumentException {

        boolean isLatLongWithinRange = checkIfLatLongAreWithinRange(
                sourceLat, sourceLong,
                personApiModel.getLatitude(), personApiModel.getLongitude());

        if (!isLatLongWithinRange) {
            throw new IllegalArgumentException("Latitude or Longitude are not valid values for id=" + personApiModel.getId());
        }

        int earthMeanRadius = 6371;

        double latDiff = Math.toRadians(sourceLat -  personApiModel.getLatitude());
        double longDiff = Math.toRadians(sourceLong - personApiModel.getLongitude());
        double a = Math.sin(latDiff / 2) * Math.sin(latDiff / 2)
                + Math.cos(Math.toRadians(sourceLat)) * Math.cos(Math.toRadians(personApiModel.getLatitude()))
                * Math.sin(longDiff / 2) * Math.sin(longDiff / 2);
        double c = 2 * Math.atan2(Math.sqrt(a), Math.sqrt(1 - a));
        double distance = earthMeanRadius * c;

        double radiusInKm = radiusInMiles * 1.609344;

        return radiusInKm >= distance;
    }

    public boolean checkIfLatLongAreWithinRange(double sourceLat, double sourceLong,
                                                   double destinationLat, double destinationLong) {
        boolean sourceLatInRange = Math.abs(sourceLat) <= 90;
        boolean destLatInRange = Math.abs(destinationLat) <= 90;
        boolean sourceLongLessThanMinus180 = sourceLong >= -180;
        boolean sourceLongLessThan80 = sourceLong <= 80;
        boolean destLongLessThanMinus180 = destinationLong >= -180;
        boolean destLongLessThan80 = destinationLong <= 80;

        return sourceLatInRange && destLatInRange
                && sourceLongLessThanMinus180 && sourceLongLessThan80
                && destLongLessThanMinus180 && destLongLessThan80;
    }
}

Мой тестовый пример выглядит так:

@RunWith(MockitoJUnitRunner.class)
public class DwpApiServiceTest {

    @Mock
    private RestTemplate restTemplate;

    @Mock
    private GeoService geoService;

    @InjectMocks
    private final DwpApiService dwpApiService = new DwpApiService();

    private final PersonApiModel personApiModel1 = new PersonApiModel();
    private final PersonApiModel personApiModel2 = new PersonApiModel();

    private final List<PersonApiModel> fakeList1 = new ArrayList<>();
    private final List<PersonApiModel> fakeList2 = new ArrayList<>();

    private final String baseUri = "XXXXXX";
    private double cityLatt = 51.5074;
    private double cityLong = -0.1278;

  @Test
    public void givenMocks_whenGetAllUsersInAndAroundCity_returnMockedPersonList() {
        personApiModel1.setId(1);
        fakeList1.add(personApiModel1);
        ResponseEntity<List<PersonApiModel>> expected1 = new ResponseEntity<>(fakeList1, HttpStatus.OK);

        personApiModel2.setId(2);
        fakeList2.add(personApiModel2);
        ResponseEntity<List<PersonApiModel>> expected2 = new ResponseEntity<>(fakeList2, HttpStatus.OK);

        PersonApiModel person = personApiModel2;

        List<PersonApiModel> expected = new ArrayList<>();
        expected.addAll(fakeList1);
        expected.addAll(fakeList2);

        String cityName = "london";
        double distanceInMiles = 20;

        Mockito.when(dwpApiService.getAllUsers())
                .thenReturn(expected1);

        Mockito.when(dwpApiService.getAllUsersInCity(cityName))
                .thenReturn(expected2);

        Mockito.when(geoService.isLocationWithinDistance(distanceInMiles, cityLatt, cityLong, person))
                .thenReturn(true);

        List<PersonApiModel> actual = dwpApiService.getAllUsersInCityOrWithinDistanceOfCity(cityName, distanceInMiles);

        Assert.assertEquals(expected, actual);
    }

Теперь имитация DwiApiService отлично работает с этим и другими тестовыми примерами. Но насмешка над GeoService, похоже, вызывает проблемы. Когда я пытаюсь запустить тест, он на самом деле вызывает не фиктивный метод, а настоящий. Ошибки не очень помогают...

[MockitoHint] DwpApiServiceTest.givenMocks_whenGetAllUsersInAndAroundCity_returnMockedPersonList (see javadoc for MockitoHint):
[MockitoHint] 1. Unused... -> at com.dwpAPI.services.DwpApiServiceTest.givenMocks_whenGetAllUsersInAndAroundCity_returnMockedPersonList(DwpApiServiceTest.java:115)
[MockitoHint]  ...args ok? -> at com.dwpAPI.services.DwpApiService.getAllUsersInCityOrWithinDistanceOfCity(DwpApiService.java:47)

Кто-нибудь знает, что здесь происходит? Трачу часы и часы на это и просто не могу понять это.


person soFreshSoClean    schedule 14.06.2020    source источник
comment
Попробуйте выполнить автоматическое подключение geoService вместо инициализации с помощью new.   -  person Smile    schedule 14.06.2020
comment
Пробовал, получая ту же ошибку.   -  person soFreshSoClean    schedule 14.06.2020
comment
Отвечает ли это на ваш вопрос? Mockito - разница между doReturn() и when()   -  person Progman    schedule 14.06.2020
comment
@Progman Пробовал, но, похоже, не сработало. Stepio ниже нашел ответ, который работает. Хотя не совсем уверен, почему!   -  person soFreshSoClean    schedule 14.06.2020


Ответы (2)


Возможная проблема заключается в том, что вы указываете точные параметры. Рассматривайте конфигурацию mockito как набор правил, поэтому точные значения там не работают (если вам действительно не повезло, и это просто волшебным образом работает для какого-то конкретного узкого случая).

Итак, попробуйте заменить этот:

when(geoService.isLocationWithinDistance(distanceInMiles, cityLatt, cityLong, person))
    .thenReturn(true);

С этим:

when(geoService.isLocationWithinDistance(eq(distanceInMiles), eq(cityLatt), eq(cityLong), eq(person)))
    .thenReturn(true);

Также у меня иногда возникают проблемы с использованием конструкции when(...).thenReturn(...) - например, она не работает должным образом с @Spy. Поэтому я обычно предпочитаю этот подход:

doReturn(true).when(geoService)
    .isLocationWithinDistance(eq(distanceInMiles), eq(cityLatt), eq(cityLong), eq(person));

А если какой-то из параметров для вас не очень важен (или все сразу), не используйте eq(...) и не используйте никакое значение - просто замените соответствующий параметр на any().

P.S.: Я изменил только одно насмешливое правило, предполагая, что вы сами будете проверять другие.

person stepio    schedule 14.06.2020
comment
Хм, я заменил when(geoService.isLocationWithinDistance(distanceInMiles, cityLatt, cityLong, person)).thenReturn(true); на when(geoService.isLocationWithinDistance(anyDouble(), anyDouble(), anyDouble(), any())).thenReturn(true); и все заработало! Не уверен, почему, поэтому копну немного глубже. Спасибо! - person soFreshSoClean; 14.06.2020

причина может заключаться в том, что вы являетесь непосредственно экземпляром этого

private GeoService geoService = new GeoService();

вам, вероятно, нужно аннотировать класс GeoService с помощью @Service и автоматически подключить его к DwpApiService

person Hamza    schedule 14.06.2020
comment
Добавлена ​​аннотация @Service к классу GeoService и @Autowired к ссылке GeoService в классе DwiApiService. Все еще получаю ту же ошибку. - person soFreshSoClean; 14.06.2020