Commit 9a8c4908 authored by Martin Pitt's avatar Martin Pitt
Browse files

tests: Re-enable test_callback_scope_call_array() check

Drop the expected failure from test_callback_scope_call_array() and just add
the explicit array length arguments. While it would look cleaner to not pass
them, it is probably not worth breaking the API for this.
parent 60963642
......@@ -675,17 +675,14 @@ class TestCallbacks(unittest.TestCase):
self.assertEqual(TestCallbacks.called, 2)
self.assertEqual(sys.getrefcount(callback), refcount)
# FIXME: TypeError: callback() takes 2 positional arguments but 4 were given
# does not remove the array length arguments
@unittest.expectedFailure
def test_callback_scope_call_array(self):
# This tests a callback that gets called multiple times from a
# single scope call in python with array arguments
TestCallbacks.callargs = []
# works with:
#def callback(one, one_length, two, two_length):
def callback(one, two):
# FIXME: would be cleaner without the explicit length args:
# def callback(one, two):
def callback(one, one_length, two, two_length):
TestCallbacks.callargs.append((one, two))
return len(TestCallbacks.callargs)
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment