如何测试类方法是否正在从类的私有对象之一调用另一个方法

Question

我正在尝试为项目创建单元测试。在我的项目中，我有一个菜单类和一个 VerticalOptions 类。

我的菜单类有一个私有的 VerticalOptions 对象和一个公共的 handleInput 方法。当我调用我的菜单的 handleInput(key) 方法时，根据我给它的键，它会做不同的事情，即调用我的 VerticalOptions 对象的不同方法。

我想做一个 unitTest 来查看被调用的方法是否正确，我该怎么做？

我已经尝试在我的菜单中添加一个 Mockito 间谍，但是因为我想测试被调用的方法是否是私有 VerticalOptions 对象中的方法，所以它实际上不起作用。

在使用 getVerticalOptions 方法获取它之后，我还尝试将间谍放在 VerticalOptions 对象上，但它也不起作用。

public void handleInput(InputKey key)

{

    switch (key) {

        case S:

        case DOWN:

            optionsInterface.cycleDown();

            break;

        case W:

        case UP:

            optionsInterface.cycleUp();

            break;

        case SPACE:

        case ENTER:

            optionsInterface.select();

            break;

        default:

            break;

    }

}

@Test

public void testInput() {

    MainMenu menu = new MainMenu(game);

    VerticalButtonInterface buttonInterface = menu.getOptionsInterface();

    VerticalButtonInterface spy = spy(buttonInterface);

    menu.handleInput(InputKey.DOWN);

    verify(spy, times(1)).cycleDown();

}

这是我得到的测试失败：

需要但未调用：verticalButtonInterface.cycleDown(); -> 在 MenuTest.testInput(MenuTest.java:60) 实际上，与此模拟的交互为零。

Answer 1

我会给你另一种观点。我见过很多人走错了路，当你这样做时，其他一切都变得很难做/测试，这正是你现在正在做的。从这里开始，你想达到什么目的？我想测试并确保某个方法被调用......这是一件好事吗？什么是不应该有的单元测试？那就是对代码的深入了解。为什么？因为每次你对代码做一点小改动，你都会因为这种深厚的知识而不得不改变测试。如果您有 1000 次测试，那么您将走上一条艰难的道路。好的，现在我们知道问题是什么了，那么我们如何解决呢？好吧，首先让我们确保我们可以在不深入了解代码的情况下进行测试。我们该怎么做？好吧，想象一下你的代码添加了一个额外的步骤，一个设置状态的标志。您可能有一个存储结果状态的标志......你有 3 个方法要调用，所以你需要 3 个不同的状态，所以创建一个变量来反映它，无论是字符串、枚举还是其他任何让你开心的东西。例如，假设我们创建了一个包含 3 个可能值的字符串：cycleDown、cycleUp 和 select。您的代码开始看起来像：public string handleInput(InputKey key){      String state = determineState(key);      SomeType someResult = executeActionForState(state);}public String determineState(string key){    String state = "";     switch (key) {    case S:    case DOWN:        state = "cycleDown";        break;    case W:    case UP:        state = "cycleUp";        break;    case SPACE:    case ENTER:        state = "select";        break;    default:        break;}return state;}public void executeActionForState(string state){     if ( state == "cycleup" ) {     }     etc etc}现在，我可能不一定像这样编写您的示例代码，这有点勉强，这取决于您对代码执行的其他操作，但这是为了展示如何将功能与 UI 方面分开。我可以很容易地测试状态方法，我可以更改它的代码，而不必更改测试，因为测试会查看输入和输出，而不是事情是如何实现的。单元测试是关于功能的，它是关于拥有一旦创建就不需要更改的简单测试。验证某个方法是否已被调用不会给您带来任何有价值的东西，因为您不知道该方法稍后会做什么。您可以通过其他方式测试 UI 内容，单元测试仅与正确的功能有关。如果你不明确这种分离，那么你将难以维护你的测试，它会变得越来越难，直到你放弃。您将测试您是否获得了正确的状态，然后测试 cycleUp 方法是否根据您的要求做了正确的事情，这就是您知道每个部分独立工作的方式。稍后您开始查看集成测试、自动化 UI 测试，但这些是不同的东西。对它的目的进行单元测试，保持简单，不与其他代码绑定，然后一切都会变得简单。你不需要模拟太多，你不需要太担心复杂的设置，你也不需要每次代码中的某些内容发生变化时都改变你的测试。现在，为了解决问题的最后一部分，私有方法，您通过观察它们的输出来测试它们。您的类中必须有一些公共的东西，在调用私有方法时会发生变化。所以测试一下。