Refactored skipIfCustomUser into the contrib.auth tests.

django/contrib/auth/tests/auth_backends.py

@@ -2,9 +2,10 @@ from __future__ import unicode_literals
 
 from django.conf import settings
 from django.contrib.auth.models import User, Group, Permission, AnonymousUser
+from django.contrib.auth.tests.utils import skipIfCustomUser
 from django.contrib.contenttypes.models import ContentType
 from django.core.exceptions import ImproperlyConfigured
-from django.test import TestCase, skipIfCustomUser
+from django.test import TestCase
 from django.test.utils import override_settings
 
 

django/contrib/auth/tests/basic.py

@@ -2,8 +2,9 @@ import locale
 
 from django.contrib.auth.management.commands import createsuperuser
 from django.contrib.auth.models import User, AnonymousUser
+from django.contrib.auth.tests.utils import skipIfCustomUser
 from django.core.management import call_command
-from django.test import TestCase, skipIfCustomUser
+from django.test import TestCase
 from django.utils.six import StringIO
 
 

django/contrib/auth/tests/context_processors.py

@@ -2,8 +2,9 @@ import os
 
 from django.conf import global_settings
 from django.contrib.auth import authenticate
+from django.contrib.auth.tests.utils import skipIfCustomUser
 from django.db.models import Q
-from django.test import TestCase, skipIfCustomUser
+from django.test import TestCase
 from django.test.utils import override_settings
 
 

django/contrib/auth/tests/decorators.py

@@ -1,7 +1,6 @@
-from django.conf import settings
 from django.contrib.auth.decorators import login_required
 from django.contrib.auth.tests.views import AuthViewsTestCase
-from django.test import skipIfCustomUser
+from django.contrib.auth.tests.utils import skipIfCustomUser
 
 
 @skipIfCustomUser

django/contrib/auth/tests/forms.py

@@ -4,9 +4,10 @@ import os
 from django.contrib.auth.models import User
 from django.contrib.auth.forms import (UserCreationForm, AuthenticationForm,
     PasswordChangeForm, SetPasswordForm, UserChangeForm, PasswordResetForm)
+from django.contrib.auth.tests.utils import skipIfCustomUser
 from django.core import mail
 from django.forms.fields import Field, EmailField
-from django.test import TestCase, skipIfCustomUser
+from django.test import TestCase
 from django.test.utils import override_settings
 from django.utils.encoding import force_text
 from django.utils import translation

django/contrib/auth/tests/management.py

@@ -5,9 +5,10 @@ from django.contrib.auth import models, management
 from django.contrib.auth.management.commands import changepassword
 from django.contrib.auth.models import User
 from django.contrib.auth.tests import CustomUser
+from django.contrib.auth.tests.utils import skipIfCustomUser
 from django.core.management import call_command
 from django.core.management.base import CommandError
-from django.test import TestCase, skipIfCustomUser
+from django.test import TestCase
 from django.test.utils import override_settings
 from django.utils import six
 from django.utils.six import StringIO

django/contrib/auth/tests/models.py

@@ -1,7 +1,8 @@
 from django.conf import settings
 from django.contrib.auth.models import (Group, User, SiteProfileNotAvailable,
     UserManager)
-from django.test import TestCase, skipIfCustomUser
+from django.contrib.auth.tests.utils import skipIfCustomUser
+from django.test import TestCase
 from django.test.utils import override_settings
 from django.utils import six
 

django/contrib/auth/tests/remote_user.py

@@ -3,7 +3,8 @@ from datetime import datetime
 from django.conf import settings
 from django.contrib.auth.backends import RemoteUserBackend
 from django.contrib.auth.models import User
-from django.test import TestCase, skipIfCustomUser
+from django.contrib.auth.tests.utils import skipIfCustomUser
+from django.test import TestCase
 from django.utils import timezone
 
 

django/contrib/auth/tests/signals.py

@@ -1,6 +1,7 @@
-from django.test import TestCase, skipIfCustomUser
-from django.test.utils import override_settings
 from django.contrib.auth import signals
+from django.contrib.auth.tests.utils import skipIfCustomUser
+from django.test import TestCase
+from django.test.utils import override_settings
 
 
 @skipIfCustomUser

django/contrib/auth/tests/tokens.py

@@ -4,7 +4,8 @@ from datetime import date, timedelta
 from django.conf import settings
 from django.contrib.auth.models import User
 from django.contrib.auth.tokens import PasswordResetTokenGenerator
-from django.test import TestCase, skipIfCustomUser
+from django.contrib.auth.tests.utils import skipIfCustomUser
+from django.test import TestCase
 from django.utils import unittest
 
 

django/contrib/auth/tests/utils.py

@@ -0,0 +1,9 @@
+from django.conf import settings
+from django.utils.unittest import skipIf
+
+
+def skipIfCustomUser(test_func):
+    """
+    Skip a test if a custom user model is in use.
+    """
+    return skipIf(settings.AUTH_USER_MODEL != 'auth.User', 'Custom user model in use')(test_func)

django/contrib/auth/tests/views.py

@@ -10,12 +10,13 @@ from django.http import QueryDict
 from django.utils.encoding import force_text
 from django.utils.html import escape
 from django.utils.http import urlquote
-from django.test import TestCase, skipIfCustomUser
+from django.test import TestCase
 from django.test.utils import override_settings
 
 from django.contrib.auth import SESSION_KEY, REDIRECT_FIELD_NAME
 from django.contrib.auth.forms import (AuthenticationForm, PasswordChangeForm,
                 SetPasswordForm, PasswordResetForm)
+from django.contrib.auth.tests.utils import skipIfCustomUser
 
 
 @override_settings(

django/test/__init__.py

@@ -5,6 +5,6 @@ Django Unit Test and Doctest framework.
 from django.test.client import Client, RequestFactory
 from django.test.testcases import (TestCase, TransactionTestCase,
     SimpleTestCase, LiveServerTestCase, skipIfDBFeature,
-    skipUnlessDBFeature, skipIfCustomUser
+    skipUnlessDBFeature
 )
 from django.test.utils import Approximate

django/test/testcases.py

@@ -48,13 +48,13 @@ from django.utils.unittest import skipIf
 from django.views.static import serve
 
 __all__ = ('DocTestRunner', 'OutputChecker', 'TestCase', 'TransactionTestCase',
-           'SimpleTestCase', 'skipIfDBFeature', 'skipUnlessDBFeature',
-           'skipIfCustomUser')
+           'SimpleTestCase', 'skipIfDBFeature', 'skipUnlessDBFeature')
 
 normalize_long_ints = lambda s: re.sub(r'(?<![\w])(\d+)L(?![\w])', '\\1', s)
 normalize_decimals = lambda s: re.sub(r"Decimal\('(\d+(\.\d*)?)'\)",
                                 lambda m: "Decimal(\"%s\")" % m.groups()[0], s)
 
+
 def to_list(value):
     """
     Puts value into a list if it's not already one.
@@ -908,13 +908,6 @@ def skipUnlessDBFeature(feature):
                          "Database doesn't support feature %s" % feature)
 
 
-def skipIfCustomUser(test_func):
-    """
-    Skip a test if a custom user model is in use.
-    """
-    return skipIf(settings.AUTH_USER_MODEL != 'auth.User', 'Custom user model in use')(test_func)
-
-
 class QuietWSGIRequestHandler(WSGIRequestHandler):
     """
     Just a regular WSGIRequestHandler except it doesn't log to the standard