0.2.3

-----

* Now `include` function finds parent `globals()` scope automatically
* added protection against infinite recursion with tests
* added tests for stackable settings definition. Check settings example in tests/settings/stacked/.

CHANGELOG.rst

@@ -2,6 +2,11 @@
 Changelog
 =========
 
+0.2.3
+-----
+
+* Now `include` function finds parent `globals()` scope automatically
+
 0.2.2
 -----
 

split_settings/__init__.py

@@ -6,6 +6,6 @@
 settings files.
 """
 
-__version__ = '0.2.2'
+__version__ = '0.2.3'
 
 __all__ = ['__version__']

split_settings/tools.py

@@ -8,6 +8,7 @@
 """
 
 import glob
+import inspect
 import os
 import sys
 import types
@@ -64,7 +65,18 @@ def include(*args, **kwargs):
         IOError: if a required settings file is not found
     """
 
-    scope = kwargs.pop('scope')
+    if 'scope' not in kwargs:
+        stack = inspect.stack()
+        if len(stack) < 2:
+            raise TypeError("Can't find parent globals(), positional argument 'scope' is required.")
+
+        scope = stack[1][0].f_globals
+    else:
+        scope = kwargs.pop('scope')
+
+    scope.setdefault('__included_files__', [])
+    included_files = scope.get('__included_files__')
+
     including_file = scope.get('__included_file__',
                                scope['__file__'].rstrip('c'))
     conf_path = os.path.dirname(including_file)
@@ -79,6 +91,11 @@ def include(*args, **kwargs):
             raise IOError('No such file: %s' % pattern)
 
         for included_file in files_to_include:
+            if included_file in included_files:
+                continue
+
+            included_files.append(included_file)
+
             scope['__included_file__'] = included_file
             with open(included_file, 'rb') as to_compile:
                 exec(compile(to_compile.read(), included_file, 'exec'), scope)

tests/settings/recursion/__init__.py

@@ -0,0 +1,14 @@
+# -*- coding: utf-8 -*-
+
+import os
+
+import django
+
+from split_settings.tools import optional, include
+
+# Must bypass this block if another settings module was specified.
+if os.environ['DJANGO_SETTINGS_MODULE'] == 'tests.settings':
+
+    include(
+        '*.py'
+    )

tests/settings/recursion/ok.py

@@ -0,0 +1 @@
+RECURSION_OK = True

tests/settings/stacked/__init__.py

@@ -0,0 +1,5 @@
+from split_settings.tools import include
+
+include(
+    'base.py'
+)

tests/settings/stacked/base.py

@@ -0,0 +1,7 @@
+from split_settings.tools import include
+
+STACKED_BASE_LOADED = True
+
+include(
+    'db/*.py'
+)

tests/settings/stacked/db/persistent.py

@@ -0,0 +1,2 @@
+if STACKED_BASE_LOADED:
+    STACKED_DB_PERSISTENT = True

tests/test_tools.py

@@ -36,8 +36,8 @@ def test_keys_count():
         scope=scope,
     )
 
-    # Keys: 'FIXTURE_VALUE', '__file__', '__doc__', '__builtins__'
-    assert len(scope.keys()) == 4
+    # Keys: 'FIXTURE_VALUE', '__file__', '__doc__', '__builtins__', '__included_files__'
+    assert len(scope.keys()) == 5
 
 
 def test_included_file_scope():
@@ -87,3 +87,36 @@ def test_unicode_passed():
     )
 
     assert 'FIXTURE_VALUE' in scope
+
+
+def test_caller_scope_automatically():
+    """
+    Tests `include` function for automatic `globals()` extraction from execution stack.
+    Now you can omit positional argument `scope`.
+    """
+
+    include(
+        six.text_type(FIXTURE)
+    )
+
+    assert 'FIXTURE_VALUE' in globals()
+
+
+def test_recursion_inclusion():
+    """
+    Tests `include` function for inclusion files only once. It protects of infinite recursion.
+    """
+
+    from tests.settings import recursion
+
+    assert hasattr(recursion, 'RECURSION_OK')
+
+def test_stacked_settings():
+    """
+    Tests `include` function for inclusion files only once. It protects of infinite recursion.
+    """
+
+    from tests.settings import stacked
+
+    assert hasattr(stacked, 'STACKED_BASE_LOADED')
+    assert hasattr(stacked, 'STACKED_DB_PERSISTENT')