Skip to content

Commit b99eecc

Browse files
committed
Bring the factory-boy and Faker dependencies forward through some minor breaking changes. Stop using the deprecated fr_QC locale in Faker. Commit ready for merge.
- Legacy-Id: 19270
1 parent 47082fe commit b99eecc

15 files changed

Lines changed: 60 additions & 56 deletions

File tree

ietf/dbtemplate/factories.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
from ietf.dbtemplate.models import DBTemplate
44

5-
class DBTemplateFactory(factory.DjangoModelFactory):
5+
class DBTemplateFactory(factory.django.DjangoModelFactory):
66
class Meta:
77
model = DBTemplate
88

ietf/doc/factories.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ def draft_name_generator(type_id,group,n):
2929
n,
3030
)
3131

32-
class BaseDocumentFactory(factory.DjangoModelFactory):
32+
class BaseDocumentFactory(factory.django.DjangoModelFactory):
3333
class Meta:
3434
model = Document
3535

@@ -259,7 +259,7 @@ class ReviewFactory(BaseDocumentFactory):
259259
name = factory.LazyAttribute(lambda o: 'review-doesnotexist-00-%s-%s'%(o.group.acronym,datetime.date.today().isoformat()))
260260
group = factory.SubFactory('ietf.group.factories.GroupFactory',type_id='review')
261261

262-
class DocAliasFactory(factory.DjangoModelFactory):
262+
class DocAliasFactory(factory.django.DjangoModelFactory):
263263
class Meta:
264264
model = DocAlias
265265

@@ -276,7 +276,7 @@ def docs(self, create, extracted, **kwargs):
276276
self.docs.add(doc)
277277

278278

279-
class DocEventFactory(factory.DjangoModelFactory):
279+
class DocEventFactory(factory.django.DjangoModelFactory):
280280
class Meta:
281281
model = DocEvent
282282

@@ -326,7 +326,7 @@ def state(obj, create, extracted, **kwargs):
326326
obj.save()
327327

328328
# All of these Ballot* factories are extremely skeletal. Flesh them out as needed by tests.
329-
class BallotTypeFactory(factory.DjangoModelFactory):
329+
class BallotTypeFactory(factory.django.DjangoModelFactory):
330330
class Meta:
331331
model = BallotType
332332
django_get_or_create = ('slug','doc_type_id')
@@ -363,14 +363,14 @@ class Meta:
363363
balloter = factory.SubFactory('ietf.person.factories.PersonFactory')
364364
pos_id = 'discuss'
365365

366-
class DocumentActionHolderFactory(factory.DjangoModelFactory):
366+
class DocumentActionHolderFactory(factory.django.DjangoModelFactory):
367367
class Meta:
368368
model = DocumentActionHolder
369369

370370
document = factory.SubFactory(WgDraftFactory)
371371
person = factory.SubFactory('ietf.person.factories.PersonFactory')
372372

373-
class DocumentAuthorFactory(factory.DjangoModelFactory):
373+
class DocumentAuthorFactory(factory.django.DjangoModelFactory):
374374
class Meta:
375375
model = DocumentAuthor
376376

ietf/group/factories.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
GroupHistory, RoleHistory
1010
from ietf.review.factories import ReviewTeamSettingsFactory
1111

12-
class GroupFactory(factory.DjangoModelFactory):
12+
class GroupFactory(factory.django.DjangoModelFactory):
1313
class Meta:
1414
model = Group
1515
django_get_or_create = ('acronym',)
@@ -39,15 +39,15 @@ class ReviewTeamFactory(GroupFactory):
3939
def settings(obj, create, extracted, **kwargs):
4040
ReviewTeamSettingsFactory.create(group=obj,**kwargs)
4141

42-
class RoleFactory(factory.DjangoModelFactory):
42+
class RoleFactory(factory.django.DjangoModelFactory):
4343
class Meta:
4444
model = Role
4545

4646
group = factory.SubFactory(GroupFactory)
4747
person = factory.SubFactory('ietf.person.factories.PersonFactory')
4848
email = factory.LazyAttribute(lambda obj: obj.person.email())
4949

50-
class GroupEventFactory(factory.DjangoModelFactory):
50+
class GroupEventFactory(factory.django.DjangoModelFactory):
5151
class Meta:
5252
model = GroupEvent
5353

@@ -56,7 +56,7 @@ class Meta:
5656
type = 'comment'
5757
desc = factory.Faker('paragraph')
5858

59-
class BaseGroupMilestoneFactory(factory.DjangoModelFactory):
59+
class BaseGroupMilestoneFactory(factory.django.DjangoModelFactory):
6060
class Meta:
6161
model = GroupMilestone
6262

@@ -72,7 +72,7 @@ class DatelessGroupMilestoneFactory(BaseGroupMilestoneFactory):
7272
group = factory.SubFactory(GroupFactory, uses_milestone_dates=False)
7373
order = factory.Sequence(lambda n: n)
7474

