Skip to content

Commit 8afde93

Browse files
committed
chore(controller): use get_user_model replace auth.User
1 parent ec55a53 commit 8afde93

33 files changed

Lines changed: 92 additions & 36 deletions

rootfs/api/models/blocklist.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,9 @@
11
from django.db import models
2-
from django.contrib.auth.models import User
2+
from django.contrib.auth import get_user_model
33
from api.models import UuidAuditedModel, App
44

5+
User = get_user_model()
6+
57

68
class Blocklist(UuidAuditedModel):
79
"""

rootfs/api/models/resource.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,6 @@ def plans(cls, serviceclass_name):
5454
if service["name"] == serviceclass_name:
5555
serviceclass_id = service["id"]
5656
break
57-
5857
plans = []
5958
if serviceclass_id is not None:
6059
for serviceplan in cls._scheduler.svcat.get_serviceplans().json()["items"]:

rootfs/api/serializers.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
from urllib.parse import urlparse
1313

1414
from django.conf import settings
15-
from django.contrib.auth.models import User
15+
from django.contrib.auth import get_user_model
1616
from django.utils import timezone
1717
from rest_framework import serializers
1818

@@ -23,6 +23,7 @@
2323
from api.exceptions import DryccException
2424

2525

26+
User = get_user_model()
2627
logger = logging.getLogger(__name__)
2728

2829
PROCTYPE_MATCH = re.compile(r'^(?P<type>[a-z0-9]+(\-[a-z0-9]+)*)$')

rootfs/api/settings/production.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@
112112
'api'
113113
)
114114

115-
DEFAULT_AUTO_FIELD = 'django.db.models.AutoField'
115+
DEFAULT_AUTO_FIELD = 'django.db.models.BigAutoField'
116116

117117
AUTHENTICATION_BACKENDS = (
118118
"django.contrib.auth.backends.ModelBackend",

rootfs/api/tests/test_api_middleware.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,12 +4,14 @@
44
Run the tests with "./manage.py test api"
55
"""
66

7-
from django.contrib.auth.models import User
7+
from django.contrib.auth import get_user_model
88
from rest_framework.authtoken.models import Token
99
from api.tests import DryccTestCase
1010

1111
from api import __version__
1212

13+
User = get_user_model()
14+
1315

1416
class APIMiddlewareTest(DryccTestCase):
1517

rootfs/api/tests/test_app.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
import requests
1212

1313
from django.conf import settings
14-
from django.contrib.auth.models import User
14+
from django.contrib.auth import get_user_model
1515
from django.core.cache import cache
1616
from django.test.utils import override_settings
1717
from rest_framework.authtoken.models import Token
@@ -23,6 +23,8 @@
2323
from api.tests import adapter, DryccTestCase
2424
import requests_mock
2525

26+
User = get_user_model()
27+
2628

2729
def mock_none(*args, **kwargs):
2830
return None

rootfs/api/tests/test_app_settings.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,16 @@
11
import requests_mock
22

33
from django.core.cache import cache
4-
from django.contrib.auth.models import User
4+
from django.contrib.auth import get_user_model
55
from rest_framework.authtoken.models import Token
66

77
from api.models import App
88
from unittest import mock
99
from scheduler import KubeException
1010
from api.tests import adapter, DryccTransactionTestCase
1111

12+
User = get_user_model()
13+
1214

1315
@requests_mock.Mocker(real_http=True, adapter=adapter)
1416
class TestAppSettings(DryccTransactionTestCase):

rootfs/api/tests/test_build.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
"""
66
import json
77

8-
from django.contrib.auth.models import User
8+
from django.contrib.auth import get_user_model
99
from django.core.cache import cache
1010
from django.conf import settings
1111
from django.test.utils import override_settings
@@ -18,6 +18,8 @@
1818
from api.tests import adapter, DryccTransactionTestCase
1919
import requests_mock
2020

21+
User = get_user_model()
22+
2123

2224
@requests_mock.Mocker(real_http=True, adapter=adapter)
2325
class BuildTest(DryccTransactionTestCase):

rootfs/api/tests/test_certificate.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,13 @@
1-
from django.contrib.auth.models import User
1+
from django.contrib.auth import get_user_model
22
from django.core.cache import cache
33
from rest_framework.authtoken.models import Token
44
from django.core.exceptions import SuspiciousOperation
55

66
from api.models import Certificate
77
from api.tests import TEST_ROOT, DryccTestCase
88

9+
User = get_user_model()
10+
911

1012
class CertificateTest(DryccTestCase):
1113

rootfs/api/tests/test_certificate_use_case_1.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,12 @@
1-
from django.contrib.auth.models import User
1+
from django.contrib.auth import get_user_model
22
from django.core.cache import cache
33
from rest_framework.authtoken.models import Token
44

55
from api.models import App, Certificate, Domain
66
from api.tests import TEST_ROOT, DryccTestCase
77

8+
User = get_user_model()
9+
810

911
class CertificateUseCase1Test(DryccTestCase):
1012

0 commit comments

Comments
 (0)