5

我正在使用 Python 对类进行单元测试unittest。据我了解,在每次测试之前unittest调用该setUp函数,以便单元测试对象的状态相同并且执行测试的顺序无关紧要。

现在我有这门课我正在测试......

#! usr/bin/python2

class SpamTest(object):

    def __init__(self, numlist = []):
        self.__numlist = numlist

    @property
    def numlist(self):
        return self.__numlist

    @numlist.setter
    def numlist(self, numlist):
        self.__numlist = numlist

    def add_num(self, num):
        self.__numlist.append(num)

    def incr(self, delta):
        self.numlist = map(lambda x: x + 1, self.numlist)

    def __eq__(self, st2):
        i = 0
        limit = len(self.numlist)

        if limit != len(st2.numlist):
            return False

        while i < limit:
            if self.numlist[i] != st2.numlist[i]:
                return False

            i += 1

        return True

通过以下单元测试...

#! usr/bin/python2

from test import SpamTest

import unittest

class Spammer(unittest.TestCase):

    def setUp(self):
        self.st = SpamTest()
        #self.st.numlist = [] <--TAKE NOTE OF ME!
        self.st.add_num(1)
        self.st.add_num(2)
        self.st.add_num(3)
        self.st.add_num(4)

    def test_translate(self):
        eggs = SpamTest([2, 3, 4, 5])
        self.st.incr(1)
        self.assertTrue(self.st.__eq__(eggs))

    def test_set(self):
        nl = [1, 4, 1, 5, 9]
        self.st.numlist = nl
        self.assertEqual(self.st.numlist, nl)

if __name__ == "__main__":
    tests = unittest.TestLoader().loadTestsFromTestCase(Spammer)
    unittest.TextTestRunner(verbosity = 2).run(tests)

此测试对 test_translate 失败。

我可以做两件事来使测试成功:

(1) 取消setUp函数第二行的注释。或者,

(2) 更改translate首先发生的测试的名称。我注意到unittest按字母顺序执行测试。translate例如,更改为atranslate使其首先执行会使所有测试成功。

对于 (1),我无法想象这会如何影响测试,因为在 的第一行setUp,我们为 self.st 创建了一个新对象。至于(2),我的抱怨是相似的,因为,嘿,setUp我分配了一个新对象,self.st所以无论我做什么都不应该影响self.st.test_settest_translate

那么,我在这里缺少什么?

4

2 回答 2

10

在不研究解决方案的细节的情况下,您应该阅读Fredrik Lundh的 Python 中的默认参数值。

它很可能解释了您将空列表作为默认参数的问题。原因是列表仅在第一次为空,除非您稍后明确将其设为空。初始为空的默认列表是列表类型的单个实例,在没有传递显式参数时会被重用。

阅读上述文章以解决您对默认参数的想法是个好主意。原因是合乎逻辑的,但可能出乎意料。

一般推荐的修复方法是使用None默认值,__init__如果没有传递参数,则在正文中设置空列表,如下所示:

class SpamTest(object):

    def __init__(self, numlist=None):
        if numlist is None:
            numlist = []         # this is the new instance -- the empty list
        self.__numlist = numlist
于 2012-07-08T09:41:47.717 回答
4

这是由于使用可变对象(如列表)时默认参数在 Python 中的行为方式:Python 中的默认参数值

在行中:

def __init__(self, numlist = []):

numlist 的默认参数仅评估一次,因此您只有一个列表实例,该实例在SpamTest类的所有实例中共享。

因此,即使setUp每次测试都调用该测试,它也不会创建一个新的空列表,并且您在该列表实例上工作的测试最终会相互踩踏。

解决方法是使用像这样的非可变对象,例如None

def __init__(self, numlist = None):
    if numlist is None:
        numlist = []
    self.__numlist = numlist

设置属性时它起作用的原因是您在那里提供了一个全新的空列表,替换了在构造函数中创建的列表。

于 2012-07-08T09:38:19.937 回答