openlp-core team mailing list archive

[Raoul Snyman <raoul@xxxxxxxxxxx>]

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.