CliffM added the comment: Sorry -- I could have been clearer :
The conditional: if member.value == value: Is redundant as the tests stand. If you comment it out -- everything works. So therefore we are missing a test. The current test works, as red is the first value to pop out of the value() list. This makes the if-clause fragile for future maintenance. So we need another test to ensure the loop is covered. It's a coverage issue -- where although the code is executed by the test, and the code is correct, the test is not complete enough for the code. ---------- _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue19252> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com