diff options
author | Aymeric Augustin <aymeric.augustin@m4x.org> | 2013-12-11 23:31:34 +0100 |
---|---|---|
committer | Aymeric Augustin <aymeric.augustin@m4x.org> | 2013-12-17 10:17:44 +0100 |
commit | 8662654d6d50e4d89f771a757ae5fc83c6b74db0 (patch) | |
tree | 1b9d98842d25a5d15a389bcc0dcefdc4fcffef4f /tests/proxy_model_inheritance | |
parent | 334551339de38569ac3530886e3f9cc681190224 (diff) | |
download | django-8662654d6d50e4d89f771a757ae5fc83c6b74db0.tar.gz |
Removed module-level functions for the app cache.
Since the original ones in django.db.models.loading were kept only for
backwards compatibility, there's no need to recreate them. However, many
internals of Django still relied on them.
They were also imported in django.db.models. They never appear in the
documentation, except a quick mention of get_models and get_app in the
1.2 release notes to document an edge case in GIS. I don't think that
makes them a public API.
This commit doesn't change the overall amount of global state but
clarifies that it's tied to the app_cache object instead of hiding it
behind half a dozen functions.
Diffstat (limited to 'tests/proxy_model_inheritance')
-rw-r--r-- | tests/proxy_model_inheritance/tests.py | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/tests/proxy_model_inheritance/tests.py b/tests/proxy_model_inheritance/tests.py index 0fad8c594f..23058d122b 100644 --- a/tests/proxy_model_inheritance/tests.py +++ b/tests/proxy_model_inheritance/tests.py @@ -3,7 +3,7 @@ from __future__ import unicode_literals import os import sys -from django.apps.cache import cache, load_app +from django.apps import app_cache from django.conf import settings from django.core.management import call_command from django.test import TestCase, TransactionTestCase @@ -28,21 +28,21 @@ class ProxyModelInheritanceTests(TransactionTestCase): self.old_sys_path = sys.path[:] sys.path.append(os.path.dirname(os.path.abspath(upath(__file__)))) for app in settings.INSTALLED_APPS: - load_app(app) + app_cache.load_app(app) def tearDown(self): sys.path = self.old_sys_path - del cache.app_labels['app1'] - del cache.app_labels['app2'] - del cache.app_models['app1'] - del cache.app_models['app2'] + del app_cache.app_labels['app1'] + del app_cache.app_labels['app2'] + del app_cache.app_models['app1'] + del app_cache.app_models['app2'] def test_table_exists(self): try: - cache.set_available_apps(settings.INSTALLED_APPS) + app_cache.set_available_apps(settings.INSTALLED_APPS) call_command('migrate', verbosity=0) finally: - cache.unset_available_apps() + app_cache.unset_available_apps() from .app1.models import ProxyModel from .app2.models import NiceModel self.assertEqual(NiceModel.objects.all().count(), 0) |