From 7086c88c7c24bc6dfc9bc36905166783caf689aa Mon Sep 17 00:00:00 2001 From: Raoul Snyman Date: Mon, 9 Oct 2017 18:08:09 -0700 Subject: [PATCH] Fix a test that was failing (not sure why) --- .../interfaces/openlp_core_ui/test_thememanager.py | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/tests/interfaces/openlp_core_ui/test_thememanager.py b/tests/interfaces/openlp_core_ui/test_thememanager.py index 246385656..7f3927cf5 100644 --- a/tests/interfaces/openlp_core_ui/test_thememanager.py +++ b/tests/interfaces/openlp_core_ui/test_thememanager.py @@ -58,20 +58,21 @@ class TestThemeManager(TestCase, TestMixin): Test the thememanager initialise - basic test """ # GIVEN: A new a call to initialise + self.theme_manager.setup_ui = MagicMock() self.theme_manager.build_theme_path = MagicMock() self.theme_manager.load_first_time_themes = MagicMock() + self.theme_manager.upgrade_themes = MagicMock() Settings().setValue('themes/global theme', 'my_theme') # WHEN: the initialisation is run self.theme_manager.bootstrap_initialise() # THEN: - self.assertEqual(1, self.theme_manager.build_theme_path.call_count, - 'The function build_theme_path should have been called') - self.assertEqual(1, self.theme_manager.load_first_time_themes.call_count, - 'The function load_first_time_themes should have been called only once') - self.assertEqual(self.theme_manager.global_theme, 'my_theme', - 'The global theme should have been set to my_theme') + self.theme_manager.setup_ui.assert_called_once_with(self.theme_manager) + assert self.theme_manager.global_theme == 'my_theme' + self.theme_manager.build_theme_path.assert_called_once_with() + self.theme_manager.load_first_time_themes.assert_called_once_with() + self.theme_manager.upgrade_themes.assert_called_once_with() @patch('openlp.core.ui.thememanager.create_paths') @patch('openlp.core.ui.thememanager.AppLocation.get_section_data_path')