为“ assertEqual”和“ assertNotEqual”编写测试：我应该打扰吗？

Question

我有一个Coordinate类，它具有一个add(Coordinate)方法。 在为此类编写单元测试时，我有一些测试来assertEqual结果：

a = Coordinate(1,2,3)
b = Coordinate(5,6,7)

result = a.add(b) 

assertEqual(result.x, 6)
assertEqual(result.y, 8)
assertEqual(result.z,10) 

我可以很容易地“伪造”这个：

def add(self, other):
    return Coordinate(6,8,10)

这是测试失败的最简单解决方案。 下一步是编写第二个测试，以防止我以这种方式伪造它。 我可以：

• 用不同的数字编写另一个assertEqual测试（因此伪造的Coordinate(6,8,10)无法通过，或
• 用两个不同的输入编写一个assertNotEqual测试，确保结果不是6,8,10 。

如果我编写一个assertEquals测试，那么我将获得两个看起来非常非常相似的测试。 这是一个问题吗？ 如果我在项目中看到类似的代码，我很想重构它。 我是否也应该为测试代码执行此操作-如果是这样，这是否意味着每对测试最终都将被重构？

如果我写一个assertNotEqual ，则测试仅测试“伪造的结果”-我非常确定，永远不会出现算法错误。 本质上，一旦我编写了测试，就停止伪造结果，以便两个测试都通过， 可以安全地删除 assertNotEquals测试，并且我对代码仍然有信心-因此，我将编写测试，修复假冒的内容，然后删除测试，这似乎很愚蠢。

在这种情况下我该怎么办？

Answer 1

我不会太担心能够像您所描述的那样“假传递”测试-始终可以在方法内部编写复杂的if语句来使每个测试用例通过。 相反，请查看add方法中涉及的逻辑，并确保您的测试用例涵盖了所有代码分支-在我看来，这应该足够好。

Answer 2

编写防止有人恶意编写代码以根据测试代码使用的特殊情况伪造测试代码的测试是不可能的。 所以你不应该尝试。

当我说“不可能”时，我并不是说“非常努力”，这可能是说谎者悖论破坏了正式系统的一个实例。

Answer 3

另一个具有不同数字的assertEqual测试就足够了。 如果可能，请采用“边界线”或不常见的情况，例如：

a = Coordinate(1,2,3)
b = Coordinate(-5,-6,-7)
...

AssertNotEqual测试对于读者IMO而言是荒谬的，而且不直观。

无论如何，我不会为这种测试担心太多。 作为读者，很明显，编写它们的开发人员只想测试几个案例，并且需要一个真正的重构极端主义者来重构它们。 我的意思是，它只有2个测试，几乎没有重复，意图很明显，并且不像当对象更改时您必须重写300行代码...

Answer 4

注意：为简单起见，我将把“坐标”更改为“整数”。 另外，下面的语法也组成..因此将不会执行。 但这应该使我明白

testAdd()
  assertThat(2.Add(2), isEqualTo(4))

然后，您可以通过始终使Add返回4来伪造此内容。 接下来进行三角剖分（通过更改某些参数来驱动通用代码）。

testAdd()
  assertThat(2.Add(2), isEqualTo(4))
  assertThat(3.Add(5), isEqualTo(8))

现在，您需要使Add进行一些实际工作。。绿色后，您可以保留这两个测试，也可以删除简单/平凡的测试-只要不让您的信心下降即可。 如果重复困扰您，您可以查看测试运行程序是否支持“参数化测试”

[2,2,4]
[3,5,8]
testAdd(operand1, operand2, expectedResult)
  assertThat(operand1.Add(operand2), isEqualTo(expectedResult))

Answer 5

Coordinate.Add（）方法是什么样的？

似乎该方法应该处理如何添加（不要伪造它），然后您所要做的就是：

# In your test setUp code
testCoord = Coordinate(6, 8, 10)  # Explicitly setting the value you expect the test to return.
addCoord1 = Coordinate(1, 2, 3)
addCoord2 = Coordinate(5, 6, 7)

# The test
addCoord1.add(addCoord2)
assertEqual(testCoord, addCoord1, "Testing addition of coordinates using add() method.")

如果我错了，请纠正我，但是您使用的是单元测试框架吗？ 我个人将对象创建放在unittest的测试用例的setUp中。