75-
class GroupHistoryFactory(factory.DjangoModelFactory):
75+
class GroupHistoryFactory(factory.django.DjangoModelFactory):
7676
class Meta:
7777
model=GroupHistory
7878

@@ -86,7 +86,7 @@ class Meta:
8686
group = factory.SubFactory(GroupFactory)
8787
acronym = factory.LazyAttribute(lambda obj: obj.group.acronym)
8888

89-
class RoleHistoryFactory(factory.DjangoModelFactory):
89+
class RoleHistoryFactory(factory.django.DjangoModelFactory):
9090
class Meta:
9191
model=RoleHistory
9292

ietf/iesg/factories.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
from ietf.iesg.models import TelechatAgendaItem
88

99

10-
class IESGMgmtItemFactory(factory.DjangoModelFactory):
10+
class IESGMgmtItemFactory(factory.django.DjangoModelFactory):
1111
class Meta:
1212
model = TelechatAgendaItem
1313

ietf/ietfauth/factories.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111

1212
from ietf.person.factories import UserFactory, PersonFactory
1313

14-
class OidClientRecordFactory(factory.DjangoModelFactory):
14+
class OidClientRecordFactory(factory.django.DjangoModelFactory):
1515
class Meta:
1616
model = OidClientRecord
1717

ietf/ipr/factories.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -14,13 +14,13 @@
1414
def _fake_patent_info():
1515
return "Date: %s\nNotes: %s\nTitle: %s\nNumber: %s\nInventor: %s\n" % (
1616
(datetime.datetime.today()-datetime.timedelta(days=365)).strftime("%Y-%m-%d"),
17-
factory.Faker('paragraph').generate({}),
18-
factory.Faker('sentence', nb_words=8).generate({}),
17+
factory.Faker('paragraph'),
18+
factory.Faker('sentence', nb_words=8),
1919
'US9999999',
20-
factory.Faker('name').generate({}),
20+
factory.Faker('name'),
2121
)
2222

23-
class IprDisclosureBaseFactory(factory.DjangoModelFactory):
23+
class IprDisclosureBaseFactory(factory.django.DjangoModelFactory):
2424
class Meta:
2525
model = IprDisclosureBase
2626

@@ -83,7 +83,7 @@ class Meta:
8383
holder_contact_email = factory.Faker('email')
8484
holder_contact_name = factory.Faker('name')
8585

86-
class IprEventFactory(factory.DjangoModelFactory):
86+
class IprEventFactory(factory.django.DjangoModelFactory):
8787
class Meta:
8888
model = IprEvent
8989

ietf/liaisons/factories.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
from ietf.group.factories import GroupFactory
44
from ietf.liaisons.models import LiaisonStatement, LiaisonStatementEvent, LiaisonStatementAttachment
55

6-
class LiaisonStatementFactory(factory.DjangoModelFactory):
6+
class LiaisonStatementFactory(factory.django.DjangoModelFactory):
77
class Meta:
88
model = LiaisonStatement
99

@@ -30,7 +30,7 @@ def to_groups(obj, create, extracted, **kwargs):
3030
obj.to_groups.add(GroupFactory(type_id='wg'))
3131

3232

33-
class LiaisonStatementEventFactory(factory.DjangoModelFactory):
33+
class LiaisonStatementEventFactory(factory.django.DjangoModelFactory):
3434
class Meta:
3535
model = LiaisonStatementEvent
3636

@@ -40,7 +40,7 @@ class Meta:
4040
desc = factory.Faker('sentence')
4141

4242

43-
class LiaisonStatementAttachmentFactory(factory.DjangoModelFactory):
43+
class LiaisonStatementAttachmentFactory(factory.django.DjangoModelFactory):
4444
class Meta:
4545
model = LiaisonStatementAttachment
4646

ietf/mailinglists/factories.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77

88
from ietf.mailinglists.models import List
99

10-
class ListFactory(factory.DjangoModelFactory):
10+
class ListFactory(factory.django.DjangoModelFactory):
1111
class Meta:
1212
model = List
1313

ietf/meeting/factories.py

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
from ietf.group.factories import GroupFactory
1414
from ietf.person.factories import PersonFactory
1515

16-
class MeetingFactory(factory.DjangoModelFactory):
16+
class MeetingFactory(factory.django.DjangoModelFactory):
1717
class Meta:
1818
model = Meeting
1919

@@ -75,7 +75,7 @@ def populate_schedule(obj, create, extracted, **kwargs): # pylint: disable=no-se
7575
obj.schedule = ScheduleFactory(meeting=obj)
7676
obj.save()
7777

78-
class SessionFactory(factory.DjangoModelFactory):
78+
class SessionFactory(factory.django.DjangoModelFactory):
7979
class Meta:
8080
model = Session
8181

@@ -116,15 +116,15 @@ def add_to_schedule(obj, create, extracted, **kwargs): # pylint: disable=no-self
116116
ts = obj.meeting.timeslot_set.all()
117117
obj.timeslotassignments.create(timeslot=ts[random.randrange(len(ts))],schedule=obj.meeting.schedule)
118118

