diff --git a/tests/functional/openlp_plugins/media/test_mediaplugin.py b/tests/functional/openlp_plugins/media/test_mediaplugin.py index 9bc53dee1..9dbab37cf 100644 --- a/tests/functional/openlp_plugins/media/test_mediaplugin.py +++ b/tests/functional/openlp_plugins/media/test_mediaplugin.py @@ -40,7 +40,7 @@ class MediaPluginTest(TestCase, TestMixin): @patch(u'openlp.plugins.media.mediaplugin.Plugin.initialise') @patch(u'openlp.plugins.media.mediaplugin.Settings') - def initialise_test(self, mocked_settings, mocked_initialise): + def initialise_test(self, _mocked_settings, mocked_initialise): """ Test that the initialise() method overwrites the built-in one, but still calls it """ @@ -48,7 +48,7 @@ class MediaPluginTest(TestCase, TestMixin): media_plugin = MediaPlugin() mocked_settings = MagicMock() mocked_settings.get_files_from_config.return_value = True # Not the real value, just need something "true-ish" - mocked_settings.return_value = mocked_settings + _mocked_settings.return_value = mocked_settings # WHEN: initialise() is called media_plugin.initialise()