Python - 猴子修补怪异的bug

时间:2014-11-18 15:34:45

标签: python unit-testing monkeypatching

My Fake Mock Class如下所示:

class FakeResponse:
    method = None               # 
    url = None                  # static class variables

    def __init__(self, method, url, data):#, response):
        self.status_code = 200       # always return 200 OK
        FakeResponse.method = method # 
        FakeResponse.url = url       #

    @staticmethod
    def check(method, url, values):
       """ checks method and URL.      
       """
       print "url fake: ", FakeResponse.url
       assert FakeResponse.method == method
       assert FakeResponse.url == url

我有另一个适用于所有测试用例的装饰器:

@pytest.fixture(autouse=True)
def no_requests(monkeypatch):
    monkeypatch.setattr('haas.cli.do_put',
                    lambda url,data: FakeResponse('PUT', url, data))
    monkeypatch.setattr("haas.cli.do_post",
                    lambda url,data: FakeResponse('POST', url, data))
    monkeypatch.setattr("haas.cli.do_delete",
                    lambda url: FakeResponse('DELETE', url, None))

我正在使用Py.test来测试代码。

一些示例测试用例是:

class Test:
    #test case passes
    def test_node_connect_network(self):
       cli.node_connect_network('node-99','eth0','hammernet')
       FakeResponse.check('POST','http://abc:5000/node/node-99/nic/eth0/connect_network',
                        {'network':'hammernet'})

    # test case fails
    def test_port_register(self):
        cli.port_register('1') # This make a indirect REST call to the original API
        FakeResponse.check('PUT','http://abc:5000/port/1', None)

    # test case fails
    def test_port_delete(self):
        cli.port_delete('port', 1)
        FakeResponse.check('DELETE','http://abc:5000/port/1', None)  

我收到的示例错误消息:

    method = 'PUT', url = 'http://abc:5000/port/1', values = None

    @staticmethod
    def check(method, url, values):
        """ checks method and URL.
           'values': if None, verifies no data was sent.
           if list of (name,value) pairs, verifies that each pair is in 'values'
        """
        print "url fake: ", FakeResponse.url
>       assert FakeResponse.method == method
E       assert 'POST' == 'PUT'
E         - POST
E         + PUT
haas/tests/unit/cli_v1.py:54: AssertionError
--------------------------------------------- Captured stdout call -------------------------------------
port_register <port>
Register a <port> on a switch
url fake:  http://abc:5000/node/node-99/nic/eth0/connect_network
--------------------------------------------- Captured stderr call -------------------------------------
Wrong number of arguements.  Usage: 

然而,如果我考虑到以下列方式调用第二个测试用例 check函数采用“self”参数,并且不使用@staticmethod,然后测试用例起作用:

def test_port_register(self):
    cli.port_register('1')
    fp = FakeResponse('PUT','http://abc:5000/port/1', None) #Create a FakeResponse class instance
    fp.check('PUT','http://abc:5000/port/1', None) # Just call the check function with the same                                                          
                                                     arguments

问题:

  • 使用猴子补丁和@staticmethod
  • 是否有任何副作用
  • 如何在下一个函数调用中使用为先前测试函数定义的url。
  • 不应该有一个论点范围来禁止上述不必要的行为。
  • 有没有更好的方法来修补猴子。

很抱歉这篇长篇文章,我一直试图解决这个问题一个星期,并希望有一些观点  其他程序员。

1 个答案:

答案 0 :(得分:2)

问题是其中一个功能没有正确的签名。通过将传递给MonkeyPatch函数的参数更改为空字典{}而不是“无”值来解决它,这是我的代码特有的。

我最初遇到问题的原因是,当当前函数的调用(cli.port_register)在传递给port_register本身的参数失败时,所以它从先前的状态中获取参数值并执行断言使用FakeResponse电话。