在我工作的项目中,已经决定停止使用fest进行测试断言,而是使用assertj。我们使用的是Java 7,我们正在从fest版本2.0M10转移到assertj核心版本2.4.1。代码库相当大,但从fest到assertj的转换非常顺利,基本上更改了导入名称并处理了重命名的方法。然而,我注意到在转换之后,我们在一个特定的测试类中遇到了测试失败(我应该补充一下,我们使用的是JUnit v4.12)
import java.util.Collection;
import java.util.HashMap;
import java.util.Map;
import java.util.Set;
import org.junit.Before;
import org.junit.Test;
class MyMap implements Map<String, Object> {
private Map<String, Object> theMap = new HashMap<>();
@Override
public int size() {
return theMap.size();
}
@Override
public boolean isEmpty() {
return theMap.isEmpty();
}
@Override
public boolean containsKey(Object key) {
return theMap.containsKey(key);
}
@Override
public boolean containsValue(Object value) {
return theMap.containsValue(value);
}
@Override
public Object get(Object key) {
return theMap.get(key);
}
@Override
public Object put(String key, Object value) {
return theMap.put(key, value);
}
@Override
public Object remove(Object key) {
return theMap.remove(key);
}
@Override
public void putAll(Map<? extends String, ? extends Object> m) {
theMap.putAll(m);
}
@Override
public void clear() {
theMap.clear();
}
@Override
public Set<String> keySet() {
return theMap.keySet();
}
@Override
public Collection<Object> values() {
return theMap.values();
}
@Override
public Set<java.util.Map.Entry<String, Object>> entrySet() {
return theMap.entrySet();
}
}
public class TestMapComparison {
private Map<String, Object> m1 = new HashMap<>();
private MyMap m2 = new MyMap();
@Before
public void before() {
m1.put("a", "b");
m2.put("a", "b");
}
// Fails with:
// java.lang.AssertionError:
// expected: <'{'a'='b'} (MyMap@6f5fc7ad)'>
// but was: <'{'a'='b'} (HashMap@3)'>
@Test
public void test1_Fest_m1_isEqualTo_m2() {
org.fest.assertions.api.Assertions.assertThat(m1).isEqualTo(m2);
}
@Test // Pass
public void test2_AssertJ_m1_isEqualTo_m2() {
org.assertj.core.api.Assertions.assertThat(m1).isEqualTo(m2);
}
@Test // Pass
public void test3_Fest_m2_isEqualTo_m1() {
org.fest.assertions.api.Assertions.assertThat(m2).isEqualTo(m1);
}
// Fails with:
// java.lang.AssertionError:
// Expecting: <"{"a"="b"} (MyMap@5aedacd2)">
// to be equal to:
// <"{"a"="b"} (HashMap@3)"> but was not.
@Test
public void test4_AssertJ_m2_isEqualTo_m1() {
org.assertj.core.api.Assertions.assertThat(m2).isEqualTo(m1);
}
}
我为这么长的代码感到抱歉。从测试结果中可以看出,当在hashmap上使用isEqualTo()时,fest和assertj之间似乎有区别,其中一个hashmap封装在实现接口映射的类中。我的问题是如何处理?我可以翻转断言中的顺序,即拥有assertThat(b).isEqualTo(a),而不是拥有assertThat(a).isEqualTo(b)。但在一个有很多断言的大型测试类中,必须对特定断言进行这样的翻转,这对我来说有点奇怪。fest和assertj之间的差异是意料之中的还是意料之外的?这两种行为是预期的吗(因为两者都在一个表达式上失败)?我应该如何更新测试用例,是否有更好的方法来对上面代码中的场景进行相等性检查?感谢您阅读本文,并感谢您的任何建议!
好吧,错误就在你的代码中,如果你测试equals()的契约,每个库都会发现错误,它必须是对称的:A等于B,iff B等于A。
我会简单地修复你的代码,并很高兴由于迁移而发现了这个错误。我也会改进测试并进行
assertThat(a).isEqualTo(b);
assertThat(b).isEqualTo(a);
以确保合同得到履行。