openlp-core team mailing list archive

[Raoul Snyman <raoul@xxxxxxxxxxx>]

Review: Needs Fixing

779	         # WHEN: We run hook_settings_tabs()
780	-        plugin_manager.hook_settings_tabs(mocked_settings_form)
781	-
782	-        # THEN: The createMediaManagerItem() method should have been called with the mocked settings form
783	-        mocked_plugin.createSettingsTab.assert_called_with(mocked_settings_form)
784	-        self.assertEqual(mocked_settings_form.plugins, plugin_manager.plugins,
785	-            u'The plugins on the settings form should be the same as the plugins in the plugin manager')
786	+        plugin_manager.hook_settings_tabs()
787	 
788	     def hook_import_menu_with_disabled_plugin_test(self):

I don't see a "THEN"?
-- 
https://code.launchpad.net/~trb143/openlp/bootstrap/+merge/150909
Your team OpenLP Core is subscribed to branch lp:openlp.