fest和assert-j与自定义地图之间的细微差别

时间:2016-04-30 10:19:24

标签: java junit java-7 fest assertj

在我工作的项目中,已经决定停止使用fest进行测试断言,而是使用assertj。我们正在使用Java 7,我们正在从fest版本2.0M10转向assertj-core版本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);
    }
}

我很抱歉这么长的代码。从测试结果可以看出,当在散列图上使用isEqualTo()时,fest和assertj之间似乎存在差异,其中一个散列图封装在实现接口映射的类中。 我的问题是如何处理这个?我可以在断言中翻转顺序,即断言(b).isEqualTo(a)而不是断言(a).isEqualTo(b)。但是对于我来说,在一个带有许多断言的大型测试类中对特定断言进行这样的翻转时,我感到有点奇怪。预期或意外的fest和assertj之间的区别是什么?是否有任何预期的行为(因为两个表达式都失败了)?我应该如何更新测试用例,是否更好地与上面的代码中的场景进行相等性检查?感谢您阅读这篇文章,感谢任何建议!

1 个答案:

答案 0 :(得分:2)

嗯,错误在你的代码中,如果你测试了equals()的契约,每个库都会发现bug,它必须是对称的:A等于B iff B等于A.

我只是修复你的代码,并且很高兴通过迁移发现了这个bug。我也会改进测试并做

assertThat(a).isEqualTo(b);
assertThat(b).isEqualTo(a);

确保合同得到履行。