Question

我遇到一个问题，其中一些类X扩展扩展了java.util.AbstractMap，并且还重载equals(Object)。致电org.testng.Assert.assertEquals(X a, X b)会解析为assertEquals(Map<?,?>, Map<?,?>)。不是调用'equals'方法，而是比较地图条目。即使assertEquals(a, b)为假，也会导致a.equals(b)通过。

此代码演示了此问题：

import static org.testng.Assert.assertEquals;
import static org.testng.Assert.assertTrue;

import java.util.AbstractMap;
import java.util.Collections;
import java.util.Set;

import org.testng.annotations.Test;

public class AssertTest {
    static class X extends AbstractMap<String, Object> {
        private int i;

        public X(int i) {
        this.i = i;
        }

        @Override
        public Set<Entry<String, Object>> entrySet() {
        return Collections.EMPTY_SET;
        }

        @Override
        public boolean equals(Object o) {
        return o instanceof X && i == ((X)o).i;
        }

        @Override
        public int hashCode() {
        return i;
        }
    }

    @Test
    public void test() {
        X one = new X(1);
        X two = new X(2);
        assertEquals(one, two); // passes, should fail IMO
        assertTrue(one.equals(two)); // correctly fails
    }
}

解决此问题的最佳方法是什么？一种可能性是记住不要使用assertEquals来验证X的实例是否匹配，但这非常容易出错。

另一种可能性是简单地制作testng的本地副本并重命名重载。这造成了持续的维护问题。

我唯一的另一个想法是创建一个特定于项目的Assert类，该类委托给testng.Assert，但将有问题的重载重命名为'assertCollectionEquals'，'assertMapEquals'等。

Answer 1

org.testng.Assert.assertEquals(X a, X b)调用the assertEquals(Map, Map) method迭代条目集并检查所有条目是否等于，绕过Map#equals方法，正如您所注意到的那样。

简单的强制转换会避免调用该方法，而是使用X#equals方法：

assertEquals((Object) one, (Object) two);

您还可以将变量声明为对象以获得相同的结果：

Object one = new X(1);
Object two = new X(2);
assertEquals(one, two);

从某种意义上讲，它仍然不容易出错。

为了避免偶然的错误，我可以想到一些解决方法（在这里累了所以有些可能没什么意义），使用assertEquals没有强制调用x.entrySet()的事实：

编写一些代码来解析X类的所有测试文件，并确保存在一个强制转换（不是直截了当）或者X <左侧没有声明= / LI>
使用模拟框架并在@BeforeGroups方法中模拟X（如果你使用组）并将每个依赖于该成语的测试放在同一个组中，这样他们就会使用模拟的X，其中entrySet失败了测试（以及其他方法按预期工作），或至少记录/打印一些警告。如果您还需要在不起作用的相同方法中使用entrySet
使用模拟框架模拟TestNG#assertEquals(Map, Map)以获得所需的行为
在所有需要该习惯用法的测试类中，创建一个特殊的assertEquals(Map, Map)方法。如果你有静态导入所有assertEquals方法，你也必须重新实现其他签名： - （
让所有这些测试类继承一个实现assertEquals(Map, Map)方法的BaseXTestClass - 同样需要注意

如何解决不需要的TestNG assertEquals重载

1 个答案: