如何验证构造函数中私有方法调用的数量？

Question

我有以下课程：

public class LegacyClass()
{
    public LegacyClass(final int aModeOfOperation)
    {
        if (aModeOfOperation == 0)
        {
            addSomeValues();
            addSomeValues();
        }
        else if (aModeOfOperation == 1)
        {
            addSomeValues();
        }
    }

    private void addSomeValues()
    {
    }
}

我想写一个单元测试，它会检查

1. 调用LegacyClass（0）会导致调用私有方法add某值2次，并且
2. 调用LegacyClass（1）会导致调用一次私有方法add某值。

我知道在Mockito/PowerMockito中使用间谍对象来计算私有方法调用的数量是可能的，但是AFAIK这些方法只有在测试类构建后调用所讨论的方法时才有效。

有没有可能在不改变LegacyClass的构造函数的情况下编写上面描述的单元测试？

Answer 1

我认为使用mockito是不可能的，因为您以这种方式声明spyspy（new LegacyClass（）），因此在监视对象之前会调用构造函数。也许有一种方法可以使用PoweverMock，但我通常不鼓励使用PowerMock，因为没有将代码重构为正确的设计。

相反，我强烈建议您将遗留代码重构为更可测试的代码（测试驱动开发是这里的关键）。

如果您绝对需要保留这个遗留代码与兼容的更改，但没有PowerMock；我将首先提高可见性级别，使包可见：

private void addSomeValues() { ... }

然后在测试中编写一个LegacyClass的静态子类，并重写将增加一个计数器调用超级的add某值方法。

public class LegacyCodeTest {
  ...

  static class AddSomeValuesCounteringLegacyClass extends LegacyClass {
    public int counter;
    void addSomeValues() { counter = counter + 1; super.addSomeValues(); }
  }
}

然后在您的测试中，您不满足这个类并断言调用计数；

Answer 2

我认为，如果您能够执行基于状态的测试，那么您可以从单元测试中获得更好的价值(http://blog.jayfields.com/2008/02/state-based-testing.html?m=1)关于公共方法的需求/行为，而不是测试私有方法的调用次数。这在一些隔离框架中可能是可能的，但从单元测试的角度来看，我看不出有多大价值。

想想某个单元测试，你会验证一个方法已经被调用，或者一个方法已经被调用了x个数字。这些测试是验证测试，它们仍然有效。但是关键因素是大多数模拟对象框架允许您对公共方法（至少在. net中）进行这些验证。原因是从单元测试的角度来看，私有方法并不重要，因为它们是内部实现细节。尽量避免代码原理图更改的测试失败。