我上了这个课:
public class A {
private int x;
public A(int x) {
this.x = x;
}
}
还有我要测试的其他类中的方法:
public class B {
public List underTest() {
int x = doStuff();
return Collections.singletonList(new A(x));
}
private int doStuff() { /* ... */ }
}
我想测试一下,在underTest()
的末尾,返回列表中的项目包含一个x
字段,该字段等于某个值。我写了这个:
import static org.hamcrest.MatcherAssert.assertThat;
import static org.hamcrest.Matchers.contains;
import static org.hamcrest.Matchers.hasProperty;
import static org.hamcrest.Matchers.is;
@Test
public void Test1() {
List result = bInstance.underTest();
assertThat(result, contains(hasProperty("x", is(1))));
}
但是junit抱怨item 0: No Property "x"
是我的测试用例。
我该如何测试?我唯一想到的就是为getX()
添加一个公共获取器,然后遍历返回的List
并检查每个元素。请记住,尽管该方法返回singletonList
,但返回类型仅为List
,因此将来可以 对其进行更改以在其中包含多个值。
答案 0 :(得分:2)
我认为首先值得一提的是,像这样测试类内部不是一个好主意,除非在非常特殊的情况下。您的测试将很脆弱,通常完全安全的更改(即重命名字段)现在可能会导致自动构建失败。您应该测试外部行为,而不是实现细节。
似乎最好在类equals
中实现hashCode
和A
,所以您可以简单地做到:
contains(new A(1))
话虽如此,如果您确实有充分的理由这样做(这种情况很少见),则不能使用hasProperty
。
来自the JavaDoc:
创建一个匹配器,该匹配器在被检查对象具有JavaBean时进行匹配 具有指定名称的属性,其值满足指定的属性 匹配器。
我认为这意味着您需要一个名为getX
的方法。
您不应仅出于测试目的添加此类方法,而是可以编写自己的通用Matcher
实现,该实现将使用反射来检查类的字段。
这是一个示例实现:
class ReflectiveFieldMatcher<T> extends BaseMatcher<Object>
{
private final String fieldName;
private final T expectedValue;
ReflectiveFieldMatcher(final String fieldName, final T expectedValue)
{
this.fieldName = fieldName;
this.expectedValue = expectedValue;
}
@Override
public boolean matches(final Object obj)
{
for (final Field field : obj.getClass().getFields())
{
if (field.getName().equals(fieldName))
{
field.setAccessible(true);
try
{
Object value = field.get(obj);
return expectedValue.equals(value);
}
catch (final IllegalAccessException e)
{
throw new RuntimeException(e);
}
}
}
return false;
}
@Override
public void describeTo(final Description description)
{
description.appendText("Object with field '" + fieldName + "' with value: " + expectedValue);
}
}
您的示例现在如下所示:
assertThat(result, contains(new ReflectiveFieldMatcher<>("x", 1)));