返回在嘲笑时错误执行的建议（在实际执行之前）

Question

在集成测试中，我的@afterreturn建议被错误执行，而在测试中我模拟它以抛出TimeoutException，并且传递给方面的参数为null。

我的建议是：

    @AfterReturning("execution(* xxxxxx" +
            "OrderKafkaProducerService.sendOrderPaidMessage(..)) && " +
            "args(order)")
    public void orderComplete(CheckoutOrder order) { // order is null when debugging
        metricService.orderPaidKafkaSent();
        log.trace("Counter inc: order paid kafka"); // this line of log is shown in console
        metricService.orderCompleted();
        log.trace("Order complete! {}", order.getId()); // this line is not, because NPE
    }

我的测试：

// mocking
doThrow(new ServiceException(new TimeoutException("A timeout occurred"), FAILED_PRODUCING_ORDER_MESSAGE))
    .when(orderKafkaProducerService).sendOrderPaidMessage(any()); // this is where advice is executed, which is wrong

...
// when
(API call with RestAssured, launch a real HTTP call to endpoint; service is called during this process)

// then
verify(orderKafkaProducerService).sendOrderPaidMessage(any(CheckoutOrder.class)); // it should be called
verify(metricService, never()).orderCompleted(); // but we are throwing, not returning, we should not see this advice executed

由于NPE（订单为空），此测试失败。

在调试中，我发现当我进行模拟时，我已经执行了建议，此时，any（）还没有值，为空，所以NPE。但我不认为这些建议应该在嘲笑的时候执行。测试时如何避免这种情况？？这对我来说太荒谬了。

Answer 1

目前Spring测试支持没有显式处理注入的mock或间谍（这是通过Mockito的代理子类）可能实际上是稍后的AOP目标（即代理并因此通过CGLIB再次子类化）的情况。

关于Spring、Spring Boot和Mockito，有几个与此主题相关的错误通知单。还没有人对此采取任何行动。我确实理解为什么Mockito维护人员不将特定于Spring的内容包含到他们的代码库中，但我不理解为什么Spring人员不改进他们的测试工具。

实际上，当调试失败的测试并检查kafkaService时，您会发现以下事实：

1. kafkaService.getClass（）iscom.example.template.services.KafkaService$MockitoMock92961867$$EnhancerBySpringCGLIB$$8fc4fe95
2. kafkaService.getClass（）. getSuperclass（）iscom.example.template.services.KafkaService$MockitoMock92961867$
3. kafkaService.getClass（）. getSuperclass（）. getSuperclass（）是类com.example.template.services.KafkaService

换句话说：

1. kafkaService是一个CGLIB Spring AOP代理。
2. AOP代理包装一个Mockito间谍（可能是ByteBuddy代理）。
3. Mockito间谍包裹了原始物体。

此外，更改包装顺序以使Mockito spy成为最外层的对象将不起作用，因为CGLIB故意使其覆盖方法成为最终方法，即不能再次扩展和覆盖它们。如果Mockito也有同样的限制，那么分层包装就根本不起作用。

不管怎样，你能做什么？

• 要么使用本教程中描述的复杂方法

在您的情况下，后一种解决方案如下所示：

@Test
void shouldCompleteHappyPath() {
    // fetch spy bean by unwrapping the AOP proxy, if any
    KafkaService kafkaServiceSpy = AopTestUtils.getTargetObject(kafkaService);
    // given mocked
    doNothing().when(kafkaServiceSpy).send(ArgumentMatchers.any());
    // when (real request)
    testClient.create().expectStatus().isOk();
    // then
    verify(kafkaServiceSpy).send(ArgumentMatchers.any());
    verify(metricService).incKafka();
}

这样做的效果是，当（kafkaServiceSpy）时。发送（ArgumentMatchers.any（））不再触发方面建议，因为kafkaServiceSpy不再是AOP代理。不过，自动连接的beankafkaService仍然是，因此AOP如预期的那样被触发，但在记录模拟交互时不再不受欢迎地被触发。

实际上，对于验证，您甚至可以使用kafkaService来代替间谍，并且只有在记录您以后想要验证的交互时才打开间谍：

@Test
void shouldCompleteHappyPath() {
    // given mocked
    doNothing()
      .when(
        // fetch spy bean by unwrapping the AOP proxy, if any
        AopTestUtils.<KafkaService>getTargetObject(kafkaService)
      )
      .send(ArgumentMatchers.any());
    // when(real request)
    testClient.create().expectStatus().isOk();
    // then
    verify(kafkaService).send(ArgumentMatchers.any());
    verify(metricService).incKafka();
}

附言：如果没有你的MCVE，我将永远无法调试它，并找出到底发生了什么。这再次证明，提出包括MCVE在内的问题是你能为自己做的最好的事情，因为它可以帮助你获得问题的答案，否则这些问题可能仍然没有答案。

更新：在我在类似的闭题Spring Boot#6871中提到这个问题后，一位维护人员自己创建了Spring Boot#22281，专门解决了您的问题。你可能想看看新问题，看看是否/何时可以解决。