Claudiu Popa added the comment:
I don't know why there's no review link for your patch. Anyway..
+ self.assertTrue(c.complete("A.foo", 0) in ['A.foo(', 'A.foobar('])
+ self.assertTrue(c.complete("A.foo", 1) in ['A.foo(', 'A.foobar('])
You can use self.assertIn for these.
+ self.assertEqual(c.complete("b.foo", 1), None)
self.assertIsNone.
Otherwise, the patch looks good to me.
----------
_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue22143>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe:
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
