Ticket #15064: 15064_DJANGO_SETTINGS_MODULE_ignored_2.diff

django/core/management/__init__.py

@@ -411 +411 @@
     if original_settings_path:
         os.environ['DJANGO_SETTINGS_MODULE'] = original_settings_path
     else:
-        os.environ['DJANGO_SETTINGS_MODULE'] = '%s.%s' % (project_name, settings_name)
+        #If DJANGO_SETTINGS_MODULE is already set, use it.
+        os.environ['DJANGO_SETTINGS_MODULE'] = os.environ.get(
+            'DJANGO_SETTINGS_MODULE',
+            '%s.%s' % (project_name, settings_name)
+        )
 
     # Import the project module. We add the parent directory to PYTHONPATH to
     # avoid some of the path errors new users can have.

tests/regressiontests/settings_tests/tests.py

@@ -76 +76 @@
         self.settings_module.MEDIA_URL = 'http://media.foo.com/stupid//'
         self.assertEqual('http://media.foo.com/stupid//',
                          self.settings_module.MEDIA_URL)
+
+
+class EnvironmentVariableTest(unittest.TestCase):
+    """
+    Ensures proper settings file is used.
+    """
+
+    def test_env_var_used(self):
+        """
+        If the environment variable is set, do
+        not ignore it. However, the kwarg
+        original_settings_path takes precedence.
+
+        This tests both plus the default (neither set).
+        """
+        import os
+        from django.core.management import setup_environ
+
+        #whatever was already there
+        original_module =  os.environ.get(
+            'DJANGO_SETTINGS_MODULE', 
+            'the default'
+        ) 
+
+        #environment variable set by user
+        user_override = 'custom.settings'
+
+        #optional argument to setup_environ
+        orig_path = 'original.path'
+
+        #expect default
+        setup_environ(global_settings)
+        self.assertEquals(
+            os.environ.get('DJANGO_SETTINGS_MODULE'),
+            original_module
+        )
+
+        #override with environment variable
+        os.environ['DJANGO_SETTINGS_MODULE'] = user_override
+        setup_environ(global_settings)
+
+        self.assertEquals(
+            os.environ.get('DJANGO_SETTINGS_MODULE'),
+            user_override
+        )
+
+        #pass in original_settings_path (takes precedence)
+        os.environ['DJANGO_SETTINGS_MODULE'] = user_override
+        setup_environ(global_settings, original_settings_path = orig_path)
+
+        self.assertEquals(
+            os.environ.get('DJANGO_SETTINGS_MODULE'),
+            orig_path
+        )
+
+        #return to factory defaults, since this changes
+        #things beyond just in this test
+        os.environ['DJANGO_SETTINGS_MODULE'] = original_module
+