Fixes load_values in PlainConfig (#167)

* Fixes #165 - Fixes `load_values` for `PlainConfig` where accessing item with `__getitem__` returns value so lower level method needs to be used in lookup
* Tested with Python 3.6 and 3.7
* Warn users about the state of the library
Thanks @haizaar

Author: jbasko

.bumpversion.cfg

@@ -1,5 +1,5 @@
 [bumpversion]
-current_version = 1.34.0
+current_version = 1.34.1
 commit = true
 tag = false
 

README.rst

@@ -1,6 +1,12 @@
 configmanager
 =============
 
+**Update on 2018-10-20**: I no longer use this library. It tries to do too many things. It was written in
+Python 2.7 and made work with Python 3. With type hints, dataclasses, and many other cool features in Python 3.6+
+you can express the same things in a much nicer way than this.
+
+----
+
 .. image:: https://travis-ci.org/jbasko/configmanager.svg?branch=master
     :target: https://travis-ci.org/jbasko/configmanager
 

configmanager/__init__.py

@@ -1,4 +1,4 @@
-__version__ = '1.34.0'
+__version__ = '1.34.1'
 
 from .managers import Config
 from .items import Item

configmanager/sections.py

@@ -574,14 +574,17 @@ def load_values(self, dictionary, as_defaults=False, flat=False):
                         self[name] = self.create_item(name, default=value)
                 else:
                     # Skip unknown names if not interpreting dictionary as defaults
-                    continue
-            elif is_config_item(self[name]):
+                    pass
+                continue
+
+            resolution = self._get_item_or_section(name, handle_not_found=False)
+            if is_config_item(resolution):
                 if as_defaults:
-                    self[name].default = value
+                    resolution.default = value
                 else:
-                    self[name].value = value
+                    resolution.value = value
             else:
-                self[name].load_values(value, as_defaults=as_defaults)
+                resolution.load_values(value, as_defaults=as_defaults)
 
     def create_item(self, *args, **kwargs):
         """

setup.py

@@ -39,6 +39,8 @@ def read(fname):
         'Programming Language :: Python :: 2.7',
         'Programming Language :: Python :: 3',
         'Programming Language :: Python :: 3.5',
+        'Programming Language :: Python :: 3.6',
+        'Programming Language :: Python :: 3.7',
         'License :: OSI Approved :: MIT License',
     ],
 )

tests/test_yaml.py

@@ -1,7 +1,7 @@
 import pytest
 import six
 
-from configmanager import Config
+from configmanager import Config, PlainConfig
 
 
 @pytest.fixture
@@ -31,6 +31,15 @@ def test_config_written_to_and_read_from_yaml_file(yaml_path1):
     assert config2.dump_values() == original_values
 
 
+def test_plain_config_loaded_from_yaml_file(yaml_path1):
+    with open(yaml_path1, "w") as f:
+        f.write("name: bar\n")
+
+    config = PlainConfig(schema={"name": "foo"})
+    config.yaml.load(yaml_path1)
+    assert config["name"] == "bar"
+
+
 def test_config_written_to_and_read_from_yaml_string():
     config_str = (
         'uploads:\n'

tox.ini

@@ -1,6 +1,6 @@
 # For more information about tox, see https://tox.readthedocs.io/en/latest/
 [tox]
-envlist = py27,py35
+envlist = py{27,35,36,37}
 skip_missing_interpreters = True
 
 [testenv]