Browse Source

[213] Added check for missing specs

Michal 'vorner' Vaner 13 years ago
parent
commit
86123d1dc3

+ 2 - 1
src/lib/python/isc/bind10/component.py

@@ -395,7 +395,8 @@ class Configurator:
         # Handle transitions of configuration of what is here
         for cname in new.keys():
             if cname in old:
-                for option in ['special', 'process', 'kind']:
+                for option in ['special', 'process', 'kind', 'address',
+                               'params']:
                     if new[cname].get(option) != old[cname].get(option):
                         raise NotImplementedError('Changing configuration of' +
                                                   ' a running component is ' +

+ 4 - 0
src/lib/python/isc/bind10/tests/component_test.py

@@ -696,6 +696,10 @@ class ConfiguratorTest(BossUtils, unittest.TestCase):
                           'not_a_test')
         self.assertRaises(NotImplementedError, self.__do_switch, 'process',
                           'different')
+        self.assertRaises(NotImplementedError, self.__do_switch, 'address',
+                          'different')
+        self.assertRaises(NotImplementedError, self.__do_switch, 'params',
+                          ['different'])
         # This does not change anything on running component, so no need to
         # raise
         self.assertEqual([], self.__do_switch('priority', 5))