Review: Needs Fixing

63      + assert default_value is False, u'The default value defined has not 
been returned'

The general consensus among TDD practitioners is that the failure message 
contains "... should ..." wording. In other words, it tells the developer what 
value it was expecting but didn't get.

For example:

    assert default_Value is False, u'The default value should be False'
-- 
https://code.launchpad.net/~trb143/openlp/settings_migration/+merge/148249
Your team OpenLP Core is subscribed to branch lp:openlp.

_______________________________________________
Mailing list: https://launchpad.net/~openlp-core
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp

Reply via email to