From c451a8eafa71d7c3ed67b393551f7e20159188f8 Mon Sep 17 00:00:00 2001
From: Andreas Preikschat <andreaspreikschat@openlp.org>
Date: Mon, 31 Mar 2014 19:01:09 +0200
Subject: [PATCH] added test

---
 openlp/core/common/registry.py                     |  1 +
 scripts/jenkins_script.py                          |  2 +-
 .../functional/openlp_core_common/test_registry.py | 14 ++++++++++++++
 3 files changed, 16 insertions(+), 1 deletion(-)

diff --git a/openlp/core/common/registry.py b/openlp/core/common/registry.py
index 014a534f7..33eb64880 100644
--- a/openlp/core/common/registry.py
+++ b/openlp/core/common/registry.py
@@ -151,6 +151,7 @@ class Registry(object):
                     if result:
                         results.append(result)
                 except TypeError:
+                    print("sdf")
                     # Who has called me can help in debugging
                     trace_error_handler(log)
                     log.exception('Exception for function %s', function)
diff --git a/scripts/jenkins_script.py b/scripts/jenkins_script.py
index 1a655799b..0e29271a2 100644
--- a/scripts/jenkins_script.py
+++ b/scripts/jenkins_script.py
@@ -30,7 +30,7 @@
 """
 This script helps to trigger builds of branches. To use it you have to install the jenkins-webapi package:
 
-    pip3 isntall jenkins-webapi
+    pip3 install jenkins-webapi
 
 You probably want to create an alias. Add this to your ~/.bashrc file and then logout and login (to apply the alias):
 
diff --git a/tests/functional/openlp_core_common/test_registry.py b/tests/functional/openlp_core_common/test_registry.py
index a57d7ea85..e27b69d10 100644
--- a/tests/functional/openlp_core_common/test_registry.py
+++ b/tests/functional/openlp_core_common/test_registry.py
@@ -100,6 +100,20 @@ class TestRegistry(TestCase):
         # THEN: I expect then function to have been called and a return given
         self.assertEqual(return_value[0], 'function_2', 'A return value is provided and matches')
 
+    def remove_function_test(self):
+        """
+        Test the remove_function() method
+        """
+        # GIVEN: An existing registry register a function
+        Registry.create()
+        Registry().register_function('test1', self.dummy_function_1)
+
+        # WHEN: Remove the function.
+        Registry().remove_function('test1', self.dummy_function_1)
+
+        # THEN: The method should not be available.
+        assert not Registry().functions_list['test1'], 'The function should not be in the dict anymore.'
+
     def dummy_function_1(self):
         return "function_1"