119-
class ScheduleFactory(factory.DjangoModelFactory):
119+
class ScheduleFactory(factory.django.DjangoModelFactory):
120120
class Meta:
121121
model = Schedule
122122

123123
meeting = factory.SubFactory(MeetingFactory)
124124
name = factory.Sequence(lambda n: 'schedule_%d'%n)
125125
owner = factory.SubFactory(PersonFactory)
126126

127-
class RoomFactory(factory.DjangoModelFactory):
127+
class RoomFactory(factory.django.DjangoModelFactory):
128128
class Meta:
129129
model = Room
130130

@@ -140,7 +140,7 @@ def session_types(obj, create, extracted, **kwargs): # pylint: disable=no-self-a
140140
obj.session_types.add(st)
141141

142142

143-
class TimeSlotFactory(factory.DjangoModelFactory):
143+
class TimeSlotFactory(factory.django.DjangoModelFactory):
144144
class Meta:
145145
model = TimeSlot
146146

@@ -164,7 +164,7 @@ def time(self):
164164
def duration(self):
165165
return datetime.timedelta(minutes=30+random.randrange(9)*15)
166166

167-
class SessionPresentationFactory(factory.DjangoModelFactory):
167+
class SessionPresentationFactory(factory.django.DjangoModelFactory):
168168
class Meta:
169169
model = SessionPresentation
170170

@@ -174,7 +174,7 @@ class Meta:
174174
def rev(self):
175175
return self.document.rev
176176

177-
class FloorPlanFactory(factory.DjangoModelFactory):
177+
class FloorPlanFactory(factory.django.DjangoModelFactory):
178178
class Meta:
179179
model = FloorPlan
180180

@@ -190,7 +190,7 @@ class Meta:
190190
)
191191
)
192192

193-
class SlideSubmissionFactory(factory.DjangoModelFactory):
193+
class SlideSubmissionFactory(factory.django.DjangoModelFactory):
194194
class Meta:
195195
model = SlideSubmission
196196

ietf/nomcom/factories.py

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,8 @@
55
import factory
66
import random
77

8+
from faker import Faker
9+
810
from ietf.nomcom.models import NomCom, Position, Feedback, Nominee, NomineePosition, Nomination, Topic
911
from ietf.group.factories import GroupFactory
1012
from ietf.person.factories import PersonFactory, UserFactory
@@ -79,7 +81,7 @@ def nomcom_kwargs_for_year(year=None, *args, **kwargs):
7981
return kwargs
8082

8183

82-
class NomComFactory(factory.DjangoModelFactory):
84+
class NomComFactory(factory.django.DjangoModelFactory):
8385
class Meta:
8486
model = NomCom
8587

@@ -137,7 +139,7 @@ def populate_topics(obj, create, extracted, **kwargs): # pylint: disable=no-self
137139
for i in range(3):
138140
TopicFactory(nomcom=obj)
139141

140-
class PositionFactory(factory.DjangoModelFactory):
142+
class PositionFactory(factory.django.DjangoModelFactory):
141143
class Meta:
142144
model = Position
143145

@@ -146,23 +148,23 @@ class Meta:
146148
accepting_nominations = True
147149
accepting_feedback = True
148150

149-
class NomineeFactory(factory.DjangoModelFactory):
151+
class NomineeFactory(factory.django.DjangoModelFactory):
150152
class Meta:
151153
model = Nominee
152154

153155
nomcom = factory.SubFactory(NomComFactory)
154156
person = factory.SubFactory(PersonFactory)
155157
email = factory.LazyAttribute(lambda obj: obj.person.email())
156158

157-
class NomineePositionFactory(factory.DjangoModelFactory):
159+
class NomineePositionFactory(factory.django.DjangoModelFactory):
158160
class Meta:
159161
model = NomineePosition
160162

161163
position = factory.SubFactory(PositionFactory)
162164
nominee = factory.SubFactory(NomineeFactory)
163165
state_id = 'accepted'
164166

165-
class FeedbackFactory(factory.DjangoModelFactory):
167+
class FeedbackFactory(factory.django.DjangoModelFactory):
166168
class Meta:
167169
model = Feedback
168170

@@ -172,10 +174,10 @@ class Meta:
172174

173175
@factory.post_generation
174176
def comments(obj, create, extracted, **kwargs):
175-
comment_text = factory.Faker('paragraph').generate({})
177+
comment_text = Faker().paragraph()
176178
obj.comments = obj.nomcom.encrypt(comment_text)
177179

178-
class TopicFactory(factory.DjangoModelFactory):
180+
class TopicFactory(factory.django.DjangoModelFactory):
179181
class Meta:
180182
model = Topic
181183

@@ -184,7 +186,7 @@ class Meta:
184186
accepting_feedback = True
185187
audience_id = 'general'
186188

187-
class NominationFactory(factory.DjangoModelFactory):
189+
class NominationFactory(factory.django.DjangoModelFactory):
188190
class Meta:
189191
model = Nomination
190192

0 commit comments

Comments
 (0)