From c89b54e4a4239f5c31d4b33ba991845f098a22f9 Mon Sep 17 00:00:00 2001 From: Ken Roberts Date: Fri, 7 Nov 2014 14:24:20 -0800 Subject: [PATCH 1/5] Add basic test for projector.sourceselectform --- .../test_projectorsourceform.py | 77 +++++++++++++++++++ 1 file changed, 77 insertions(+) create mode 100644 tests/interfaces/openlp_core_ui/test_projectorsourceform.py diff --git a/tests/interfaces/openlp_core_ui/test_projectorsourceform.py b/tests/interfaces/openlp_core_ui/test_projectorsourceform.py new file mode 100644 index 000000000..88b475d7e --- /dev/null +++ b/tests/interfaces/openlp_core_ui/test_projectorsourceform.py @@ -0,0 +1,77 @@ +# -*- coding: utf-8 -*- +# vim: autoindent shiftwidth=4 expandtab textwidth=120 tabstop=4 softtabstop=4 + +############################################################################### +# OpenLP - Open Source Lyrics Projection # +# --------------------------------------------------------------------------- # +# Copyright (c) 2008-2014 Raoul Snyman # +# Portions copyright (c) 2008-2014 Tim Bentley, Gerald Britton, Jonathan # +# Corwin, Samuel Findlay, Michael Gorven, Scott Guerrieri, Matthias Hub, # +# Meinert Jordan, Armin Köhler, Erik Lundin, Edwin Lunando, Brian T. Meyer. # +# Joshua Miller, Stevan Pettit, Andreas Preikschat, Mattias Põldaru, # +# Christian Richter, Philip Ridout, Ken Roberts, Simon Scudder, # +# Jeffrey Smith, Maikel Stuivenberg, Martin Thompson, Jon Tibble, # +# Dave Warnock, Frode Woldsund, Martin Zibricky, Patrick Zimmermann # +# --------------------------------------------------------------------------- # +# This program is free software; you can redistribute it and/or modify it # +# under the terms of the GNU General Public License as published by the Free # +# Software Foundation; version 2 of the License. # +# # +# This program is distributed in the hope that it will be useful, but WITHOUT # +# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or # +# FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for # +# more details. # +# # +# You should have received a copy of the GNU General Public License along # +# with this program; if not, write to the Free Software Foundation, Inc., 59 # +# Temple Place, Suite 330, Boston, MA 02111-1307 USA # +############################################################################### +""" + :mod: `tests.interfaces.openlp_core_ui.test_projectorsourceform` module + + Tests for the Projector Source Select form. +""" +import logging +log = logging.getLogger(__name__) +log.debug('test_projectorsourceform loaded') + +from unittest import TestCase + +from tests.helpers.testmixin import TestMixin +from openlp.core.lib.projector.constants import PJLINK_DEFAULT_CODES, PJLINK_DEFAULT_SOURCES + +from openlp.core.ui.projector.sourceselectform import source_group + +def build_source_dict(): + """ + Builds a source dictionary to verify source_group returns a valid dictionary of dictionary items + + :returns: dictionary of valid PJLink source codes grouped by PJLink source group + """ + test_group = {} + for group in PJLINK_DEFAULT_SOURCES.keys(): + test_group[group] = {} + for key in PJLINK_DEFAULT_CODES: + test_group[key[0]][key] = PJLINK_DEFAULT_CODES[key] + return test_group + +class ProjectorSourceFormTest(TestCase, TestMixin): + """ + Test class for the Projector Source Select form module + """ + def source_dict_test(self): + """ + Test that source list dict returned from sourceselectform module is a valid dict with proper entries + """ + # GIVEN: A list of inputs + codes = [] + for item in PJLINK_DEFAULT_CODES.keys(): + codes.append(item) + codes.sort() + + # WHEN: projector.sourceselectform.source_select() is called + check = source_group(codes, PJLINK_DEFAULT_CODES) + + # THEN: return dictionary should match test dictionary + self.assertEquals(check, build_source_dict(), + "Source group dictionary should match test dictionary") From f2cd0a974352d05f70a750ac44e9e12eacbb7efb Mon Sep 17 00:00:00 2001 From: Ken Roberts Date: Fri, 7 Nov 2014 14:44:40 -0800 Subject: [PATCH 2/5] Fix "Discard" and "Reset" button calls, pep8 --- openlp/core/ui/projector/sourceselectform.py | 39 ++++++++++--------- .../test_projectorsourceform.py | 2 + 2 files changed, 23 insertions(+), 18 deletions(-) diff --git a/openlp/core/ui/projector/sourceselectform.py b/openlp/core/ui/projector/sourceselectform.py index 6e7ab7f76..6c03473ad 100644 --- a/openlp/core/ui/projector/sourceselectform.py +++ b/openlp/core/ui/projector/sourceselectform.py @@ -152,16 +152,15 @@ def set_button_tooltip(bar): """ for button in bar.buttons(): if bar.standardButton(button) == QDialogButtonBox.Cancel: - tip = "Ignoring current changes and return to OpenLP" + button.setToolTip("Ignoring current changes and return to OpenLP") elif bar.standardButton(button) == QDialogButtonBox.Reset: - tip = "Delete all user-defined text and revert to PJLink default text" + button.setToolTip("Delete all user-defined text and revert to PJLink default text") elif bar.standardButton(button) == QDialogButtonBox.Discard: - tip = "Discard changes and reset to previous user-defined text" + button.setToolTip("Discard changes and reset to previous user-defined text") elif bar.standardButton(button) == QDialogButtonBox.Ok: - tip = "Save changes and return to OpenLP" + button.setToolTip("Save changes and return to OpenLP") else: - tip = "" - button.setToolTip(tip) + log.debug('No tooltip for button {}'.format(button.text())) class FingerTabBarWidget(QTabBar): @@ -286,6 +285,10 @@ class SourceSelectTabs(QDialog): thistab = self.tabwidget.addTab(tab, PJLINK_DEFAULT_SOURCES[key]) if buttonchecked: self.tabwidget.setCurrentIndex(thistab) + self.button_box = QDialogButtonBox(QtGui.QDialogButtonBox.Reset | + QtGui.QDialogButtonBox.Discard | + QtGui.QDialogButtonBox.Ok | + QtGui.QDialogButtonBox.Cancel) else: for key in keys: (tab, button_count, buttonchecked) = Build_Tab(group=self.button_group, @@ -297,10 +300,8 @@ class SourceSelectTabs(QDialog): thistab = self.tabwidget.addTab(tab, PJLINK_DEFAULT_SOURCES[key]) if buttonchecked: self.tabwidget.setCurrentIndex(thistab) - self.button_box = QDialogButtonBox(QtGui.QDialogButtonBox.Reset | - QtGui.QDialogButtonBox.Discard | - QtGui.QDialogButtonBox.Ok | - QtGui.QDialogButtonBox.Cancel) + self.button_box = QDialogButtonBox(QtGui.QDialogButtonBox.Ok | + QtGui.QDialogButtonBox.Cancel) self.button_box.clicked.connect(self.button_clicked) self.layout.addWidget(self.button_box) set_button_tooltip(self.button_box) @@ -321,9 +322,9 @@ class SourceSelectTabs(QDialog): if self.button_box.standardButton(button) == self.button_box.Cancel: self.done(0) elif self.button_box.standardButton(button) == self.button_box.Reset: - self.delete_sources() - elif self.button_box.standardButton(button) == self.button_box.Discard: self.done(100) + elif self.button_box.standardButton(button) == self.button_box.Discard: + self.delete_sources() elif self.button_box.standardButton(button) == self.button_box.Ok: return self.accept_me() else: @@ -418,6 +419,10 @@ class SourceSelectSingle(QDialog): item.setText(source_item.text) self.layout.addRow(PJLINK_DEFAULT_CODES[key], item) self.button_group.append(item) + self.button_box = QDialogButtonBox(QtGui.QDialogButtonBox.Reset | + QtGui.QDialogButtonBox.Discard | + QtGui.QDialogButtonBox.Ok | + QtGui.QDialogButtonBox.Cancel) else: for key in keys: source_text = self.projectordb.get_source_by_code(code=key, projector_id=self.projector.db_item.id) @@ -427,10 +432,8 @@ class SourceSelectSingle(QDialog): self.layout.addWidget(button) self.button_group.addButton(button, int(key)) button_list.append(key) - self.button_box = QDialogButtonBox(QtGui.QDialogButtonBox.Reset | - QtGui.QDialogButtonBox.Discard | - QtGui.QDialogButtonBox.Ok | - QtGui.QDialogButtonBox.Cancel) + self.button_box = QDialogButtonBox(QtGui.QDialogButtonBox.Ok | + QtGui.QDialogButtonBox.Cancel) self.button_box.clicked.connect(self.button_clicked) self.layout.addWidget(self.button_box) self.setMinimumHeight(key_count*25) @@ -452,9 +455,9 @@ class SourceSelectSingle(QDialog): if self.button_box.standardButton(button) == self.button_box.Cancel: self.done(0) elif self.button_box.standardButton(button) == self.button_box.Reset: - self.delete_sources() - elif self.button_box.standardButton(button) == self.button_box.Discard: self.done(100) + elif self.button_box.standardButton(button) == self.button_box.Discard: + self.delete_sources() elif self.button_box.standardButton(button) == self.button_box.Ok: return self.accept_me() else: diff --git a/tests/interfaces/openlp_core_ui/test_projectorsourceform.py b/tests/interfaces/openlp_core_ui/test_projectorsourceform.py index 88b475d7e..ed005a7d9 100644 --- a/tests/interfaces/openlp_core_ui/test_projectorsourceform.py +++ b/tests/interfaces/openlp_core_ui/test_projectorsourceform.py @@ -42,6 +42,7 @@ from openlp.core.lib.projector.constants import PJLINK_DEFAULT_CODES, PJLINK_DEF from openlp.core.ui.projector.sourceselectform import source_group + def build_source_dict(): """ Builds a source dictionary to verify source_group returns a valid dictionary of dictionary items @@ -55,6 +56,7 @@ def build_source_dict(): test_group[key[0]][key] = PJLINK_DEFAULT_CODES[key] return test_group + class ProjectorSourceFormTest(TestCase, TestMixin): """ Test class for the Projector Source Select form module From aadbce133e9c1e39e0759b7864e46f49fe857c98 Mon Sep 17 00:00:00 2001 From: Ken Roberts Date: Mon, 10 Nov 2014 16:14:17 -0800 Subject: [PATCH 3/5] Add missing tranlsate to text/fix select/edit title --- openlp/core/ui/projector/sourceselectform.py | 39 +++++++++++++------- 1 file changed, 26 insertions(+), 13 deletions(-) diff --git a/openlp/core/ui/projector/sourceselectform.py b/openlp/core/ui/projector/sourceselectform.py index 6c03473ad..e7e0c2201 100644 --- a/openlp/core/ui/projector/sourceselectform.py +++ b/openlp/core/ui/projector/sourceselectform.py @@ -152,13 +152,17 @@ def set_button_tooltip(bar): """ for button in bar.buttons(): if bar.standardButton(button) == QDialogButtonBox.Cancel: - button.setToolTip("Ignoring current changes and return to OpenLP") + button.setToolTip(translate('OpenLP.SourceSelectForm', + "Ignoring current changes and return to OpenLP")) elif bar.standardButton(button) == QDialogButtonBox.Reset: - button.setToolTip("Delete all user-defined text and revert to PJLink default text") + button.setToolTip(translate('OpenLP.SourceSelectForm', + "Delete all user-defined text and revert to PJLink default text")) elif bar.standardButton(button) == QDialogButtonBox.Discard: - button.setToolTip("Discard changes and reset to previous user-defined text") + button.setToolTip(translate('OpenLP.SourceSelectForm', + "Discard changes and reset to previous user-defined text")) elif bar.standardButton(button) == QDialogButtonBox.Ok: - button.setToolTip("Save changes and return to OpenLP") + button.setToolTip(translate('OpenLP.SourceSelectForm', + "Save changes and return to OpenLP")) else: log.debug('No tooltip for button {}'.format(button.text())) @@ -236,12 +240,13 @@ class SourceSelectTabs(QDialog): """ log.debug('Initializing SourceSelectTabs()') super(SourceSelectTabs, self).__init__(parent) + self.setMinimumWidth(350) self.projectordb = projectordb self.edit = edit if self.edit: - title = translate('OpenLP.SourceSelectForm', 'Select Projector Source') - else: title = translate('OpenLP.SourceSelectForm', 'Edit Projector Source Text') + else: + title = translate('OpenLP.SourceSelectForm', 'Select Projector Source') self.setWindowTitle(title) self.setObjectName('source_select_tabs') self.setWindowIcon(build_icon(':/icon/openlp-log-32x32.png')) @@ -332,9 +337,11 @@ class SourceSelectTabs(QDialog): def delete_sources(self): msg = QtGui.QMessageBox() - msg.setText('Delete entries for this projector') - msg.setInformativeText('Are you sure you want to delete ALL user-defined ' - 'source input text for this projector?') + msg.setText(translate('OpenLP.SourceSelectForm', 'Delete entries for this projector')) + msg.setInformativeText(translate('OpenLP.SourceSelectForm', + 'Are you sure you want to delete ALL user-defined '), + translate('OpenLP.SourceSelectForm', + 'source input text for this projector?')) msg.setStandardButtons(msg.Cancel | msg.Ok) msg.setDefaultButton(msg.Cancel) ans = msg.exec_() @@ -383,7 +390,11 @@ class SourceSelectSingle(QDialog): log.debug('Initializing SourceSelectSingle()') self.projectordb = projectordb super(SourceSelectSingle, self).__init__(parent) - self.setWindowTitle(translate('OpenLP.SourceSelectSingle', 'Select Projector Source')) + self.edit = edit + if self.edit: + title = translate('OpenLP.SourceSelectForm', 'Edit Projector Source Text') + else: + title = translate('OpenLP.SourceSelectForm', 'Select Projector Source') self.setObjectName('source_select_single') self.setWindowIcon(build_icon(':/icon/openlp-log-32x32.png')) self.setModal(True) @@ -465,9 +476,11 @@ class SourceSelectSingle(QDialog): def delete_sources(self): msg = QtGui.QMessageBox() - msg.setText('Delete entries for this projector') - msg.setInformativeText('Are you sure you want to delete ALL user-defined ' - 'source input text for this projector?') + msg.setText(translate('OpenLP.SourceSelectForm', 'Delete entries for this projector')) + msg.setInformativeText(translate('OpenLP.SourceSelectForm', + 'Are you sure you want to delete ALL user-defined '), + translate('OpenLP.SourceSelectForm', + 'source input text for this projector?')) msg.setStandardButtons(msg.Cancel | msg.Ok) msg.setDefaultButton(msg.Cancel) ans = msg.exec_() From ea14deaa873ceb2076ad5979e618764d907d06e2 Mon Sep 17 00:00:00 2001 From: Ken Roberts Date: Wed, 26 Nov 2014 09:24:05 -0800 Subject: [PATCH 4/5] Fix quotes --- .bzrignore | 42 ------------------- openlp/core/ui/projector/sourceselectform.py | 8 ++-- .../openlp_core_ui/test_projectormanager.py | 4 +- 3 files changed, 5 insertions(+), 49 deletions(-) delete mode 100644 .bzrignore diff --git a/.bzrignore b/.bzrignore deleted file mode 100644 index 6b7b989a6..000000000 --- a/.bzrignore +++ /dev/null @@ -1,42 +0,0 @@ -*.pyc -*.*~ -\#*\# -*.eric4project -*.eric5project -*.ropeproject -*.e4* -.eric4project -.komodotools -*.komodoproject -list -openlp.org 2.0.e4* -documentation/build/html -documentation/build/doctrees -*.log* -dist -OpenLP.egg-info -build -resources/innosetup/Output -_eric4project -.pylint.d -*.qm -openlp/core/resources.py.old -*.qm -resources/windows/warnOpenLP.txt -openlp.cfg -.idea -openlp.pro -.kdev4 -tests.kdev4 -*.nja -*.orig -__pycache__ -*.dll -.directory -*.kate-swp -# Git files -.git -.gitignore -# Rejected diff's -*.rej -*.~\?~ diff --git a/openlp/core/ui/projector/sourceselectform.py b/openlp/core/ui/projector/sourceselectform.py index e7e0c2201..244b7adef 100644 --- a/openlp/core/ui/projector/sourceselectform.py +++ b/openlp/core/ui/projector/sourceselectform.py @@ -153,16 +153,16 @@ def set_button_tooltip(bar): for button in bar.buttons(): if bar.standardButton(button) == QDialogButtonBox.Cancel: button.setToolTip(translate('OpenLP.SourceSelectForm', - "Ignoring current changes and return to OpenLP")) + 'Ignoring current changes and return to OpenLP')) elif bar.standardButton(button) == QDialogButtonBox.Reset: button.setToolTip(translate('OpenLP.SourceSelectForm', - "Delete all user-defined text and revert to PJLink default text")) + 'Delete all user-defined text and revert to PJLink default text')) elif bar.standardButton(button) == QDialogButtonBox.Discard: button.setToolTip(translate('OpenLP.SourceSelectForm', - "Discard changes and reset to previous user-defined text")) + 'Discard changes and reset to previous user-defined text')) elif bar.standardButton(button) == QDialogButtonBox.Ok: button.setToolTip(translate('OpenLP.SourceSelectForm', - "Save changes and return to OpenLP")) + 'Save changes and return to OpenLP')) else: log.debug('No tooltip for button {}'.format(button.text())) diff --git a/tests/interfaces/openlp_core_ui/test_projectormanager.py b/tests/interfaces/openlp_core_ui/test_projectormanager.py index a46b0b93c..58638458e 100644 --- a/tests/interfaces/openlp_core_ui/test_projectormanager.py +++ b/tests/interfaces/openlp_core_ui/test_projectormanager.py @@ -94,11 +94,9 @@ class TestProjectorManager(TestCase, TestMixin): self.projector_manager.bootstrap_initialise() self.projector_manager.bootstrap_post_set_up() - # THEN: verify calls to retrieve saved projectors + # THEN: verify calls to retrieve saved projectors and edit page initialized self.assertEqual(1, self.projector_manager._load_projectors.call_count, 'Initialization should have called load_projectors()') - - # THEN: Verify edit page is initialized self.assertEqual(type(self.projector_manager.projector_form), ProjectorEditForm, 'Initialization should have created a Projector Edit Form') self.assertIs(self.projector_manager.projectordb, From f9a052ac8527921c5e5dc9ebec23926514ac3de5 Mon Sep 17 00:00:00 2001 From: Ken Roberts Date: Mon, 1 Dec 2014 07:39:05 -0800 Subject: [PATCH 5/5] Add .bzrignore from trunk --- .bzrignore | 42 ++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 42 insertions(+) create mode 100644 .bzrignore diff --git a/.bzrignore b/.bzrignore new file mode 100644 index 000000000..6b7b989a6 --- /dev/null +++ b/.bzrignore @@ -0,0 +1,42 @@ +*.pyc +*.*~ +\#*\# +*.eric4project +*.eric5project +*.ropeproject +*.e4* +.eric4project +.komodotools +*.komodoproject +list +openlp.org 2.0.e4* +documentation/build/html +documentation/build/doctrees +*.log* +dist +OpenLP.egg-info +build +resources/innosetup/Output +_eric4project +.pylint.d +*.qm +openlp/core/resources.py.old +*.qm +resources/windows/warnOpenLP.txt +openlp.cfg +.idea +openlp.pro +.kdev4 +tests.kdev4 +*.nja +*.orig +__pycache__ +*.dll +.directory +*.kate-swp +# Git files +.git +.gitignore +# Rejected diff's +*.rej +*.~\?~