Message230269
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. |
|
Date |
User |
Action |
Args |
2014-10-30 13:03:38 | Claudiu.Popa | set | recipients:
+ Claudiu.Popa, donlorenzo, ezio.melotti |
2014-10-30 13:03:38 | Claudiu.Popa | set | messageid: <1414674218.92.0.157774719925.issue22143@psf.upfronthosting.co.za> |
2014-10-30 13:03:38 | Claudiu.Popa | link | issue22143 messages |
2014-10-30 13:03:38 | Claudiu.Popa | create | |
|