Ezio Melotti added the comment:

You should use assertRaises/assertRaisesRegex to check the error message and 
add more tests to cover all the changes.  I also noticed an inconsistence 
between 'element not in container' and 'element is not in container' (note the 
extra 'is').  FWIW I prefer the version without 'is'.

stage: needs patch -> patch review

Python tracker <rep...@bugs.python.org>
Python-bugs-list mailing list

Reply via email to