模拟无效方法

Question

//原始方法：

@Autowired
    private ConversionServiceValidator validator; 

    public CRSConversionResult convertCRS(ConvertCrsVo convertCrsVo) throws Exception {

if (validator.isSameSourceAndTarget(convertCrsVo))
            throw new ValidationException(Constants.BADREQUEST);

        if (convertCrsVo.getPreferredTransforms() != null) {
            List<TransformVo> preferredTransformList = new ArrayList<>();
            for (TransformVo transformVo : convertCrsVo.getPreferredTransforms()) {
                preferredTransformList.add(getPerfByCode(transformVo));
            }
            convertCrsVo.setPreferredTransforms(preferredTransformList);
        }
        convertCrsVo.setSourceCRS(getCrsVoByCode(convertCrsVo.getSourceCRS()));
        convertCrsVo.setTargetCRS(getCrsVoByCode(convertCrsVo.getTargetCRS()));
        convertCrsVo = validator.replaceCoordinates(convertCrsVo);
        logger.info("ShellGeodeticService::convertCRS::Request to GeoCalService convertpoints::" + mapper.writeValueAsString(convertCrsVo));
        ConvertPointsResponse response = geoCalService.convertCRS(convertCrsVo);
        CRSConversionResult result = new CRSConversionResult();
        result.setCriteriaMessage(response.getCriteriaMessage());
        result.setResultPoints(response.getResultPoints());
        result.setTransformName(response.getTransformName());
        result.setTransformDescription(response.getTransformDescription());
        // added schema as per pbi 195298
        List<ConvertedTransformsResult> transformsResults = new ArrayList<>();
        if (response.getTransforms() != null || !response.getTransforms().isEmpty())
            response.getTransforms().stream().forEach(
                    t -> transformsResults.add(new ConvertedTransformsResult().getConvertedTransformsResult(t)));
        result.setTransforms(transformsResults);
        String logmessage=generateLogMessage(result,convertCrsVo);
        logger.info(logmessage);
        validator.isResponseValid(result);
        return result;
}

//上述方法的测试用例

@Test
    public void testconvertCRSJob() throws Exception{
        ConvertCrsVo convertCrsVo = TestDataFactory.getConvertCrsVo();
        CRSConversionResult crsConversionResult = TestDataFactory.getCRSConversionResult();
        ConversionServiceValidator conversionServiceValidatorMock = mock(ConversionServiceValidator.class);
Mockito.when(geoCalService.convertCRS(Mockito.any()))
        .thenReturn(TestDataFactory.getConvertPointsResponse(convertCrsVo));
Mockito.when(validator.replaceCoordinates(convertCrsVo))
        .thenReturn(TestDataFactory.getConvertCrsVo());
Mockito.when(geoCalService.search(Mockito.any(SearchFilter.class)))
        .thenReturn(TestDataFactory.getSearchResultResponseForCRS());
Mockito.when(shellGeodeticService.convertCRS(convertCrsVo))
        .thenReturn(TestDataFactory.getCRSConversionResult());  
shellGeodeticService.convertCRSJob();

        }

我得到的错误如下：

org.mockito.exceptions.misusing.CannotStubVoidMethodWithReturnValue：'isResponseValid'是void方法，并且在cannot中插入返回值！虚空通常与Throwables一起使用：doThrow（exception）.when（mock）.someVoidMethod（）;

---

如果不确定为什么会遇到错误，请继续阅读。由于上述语法的性质，可能由于以下原因而出现问题：1.您尝试存根的方法是重载。确保您正在调用正确的重载版本。2.在测试的某个地方，您正在存根final methods。抱歉，Mockito不验证/存根最终方法。3.使用when（spy.foo（））。then（）语法对间谍进行打桩。对间谍进行存根比较安全--使用doReturn | Throw（）系列方法。有关Mockito.spy（）方法的更多Javadocs。4.不支持在非公共父类上声明的模拟方法。

at com.shell.geodetic.GeodeticConvertionApiAppTests.testconvertCRSJob(GeodeticConvertionApiAppTests.java:1783)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)

有人可以帮助我如何对void方法“ isResponseValid”进行存根吗？我尝试了大约100种在SOF中看到的组合，但没有任何效果。感谢您的提前帮助。

*编辑

Class ConversionServiceValidator {

    public void isResponseValid(CRSConversionResult response) throws InvalidDataException {

            if (response.getResultPoints().isEmpty() || response.getResultPoints() == null) {
                throw new ValidationException("Request body has incorrect format");
            } else {
                for (Point point : response.getResultPoints()) {
                    if (point.getX().trim().equals("0") || point.getY().trim().equals("0")) {
                        throw new InvalidDataException(400, "Bad Request", "WARNING: Not all points could be converted",
                                response);
                    }
                }
}

Answer 1

它是一个模拟@InjectMocks ShellGeodeticService shellGeodeticService;

shellGeodeticService不是模拟对象（@InjectMocks用于被测试的类，其中将mocks注入其中）。

这意味着您不能使用

Mockito.when(shellGeodeticService.convertCRS(convertCrsVo)) 
       .thenReturn(TestDataFactory.getCRSConversionResult());

在您的测试中，在mocks中只能使用spys（或Mockito.when）。

实际上我正在尝试运行shellGeodeticService.convertCRS（）的测试用例，并且由于它在内部调用了isResponseValid方法，所以我也必须模拟它，对吗？

不，那是不正确的。如果validator是模拟的，则默认情况下，每个方法调用都不会执行任何操作。因此，除非您想引发异常，否则无需定义任何内容。

由于您的问题缺少一些细节，我认为您的测试的完整版本可能与此类似：

---

@InjectMocks
ShellGeodeticService shellGeodeticService;

@Mock
ConversionServiceValidator validator;

@Mock
... geoCalService; // some unknown class

@Test
public void testconvertCRSJob() throws Exception{

    ConvertCrsVo convertCrsVo = TestDataFactory.getConvertCrsVo();

    Mockito.when(validator.replaceCoordinates(convertCrsVo)).thenReturn(convertCrsVo);

    Mockito.when(geoCalService.convertCRS(Mockito.any())).thenReturn(TestDataFactory.getConvertPointsResponse(convertCrsVo));

    CRSConversionResult result = shellGeodeticService.convertCRS();

    // do some assertions on the result
}

因为validator是一个模拟：

[validator.isSameSourceAndTarget(convertCrsVo)返回false为默认值

[validator.isResponseValid( ... )默认不执行任何操作

因为您没有添加方法getPerfByCode和generateLogMessage，请注意，如果与模拟对象有任何进一步的交互，则需要添加它们。（例如：在您的测试代码中看不到对geoCalService.search的调用，因此我从上面显示的测试中删除了行为定义）