Serhiy Storchaka added the comment: LGTM except that arguments of assertEqual() should be swapped. ---------- stage: patch review -> commit review _______________________________________ Python tracker <report at bugs.python.org> <http://bugs.python.org/issue29099> _______________________________________