1

我遇到了一个问题，其中某些类 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”等。

4

1 回答 1

3

org.testng.Assert.assertEquals(X a, X b)调用迭代条目集的assertEquals(Map, Map)方法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 类的所有测试文件，并确保有一个适当的强制转换（不简单）或 noX在左侧声明=
  • 在方法中使用模拟框架和模拟 X @BeforeGroups（如果您使用组）并将依赖于该惯用语的每个测试放在同一个组中，以便他们将使用模拟 X，其中 entrySet 无法通过您的测试（并且其他方法作为预期），或者至少记录/打印一些警告。如果你还需要用entrySet同样的方法不起作用
  • 使用模拟框架来模拟TestNG#assertEquals(Map, Map)以获得所需的行为
  • 在所有需要该习惯用法的测试类中，创建一个临时assertEquals(Map, Map)方法。如果您有所有 assertEquals 方法的静态导入，您也必须重新实现其他签名:-(
  • have all those test classes inherit a BaseXTestClass where you implement an assertEquals(Map, Map) method - same caveat
于 2013-01-23T20:58:04.547 回答