openlp-core team mailing list archive
-
openlp-core team
-
Mailing list archive
-
Message #23164
[Merge] lp:~sam92/openlp/fix-songbook-search into lp:openlp
Samuel Mehrbrodt has proposed merging lp:~sam92/openlp/fix-songbook-search into lp:openlp.
Requested reviews:
OpenLP Core (openlp-core)
Related bugs:
Bug #1306958 in OpenLP: "Song Book search broken"
https://bugs.launchpad.net/openlp/+bug/1306958
For more details, see:
https://code.launchpad.net/~sam92/openlp/fix-songbook-search/+merge/215978
Fix Songbook search
[SUCCESS] http://ci.openlp.org/job/Branch-01-Pull/264/
[SUCCESS] http://ci.openlp.org/job/Branch-02-Functional-Tests/225/
[SUCCESS] http://ci.openlp.org/job/Branch-03-Interface-Tests/174/
[SUCCESS] http://ci.openlp.org/job/Branch-04-Windows_Tests/135/
[FAILURE] http://ci.openlp.org/job/Branch-05-Code-Analysis/91/
--
https://code.launchpad.net/~sam92/openlp/fix-songbook-search/+merge/215978
Your team OpenLP Core is requested to review the proposed merge of lp:~sam92/openlp/fix-songbook-search into lp:openlp.
=== modified file 'openlp/plugins/songs/lib/mediaitem.py'
--- openlp/plugins/songs/lib/mediaitem.py 2014-04-12 20:19:22 +0000
+++ openlp/plugins/songs/lib/mediaitem.py 2014-04-15 22:24:48 +0000
@@ -192,7 +192,7 @@
song_number = False
if not search_results:
search_keywords = search_keywords.rpartition(' ')
- search_string = '%' + search_keywords + '%'
+ search_string = '%' + search_keywords[0] + '%'
search_results = self.plugin.manager.get_all_objects(Book,
Book.name.like(search_string), Book.name.asc())
song_number = re.sub(r'[^0-9]', '', search_keywords[2])
=== modified file 'openlp/plugins/songs/lib/openlyricsimport.py'
--- openlp/plugins/songs/lib/openlyricsimport.py 2014-03-06 22:05:15 +0000
+++ openlp/plugins/songs/lib/openlyricsimport.py 2014-04-15 22:24:48 +0000
@@ -53,7 +53,7 @@
Initialise the Open Lyrics importer.
"""
log.debug('initialise OpenLyricsImport')
- SongImport.__init__(self, manager, **kwargs)
+ super(OpenLyricsImport, self).__init__(manager, **kwargs)
self.open_lyrics = OpenLyrics(self.manager)
def do_import(self):
@@ -71,7 +71,7 @@
# special characters in the path (see lp:757673 and lp:744337).
parsed_file = etree.parse(open(file_path, 'r'), parser)
xml = etree.tostring(parsed_file).decode()
- self.open_lyrics.xml_to_song(xml)
+ return self.open_lyrics.xml_to_song(xml)
except etree.XMLSyntaxError:
log.exception('XML syntax error in file %s' % file_path)
self.log_error(file_path, SongStrings.XMLSyntaxError)
=== added file 'tests/functional/openlp_plugins/songs/test_openlyricsimport.py'
--- tests/functional/openlp_plugins/songs/test_openlyricsimport.py 1970-01-01 00:00:00 +0000
+++ tests/functional/openlp_plugins/songs/test_openlyricsimport.py 2014-04-15 22:24:48 +0000
@@ -0,0 +1,96 @@
+# -*- 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, 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 #
+###############################################################################
+"""
+This module contains tests for the OpenLyrics song importer.
+"""
+
+import os
+from unittest import TestCase
+
+from tests.functional import MagicMock, patch
+from openlp.plugins.songs.lib.openlyricsimport import OpenLyricsImport
+from openlp.plugins.songs.lib.songimport import SongImport
+
+TEST_PATH = os.path.abspath(os.path.join(os.path.dirname(__file__),
+ '..', '..', '..', 'resources', 'openlyricssongs'))
+SONG_TEST_DATA = {
+ 'Mám zde přítele, Pána Ježíše.xml': {
+ 'title': 'Mám zde přítele',
+ 'verses': [
+ ('Mám zde přítele,\nPána Ježíše,\na na rámě jeho spoléhám;\nv něm své stěstí mám,\n\
+ pokoj nalézám,\nkdyž na rámě jeho spoléhám!', 'v1'),
+ ('Boží rámě\nje v soužení náš pevný hrad;\nBoží rámě,\nuč se na ně vždycky spoléhat!', 'c'),
+ ('Jak je sladké být,\nv jeho družině,\nkdyž na rámě jeho spoléhám,\njak se života\ncesta zjasňuje\n\
+ když na rámě Boží spoléhám!', 'v2'),
+ ('Čeho bych se bál,\nčeho strachoval,\nkdyž na rámě Boží spoléhám?\nMír je v duši mé,\n\
+ když On blízko je,\nkdyž na rámě jeho spoléhám.', 'v')
+ ]
+ }
+}
+
+
+class TestOpenLyricsImport(TestCase):
+ """
+ Test the functions in the :mod:`openlyricsimport` module.
+ """
+ def create_importer_test(self):
+ """
+ Test creating an instance of the OpenLyrics file importer
+ """
+ # GIVEN: A mocked out SongImport class, and a mocked out "manager"
+ with patch('openlp.plugins.songs.lib.songbeamerimport.SongImport'):
+ mocked_manager = MagicMock()
+
+ # WHEN: An importer object is created
+ importer = OpenLyricsImport(mocked_manager, filenames=[])
+
+ # THEN: The importer should be an instance of SongImport
+ self.assertIsInstance(importer, SongImport)
+
+ @patch('openlp.plugins.songs.lib.db.Song')
+ @patch('openlp.plugins.songs.lib.songbeamerimport.SongImport')
+ def file_import_test(self, mock_songimport, mock_song):
+ """
+ Test the actual import of real song files and check that the imported data is correct.
+ """
+
+ # GIVEN: Test files with a mocked out "manager" and a mocked out "import_wizard"
+ for song_file in SONG_TEST_DATA:
+ mocked_manager = MagicMock()
+ mocked_import_wizard = MagicMock()
+ importer = OpenLyricsImport(mocked_manager, filenames=[])
+ importer.import_wizard = mocked_import_wizard
+
+ # WHEN: Importing each file
+ importer.import_source = [os.path.join(TEST_PATH, song_file)]
+ song = importer.do_import()
+
+ # THEN: the song title should be as expected
+ self.assertEqual(song.title, SONG_TEST_DATA[song_file]['title'],
+ 'title for %s should be "%s"' % (song_file, SONG_TEST_DATA[song_file]['title']))
=== added directory 'tests/resources/openlyricssongs'
=== added file 'tests/resources/openlyricssongs/Mám zde přítele, Pána Ježíše.xml'
--- tests/resources/openlyricssongs/Mám zde přítele, Pána Ježíše.xml 1970-01-01 00:00:00 +0000
+++ tests/resources/openlyricssongs/Mám zde přítele, Pána Ježíše.xml 2014-04-15 22:24:48 +0000
@@ -0,0 +1,53 @@
+<?xml version='1.0' encoding='UTF-8'?>
+<song xmlns="http://openlyrics.info/namespace/2009/song"; version="0.8" createdIn="opensong2openlyrics.py 0.1" modifiedIn="convert-schema.py" modifiedDate="2013-03-12T09:39:58.853358">
+ <properties>
+ <titles>
+ <title>Mám zde přítele</title>
+ </titles>
+ <authors>
+ <author>A.J. Showalter</author>
+ <author>E.A. Hoffman</author>
+ </authors>
+ <verseOrder>v1 c v2 c v3 c</verseOrder>
+ </properties>
+ <lyrics>
+ <verse name="v1">
+ <lines>
+ Mám zde přítele,<br/>
+ Pána Ježíše,<br/>
+ a na rámě jeho spoléhám;<br/>
+ v něm své stěstí mám,<br/>
+ pokoj nalézám,<br/>
+ když na rámě jeho spoléhám!
+ </lines>
+ </verse>
+ <verse name="c">
+ <lines>
+ Boží rámě<br/>
+ je v soužení náš pevný hrad;<br/>
+ Boží rámě,<br/>
+ uč se na ně vždycky spoléhat!
+ </lines>
+ </verse>
+ <verse name="v2">
+ <lines>
+ Jak je sladké být,<br/>
+ v jeho družině,<br/>
+ když na rámě jeho spoléhám,<br/>
+ jak se života<br/>
+ cesta zjasňuje<br/>
+ když na rámě Boží spoléhám!
+ </lines>
+ </verse>
+ <verse name="v3">
+ <lines>
+ Čeho bych se bál,<br/>
+ čeho strachoval,<br/>
+ když na rámě Boží spoléhám?<br/>
+ Mír je v duši mé,<br/>
+ když On blízko je,<br/>
+ když na rámě jeho spoléhám.
+ </lines>
+ </verse>
+ </lyrics>
+</song>
References
