add option name's validation and rename Option method with objimpl_
[tiramisu.git] / test / test_parsing_group.py
index f060a97..34019f7 100644 (file)
@@ -27,13 +27,13 @@ def make_description():
 
     master = OptionDescription('ip_admin_eth0', '', [ip_admin_eth0, netmask_admin_eth0])
     interface1 = OptionDescription('interface1', '', [master])
-    interface1.set_group_type(groups.family)
+    interface1.objimpl_set_group_type(groups.family)
 
     general = OptionDescription('general', '', [numero_etab, nom_machine,
                                 nombre_interfaces, activer_proxy_client,
                                 mode_conteneur_actif, adresse_serveur_ntp,
                                 time_zone])
-    general.set_group_type(groups.family)
+    general.objimpl_set_group_type(groups.family)
     creole = OptionDescription('creole', 'first tiramisu configuration', [general, interface1])
     descr = OptionDescription('baseconfig', 'baseconifgdescr', [creole])
     return descr
@@ -62,10 +62,10 @@ def test_get_group_type():
     descr = make_description()
     config = Config(descr)
     grp = config.unwrap_from_path('creole.general')
-    assert grp.get_group_type() == groups.family
-    assert grp.get_group_type() == 'family'
-    assert isinstance(grp.get_group_type(), groups.GroupType)
-    raises(TypeError, 'grp.set_group_type(groups.default)')
+    assert grp.objimpl_get_group_type() == groups.family
+    assert grp.objimpl_get_group_type() == 'family'
+    assert isinstance(grp.objimpl_get_group_type(), groups.GroupType)
+    raises(TypeError, 'grp.objimpl_set_group_type(groups.default)')
 
 
 def test_iter_on_groups():
@@ -91,31 +91,31 @@ def test_groups_with_master():
     ip_admin_eth0 = StrOption('ip_admin_eth0', "ip réseau autorisé", multi=True)
     netmask_admin_eth0 = StrOption('netmask_admin_eth0', "masque du sous-réseau", multi=True)
     interface1 = OptionDescription('ip_admin_eth0', '', [ip_admin_eth0, netmask_admin_eth0])
-    interface1.set_group_type(groups.master)
-    assert interface1.get_group_type() == groups.master
+    interface1.objimpl_set_group_type(groups.master)
+    assert interface1.objimpl_get_group_type() == groups.master
 
 
 def test_groups_with_master_in_config():
     ip_admin_eth0 = StrOption('ip_admin_eth0', "ip réseau autorisé", multi=True)
     netmask_admin_eth0 = StrOption('netmask_admin_eth0', "masque du sous-réseau", multi=True)
     interface1 = OptionDescription('ip_admin_eth0', '', [ip_admin_eth0, netmask_admin_eth0])
-    interface1.set_group_type(groups.master)
+    interface1.objimpl_set_group_type(groups.master)
     Config(interface1)
-    assert interface1.get_group_type() == groups.master
+    assert interface1.objimpl_get_group_type() == groups.master
 
 
 def test_allowed_groups():
     ip_admin_eth0 = StrOption('ip_admin_eth0', "ip réseau autorisé", multi=True)
     netmask_admin_eth0 = StrOption('netmask_admin_eth0', "masque du sous-réseau", multi=True)
     interface1 = OptionDescription('ip_admin_eth0', '', [ip_admin_eth0, netmask_admin_eth0])
-    raises(ValueError, "interface1.set_group_type('toto')")
+    raises(ValueError, "interface1.objimpl_set_group_type('toto')")
 
 
 def test_master_not_valid_name():
     ip_admin_eth0 = StrOption('ip_admin_eth0', "ip réseau autorisé", multi=True)
     netmask_admin_eth0 = StrOption('netmask_admin_eth0', "masque du sous-réseau", multi=True)
     invalid_group = OptionDescription('interface1', '', [ip_admin_eth0, netmask_admin_eth0])
-    raises(ValueError, "invalid_group.set_group_type(groups.master)")
+    raises(ValueError, "invalid_group.objimpl_set_group_type(groups.master)")
 
 
 def test_sub_group_in_master_group():
@@ -123,14 +123,14 @@ def test_sub_group_in_master_group():
     netmask_admin_eth0 = StrOption('netmask_admin_eth0', "masque du sous-réseau", multi=True)
     subgroup = OptionDescription("subgroup", '', [])
     invalid_group = OptionDescription('ip_admin_eth0', '', [subgroup, ip_admin_eth0, netmask_admin_eth0])
