如何测试调用共享方法的两个Java公共API方法？

先看你给出的代码：

public class A {
    public List<Model> getModels() {
        List<Model> models = (some logic to find all models...)
        models.forEach(model -> doSharedLogic(model));
    }

    public Model getModel(int id) {
        Model model = (some logic to find Model by Id...)
        doSharedLogic(model);
    }    

    private void doSharedLogic(Model model) {
        // manipulations with model...       
    }
}

你遇到的问题很典型：两个公共方法依赖同一个私有共享逻辑，测试时要么重复代码，要么有遗漏风险。这里给你三个实用的解决思路：

1. 拆分测试职责，分层验证 #

把测试分成两部分，各司其职：

• 单独覆盖共享逻辑的所有场景：用反射或者PowerMock这类工具直接调用私有方法doSharedLogic，把它的业务分支、边界条件全部测一遍，这部分代码只写一次，彻底避免重复。
• 验证公共方法的流程正确性：对getModels和getModel，不用再重复测共享逻辑的细节，只需要确认它们确实调用了doSharedLogic，且调用符合预期：
  • 对getModel：测当找到对应ID的Model时，doSharedLogic被调用1次；如果没找到Model，确认方法行为符合预期（比如返回null或抛异常）且不调用共享逻辑。
  • 对getModels：测返回的每个Model都触发了一次doSharedLogic调用，空列表时不调用，以此保证方法和共享逻辑的绑定关系。

2. 重构代码，抽离共享逻辑（可选） #

如果不想碰反射这类工具，最彻底的方式是重构：把doSharedLogic的逻辑抽成一个独立的公共类（比如ModelHandler），让类A依赖这个类。这样：

• 直接对ModelHandler的公共方法写完整测试，覆盖所有业务场景；
• 对类A的两个公共方法，只需要用Mock框架验证它们正确调用了ModelHandler的方法就行，不用再管业务逻辑细节。

这种方式不仅让测试更干净，代码的可维护性也会提升。

3. 基于行为的测试，平衡重复与风险 #

如果暂时不想重构也不想用反射，可以换个测试思路：

• 选其中一个方法（比如getModel），把doSharedLogic的所有业务场景细节都在这里测一遍；
• 对另一个方法getModels，只测典型场景的行为结果：比如构造包含不同状态Model的列表，调用方法后验证每个Model都被正确处理，同时加一个测试用例确认方法确实调用了共享逻辑（比如用Mock框架统计调用次数）。

这样既避免了大部分测试代码重复，又能保证两个方法都和共享逻辑绑定，不会出现后续修改时其中一个方法脱离共享逻辑的情况。

内容的提问来源于stack exchange，提问作者Vasya Chopovenko