How to test void method that only modifies private class member variables?
我正在尝试在一个初始化一些私有字段的类中对一个方法进行单元测试:
1 2 3 4 5 6 7 8 9 10 11 12 | public void init(Properties props) throws Exception { this.language = props.getProperty(Constants.LANGUAGE,Constants.LANGUAGE_DEFAULT); this.country = props.getProperty(Constants.COUNTRY,Constants.COUNTRY_DEFAULT); try { this.credits = Integer.valueOf(props.getProperty(Constants.CREDITS_OPTION_NAME, Constants.CREDITS_DEFAULT_VALUE)); } catch (NumberFormatException e) { throw new Exception("Invalid configuration: 'credits' does not contain a valid integer value.", e); } //rest of method removed for sake of simplicity } |
我的困境是我想断言语言,国家和信用字段是在调用init之后设置的,但它们是私有的,没有公共访问器方法。 我看到有两种解决方案可用于测试:
您认为测试此方法的理想方法是什么?
I want to assert that the language, country and credits fields have been set after calling init
一些哲学:你为什么关心它们是否设置正确?如果答案是"所以该类的行为正确",那么您可以通过测试预期的后续行为来测试它们是否已经设置正确。但是,如果设置不正确无效,则它们是多余的,您应该删除它们。
另一张海报建议使用反射来访问私人领域。我建议不要这样做;标记为private的任何内容都应该是该类的实现细节,因此不应直接测试。您应该测试您的类的已发布API。然后,您可以通过更改实现(私有)详细信息,轻松地自由地重构它。
您可以使用反射来获取私有字段的值。例如:
1 2 3 | Field field = YourClass.class.getDeclaredField("language"); field.setAccessible(true); //override the access restriction of it being private field.get(yourObject); |
如果类路径上有弹簧,则可以使用其
有时人们认为不应该验证私有字段,只有对象的公共状态才是单元测试的重点。从这个角度来看,暴露一个getter可能更好,甚至更好 - 抛出一个异常。
如果对象处于无效状态(如果未设置字段),则对象本身应负责通过抛出异常来强制其有效性。
我通常会尝试避免依赖于被测试类的内部结构的测试。我宁愿通过测试一些然后使用这些字段的方法来测试它。
如果你正在进行严格的TDD,你甚至不应该添加这些字段,直到你有一个实际利用它们的测试用例:)
我个人会检查公共方法。有时您需要检查内部,但这种情况很少见。
BTW:公共getter的替代方案是在同一个包中进行单元测试并提供包本地getter。
您的JUnit是否直接调用该方法?然后对属性对象的引用是相同的,您应该能够访问语言和国家,以便在JUnit中重新计算。
私人方法的公共访问者也是一个不错的选择。