Skip to content

Commit

Permalink
call get_user_model in try block for better backwards compatibility
Browse files Browse the repository at this point in the history
  • Loading branch information
bhagany committed Feb 1, 2014
1 parent e823dc7 commit 080f6fa
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 10 deletions.
5 changes: 3 additions & 2 deletions tests/testsite/apps/api/tests/base.py
Expand Up @@ -4,6 +4,7 @@
except ImportError: import json
try:
from django.contrib.auth import get_user_model # Django 1.5+
User = get_user_model()
except:
from django.contrib.auth.models import User
from oauth2app.models import Client
Expand All @@ -30,14 +31,14 @@ class BaseTestCase(unittest.TestCase):
client_application = None

def setUp(self):
self.user = (get_user_model() or User).objects.create_user(
self.user = User.objects.create_user(
USER_USERNAME,
USER_EMAIL,
USER_PASSWORD)
self.user.first_name = USER_FIRSTNAME
self.user.last_name = USER_LASTNAME
self.user.save()
self.client = (get_user_model() or User).objects.create_user(CLIENT_USERNAME, CLIENT_EMAIL)
self.client = User.objects.create_user(CLIENT_USERNAME, CLIENT_EMAIL)
self.client_application = Client.objects.create(
name="TestApplication",
user=self.client)
Expand Down
6 changes: 3 additions & 3 deletions tests/testsite/apps/api/tests/mac.py
Expand Up @@ -8,6 +8,7 @@
from django.utils import unittest
try:
from django.contrib.auth import get_user_model # Django 1.5+
User = get_user_model()
except:
from django.contrib.auth.models import User
from oauth2app.models import Client
Expand All @@ -31,14 +32,14 @@ class MACTestCase(unittest.TestCase):
client_application = None

def setUp(self):
self.user = (get_user_model() or User).objects.create_user(
self.user = User.objects.create_user(
USER_USERNAME,
USER_EMAIL,
USER_PASSWORD)
self.user.first_name = USER_FIRSTNAME
self.user.last_name = USER_LASTNAME
self.user.save()
self.client = (get_user_model() or User).objects.create_user(CLIENT_USERNAME, CLIENT_EMAIL)
self.client = User.objects.create_user(CLIENT_USERNAME, CLIENT_EMAIL)
self.client_application = Client.objects.create(
name="TestApplication",
user=self.client)
Expand Down Expand Up @@ -70,4 +71,3 @@ def test_00_mac(self):
parameters,
HTTP_AUTHORIZATION="Basic %s" % basic_auth)
token = json.loads(response.content)

5 changes: 3 additions & 2 deletions tests/testsite/apps/api/tests/responsetype.py
Expand Up @@ -7,6 +7,7 @@
from django.test.client import Client as DjangoTestClient
try:
from django.contrib.auth import get_user_model # Django 1.5+
User = get_user_model()
except:
from django.contrib.auth.models import User
from oauth2app.models import Client
Expand All @@ -29,14 +30,14 @@ class ResponseTypeTestCase(unittest.TestCase):
client_application = None

def setUp(self):
self.user = (get_user_model() or User).objects.create_user(
self.user = User.objects.create_user(
USER_USERNAME,
USER_EMAIL,
USER_PASSWORD)
self.user.first_name = USER_FIRSTNAME
self.user.last_name = USER_LASTNAME
self.user.save()
self.client = (get_user_model() or User).objects.create_user(CLIENT_USERNAME, CLIENT_EMAIL)
self.client = User.objects.create_user(CLIENT_USERNAME, CLIENT_EMAIL)
self.client_application = Client.objects.create(
name="TestApplication",
user=self.client)
Expand Down
6 changes: 3 additions & 3 deletions tests/testsite/apps/api/tests/scope.py
Expand Up @@ -9,6 +9,7 @@
from django.test.client import Client as DjangoTestClient
try:
from django.contrib.auth import get_user_model # Django 1.5+
User = get_user_model()
except:
from django.contrib.auth.models import User
from oauth2app.models import Client
Expand All @@ -31,14 +32,14 @@ class ScopeTestCase(unittest.TestCase):
client_application = None

def setUp(self):
self.user = (get_user_model() or User).objects.create_user(
self.user = User.objects.create_user(
USER_USERNAME,
USER_EMAIL,
USER_PASSWORD)
self.user.first_name = USER_FIRSTNAME
self.user.last_name = USER_LASTNAME
self.user.save()
self.client = (get_user_model() or User).objects.create_user(CLIENT_USERNAME, CLIENT_EMAIL)
self.client = User.objects.create_user(CLIENT_USERNAME, CLIENT_EMAIL)
self.client_application = Client.objects.create(
name="TestApplication",
user=self.client)
Expand Down Expand Up @@ -176,4 +177,3 @@ def test_02_dual_scope(self):
HTTP_AUTHORIZATION="Bearer %s" % token)
self.assertEqual(response.status_code, 200)
self.assertEqual(response.content, USER_FIRSTNAME)

0 comments on commit 080f6fa

Please sign in to comment.