13042016

lodel/settings/settings_loader.py

@@ -28,8 +28,10 @@ class SettingsLoader(object):
         l_dir = glob.glob(self.__conf_path+'/*.ini')  
 
         for f_ini in l_dir:  
-            config = configparser.ConfigParser(default_section = 'lodel2')
+            #To avoid the DEFAULT section whose values are found in all sections, we have to give it an unsual name
+            config = configparser.ConfigParser(default_section = 'lodel2_default_passaway_tip',interpolation=None)
             config.read(f_ini)
+
             for sect in config:
                 if sect in conf:
                     for param in config[sect]:
@@ -37,13 +39,12 @@ class SettingsLoader(object):
                             conf[sect][param] = config[sect][param]
                             if sect != 'DEFAULT': self.__conf_sv[sect + ':' + param]=f_ini
                         else:
-                            print(conf)
                             raise SettingsError("Key attribute already defined : %s " % sect + '.' + param + ' dans ' + f_ini + ' et ' + self.__conf_sv[sect + ':' + param])                        
                 else:
                     opts={}
                     for key in config[sect]:
                         opts[key] = config[sect].get(key)
-                        if sect != 'DEFAULT': self.__conf_sv[sect + ':' + key]=f_ini
+                        if sect != 'lodel2_default_passaway_tip': self.__conf_sv[sect + ':' + key]=f_ini
                     conf.update({sect: opts})
         os.close(dir_conf)
         return conf

tests/settings/settings_examples/conf.d/a.ini

@@ -0,0 +1,8 @@
+[lodel2.A]
+a=a1
+b=b1,b2,b3
+c=toto
+[lodel2.A.e]
+titi=tata
+[lodel2.C]
+a=test

tests/settings/settings_examples/conf.d/b.ini

@@ -0,0 +1,3 @@
+[lodel2.A]
+fhui=njl
+

tests/settings/settings_examples/conf.d/c.ini

@@ -0,0 +1,2 @@
+[lodel2.A.e]
+a=ft

tests/settings/settings_examples/conf.d/d.ini

@@ -0,0 +1,8 @@
+[lodel2.C]
+ca=a2
+cb=b4,b2,b3
+cc=titi
+[lodel2.B]
+ab=art
+bb=bj,kl,mn
+cb=tatat

tests/settings/settings_examples/conf_raise.d/a.ini

@@ -0,0 +1,8 @@
+[A]
+a=a1
+b=b1,b2,b3
+c=toto
+[C]
+ca=a2
+cb=b4,b2,b3
+cc=titi

tests/settings/settings_examples/conf_raise.d/b.ini

@@ -0,0 +1,8 @@
+[B]
+ab=art
+bb=bj,kl,mn
+cb=tatat
+[A]
+fhui=njl
+a=gh
+

tests/settings/test_settings_loader.py

@@ -8,34 +8,36 @@ from lodel.settings.settings_loader import SettingsLoader
 #A dummy validator that only returns the value
 def dummy_validator(value): return value
 #A dummy validator that always fails
-def dummy_validator_fails(value):  raise ValueError("Fake validaion error") 
+def dummy_validator_fails(value):  raise ValueError("Fake validation error") 
 
 class SettingsLoaderTestCase(unittest.TestCase):
 
     def test_merge_getsection(self):
         """Tests merge and getSection functions """
-        settings = SettingsLoader('tests/settings/conf.d')
+        settings = SettingsLoader('tests/settings/settings_examples/conf.d')
         def maFonction(a):
             return a
-        e=settings.getoption('A','a',maFonction)
+        e=settings.getoption('lodel2.A','a',maFonction)
         self.assertEqual(e,'a1')
-        f=settings.getoption('B','bb',maFonction)
+        f=settings.getoption('lodel2.B','bb',maFonction)
         self.assertEqual(f,"bj,kl,mn")
         g=settings.getremains()
         self.assertIsNotNone(g)
-        e=settings.getoption('A','b',maFonction)
-        e=settings.getoption('A','c',maFonction)
-        e=settings.getoption('A','fhui',maFonction)
-        f=settings.getoption('B','ab',maFonction)
-        f=settings.getoption('B','cb',maFonction)
-        f=settings.getoption('C','cb',maFonction)
-        f=settings.getoption('C','ca',maFonction)
-        f=settings.getoption('C','cc',maFonction)
-        f=settings.getoption('C','a',maFonction)
+        e=settings.getoption('lodel2.A','b',maFonction)
+        e=settings.getoption('lodel2.A','c',maFonction)
+        e=settings.getoption('lodel2.A','fhui',maFonction)
+        f=settings.getoption('lodel2.B','ab',maFonction)
+        f=settings.getoption('lodel2.B','cb',maFonction)
+        f=settings.getoption('lodel2.C','cb',maFonction)
+        f=settings.getoption('lodel2.C','ca',maFonction)
+        f=settings.getoption('lodel2.C','cc',maFonction)
+        f=settings.getoption('lodel2.C','a',maFonction)
+        f=settings.getoption('lodel2.A.e','a',maFonction)
+        f=settings.getoption('lodel2.A.e','titi',maFonction)
         g=settings.getremains()
         self.assertEqual(g,[])
         with self.assertRaises(SettingsError):
-            loader = SettingsLoader('tests/settings/conf_raise.d')
+            loader = SettingsLoader('tests/settings/settings_examples/conf_raise.d')
     
     def test_merge(self):
         """ Test merge of multiple configuration files """