Code review comment for lp:~thelinuxguy/openlp/backport-fix

Add this to your merge proposal:
--------------------------------
lp:~thelinuxguy/openlp/backport-fix (revision 2681)
[SUCCESS] https://ci.openlp.io/job/Branch-01-Pull/1951/
[FAILURE] https://ci.openlp.io/job/Branch-02-Functional-Tests/1862/
Stopping after failure

Test that when the logout method is called, it logs the user out of SongSelect ... ok
Test that when the logout method is called, it logs the user out of SongSelect ... ok
Test that saving a song with an existing author performs the correct actions ... ok
Test that saving a song with a new author performs the correct actions ... ok
Test that saving a song with topics performs the correct actions ... ok
Test that saving a song with an author name of only one word performs the correct actions ... ok
Test that when the search finds MAX (2) results, it simply returns a list with those (2) ... ok
Test that when the search finds no results, it simply returns an empty list ... ok
Test that when the search finds no results, it simply returns an empty list ... ok
Test that when the search finds 2 results, it simply returns a list with 2 results ... ok
Test that the search is stopped with stop() is called ... ok
Test that loading a SongShow Plus file works correctly on various files ... ok
Test creating an instance of the SongShow Plus file importer ... ok
Test SongShowPlusImport.do_import handles different invalid import_source values ... ok
Test to_openlp_verse_tag method by simulating adding a verse ... ok
Test to_openlp_verse_tag method by simulating adding a verse to the verse order ... ok
Test SongShowPlusImport.do_import handles different invalid import_source values ... ok
Test that loading an SundayPlus file works correctly on various files ... ok
Test that loading an VideoPsalm file works correctly on various files ... ok
Test that loading a Words of Worship file works correctly ... ok
Test that loading an Worship Assistant file works correctly ... ok
Failure: SkipTest (Not Windows, skipping test) ... SKIP: Not Windows, skipping test
Test that loading an ZionWorx file works correctly on various files ... ok
Test creating an instance of the ZionWorx file importer ... ok
Test the about text of the song usage plugin ... ok
Test that check_pre_condition returns true for valid manager session ... ok
Test the initialisation of the SongUsagePlugin class ... ok
Test that all the default settings are correct ... ok
Test that clicking on the dock icon on Mac OS X restores the main window if it is minimized ... ok
Test that we don't try to backup on a new install ... ok
Test that we try to backup on a new install ... ok
Test the reimplemented event method ... ok

======================================================================
ERROR: Test that a CP1252 encoded file get's decoded properly.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/var/lib/jenkins/shared-repo/branch/tests/functional/openlp_plugins/songs/test_songbeamerimport.py", line 57, in test_cp1252_encoded_file
    self.load_external_result_data(os.path.join(TEST_PATH, 'cp1252song.json')))
  File "/var/lib/jenkins/shared-repo/branch/tests/helpers/songfileimport.py", line 112, in file_import
    self.assertIsNone(importer.do_import(), 'do_import should return None when it has completed')
  File "/var/lib/jenkins/shared-repo/branch/openlp/plugins/songs/lib/importers/songbeamer.py", line 116, in do_import
    self.input_file_encoding = chardet.detect(detect_file.read())['encoding']
NameError: name 'chardet' is not defined
-------------------- >> begin captured logging << --------------------
openlp.core.common.registry: INFO: Registry Initialising
openlp.plugins.songs.lib.importers.songimport: DEBUG: [['/var/lib/jenkins/shared-repo/branch/tests/resources/songbeamersongs/cp1252song.sng']]
--------------------- >> end captured logging << ---------------------

======================================================================
ERROR: Test that loading an OpenSong file works correctly on various files
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/var/lib/jenkins/shared-repo/branch/tests/functional/openlp_plugins/songs/test_songbeamerimport.py", line 50, in test_song_import
    self.load_external_result_data(os.path.join(TEST_PATH, 'Lobsinget dem Herrn.json')))
  File "/var/lib/jenkins/shared-repo/branch/tests/helpers/songfileimport.py", line 112, in file_import
    self.assertIsNone(importer.do_import(), 'do_import should return None when it has completed')
  File "/var/lib/jenkins/shared-repo/branch/openlp/plugins/songs/lib/importers/songbeamer.py", line 116, in do_import
    self.input_file_encoding = chardet.detect(detect_file.read())['encoding']
NameError: name 'chardet' is not defined
-------------------- >> begin captured logging << --------------------
openlp.core.common.registry: INFO: Registry Initialising
openlp.plugins.songs.lib.importers.songimport: DEBUG: [['/var/lib/jenkins/shared-repo/branch/tests/resources/songbeamersongs/Lobsinget dem Herrn.sng']]