◐ Shell
clean mode source ↗

Message 230269 - Python tracker

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.