Fix configparser.NoSectionError

Josef-Friedrich · Jun 17, 2022 · 1093ac7 · 1093ac7
1 parent 18a791e
commit 1093ac7
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 7 deletions.
diff --git a/audiorename/job.py b/audiorename/job.py
@@ -104,7 +104,8 @@ def __init__(self, job: 'Job', section: str,
                         attr = job.config.getint(section, option)
                     else:
                         attr = job.config.get(section, option)
-                except configparser.NoOptionError:
+                except (configparser.NoOptionError,
+                        configparser.NoSectionError):
                     pass
 
             if attr is not None:

diff --git a/test/files/config/minimal.ini b/test/files/config/minimal.ini
@@ -0,0 +1,2 @@
+[rename]
+backup_folder = /tmp/minimal
diff --git a/test/test_job.py b/test/test_job.py
@@ -118,15 +118,20 @@ def test_remap_classical(self):
             job(remap_classical=True).metadata_actions.remap_classical, True)
 
 
-class TestJobWithConfigParser(unittest.TestCase):
+def make_job_with_config(config_file: str) -> Job:
+    args = ArgsDefault()
+    args.config = helper.get_testfile('config', config_file)
+    return Job(args)
+
 
-    def get_job(self, *config_file: str) -> Job:
-        args = ArgsDefault()
-        args.config = helper.get_testfile(*config_file)
-        return Job(args)
+class TestJobWithConfigParser(unittest.TestCase):
 
     def setUp(self):
-        self.job = self.get_job('config', 'all-true.ini')
+        self.job = make_job_with_config('all-true.ini')
+
+    def test_minimal_config_file(self):
+        job = make_job_with_config('minimal.ini')
+        self.assertEqual(job.rename.backup_folder, '/tmp/minimal')
 
     def test_section_selection(self):
         self.assertEqual(self.job.selection.source, '/tmp')