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

Fix lint warning in tests and importer.py

Browse Source
This commit is contained in:
John Lines 2019-06-30 20:13:33 +01:00
parent e566a6788c
commit d52d5bb0e2
3 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
tests/functional/openlp_plugins/songs/test_singingthefaithimport.py
View File

@ -44,7 +44,5 @@ class TestSingingTheFaithFileImport(SongImportTestHelper):
self.file_import([TEST_PATH / 'H1.txt'], self.file_import([TEST_PATH / 'H1.txt'],
self.load_external_result_data(TEST_PATH / 'STF001.json')) self.load_external_result_data(TEST_PATH / 'STF001.json'))
# Whole song - currently not working - test needs debugging. # Whole song - currently not working - test needs debugging.
## self.file_import([TEST_PATH / 'H2.txt'], # self.file_import([TEST_PATH / 'H2.txt'],
## self.load_external_result_data(TEST_PATH / 'STF002.json')) # self.load_external_result_data(TEST_PATH / 'STF002.json'))

3
tests/helpers/songfileimport.py
View File

@ -123,7 +123,8 @@ class SongImportTestHelper(TestCase):
log.debug("Song copyright imported: %s" % importer.song_number) log.debug("Song copyright imported: %s" % importer.song_number)
log.debug("Topics imported: %s" % importer.topics) log.debug("Topics imported: %s" % importer.topics)
assert importer.title == title, 'title for %s should be "%s" and is "%s"' % (source_file_name, title, importer.title) assert importer.title == title, \
'title for %s should be "%s" and is "%s"' % (source_file_name, title, importer.title)
for author in author_calls: for author in author_calls:
if isinstance(author, str): if isinstance(author, str):
self.mocked_add_author.assert_any_call(author) self.mocked_add_author.assert_any_call(author)