forked from openlp/openlp
Merged with trunk and reverted changes to the tag test
This commit is contained in:
commit
29f8eedeaf
@ -96,6 +96,8 @@ class DisplayController(QtGui.QWidget):
|
||||
"""
|
||||
This is the generic function to send signal for control widgets, created from within other plugins
|
||||
This function is needed to catch the current controller
|
||||
|
||||
:param args: Arguments to send to the plugins
|
||||
"""
|
||||
sender = self.sender().objectName() if self.sender().objectName() else self.sender().text()
|
||||
controller = self
|
||||
@ -143,11 +145,19 @@ class SlideController(DisplayController, RegistryProperties):
|
||||
self.panel_layout = QtGui.QVBoxLayout(self.panel)
|
||||
self.panel_layout.setSpacing(0)
|
||||
self.panel_layout.setMargin(0)
|
||||
# Type label for the top of the slide controller
|
||||
# Type label at the top of the slide controller
|
||||
self.type_label = QtGui.QLabel(self.panel)
|
||||
self.type_label.setStyleSheet('font-weight: bold; font-size: 12pt;')
|
||||
self.type_label.setAlignment(QtCore.Qt.AlignCenter)
|
||||
if self.is_live:
|
||||
self.type_label.setText(UiStrings().Live)
|
||||
else:
|
||||
self.type_label.setText(UiStrings().Preview)
|
||||
self.panel_layout.addWidget(self.type_label)
|
||||
# Info label for the title of the current item, at the top of the slide controller
|
||||
self.info_label = QtGui.QLabel(self.panel)
|
||||
self.info_label.setAlignment(QtCore.Qt.AlignCenter)
|
||||
self.panel_layout.addWidget(self.info_label)
|
||||
# Splitter
|
||||
self.splitter = QtGui.QSplitter(self.panel)
|
||||
self.splitter.setOrientation(QtCore.Qt.Vertical)
|
||||
@ -402,12 +412,17 @@ class SlideController(DisplayController, RegistryProperties):
|
||||
"""
|
||||
try:
|
||||
from openlp.plugins.songs.lib import VerseType
|
||||
SONGS_PLUGIN_AVAILABLE = True
|
||||
is_songs_plugin_available = True
|
||||
except ImportError:
|
||||
SONGS_PLUGIN_AVAILABLE = False
|
||||
class VerseType(object):
|
||||
"""
|
||||
This empty class is mostly just to satisfy Python, PEP8 and PyCharm
|
||||
"""
|
||||
pass
|
||||
is_songs_plugin_available = False
|
||||
sender_name = self.sender().objectName()
|
||||
verse_type = sender_name[15:] if sender_name[:15] == 'shortcutAction_' else ''
|
||||
if SONGS_PLUGIN_AVAILABLE:
|
||||
if is_songs_plugin_available:
|
||||
if verse_type == 'V':
|
||||
self.current_shortcut = VerseType.translated_tags[VerseType.Verse]
|
||||
elif verse_type == 'C':
|
||||
@ -777,6 +792,7 @@ class SlideController(DisplayController, RegistryProperties):
|
||||
if service_item.is_command():
|
||||
Registry().execute(
|
||||
'%s_start' % service_item.name.lower(), [self.service_item, self.is_live, self.hide_mode(), slide_no])
|
||||
self.info_label.setText(self.service_item.title)
|
||||
self.slide_list = {}
|
||||
if self.is_live:
|
||||
self.song_menu.menu().clear()
|
||||
|
@ -36,7 +36,7 @@ from PyQt4 import QtCore, QtGui
|
||||
from sqlalchemy.sql import or_
|
||||
|
||||
from openlp.core.common import Registry, AppLocation, Settings, check_directory_exists, UiStrings, translate
|
||||
from openlp.core.lib import MediaManagerItem, ItemCapabilities, PluginStatus, ServiceItem, ServiceItemContext, \
|
||||
from openlp.core.lib import MediaManagerItem, ItemCapabilities, PluginStatus, ServiceItemContext, \
|
||||
check_item_selected, create_separated_list
|
||||
from openlp.core.lib.ui import create_widget_action
|
||||
from openlp.plugins.songs.forms.editsongform import EditSongForm
|
||||
@ -126,6 +126,7 @@ class SongMediaItem(MediaManagerItem):
|
||||
self.update_service_on_edit = Settings().value(self.settings_section + '/update service on edit')
|
||||
self.add_song_from_service = Settings().value(self.settings_section + '/add song from service')
|
||||
self.display_songbook = Settings().value(self.settings_section + '/display songbook')
|
||||
self.display_copyright_symbol = Settings().value(self.settings_section + '/display copyright symbol')
|
||||
|
||||
def retranslateUi(self):
|
||||
self.search_text_label.setText('%s:' % UiStrings().Search)
|
||||
@ -506,7 +507,11 @@ class SongMediaItem(MediaManagerItem):
|
||||
if authors_translation:
|
||||
item.raw_footer.append("%s: %s" % (AuthorType.Types[AuthorType.Translation],
|
||||
create_separated_list(authors_translation)))
|
||||
item.raw_footer.append(song.copyright)
|
||||
if song.copyright:
|
||||
if self.display_copyright_symbol:
|
||||
item.raw_footer.append("%s %s" % (SongStrings.CopyrightSymbol, song.copyright))
|
||||
else:
|
||||
item.raw_footer.append(song.copyright)
|
||||
if self.display_songbook and song.book:
|
||||
item.raw_footer.append("%s #%s" % (song.book.name, song.song_number))
|
||||
if Settings().value('core/ccli number'):
|
||||
|
@ -31,6 +31,7 @@ from PyQt4 import QtCore, QtGui
|
||||
|
||||
from openlp.core.common import Settings, translate
|
||||
from openlp.core.lib import SettingsTab
|
||||
from openlp.plugins.songs.lib.ui import SongStrings
|
||||
|
||||
|
||||
class SongsTab(SettingsTab):
|
||||
@ -62,6 +63,9 @@ class SongsTab(SettingsTab):
|
||||
self.display_songbook_check_box = QtGui.QCheckBox(self.mode_group_box)
|
||||
self.display_songbook_check_box.setObjectName('songbook_check_box')
|
||||
self.mode_layout.addWidget(self.display_songbook_check_box)
|
||||
self.display_copyright_check_box = QtGui.QCheckBox(self.mode_group_box)
|
||||
self.display_copyright_check_box.setObjectName('copyright_check_box')
|
||||
self.mode_layout.addWidget(self.display_copyright_check_box)
|
||||
self.left_layout.addWidget(self.mode_group_box)
|
||||
self.left_layout.addStretch()
|
||||
self.right_layout.addStretch()
|
||||
@ -70,6 +74,7 @@ class SongsTab(SettingsTab):
|
||||
self.update_on_edit_check_box.stateChanged.connect(self.on_update_on_edit_check_box_changed)
|
||||
self.add_from_service_check_box.stateChanged.connect(self.on_add_from_service_check_box_changed)
|
||||
self.display_songbook_check_box.stateChanged.connect(self.on_songbook_check_box_changed)
|
||||
self.display_copyright_check_box.stateChanged.connect(self.on_copyright_check_box_changed)
|
||||
|
||||
def retranslateUi(self):
|
||||
self.mode_group_box.setTitle(translate('SongsPlugin.SongsTab', 'Songs Mode'))
|
||||
@ -80,6 +85,9 @@ class SongsTab(SettingsTab):
|
||||
self.add_from_service_check_box.setText(translate('SongsPlugin.SongsTab',
|
||||
'Import missing songs from service files'))
|
||||
self.display_songbook_check_box.setText(translate('SongsPlugin.SongsTab', 'Display songbook in footer'))
|
||||
self.display_copyright_check_box.setText(translate('SongsPlugin.SongsTab',
|
||||
'Display "%s" symbol before copyright info' %
|
||||
SongStrings.CopyrightSymbol))
|
||||
|
||||
def on_search_as_type_check_box_changed(self, check_state):
|
||||
self.song_search = (check_state == QtCore.Qt.Checked)
|
||||
@ -96,6 +104,9 @@ class SongsTab(SettingsTab):
|
||||
def on_songbook_check_box_changed(self, check_state):
|
||||
self.display_songbook = (check_state == QtCore.Qt.Checked)
|
||||
|
||||
def on_copyright_check_box_changed(self, check_state):
|
||||
self.display_copyright_symbol = (check_state == QtCore.Qt.Checked)
|
||||
|
||||
def load(self):
|
||||
settings = Settings()
|
||||
settings.beginGroup(self.settings_section)
|
||||
@ -104,11 +115,13 @@ class SongsTab(SettingsTab):
|
||||
self.update_edit = settings.value('update service on edit')
|
||||
self.update_load = settings.value('add song from service')
|
||||
self.display_songbook = settings.value('display songbook')
|
||||
self.display_copyright_symbol = settings.value('display copyright symbol')
|
||||
self.search_as_type_check_box.setChecked(self.song_search)
|
||||
self.tool_bar_active_check_box.setChecked(self.tool_bar)
|
||||
self.update_on_edit_check_box.setChecked(self.update_edit)
|
||||
self.add_from_service_check_box.setChecked(self.update_load)
|
||||
self.display_songbook_check_box.setChecked(self.display_songbook)
|
||||
self.display_copyright_check_box.setChecked(self.display_copyright_symbol)
|
||||
settings.endGroup()
|
||||
|
||||
def save(self):
|
||||
@ -119,6 +132,7 @@ class SongsTab(SettingsTab):
|
||||
settings.setValue('update service on edit', self.update_edit)
|
||||
settings.setValue('add song from service', self.update_load)
|
||||
settings.setValue('display songbook', self.display_songbook)
|
||||
settings.setValue('display copyright symbol', self.display_copyright_symbol)
|
||||
settings.endGroup()
|
||||
if self.tab_visited:
|
||||
self.settings_form.register_post_process('songs_config_updated')
|
||||
|
@ -64,6 +64,7 @@ __default_settings__ = {
|
||||
'songs/add song from service': True,
|
||||
'songs/display songbar': True,
|
||||
'songs/display songbook': False,
|
||||
'songs/display copyright symbol': False,
|
||||
'songs/last directory import': '',
|
||||
'songs/last directory export': '',
|
||||
'songs/songselect username': '',
|
||||
|
@ -30,10 +30,13 @@
|
||||
Package to test the openlp.core.ui.slidecontroller package.
|
||||
"""
|
||||
from unittest import TestCase
|
||||
from openlp.core import Registry
|
||||
from openlp.core.lib import ServiceItemAction
|
||||
|
||||
from openlp.core.ui import SlideController
|
||||
from openlp.core.ui.slidecontroller import WIDE_MENU, NON_TEXT_MENU
|
||||
|
||||
from tests.interfaces import MagicMock
|
||||
from tests.interfaces import MagicMock, patch
|
||||
|
||||
|
||||
class TestSlideController(TestCase):
|
||||
@ -42,37 +45,514 @@ class TestSlideController(TestCase):
|
||||
"""
|
||||
Test the initial slide controller state .
|
||||
"""
|
||||
# GIVEN: A new slideController instance.
|
||||
# GIVEN: A new SlideController instance.
|
||||
slide_controller = SlideController(None)
|
||||
|
||||
# WHEN: the default controller is built.
|
||||
# THEN: The controller should not be a live controller.
|
||||
self.assertEqual(slide_controller.is_live, False, 'The base slide controller should not be a live controller')
|
||||
|
||||
def toggle_blank_test(self):
|
||||
def text_service_item_blank_test(self):
|
||||
"""
|
||||
Test the setting of the display blank icons by display type.
|
||||
Test that loading a text-based service item into the slide controller sets the correct blank menu
|
||||
"""
|
||||
# GIVEN: A new slideController instance.
|
||||
# GIVEN: A new SlideController instance.
|
||||
slide_controller = SlideController(None)
|
||||
service_item = MagicMock()
|
||||
toolbar = MagicMock()
|
||||
toolbar.set_widget_visible = self.dummy_widget_visible
|
||||
toolbar.set_widget_visible = MagicMock()
|
||||
slide_controller.toolbar = toolbar
|
||||
slide_controller.service_item = service_item
|
||||
|
||||
# WHEN a text based service item is used
|
||||
# WHEN: a text based service item is used
|
||||
slide_controller.service_item.is_text = MagicMock(return_value=True)
|
||||
slide_controller.set_blank_menu()
|
||||
|
||||
# THEN: then call set up the toolbar to blank the display screen.
|
||||
self.assertEqual(len(self.test_widget), 3, 'There should be three icons to display on the screen')
|
||||
# THEN: the call to set the visible items on the toolbar should be correct
|
||||
toolbar.set_widget_visible.assert_called_with(WIDE_MENU, True)
|
||||
|
||||
def non_text_service_item_blank_test(self):
|
||||
"""
|
||||
Test that loading a non-text service item into the slide controller sets the correct blank menu
|
||||
"""
|
||||
# GIVEN: A new SlideController instance.
|
||||
slide_controller = SlideController(None)
|
||||
service_item = MagicMock()
|
||||
toolbar = MagicMock()
|
||||
toolbar.set_widget_visible = MagicMock()
|
||||
slide_controller.toolbar = toolbar
|
||||
slide_controller.service_item = service_item
|
||||
|
||||
# WHEN a non text based service item is used
|
||||
slide_controller.service_item.is_text = MagicMock(return_value=False)
|
||||
slide_controller.set_blank_menu()
|
||||
|
||||
# THEN: then call set up the toolbar to blank the display screen.
|
||||
self.assertEqual(len(self.test_widget), 2, 'There should be only two icons to display on the screen')
|
||||
toolbar.set_widget_visible.assert_called_with(NON_TEXT_MENU, True)
|
||||
|
||||
def dummy_widget_visible(self, widget, visible=True):
|
||||
self.test_widget = widget
|
||||
def receive_spin_delay_test(self):
|
||||
"""
|
||||
Test that the spin box is updated accordingly after a call to receive_spin_delay()
|
||||
"""
|
||||
with patch('openlp.core.ui.slidecontroller.Settings') as MockedSettings:
|
||||
# GIVEN: A new SlideController instance.
|
||||
mocked_value = MagicMock(return_value=1)
|
||||
MockedSettings.return_value = MagicMock(value=mocked_value)
|
||||
mocked_delay_spin_box = MagicMock()
|
||||
slide_controller = SlideController(None)
|
||||
slide_controller.delay_spin_box = mocked_delay_spin_box
|
||||
|
||||
# WHEN: The receive_spin_delay() method is called
|
||||
slide_controller.receive_spin_delay()
|
||||
|
||||
# THEN: The Settings()value() and delay_spin_box.setValue() methods should have been called correctly
|
||||
mocked_value.assert_called_with('core/loop delay')
|
||||
mocked_delay_spin_box.setValue.assert_called_with(1)
|
||||
|
||||
def toggle_display_blank_test(self):
|
||||
"""
|
||||
Check that the toggle_display('blank') method calls the on_blank_display() method
|
||||
"""
|
||||
# GIVEN: A new SlideController instance.
|
||||
mocked_on_blank_display = MagicMock()
|
||||
mocked_on_theme_display = MagicMock()
|
||||
mocked_on_hide_display = MagicMock()
|
||||
slide_controller = SlideController(None)
|
||||
slide_controller.on_blank_display = mocked_on_blank_display
|
||||
slide_controller.on_theme_display = mocked_on_theme_display
|
||||
slide_controller.on_hide_display = mocked_on_hide_display
|
||||
|
||||
# WHEN: toggle_display() is called with an argument of "blank"
|
||||
slide_controller.toggle_display('blank')
|
||||
|
||||
# THEN: Only on_blank_display() should have been called with an argument of True
|
||||
mocked_on_blank_display.assert_called_once_with(True)
|
||||
self.assertEqual(0, mocked_on_theme_display.call_count, 'on_theme_display should not have been called')
|
||||
self.assertEqual(0, mocked_on_hide_display.call_count, 'on_hide_display should not have been called')
|
||||
|
||||
def toggle_display_hide_test(self):
|
||||
"""
|
||||
Check that the toggle_display('hide') method calls the on_blank_display() method
|
||||
"""
|
||||
# GIVEN: A new SlideController instance.
|
||||
mocked_on_blank_display = MagicMock()
|
||||
mocked_on_theme_display = MagicMock()
|
||||
mocked_on_hide_display = MagicMock()
|
||||
slide_controller = SlideController(None)
|
||||
slide_controller.on_blank_display = mocked_on_blank_display
|
||||
slide_controller.on_theme_display = mocked_on_theme_display
|
||||
slide_controller.on_hide_display = mocked_on_hide_display
|
||||
|
||||
# WHEN: toggle_display() is called with an argument of "hide"
|
||||
slide_controller.toggle_display('hide')
|
||||
|
||||
# THEN: Only on_blank_display() should have been called with an argument of True
|
||||
mocked_on_blank_display.assert_called_once_with(True)
|
||||
self.assertEqual(0, mocked_on_theme_display.call_count, 'on_theme_display should not have been called')
|
||||
self.assertEqual(0, mocked_on_hide_display.call_count, 'on_hide_display should not have been called')
|
||||
|
||||
def toggle_display_theme_test(self):
|
||||
"""
|
||||
Check that the toggle_display('theme') method calls the on_theme_display() method
|
||||
"""
|
||||
# GIVEN: A new SlideController instance.
|
||||
mocked_on_blank_display = MagicMock()
|
||||
mocked_on_theme_display = MagicMock()
|
||||
mocked_on_hide_display = MagicMock()
|
||||
slide_controller = SlideController(None)
|
||||
slide_controller.on_blank_display = mocked_on_blank_display
|
||||
slide_controller.on_theme_display = mocked_on_theme_display
|
||||
slide_controller.on_hide_display = mocked_on_hide_display
|
||||
|
||||
# WHEN: toggle_display() is called with an argument of "theme"
|
||||
slide_controller.toggle_display('theme')
|
||||
|
||||
# THEN: Only on_theme_display() should have been called with an argument of True
|
||||
mocked_on_theme_display.assert_called_once_with(True)
|
||||
self.assertEqual(0, mocked_on_blank_display.call_count, 'on_blank_display should not have been called')
|
||||
self.assertEqual(0, mocked_on_hide_display.call_count, 'on_hide_display should not have been called')
|
||||
|
||||
def toggle_display_desktop_test(self):
|
||||
"""
|
||||
Check that the toggle_display('desktop') method calls the on_hide_display() method
|
||||
"""
|
||||
# GIVEN: A new SlideController instance.
|
||||
mocked_on_blank_display = MagicMock()
|
||||
mocked_on_theme_display = MagicMock()
|
||||
mocked_on_hide_display = MagicMock()
|
||||
slide_controller = SlideController(None)
|
||||
slide_controller.on_blank_display = mocked_on_blank_display
|
||||
slide_controller.on_theme_display = mocked_on_theme_display
|
||||
slide_controller.on_hide_display = mocked_on_hide_display
|
||||
|
||||
# WHEN: toggle_display() is called with an argument of "desktop"
|
||||
slide_controller.toggle_display('desktop')
|
||||
|
||||
# THEN: Only on_hide_display() should have been called with an argument of True
|
||||
mocked_on_hide_display.assert_called_once_with(True)
|
||||
self.assertEqual(0, mocked_on_blank_display.call_count, 'on_blank_display should not have been called')
|
||||
self.assertEqual(0, mocked_on_theme_display.call_count, 'on_theme_display should not have been called')
|
||||
|
||||
def toggle_display_show_test(self):
|
||||
"""
|
||||
Check that the toggle_display('show') method calls all the on_X_display() methods
|
||||
"""
|
||||
# GIVEN: A new SlideController instance.
|
||||
mocked_on_blank_display = MagicMock()
|
||||
mocked_on_theme_display = MagicMock()
|
||||
mocked_on_hide_display = MagicMock()
|
||||
slide_controller = SlideController(None)
|
||||
slide_controller.on_blank_display = mocked_on_blank_display
|
||||
slide_controller.on_theme_display = mocked_on_theme_display
|
||||
slide_controller.on_hide_display = mocked_on_hide_display
|
||||
|
||||
# WHEN: toggle_display() is called with an argument of "show"
|
||||
slide_controller.toggle_display('show')
|
||||
|
||||
# THEN: All the on_X_display() methods should have been called with an argument of False
|
||||
mocked_on_blank_display.assert_called_once_with(False)
|
||||
mocked_on_theme_display.assert_called_once_with(False)
|
||||
mocked_on_hide_display.assert_called_once_with(False)
|
||||
|
||||
def live_escape_test(self):
|
||||
"""
|
||||
Test that when the live_escape() method is called, the display is set to invisible and any media is stopped
|
||||
"""
|
||||
# GIVEN: A new SlideController instance and mocked out display and media_controller
|
||||
mocked_display = MagicMock()
|
||||
mocked_media_controller = MagicMock()
|
||||
Registry.create()
|
||||
Registry().register('media_controller', mocked_media_controller)
|
||||
slide_controller = SlideController(None)
|
||||
slide_controller.display = mocked_display
|
||||
|
||||
# WHEN: live_escape() is called
|
||||
slide_controller.live_escape()
|
||||
|
||||
# THEN: the display should be set to invisible and the media controller stopped
|
||||
mocked_display.setVisible.assert_called_once_with(False)
|
||||
mocked_media_controller.media_stop.assert_called_once_with(slide_controller)
|
||||
|
||||
def service_previous_test(self):
|
||||
"""
|
||||
Check that calling the service_previous() method adds the previous key to the queue and processes the queue
|
||||
"""
|
||||
# GIVEN: A new SlideController instance.
|
||||
mocked_keypress_queue = MagicMock()
|
||||
mocked_process_queue = MagicMock()
|
||||
slide_controller = SlideController(None)
|
||||
slide_controller.keypress_queue = mocked_keypress_queue
|
||||
slide_controller._process_queue = mocked_process_queue
|
||||
|
||||
# WHEN: The service_previous() method is called
|
||||
slide_controller.service_previous()
|
||||
|
||||
# THEN: The keypress is added to the queue and the queue is processed
|
||||
mocked_keypress_queue.append.assert_called_once_with(ServiceItemAction.Previous)
|
||||
mocked_process_queue.assert_called_once_with()
|
||||
|
||||
def service_next_test(self):
|
||||
"""
|
||||
Check that calling the service_next() method adds the next key to the queue and processes the queue
|
||||
"""
|
||||
# GIVEN: A new SlideController instance and mocked out methods
|
||||
mocked_keypress_queue = MagicMock()
|
||||
mocked_process_queue = MagicMock()
|
||||
slide_controller = SlideController(None)
|
||||
slide_controller.keypress_queue = mocked_keypress_queue
|
||||
slide_controller._process_queue = mocked_process_queue
|
||||
|
||||
# WHEN: The service_next() method is called
|
||||
slide_controller.service_next()
|
||||
|
||||
# THEN: The keypress is added to the queue and the queue is processed
|
||||
mocked_keypress_queue.append.assert_called_once_with(ServiceItemAction.Next)
|
||||
mocked_process_queue.assert_called_once_with()
|
||||
|
||||
def update_slide_limits_test(self):
|
||||
"""
|
||||
Test that calling the update_slide_limits() method updates the slide limits
|
||||
"""
|
||||
# GIVEN: A mocked out Settings object, a new SlideController and a mocked out main_window
|
||||
with patch('openlp.core.ui.slidecontroller.Settings') as MockedSettings:
|
||||
mocked_value = MagicMock(return_value=10)
|
||||
MockedSettings.return_value = MagicMock(value=mocked_value)
|
||||
mocked_main_window = MagicMock(advanced_settings_section='advanced')
|
||||
Registry.create()
|
||||
Registry().register('main_window', mocked_main_window)
|
||||
slide_controller = SlideController(None)
|
||||
|
||||
# WHEN: update_slide_limits() is called
|
||||
slide_controller.update_slide_limits()
|
||||
|
||||
# THEN: The value of slide_limits should be 10
|
||||
mocked_value.assert_called_once_with('advanced/slide limits')
|
||||
self.assertEqual(10, slide_controller.slide_limits, 'Slide limits should have been updated to 10')
|
||||
|
||||
def enable_tool_bar_live_test(self):
|
||||
"""
|
||||
Check that when enable_tool_bar on a live slide controller is called, enable_live_tool_bar is called
|
||||
"""
|
||||
# GIVEN: Mocked out enable methods and a real slide controller which is set to live
|
||||
mocked_enable_live_tool_bar = MagicMock()
|
||||
mocked_enable_preview_tool_bar = MagicMock()
|
||||
slide_controller = SlideController(None)
|
||||
slide_controller.is_live = True
|
||||
slide_controller.enable_live_tool_bar = mocked_enable_live_tool_bar
|
||||
slide_controller.enable_preview_tool_bar = mocked_enable_preview_tool_bar
|
||||
mocked_service_item = MagicMock()
|
||||
|
||||
# WHEN: enable_tool_bar() is called
|
||||
slide_controller.enable_tool_bar(mocked_service_item)
|
||||
|
||||
# THEN: The enable_live_tool_bar() method is called, not enable_preview_tool_bar()
|
||||
mocked_enable_live_tool_bar.assert_called_once_with(mocked_service_item)
|
||||
self.assertEqual(0, mocked_enable_preview_tool_bar.call_count, 'The preview method should not have been called')
|
||||
|
||||
def enable_tool_bar_preview_test(self):
|
||||
"""
|
||||
Check that when enable_tool_bar on a preview slide controller is called, enable_preview_tool_bar is called
|
||||
"""
|
||||
# GIVEN: Mocked out enable methods and a real slide controller which is set to live
|
||||
mocked_enable_live_tool_bar = MagicMock()
|
||||
mocked_enable_preview_tool_bar = MagicMock()
|
||||
slide_controller = SlideController(None)
|
||||
slide_controller.is_live = False
|
||||
slide_controller.enable_live_tool_bar = mocked_enable_live_tool_bar
|
||||
slide_controller.enable_preview_tool_bar = mocked_enable_preview_tool_bar
|
||||
mocked_service_item = MagicMock()
|
||||
|
||||
# WHEN: enable_tool_bar() is called
|
||||
slide_controller.enable_tool_bar(mocked_service_item)
|
||||
|
||||
# THEN: The enable_preview_tool_bar() method is called, not enable_live_tool_bar()
|
||||
mocked_enable_preview_tool_bar.assert_called_once_with(mocked_service_item)
|
||||
self.assertEqual(0, mocked_enable_live_tool_bar.call_count, 'The live method should not have been called')
|
||||
|
||||
def refresh_service_item_text_test(self):
|
||||
"""
|
||||
Test that the refresh_service_item() method refreshes a text service item
|
||||
"""
|
||||
# GIVEN: A mock service item and a fresh slide controller
|
||||
mocked_service_item = MagicMock()
|
||||
mocked_service_item.is_text.return_value = True
|
||||
mocked_service_item.is_image.return_value = False
|
||||
mocked_process_item = MagicMock()
|
||||
slide_controller = SlideController(None)
|
||||
slide_controller.service_item = mocked_service_item
|
||||
slide_controller._process_item = mocked_process_item
|
||||
slide_controller.selected_row = 5
|
||||
|
||||
# WHEN: The refresh_service_item method() is called
|
||||
slide_controller.refresh_service_item()
|
||||
|
||||
# THEN: The item should be re-processed
|
||||
mocked_service_item.is_text.assert_called_once_with()
|
||||
self.assertEqual(0, mocked_service_item.is_image.call_count, 'is_image should not have been called')
|
||||
mocked_service_item.render.assert_called_once_with()
|
||||
mocked_process_item.assert_called_once_with(mocked_service_item, 5)
|
||||
|
||||
def refresh_service_item_image_test(self):
|
||||
"""
|
||||
Test that the refresh_service_item() method refreshes a image service item
|
||||
"""
|
||||
# GIVEN: A mock service item and a fresh slide controller
|
||||
mocked_service_item = MagicMock()
|
||||
mocked_service_item.is_text.return_value = False
|
||||
mocked_service_item.is_image.return_value = True
|
||||
mocked_process_item = MagicMock()
|
||||
slide_controller = SlideController(None)
|
||||
slide_controller.service_item = mocked_service_item
|
||||
slide_controller._process_item = mocked_process_item
|
||||
slide_controller.selected_row = 5
|
||||
|
||||
# WHEN: The refresh_service_item method() is called
|
||||
slide_controller.refresh_service_item()
|
||||
|
||||
# THEN: The item should be re-processed
|
||||
mocked_service_item.is_text.assert_called_once_with()
|
||||
mocked_service_item.is_image.assert_called_once_with()
|
||||
mocked_service_item.render.assert_called_once_with()
|
||||
mocked_process_item.assert_called_once_with(mocked_service_item, 5)
|
||||
|
||||
def refresh_service_item_not_image_or_text_test(self):
|
||||
"""
|
||||
Test that the refresh_service_item() method does not refresh a service item if it's neither text or an image
|
||||
"""
|
||||
# GIVEN: A mock service item and a fresh slide controller
|
||||
mocked_service_item = MagicMock()
|
||||
mocked_service_item.is_text.return_value = False
|
||||
mocked_service_item.is_image.return_value = False
|
||||
mocked_process_item = MagicMock()
|
||||
slide_controller = SlideController(None)
|
||||
slide_controller.service_item = mocked_service_item
|
||||
slide_controller._process_item = mocked_process_item
|
||||
slide_controller.selected_row = 5
|
||||
|
||||
# WHEN: The refresh_service_item method() is called
|
||||
slide_controller.refresh_service_item()
|
||||
|
||||
# THEN: The item should be re-processed
|
||||
mocked_service_item.is_text.assert_called_once_with()
|
||||
mocked_service_item.is_image.assert_called_once_with()
|
||||
self.assertEqual(0, mocked_service_item.render.call_count, 'The render() method should not have been called')
|
||||
self.assertEqual(0, mocked_process_item.call_count,
|
||||
'The mocked_process_item() method should not have been called')
|
||||
|
||||
def add_service_item_with_song_edit_test(self):
|
||||
"""
|
||||
Test the add_service_item() method when song_edit is True
|
||||
"""
|
||||
# GIVEN: A slide controller and a new item to add
|
||||
mocked_item = MagicMock()
|
||||
mocked_process_item = MagicMock()
|
||||
slide_controller = SlideController(None)
|
||||
slide_controller._process_item = mocked_process_item
|
||||
slide_controller.song_edit = True
|
||||
slide_controller.selected_row = 2
|
||||
|
||||
# WHEN: The item is added to the service
|
||||
slide_controller.add_service_item(mocked_item)
|
||||
|
||||
# THEN: The item is processed, the slide number is correct, and the song is not editable (or something)
|
||||
mocked_item.render.assert_called_once_with()
|
||||
self.assertFalse(slide_controller.song_edit, 'song_edit should be False')
|
||||
mocked_process_item.assert_called_once_with(mocked_item, 2)
|
||||
|
||||
def add_service_item_without_song_edit_test(self):
|
||||
"""
|
||||
Test the add_service_item() method when song_edit is False
|
||||
"""
|
||||
# GIVEN: A slide controller and a new item to add
|
||||
mocked_item = MagicMock()
|
||||
mocked_process_item = MagicMock()
|
||||
slide_controller = SlideController(None)
|
||||
slide_controller._process_item = mocked_process_item
|
||||
slide_controller.song_edit = False
|
||||
slide_controller.selected_row = 2
|
||||
|
||||
# WHEN: The item is added to the service
|
||||
slide_controller.add_service_item(mocked_item)
|
||||
|
||||
# THEN: The item is processed, the slide number is correct, and the song is not editable (or something)
|
||||
mocked_item.render.assert_called_once_with()
|
||||
self.assertFalse(slide_controller.song_edit, 'song_edit should be False')
|
||||
mocked_process_item.assert_called_once_with(mocked_item, 0)
|
||||
|
||||
def replace_service_manager_item_different_items_test(self):
|
||||
"""
|
||||
Test that when the service items are not the same, nothing happens
|
||||
"""
|
||||
# GIVEN: A slide controller and a new item to add
|
||||
mocked_item = MagicMock()
|
||||
mocked_preview_widget = MagicMock()
|
||||
mocked_process_item = MagicMock()
|
||||
slide_controller = SlideController(None)
|
||||
slide_controller.preview_widget = mocked_preview_widget
|
||||
slide_controller._process_item = mocked_process_item
|
||||
slide_controller.service_item = None
|
||||
|
||||
# WHEN: The service item is replaced
|
||||
slide_controller.replace_service_manager_item(mocked_item)
|
||||
|
||||
# THEN: The service item should not be processed
|
||||
self.assertEqual(0, mocked_process_item.call_count, 'The _process_item() method should not have been called')
|
||||
self.assertEqual(0, mocked_preview_widget.current_slide_number.call_count,
|
||||
'The preview_widgetcurrent_slide_number.() method should not have been called')
|
||||
|
||||
def replace_service_manager_item_same_item_test(self):
|
||||
"""
|
||||
Test that when the service item is the same, the service item is reprocessed
|
||||
"""
|
||||
# GIVEN: A slide controller and a new item to add
|
||||
mocked_item = MagicMock()
|
||||
mocked_preview_widget = MagicMock()
|
||||
mocked_preview_widget.current_slide_number.return_value = 7
|
||||
mocked_process_item = MagicMock()
|
||||
slide_controller = SlideController(None)
|
||||
slide_controller.preview_widget = mocked_preview_widget
|
||||
slide_controller._process_item = mocked_process_item
|
||||
slide_controller.service_item = mocked_item
|
||||
|
||||
# WHEN: The service item is replaced
|
||||
slide_controller.replace_service_manager_item(mocked_item)
|
||||
|
||||
# THEN: The service item should not be processed
|
||||
mocked_preview_widget.current_slide_number.assert_called_with()
|
||||
mocked_process_item.assert_called_once_with(mocked_item, 7)
|
||||
|
||||
def on_slide_selected_index_no_service_item_test(self):
|
||||
"""
|
||||
Test that when there is no service item, the on_slide_selected_index() method returns immediately
|
||||
"""
|
||||
# GIVEN: A mocked service item and a slide controller without a service item
|
||||
mocked_item = MagicMock()
|
||||
slide_controller = SlideController(None)
|
||||
slide_controller.service_item = None
|
||||
|
||||
# WHEN: The method is called
|
||||
slide_controller.on_slide_selected_index([10])
|
||||
|
||||
# THEN: It should have exited early
|
||||
self.assertEqual(0, mocked_item.is_command.call_count, 'The service item should have not been called')
|
||||
|
||||
def on_slide_selected_index_service_item_command_test(self):
|
||||
"""
|
||||
Test that when there is a command service item, the command is executed
|
||||
"""
|
||||
# GIVEN: A mocked service item and a slide controller with a service item
|
||||
mocked_item = MagicMock()
|
||||
mocked_item.is_command.return_value = True
|
||||
mocked_item.name = 'Mocked Item'
|
||||
mocked_execute = MagicMock()
|
||||
mocked_update_preview = MagicMock()
|
||||
mocked_preview_widget = MagicMock()
|
||||
mocked_slide_selected = MagicMock()
|
||||
Registry.execute = mocked_execute
|
||||
Registry.create()
|
||||
slide_controller = SlideController(None)
|
||||
slide_controller.service_item = mocked_item
|
||||
slide_controller.update_preview = mocked_update_preview
|
||||
slide_controller.preview_widget = mocked_preview_widget
|
||||
slide_controller.slide_selected = mocked_slide_selected
|
||||
slide_controller.is_live = True
|
||||
|
||||
# WHEN: The method is called
|
||||
slide_controller.on_slide_selected_index([9])
|
||||
|
||||
# THEN: It should have sent a notification
|
||||
mocked_item.is_command.assert_called_once_with()
|
||||
mocked_execute.assert_called_once_with('mocked item_slide', [mocked_item, True, 9])
|
||||
mocked_update_preview.assert_called_once_with()
|
||||
self.assertEqual(0, mocked_preview_widget.change_slide.call_count, 'Change slide should not have been called')
|
||||
self.assertEqual(0, mocked_slide_selected.call_count, 'slide_selected should not have been called')
|
||||
|
||||
def on_slide_selected_index_service_item_not_command_test(self):
|
||||
"""
|
||||
Test that when there is a service item but it's not a command, the preview widget is updated
|
||||
"""
|
||||
# GIVEN: A mocked service item and a slide controller with a service item
|
||||
mocked_item = MagicMock()
|
||||
mocked_item.is_command.return_value = False
|
||||
mocked_item.name = 'Mocked Item'
|
||||
mocked_execute = MagicMock()
|
||||
mocked_update_preview = MagicMock()
|
||||
mocked_preview_widget = MagicMock()
|
||||
mocked_slide_selected = MagicMock()
|
||||
Registry.execute = mocked_execute
|
||||
Registry.create()
|
||||
slide_controller = SlideController(None)
|
||||
slide_controller.service_item = mocked_item
|
||||
slide_controller.update_preview = mocked_update_preview
|
||||
slide_controller.preview_widget = mocked_preview_widget
|
||||
slide_controller.slide_selected = mocked_slide_selected
|
||||
|
||||
# WHEN: The method is called
|
||||
slide_controller.on_slide_selected_index([7])
|
||||
|
||||
# THEN: It should have sent a notification
|
||||
mocked_item.is_command.assert_called_once_with()
|
||||
self.assertEqual(0, mocked_execute.call_count, 'Execute should not have been called')
|
||||
self.assertEqual(0, mocked_update_preview.call_count, 'Update preview should not have been called')
|
||||
mocked_preview_widget.change_slide.assert_called_once_with(7)
|
||||
mocked_slide_selected.assert_called_once_with()
|
||||
|
@ -28,6 +28,7 @@ class TestMediaItem(TestCase, TestMixin):
|
||||
patch('openlp.plugins.songs.forms.editsongform.EditSongForm.__init__'):
|
||||
self.media_item = SongMediaItem(None, MagicMock())
|
||||
self.media_item.display_songbook = False
|
||||
self.media_item.display_copyright_symbol = False
|
||||
self.get_application()
|
||||
self.build_settings()
|
||||
QtCore.QLocale.setDefault(QtCore.QLocale('en_GB'))
|
||||
@ -154,6 +155,39 @@ class TestMediaItem(TestCase, TestMixin):
|
||||
# THEN: The songbook should be in the footer
|
||||
self.assertEqual(service_item.raw_footer, ['My Song', 'My copyright', 'My songbook #12'])
|
||||
|
||||
def build_song_footer_copyright_enabled_test(self):
|
||||
"""
|
||||
Test building song footer with displaying the copyright symbol
|
||||
"""
|
||||
# GIVEN: A Song and a Service Item; displaying the copyright symbol is enabled
|
||||
self.media_item.display_copyright_symbol = True
|
||||
mock_song = MagicMock()
|
||||
mock_song.title = 'My Song'
|
||||
mock_song.copyright = 'My copyright'
|
||||
service_item = ServiceItem(None)
|
||||
|
||||
# WHEN: I generate the Footer with default settings
|
||||
self.media_item.generate_footer(service_item, mock_song)
|
||||
|
||||
# THEN: The copyright symbol should be in the footer
|
||||
self.assertEqual(service_item.raw_footer, ['My Song', '© My copyright'])
|
||||
|
||||
def build_song_footer_copyright_disabled_test(self):
|
||||
"""
|
||||
Test building song footer without displaying the copyright symbol
|
||||
"""
|
||||
# GIVEN: A Song and a Service Item; displaying the copyright symbol should be disabled by default
|
||||
mock_song = MagicMock()
|
||||
mock_song.title = 'My Song'
|
||||
mock_song.copyright = 'My copyright'
|
||||
service_item = ServiceItem(None)
|
||||
|
||||
# WHEN: I generate the Footer with default settings
|
||||
self.media_item.generate_footer(service_item, mock_song)
|
||||
|
||||
# THEN: The copyright symbol should not be in the footer
|
||||
self.assertEqual(service_item.raw_footer, ['My Song', 'My copyright'])
|
||||
|
||||
def authors_match_test(self):
|
||||
"""
|
||||
Test the author matching when importing a song from a service
|
||||
|
@ -30,7 +30,7 @@
|
||||
Package to test for proper bzr tags.
|
||||
"""
|
||||
import os
|
||||
|
||||
import re
|
||||
from unittest import TestCase
|
||||
|
||||
from subprocess import Popen, PIPE
|
||||
@ -50,12 +50,12 @@ TAGS = [
|
||||
['1.9.11', '2039'],
|
||||
['1.9.12', '2063'],
|
||||
['2.0', '2118'],
|
||||
['2.0.1', '?'],
|
||||
['2.0.2', '?'],
|
||||
['2.0.3', '?'],
|
||||
['2.0.4', '?'],
|
||||
['2.1.0', '2119']
|
||||
]
|
||||
# Depending on the repository, we sometimes have the 2.0.x tags in the repo too. They come up with a revision number of
|
||||
# "?", which I suspect is due to the fact that we're using shared repositories. This regular expression matches all
|
||||
# 2.0.x tags.
|
||||
TAG_SEARCH = re.compile('2\.0\.\d')
|
||||
|
||||
|
||||
class TestBzrTags(TestCase):
|
||||
@ -69,8 +69,9 @@ class TestBzrTags(TestCase):
|
||||
|
||||
# WHEN getting the branches tags
|
||||
bzr = Popen(('bzr', 'tags', '--directory=' + path), stdout=PIPE)
|
||||
stdout = bzr.communicate()[0]
|
||||
tags = [line.decode('utf-8').split() for line in stdout.splitlines()]
|
||||
std_out = bzr.communicate()[0]
|
||||
tags = [line.decode('utf-8').split() for line in std_out.splitlines()]
|
||||
tags = [t_r for t_r in tags if t_r[1] != '?' or not (t_r[1] == '?' and TAG_SEARCH.search(t_r[0]))]
|
||||
|
||||
# THEN the tags should match the accepted tags
|
||||
self.assertEqual(TAGS, tags, 'List of tags should match')
|
||||
|
Loading…
Reference in New Issue
Block a user