-    raises(ValueError, "invalid_group.set_group_type(groups.master)")
+    raises(ValueError, "invalid_group.objimpl_set_group_type(groups.master)")
 
 
 def test_group_always_has_multis():
     ip_admin_eth0 = StrOption('ip_admin_eth0', "ip réseau autorisé", multi=True)
     netmask_admin_eth0 = StrOption('netmask_admin_eth0', "masque du sous-réseau")
     group = OptionDescription('ip_admin_eth0', '', [ip_admin_eth0, netmask_admin_eth0])
-    raises(ValueError, "group.set_group_type(groups.master)")
+    raises(ValueError, "group.objimpl_set_group_type(groups.master)")
 
 
 #____________________________________________________________
@@ -138,13 +138,13 @@ def test_values_with_master_and_slaves():
     ip_admin_eth0 = StrOption('ip_admin_eth0', "ip réseau autorisé", multi=True)
     netmask_admin_eth0 = StrOption('netmask_admin_eth0', "masque du sous-réseau", multi=True)
     interface1 = OptionDescription('ip_admin_eth0', '', [ip_admin_eth0, netmask_admin_eth0])
-    interface1.set_group_type(groups.master)
+    interface1.objimpl_set_group_type(groups.master)
     maconfig = OptionDescription('toto', '', [interface1])
     cfg = Config(maconfig)
     opt = cfg.unwrap_from_path("ip_admin_eth0.ip_admin_eth0")
     opt_slave = cfg.unwrap_from_path("ip_admin_eth0.netmask_admin_eth0")
     owner = cfg._cfgimpl_context._cfgimpl_settings.getowner()
-    assert interface1.get_group_type() == groups.master
+    assert interface1.objimpl_get_group_type() == groups.master
     assert cfg.cfgimpl_get_values().getowner(opt) == owners.default
     assert cfg.cfgimpl_get_values().getowner(opt_slave) == owners.default
     assert cfg.ip_admin_eth0.netmask_admin_eth0 == []
@@ -159,13 +159,13 @@ def test_reset_values_with_master_and_slaves():
     ip_admin_eth0 = StrOption('ip_admin_eth0', "ip réseau autorisé", multi=True)
     netmask_admin_eth0 = StrOption('netmask_admin_eth0', "masque du sous-réseau", multi=True)
     interface1 = OptionDescription('ip_admin_eth0', '', [ip_admin_eth0, netmask_admin_eth0])
-    interface1.set_group_type(groups.master)
+    interface1.objimpl_set_group_type(groups.master)
     maconfig = OptionDescription('toto', '', [interface1])
     cfg = Config(maconfig)
     opt = cfg.unwrap_from_path("ip_admin_eth0.ip_admin_eth0")
     opt_slave = cfg.unwrap_from_path("ip_admin_eth0.netmask_admin_eth0")
     owner = cfg._cfgimpl_context._cfgimpl_settings.getowner()
-    assert interface1.get_group_type() == groups.master
+    assert interface1.objimpl_get_group_type() == groups.master
     assert cfg.cfgimpl_get_values().getowner(opt) == owners.default
     assert cfg.cfgimpl_get_values().getowner(opt_slave) == owners.default
     cfg.ip_admin_eth0.ip_admin_eth0.append("192.168.230.145")
@@ -182,7 +182,7 @@ def test_values_with_master_and_slaves_slave():
     ip_admin_eth0 = StrOption('ip_admin_eth0', "ip réseau autorisé", multi=True)
     netmask_admin_eth0 = StrOption('netmask_admin_eth0', "masque du sous-réseau", multi=True)
     interface1 = OptionDescription('ip_admin_eth0', '', [ip_admin_eth0, netmask_admin_eth0])
-    interface1.set_group_type(groups.master)
+    interface1.objimpl_set_group_type(groups.master)
     maconfig = OptionDescription('toto', '', [interface1])
     cfg = Config(maconfig)
     assert cfg.ip_admin_eth0.netmask_admin_eth0 == []
@@ -204,7 +204,7 @@ def test_values_with_master_and_slaves_master():
     ip_admin_eth0 = StrOption('ip_admin_eth0', "ip réseau autorisé", multi=True)
     netmask_admin_eth0 = StrOption('netmask_admin_eth0', "masque du sous-réseau", multi=True)
     interface1 = OptionDescription('ip_admin_eth0', '', [ip_admin_eth0, netmask_admin_eth0])
-    interface1.set_group_type(groups.master)
+    interface1.objimpl_set_group_type(groups.master)
     maconfig = OptionDescription('toto', '', [interface1])
     cfg = Config(maconfig)
     cfg.ip_admin_eth0.ip_admin_eth0.append("192.168.230.145")