基于Spock交互的测试：方法调用太少

Question

AuthApiKeyPair updateApiKeyPair(AuthApiKeyPair apiKeyPair, Boolean createNewKey) {

    AuthApiKeyPair newKeyPair

    if (createNewKey) {
        deleteApiKeyPair(apiKeyPair)

        //The key will be created with the same info as the previous key.
        newKeyPair = createApiKeyPair(apiKeyPair.label, apiKeyPair.accountMode, apiKeyPair.source)
    }

    newKeyPair
}

测试：

def "should soft delete key pair and create new one"() {
    setup:
    AuthApiKeyPair apiKeyPair = AuthApiKeyPair.build(acquirerId: 123, source: PaymentSource.BOARDING_API, label: 'Boarding API key')

    when:
    service.updateApiKeyPair(apiKeyPair, true)

    then:
    1 * service.deleteApiKeyPair(apiKeyPair)
    1 * service.createApiKeyPair(apiKeyPair.label, apiKeyPair.accountMode, apiKeyPair.source)
}

Answer 1

如果您考虑一下您的测试，您就会意识到，在最好的情况下，它测试的是Spock的模拟机制，而不是业务代码。您还没有用您的测试规范向我们展示完整的类，但是根据您的场景，我们可以假设您的测试中的service只是一个模拟。如果这是真的，那么就不能指望这两个调用：

then:
1 * service.deleteApiKeyPair(apiKeyPair)
1 * service.createApiKeyPair(apiKeyPair.label, apiKeyPair.accountMode, apiKeyPair.source)

发生。仅仅是因为mocked类不执行真正的方法。

null

（在使用此特性之前要三思而后行。根据规范更改代码的设计可能会更好。）

来源:http://spockframework.org/spock/docs/1.0/interaction_based_testing.html#spies

我不知道Grails是否有一个用于创建Spy而不是Mock的测试注释，但您可以始终遵循官方文档，创建普通的Spock单元测试，将服务实例化为Spy，然后可以尝试计算调用。不过，我不建议这么做，只是为了记录在案而提到这一点。