Message199853
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. |
|
Date |
User |
Action |
Args |
2013-10-14 08:53:27 | CliffM | set | recipients:
+ CliffM, ethan.furman |
2013-10-14 08:53:27 | CliffM | set | messageid: <1381740807.48.0.594318314007.issue19252@psf.upfronthosting.co.za> |
2013-10-14 08:53:27 | CliffM | link | issue19252 messages |
2013-10-14 08:53:27 | CliffM | create | |
|