config tests

also a couple of fixes to manager

lib/ansible/config/manager.py

@@ -164,7 +164,7 @@ class ConfigManager(object):
     UNABLE = []
     DEPRECATED = []
 
-    def __init__(self, conf_file=None):
+    def __init__(self, conf_file=None, defs_file=None):
 
         self._base_defs = {}
         self._plugins = {}
@@ -173,19 +173,24 @@ class ConfigManager(object):
         self._config_file = conf_file
         self.data = ConfigData()
 
-        # FIXME: make dynamic? scan for more? make it's own method?
+        if defs_file is None:
-        # Create configuration definitions from source
+            # Create configuration definitions from source
-        bconfig_def = to_bytes('%s/base.yml' % os.path.dirname(__file__))
+            b_defs_file = to_bytes('%s/base.yml' % os.path.dirname(__file__))
-        if os.path.exists(bconfig_def):
+        else:
-            with open(bconfig_def, 'rb') as config_def:
+            b_defs_file = to_bytes(defs_file)
+
+        # consume definitions
+        if os.path.exists(b_defs_file):
+            with open(b_defs_file, 'rb') as config_def:
                 self._base_defs = yaml.safe_load(config_def)
         else:
-            raise AnsibleError("Missing base configuration definition file (bad install?): %s" % to_native(bconfig_def))
+            raise AnsibleError("Missing base configuration definition file (bad install?): %s" % to_native(b_defs_file))
 
         if self._config_file is None:
             # set config using ini
             self._config_file = find_ini_config_file()
 
+        # consume configuration
         if self._config_file:
             if os.path.exists(self._config_file):
                 # initialize parser and read config
@@ -270,9 +275,12 @@ class ConfigManager(object):
 
         if cfile is None:
             cfile = self._config_file
+        else:
+            self._parse_config_file(cfile)
 
         # Note: sources that are lists listed in low to high precedence (last one wins)
         value = None
+        origin = None
         defs = {}
         if plugin_type is None:
             defs = self._base_defs
@@ -281,60 +289,63 @@ class ConfigManager(object):
         else:
             defs = self._plugins[plugin_type][plugin_name]
 
