Merge lp:~raoul-snyman/openlp/bug-1668669 into lp:openlp

[Raoul Snyman]

Fix bug #1668669 by reducing the height of the settings window

Add this to your merge proposal:
--------------------------------
lp:~raoul-snyman/openlp/bug-1668669 (revision 2724)
[SUCCESS] https://ci.openlp.io/job/Branch-01-Pull/1909/
[SUCCESS] https://ci.openlp.io/job/Branch-02-Functional-Tests/1820/
[SUCCESS] https://ci.openlp.io/job/Branch-03-Interface-Tests/1760/
[SUCCESS] https://ci.openlp.io/job/Branch-04a-Windows_Functional_Tests/1494/
[SUCCESS] https://ci.openlp.io/job/Branch-04b-Windows_Interface_Tests/1084/
[SUCCESS] https://ci.openlp.io/job/Branch-05a-Code_Analysis/1152/
[SUCCESS] https://ci.openlp.io/job/Branch-05b-Test_Coverage/1020/
[SUCCESS] https://ci.openlp.io/job/Branch-05c-Code_Analysis2/162/

[Tim Bentley (trb143)]

review: Approve

=== modified file 'openlp/core/ui/settingsdialog.py'
--- openlp/core/ui/settingsdialog.py	2016-12-31 11:01:36 +0000
+++ openlp/core/ui/settingsdialog.py	2017-03-02 00:27:23 +0000
@@ -39,7 +39,7 @@
         """
         settings_dialog.setObjectName('settings_dialog')
         settings_dialog.setWindowIcon(build_icon(':/icon/openlp-logo.svg'))
-        settings_dialog.resize(800, 700)
+        settings_dialog.resize(920, 625)
         self.dialog_layout = QtWidgets.QGridLayout(settings_dialog)
         self.dialog_layout.setObjectName('dialog_layout')
         self.dialog_layout.setContentsMargins(8, 8, 8, 8)

=== modified file 'tests/functional/openlp_core_ui/test_settingsform.py'
--- tests/functional/openlp_core_ui/test_settingsform.py	2016-12-31 11:01:36 +0000
+++ tests/functional/openlp_core_ui/test_settingsform.py	2017-03-02 00:27:23 +0000
@@ -151,3 +151,17 @@
         # THEN: The general tab's cancel() method should have been called, but not the themes tab
         mocked_general_cancel.assert_called_with()
         self.assertEqual(0, mocked_theme_cancel.call_count, 'The Themes tab\'s cancel() should not have been called')
+
+    def test_register_post_process(self):
+        """
+        Test that the register_post_process() method works correctly
+        """
+        # GIVEN: A settings form instance
+        settings_form = SettingsForm(None)
+        fake_function = MagicMock()
+
+        # WHEN: register_post_process() is called
+        settings_form.register_post_process(fake_function)
+
+        # THEN: The fake function should be in the settings form's list
+        assert fake_function in settings_form.processes