diff --git a/tests/functional/openlp_core_lib/test_formattingtags.py b/tests/functional/openlp_core_lib/test_formattingtags.py index 5a95e195f..0d3b8b6b6 100644 --- a/tests/functional/openlp_core_lib/test_formattingtags.py +++ b/tests/functional/openlp_core_lib/test_formattingtags.py @@ -10,6 +10,12 @@ from openlp.core.lib import FormattingTags class TestFormattingTags(TestCase): + def tearDown(self): + """ + Clean up the FormattingTags class. + """ + FormattingTags.html_expands = [] + def get_html_tags_no_user_tags_test(self): """ Test the get_html_tags static method. @@ -18,7 +24,7 @@ class TestFormattingTags(TestCase): patch(u'openlp.core.lib.settings') as mocked_settings, \ patch(u'openlp.core.lib.formattingtags.cPickle') as mocked_cPickle: # GIVEN: Our mocked modules and functions. - mocked_translate.side_effect = lambda module, string_to_translate: string_to_translate + mocked_translate.side_effect = lambda module, string_to_translate, comment: string_to_translate mocked_settings.value.return_value = u'' mocked_cPickle.load.return_value = [] @@ -35,32 +41,32 @@ class TestFormattingTags(TestCase): """ Add a tag and check if it still exists after reloading the tags list. """ - # FIXME: not working yet. with patch(u'openlp.core.lib.translate') as mocked_translate, \ patch(u'openlp.core.lib.settings') as mocked_settings, \ patch(u'openlp.core.lib.formattingtags.cPickle') as mocked_cPickle: # GIVEN: Our mocked modules and functions. mocked_translate.side_effect = lambda module, string_to_translate: string_to_translate mocked_settings.value.return_value = u'' - mocked_cPickle.load.return_value = [] tags = [{ u'end tag': '{/aa}', u'start html': '', u'start tag': '{aa}', u'protected': False, u'end html': '', - u'desc': 'name'} - ] + u'desc': 'name' + }] + mocked_cPickle.load.return_value = tags # WHEN: Get the display tags. - FormattingTags.add_html_tags(tags) FormattingTags.load_tags() old_tags_list = FormattingTags.get_html_tags() + + FormattingTags.add_html_tags(tags) FormattingTags.load_tags() new_tags_list = FormattingTags.get_html_tags() - # THEN: Lists should be identically. - assert old_tags_list == new_tags_list, u'The formatting tag lists with user tags should be identically.' + # THEN: Lists should not be identically. + assert len(old_tags_list) - 1 == len(new_tags_list), u'The lists should be different.'