-        # Use 'variable overrides' if present, highest precedence, but only present when querying running play
+        if config in defs:
-        if variables:
+            # Use 'variable overrides' if present, highest precedence, but only present when querying running play
-            value, origin = self._loop_entries(variables, defs[config]['vars'])
+            if variables:
-            origin = 'var: %s' % origin
+                value, origin = self._loop_entries(variables, defs[config]['vars'])
-
+                origin = 'var: %s' % origin
-        # env vars are next precedence
+
-        if value is None and defs[config].get('env'):
+            # env vars are next precedence
-            value, origin = self._loop_entries(os.environ, defs[config]['env'])
+            if value is None and defs[config].get('env'):
-            origin = 'env: %s' % origin
+                value, origin = self._loop_entries(os.environ, defs[config]['env'])
-
+                origin = 'env: %s' % origin
-        # try config file entries next, if we have one
+
-        if value is None and cfile is not None:
+            # try config file entries next, if we have one
-            ftype = get_config_type(cfile)
+            if value is None and cfile is not None:
-            if ftype and defs[config].get(ftype):
+                ftype = get_config_type(cfile)
-                if ftype == 'ini':
+                if ftype and defs[config].get(ftype):
-                    # load from ini config
+                    if ftype == 'ini':
-                    try:  # FIXME: generaelize _loop_entries to allow for files also, most of this code is dupe
+                        # load from ini config
-                        for ini_entry in defs[config]['ini']:
+                        try:  # FIXME: generalize _loop_entries to allow for files also, most of this code is dupe
-                            temp_value = get_ini_config_value(self._parser, ini_entry)
+                            for ini_entry in defs[config]['ini']:
-                            if temp_value is not None:
+                                temp_value = get_ini_config_value(self._parser, ini_entry)
-                                value = temp_value
+                                if temp_value is not None:
-                                origin = cfile
+                                    value = temp_value
-                                if 'deprecated' in ini_entry:
+                                    origin = cfile
-                                    self.DEPRECATED.append(('[%s]%s' % (ini_entry['section'], ini_entry['key']), ini_entry['deprecated']))
+                                    if 'deprecated' in ini_entry:
-                    except Exception as e:
+                                        self.DEPRECATED.append(('[%s]%s' % (ini_entry['section'], ini_entry['key']), ini_entry['deprecated']))
-                        sys.stderr.write("Error while loading ini config %s: %s" % (cfile, to_native(e)))
+                        except Exception as e:
-                elif ftype == 'yaml':
+                            sys.stderr.write("Error while loading ini config %s: %s" % (cfile, to_native(e)))
-                    # FIXME: implement, also , break down key from defs (. notation???)
+                    elif ftype == 'yaml':
-                    origin = cfile
+                        # FIXME: implement, also , break down key from defs (. notation???)
-
+                        origin = cfile
-        '''
+
-        # for plugins, try using existing constants, this is for backwards compatiblity
+            '''
-        if plugin_name and defs[config].get('constants'):
+            # for plugins, try using existing constants, this is for backwards compatiblity
-            value, origin = self._loop_entries(self.data, defs[config]['constants'])
+            if plugin_name and defs[config].get('constants'):
-            origin = 'constant: %s' % origin
+                value, origin = self._loop_entries(self.data, defs[config]['constants'])
-        '''
+                origin = 'constant: %s' % origin
-
+            '''
-        # set default if we got here w/o a value
+
-        if value is None:
+            # set default if we got here w/o a value
-            value = defs[config].get('default')
+            if value is None:
-            origin = 'default'
+                value = defs[config].get('default')
-            # skip typing as this is a temlated default that will be resolved later in constants, which has needed vars
+                origin = 'default'
-            if plugin_type is None and isinstance(value, string_types) and (value.startswith('{{') and value.endswith('}}')):
+                # skip typing as this is a temlated default that will be resolved later in constants, which has needed vars
-                return value, origin
+                if plugin_type is None and isinstance(value, string_types) and (value.startswith('{{') and value.endswith('}}')):
-
+                    return value, origin
-        # ensure correct type
+
-        try:
+            # ensure correct type
-            value = ensure_type(value, defs[config].get('type'), origin=origin)
+            try:
-        except Exception as e:
+                value = ensure_type(value, defs[config].get('type'), origin=origin)
-            self.UNABLE.append(config)
+            except Exception as e:
-
+                self.UNABLE.append(config)
-        # deal with deprecation of the setting
+
-        if 'deprecated' in defs[config] and origin != 'default':
+            # deal with deprecation of the setting
-            self.DEPRECATED.append((config, defs[config].get('deprecated')))
+            if 'deprecated' in defs[config] and origin != 'default':
+                self.DEPRECATED.append((config, defs[config].get('deprecated')))
+        else:
+            raise AnsibleError('Requested option %s was not defined in configuration' % to_native(config))
 
         return value, origin
 

test/units/config/test.cfg

@@ -0,0 +1,4 @@
+[defaults]
+inikey=fromini
+matterless=lessfromini
+mattermore=morefromini

test/units/config/test.yml

@@ -0,0 +1,55 @@
+# mock config defs with diff use cases
+config_entry: &entry
+    name: test config
+    default: DEFAULT
+    description:
+      - This does nothing, its for testing
+    env:
+      - name: ENVVAR
+    ini:
+      - section: defaults
+        key: inikey
+    type: string
+config_entry_multi: &entry_multi
+    name: has more than one entry per config source
+    default: DEFAULT
+    description:
+      - This does nothing, its for testing
+    env:
+      - name: MATTERLESS
+      - name: MATTERMORE
+    ini:
+      - section: defaults
+        key: matterless
+      - section: defaults
+        key: mattermore
+    type: string
+config_entry_bool:
+    <<: *entry
+    type: bool
+    default: False
+config_entry_list:
+    <<: *entry
+    type: list
+    default: [DEFAULT]
+config_entry_deprecated:
+    <<: *entry
+    deprecated: &dep
+        why: 'cause i wanna'
+        version: 9.2
+        alternative: 'none whatso ever'
+config_entry_multi_deprecated:
+    <<: *entry_multi
+    deprecated: *dep
+config_entry_multi_deprecated_source:
+    <<: *entry_multi
+    env:
+      - name: MATTERLESS
+        deprecated: *dep
+      - name: MATTERMORE
+    ini:
+      - section: defaults
+        key: matterless
+        deprecated: *dep
+      - section: defaults
+        key: mattermore

