简单api调用的Mockito JUnit测试

Question

我认为自己是单元测试的新手，对McCito和JUnit完全陌生。我必须为一些简单的api调用编写单元测试。但我的测试对我来说似乎有点毫无意义，我不知道我哪里出了问题。我向现有的web服务ManagerWS添加了一个方法。java，见下文。

经理。java方法：

public String healthCheck(String userId) { 
    String healthCheckUrlEndpoint  = this.baseUrl()+"/health";
    logger.debug("Calling health check: {}", healthCheckUrlEndpoint);
    HttpHeaders healthCheckHeaders = new HttpHeaders();
    healthCheckHeaders.setContentType(MediaType.APPLICATION_JSON_UTF8);
    healthCheckHeaders.add(USER_KEY, userId);
    healthCheckHeaders.add(TOKEN_NAME, TOKEN_VALUE);
    healthCheckHeaders.add(Constants.ACCEPT_LANGUAGE_HEADER, LocaleContextHolder.getLocale().toString());
    healthCheckHeaders.add(CORRELATION_HEADER, myService.get(AppLoggingMDCService.LOG_KEY_REQUEST_ID));
    HttpEntity<Object> request = new HttpEntity<Object>(healthCheckHeaders);
    ResponseEntity<String> response;        
    try {
        response = makeRequest(HttpMethod.GET, healthCheckUrlEndpoint, request, String.class); 
    } catch (Exception e) {
        logger.error("Exception encountered during health check", e);
        throw e;
    }       
    logger.debug("RESPONSE : http status: {} - body: {}", response.getStatusCode(), response.getBody());
    return response.getStatusCode().toString();
}

逻辑很简单。构造url，创建标头并将标头添加到请求中。发出请求并从响应中提取状态代码。这是我的测试。注意：类使用@RunAnd（SpringRunner.class），我使用@Mock作为依赖项，使用@InjectMocks作为本地实例ManagerWS。java是正在测试的服务calss。

试验等级试验方法：

    @Test
    public void testHealthCheck() throws Exception {
    //Given
    managerWS = new ManagerWS(templateFactory, configParamService, mdcService, env);    
    String url = "http://baseurl/health";
    HttpHeaders headers = new HttpHeaders();    
    HttpEntity<Object> request = new HttpEntity<Object>(headers);   
    ResponseEntity<String> response = new ResponseEntity<>(HttpStatus.OK);
    //when
    when(managerWS.makeRequest(HttpMethod.GET, url, request, String.class)).thenReturn(response);
    String actualStatus = response.getStatusCode().toString();  
    //then
    Assert.assertEquals("200",actualStatus);
}

对我来说，这个测试似乎很愚蠢（因为缺少一个好词）。我将状态设置为“200”，并断言我设置的是“200”。这没有多大意义。对我来说，它实际上什么都不做。我尝试使用spy（ManagerWS.class）。但我实际上是在不完全理解的情况下抓住救命稻草。

SonarQube仍然抱怨“没有被单元测试覆盖”。我完全不知道如何写这个测试。我还必须为另外三个电话做类似的测试。

我完全是测试新手，我看不出我的错误。请告知。

Answer 1

SonarQube仍然抱怨“单元测试未涵盖”。

您的单元测试不从要测试的方法的入口点进行测试：healthCheck（String），因此单元测试不包括它。此外，您还模拟了要测试的方法部分：

when(managerWS.makeRequest(HttpMethod.GET, url, request, String.class)).thenReturn(response);

事实上，为这段代码编写单元测试看起来也是错误的，或者至少看起来像一个没有什么价值的白盒测试。
为什么？你的逻辑取决于：

response = makeRequest(HttpMethod.GET, healthCheckUrlEndpoint, request, String.class); 

但是，您可以知道它是否只在运行HTTP服务器时工作。因此，您可以做的一件事就是模拟所有内容，监视被测对象，并验证实现中的每个语句是否都已执行：没有可读的测试，没有健壮性和很少/没有值。

基本上依赖于副作用的方法更适合作为集成测试进行测试：

  ManagerWS managerWS; // real ManagerWS implementation without mock

  @Test
  public void healthCheck() throws Exception {
    //Given
    String url = "http://baseurl/health";
    // When
    String actual managerWS.healthCheck(url);
    // Then
    String expected = "...";
    Assertions.assertEquals(expected, actual);
  }

作为旁注，如果您使用Spring，我建议您查看测试切片@WebMvcTest，该切片主要关注被测组件的web部件。它主要允许测试HTTP部分/逻辑（头、请求、响应）。