From c2b4967e76fd671e6199e4dd54d2a2c1f096b8eb Mon Sep 17 00:00:00 2001 From: Tomasz Kontusz Date: Wed, 3 Jun 2015 17:05:03 +0200 Subject: [PATCH] Fixed ImportError message in utils.module_loading.import_string() --- django/utils/module_loading.py | 2 +- tests/utils_tests/test_module_loading.py | 4 +++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/django/utils/module_loading.py b/django/utils/module_loading.py index 5dc3aefe758..c42d2081785 100644 --- a/django/utils/module_loading.py +++ b/django/utils/module_loading.py @@ -23,7 +23,7 @@ def import_string(dotted_path): return getattr(module, class_name) except AttributeError: msg = 'Module "%s" does not define a "%s" attribute/class' % ( - dotted_path, class_name) + module_path, class_name) six.reraise(ImportError, ImportError(msg), sys.exc_info()[2]) diff --git a/tests/utils_tests/test_module_loading.py b/tests/utils_tests/test_module_loading.py index 8d600cb6f0e..fb9ccbf5d6a 100644 --- a/tests/utils_tests/test_module_loading.py +++ b/tests/utils_tests/test_module_loading.py @@ -115,7 +115,9 @@ class ModuleImportTestCase(unittest.TestCase): # Test exceptions raised self.assertRaises(ImportError, import_string, 'no_dots_in_path') - self.assertRaises(ImportError, import_string, 'utils_tests.unexistent') + msg = 'Module "utils_tests" does not define a "unexistent" attribute' + with six.assertRaisesRegex(self, ImportError, msg): + import_string('utils_tests.unexistent') @modify_settings(INSTALLED_APPS={'append': 'utils_tests.test_module'})