test/units/config/test2.cfg

@@ -0,0 +1,4 @@
+[defaults]
+inikey=fromini2
+matterless=lessfromini2
+mattermore=morefromini2

test/units/config/test_data.py

@@ -0,0 +1,41 @@
+# Make coding more python3-ish
+from __future__ import (absolute_import, division, print_function)
+__metaclass__ = type
+
+from ansible.compat.tests import unittest
+
+from ansible.config.data import ConfigData
+from ansible.config.manager import Setting
+
+
+mykey = Setting('mykey', 'myvalue', 'test', 'string')
+mykey2 = Setting('mykey2', 'myvalue2', ['test', 'test2'], 'list')
+mykey3 = Setting('mykey3', 'myvalue3', 11111111111, 'integer')
+
+
+class TestConfigData(unittest.TestCase):
+
+    def setUp(self):
+        self.cdata = ConfigData()
+
+    def tearDown(self):
+        self.cdata = None
+
+    def test_update_setting(self):
+        for setting in [mykey, mykey2, mykey3]:
+            self.cdata.update_setting(setting)
+            self.assertEqual(setting, self.cdata._global_settings.get(setting.name))
+
+    def test_update_setting_with_plugin(self):
+        pass
+
+    def test_get_setting(self):
+        self.cdata._global_settings = {'mykey': mykey}
+        self.assertEqual(mykey, self.cdata.get_setting('mykey'))
+
+    def test_get_settings(self):
+        all_settings = {'mykey': mykey, 'mykey2': mykey2}
+        self.cdata._global_settings = all_settings
+
+        for setting in self.cdata.get_settings():
+            self.assertEqual(all_settings[setting.name], setting)

test/units/config/test_manager.py

@@ -0,0 +1,47 @@
+# Make coding more python3-ish
+from __future__ import (absolute_import, division, print_function)
+__metaclass__ = type
+
+import os
+
+from ansible.compat.tests import unittest
+
+from ansible.config.manager import ConfigManager, Setting
+
+curdir = os.path.dirname(__file__)
+cfg_file = os.path.join(curdir, 'test.cfg')
+cfg_file2 = os.path.join(curdir, 'test2.cfg')
+
+expected_ini = {'CONFIG_FILE': Setting(name='CONFIG_FILE', value=cfg_file, origin='', type='string'),
+                'config_entry': Setting(name='config_entry', value=u'fromini', origin=cfg_file, type='string'),
+                'config_entry_bool': Setting(name='config_entry_bool', value=False, origin=cfg_file, type='bool'),
+                'config_entry_list': Setting(name='config_entry_list', value=['fromini'], origin=cfg_file, type='list'),
+                'config_entry_deprecated': Setting(name='config_entry_deprecated', value=u'fromini', origin=cfg_file, type='string'),
+                'config_entry_multi': Setting(name='config_entry_multi', value=u'morefromini', origin=cfg_file, type='string'),
+                'config_entry_multi_deprecated': Setting(name='config_entry_multi_deprecated', value=u'morefromini', origin=cfg_file, type='string'),
+                'config_entry_multi_deprecated_source': Setting(name='config_entry_multi_deprecated_source', value=u'morefromini',
+                                                                origin=cfg_file, type='string')}
+
+
+class TestConfigData(unittest.TestCase):
+
+    def setUp(self):
+        self.manager = ConfigManager(os.path.join(curdir, 'test.cfg'), os.path.join(curdir, 'test.yml'))
+
+    def tearDown(self):
+        self.manager = None
+
+    def test_initial_load(self):
+        self.assertEquals(self.manager.data._global_settings, expected_ini)
+
+    def test_value_and_origin_from_ini(self):
+        self.assertEquals(self.manager.get_config_value_and_origin('config_entry'), ('fromini', cfg_file))
+
+    def test_value_from_ini(self):
+        self.assertEquals(self.manager.get_config_value('config_entry'), 'fromini')
+
+    def test_value_and_origin_from_alt_ini(self):
+        self.assertEquals(self.manager.get_config_value_and_origin('config_entry', cfile=cfg_file2), ('fromini2', cfg_file2))
+
+    def test_value_from_alt_ini(self):
+        self.assertEquals(self.manager.get_config_value('config_entry', cfile=cfg_file2), 'fromini2')