Catch all archive extract errors and skip tests without unrar

Author: Thomas Scholtes

beets/importer.py

@@ -665,7 +665,7 @@ def read_tasks(session):
             archive_task = ArchiveImportTask(toppath)
             try:
                 archive_task.extract()
-            except IOError as exc:
+            except Exception as exc:
                 log.error('extraction failed: {0}'.format(exc))
                 continue
             toppath = archive_task.toppath

test/test_importer.py

@@ -23,7 +23,7 @@
 
 import _common
 from _common import unittest
-from helper import TestImportSession, TestHelper
+from helper import TestImportSession, TestHelper, has_program
 from beets import library
 from beets import importer
 from beets.mediafile import MediaFile
@@ -342,6 +342,7 @@ def create_archive(self):
         return path
 
 
+@unittest.skipIf(not has_program('unrar'), 'unrar program not found')
 class ImportRarTest(ImportZipTest):
 
     def create_archive(self):