使用对象实例化测试一段代码的良好实践

Question

是否有任何指导方针（或最佳实践）来整理以下内容？

我有以下方法可以测试：

fun findProfileByPersonalInfo(personInfo: PersonalInfo): Profile? {
    localProfileRepository.findByPersonalInfo(personalInfo)?.let {
        return it
    }
    val searchParams = RemoteProfileSearchParameters()
    //... Some mapping from personalInfo to searchParams
    remoteProfileRepository.findRemoteProfile(searchParams)?.let {
        val profile = profileFactory.create(remoteProfile)
        return save(profile)
    }
    return null
}

当我开始为这个函数编写单元测试时，我意识到：

1. 测试线val profile = profileFactory.create(remoteProfile) 既简单又方便，因为我可以模拟此调用的结果并分别测试从remoteProfile 到profile 的映射。有了这个，如果我改变映射的逻辑，我不会破坏我对 findProfileByPersonalInfo 的测试，这很好。
2. 反之亦然，如果personalInfo到searchParams的映射逻辑属于findProfileByPersonalInfo，感觉不太对也很难维护。

问题是：

1. 是否可以（几乎）总是将实例化逻辑从被测函数中移出，以使其更易于测试和维护？毫无疑问，唯一的职责是实例化对象的函数是个例外。
2. 为每个属于上述情况的对象创建某种类型的工厂是不是太过分了？系统的整体设计似乎因此受到影响。

非常感谢任何对最佳实践或讨论的引用。非常感谢你！

Answer 1

1. 这有点宽泛，一般来说，您希望方法不要做太多事情，并将代码的可重用部分移动到单独的方法中。根据您展示的有限上下文，在这种情况下，出于可测试性和可读性的原因，我肯定会将实例移出。

2. 你并不总是需要一个成熟的工厂，根据上下文一个简单的构造函数RemoteProfileSearchParameters(personInfo: PersonalInfo) 可能是最明智的。根据复杂性，这可能会被重构为工厂方法或实际工厂对象，但通常这不会是您的第一个 goto。