Merge branch 'master' into orm
[tiramisu.git] / test / test_config.py
index 5cdbe7d..f660cf5 100644 (file)
@@ -43,7 +43,7 @@ def test_base_config():
     cfg = Config(descr)
     assert cfg.dummy is False
     dm = cfg.unwrap_from_path('dummy')
-    assert dm._name == 'dummy'
+    assert dm.impl_getname() == 'dummy'
 
 
 def test_not_config():
@@ -82,11 +82,11 @@ def test_base_config_and_groups():
     assert config.gc.name == 'ref'
     assert config.bool is False
     nm = config.unwrap_from_path('gc.name')
-    assert nm._name == 'name'
+    assert nm.impl_getname() == 'name'
     gc = config.unwrap_from_path('gc')
-    assert gc._name == 'gc'
-    #nm = config.unwrap_from_name('name')
-    #assert nm._name == 'name'
+    assert gc.impl_getname() == 'gc'
+    #nm = config.unwrap_fromimpl_getname()('name')
+    #assert nm.impl_getname() == 'name'
 
 
 def test_base_config_in_a_tree():
@@ -147,6 +147,7 @@ def test_information_config():
     assert config.impl_get_information('noinfo', 'default') == 'default'
 
 
+#FIXME test impl_get_xxx pour OD ou ne pas cacher
 def test_config_impl_get_path_by_opt():
     descr = make_description()
     config = Config(descr)
@@ -231,11 +232,14 @@ def test_duplicated_option():
     g1 = IntOption('g1', '', 1)
     #in same OptionDescription
     raises(ConflictError, "d1 = OptionDescription('od', '', [g1, g1])")
+
+
+def test_duplicated_option_diff_od():
+    g1 = IntOption('g1', '', 1)
     d1 = OptionDescription('od1', '', [g1])
-    d2 = OptionDescription('od2', '', [g1])
-    root = OptionDescription('root', '', [d1, d2])
     #in different OptionDescription
-    raises(ConflictError, "config = Config(root)")
+    raises(ConflictError, "d2 = OptionDescription('od2', '', [g1])")
+
 
 
 def test_cannot_assign_value_to_option_description():