raoul/openlp
1
0
Fork 0
forked from openlp/openlp
Code Issues Pull Requests Projects Releases Wiki Activity

review corrections

Browse Source
This commit is contained in:
Tim Bentley 2013-03-12 07:17:14 +00:00
parent 46fe786f8b
commit dcec6ebbac
2 changed files with 4 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
openlp/core/ui/servicemanager.py
View File

@ -1127,9 +1127,7 @@ class ServiceManager(QtGui.QWidget, ServiceManagerDialog):
"""
item = self.find_service_item()[0]
if item != -1:
print self.service_items
self.service_items.remove(self.service_items[item])
print self.service_items
self.repaint_service_list(item - 1, -1)
self.set_modified()

8
tests/interfaces/openlp_core_ui/test_settings_form.py
View File

@ -82,7 +82,7 @@ class TestSettingsForm(TestCase):
QtTest.QTest.mouseClick(okWidget, QtCore.Qt.LeftButton)
# THEN the processing stack should be empty
assert len(self.form.processes) == 0, u'The defined processes have been called'
assert len(self.form.processes) == 0, u'he one requested process should have been added to the stack'
def register_multiple_functions_test(self):
"""
@ -93,18 +93,18 @@ class TestSettingsForm(TestCase):
# WHEN testing the processing stack
# THEN the processing stack should have one item
assert len(self.form.processes) == 1, u'The one defined process is ready to be called'
assert len(self.form.processes) == 1, u'The one requested process should have been added to the stack'
# GIVEN: Registering a new function
self.form.register_post_process(u'function2')
# WHEN testing the processing stack
# THEN the processing stack should have two items
assert len(self.form.processes) == 2, u'The two defined processes is ready to be called'
assert len(self.form.processes) == 2, u'The two requested processes should have been added to the stack'
# GIVEN: Registering a process for the second time
self.form.register_post_process(u'function1')
# WHEN testing the processing stack
# THEN the processing stack should still have two items
assert len(self.form.processes) == 2, u'No new processes have been added to the stack'
assert len(self.form.processes) == 2, u'No new processes should have been added to the stack'