Skip to content

Commit 74a9c3a

Browse files
Use django_dynamic_fixture's new/get instead of N/G
Based on feedback: #1503 (comment)
1 parent 43da778 commit 74a9c3a

File tree

2 files changed

+14
-13
lines changed

2 files changed

+14
-13
lines changed

readthedocs/rtd_tests/tests/test_subproject.py

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,15 @@
11
from django.contrib.auth.models import User
22
from django.test import TestCase
3-
from django_dynamic_fixture import G
3+
from django_dynamic_fixture import get
44

55
from readthedocs.projects.forms import SubprojectForm
66
from readthedocs.projects.models import Project
77

88

99
class SubprojectFormTests(TestCase):
1010
def test_name_validation(self):
11-
user = G(User)
12-
project = G(Project, slug='mainproject')
11+
user = get(User)
12+
project = get(Project, slug='mainproject')
1313

1414
form = SubprojectForm({},
1515
parent=project, user=user)
@@ -25,10 +25,10 @@ def test_adding_subproject_fails_when_user_is_not_admin(self):
2525
# Make sure that a user cannot add a subproject that he is not the
2626
# admin of.
2727

28-
user = G(User)
29-
project = G(Project, slug='mainproject')
28+
user = get(User)
29+
project = get(Project, slug='mainproject')
3030
project.users.add(user)
31-
subproject = G(Project, slug='subproject')
31+
subproject = get(Project, slug='subproject')
3232

3333
form = SubprojectForm({'subproject': subproject.slug},
3434
parent=project, user=user)
@@ -37,10 +37,10 @@ def test_adding_subproject_fails_when_user_is_not_admin(self):
3737
self.assertTrue('subproject' in form.errors)
3838

3939
def test_admin_of_subproject_can_add_it(self):
40-
user = G(User)
41-
project = G(Project, slug='mainproject')
40+
user = get(User)
41+
project = get(Project, slug='mainproject')
4242
project.users.add(user)
43-
subproject = G(Project, slug='subproject')
43+
subproject = get(Project, slug='subproject')
4444
subproject.users.add(user)
4545

4646
# Works now as user is admin of subproject.

readthedocs/rtd_tests/tests/test_views.py

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,8 @@
22
from django.core.urlresolvers import reverse
33
from django.test import TestCase
44
from django.utils.six.moves.urllib.parse import urlsplit
5-
from django_dynamic_fixture import G, N
5+
from django_dynamic_fixture import get
6+
from django_dynamic_fixture import new
67

78
from readthedocs.builds.constants import LATEST
89
from readthedocs.projects.models import Project
@@ -173,12 +174,12 @@ def test_project_redirects_delete(self):
173174

174175
class SubprojectViewTests(TestCase):
175176
def setUp(self):
176-
self.user = N(User, username='test')
177+
self.user = new(User, username='test')
177178
self.user.set_password('test')
178179
self.user.save()
179180

180-
self.project = G(Project, slug='my-mainproject')
181-
self.subproject = G(Project, slug='my-subproject')
181+
self.project = get(Project, slug='my-mainproject')
182+
self.subproject = get(Project, slug='my-subproject')
182183
self.project.add_subproject(self.subproject)
183184

184185
self.client.login(username='test', password='test')

0 commit comments

Comments
 (0)