Merged in [12675] from rjsparks@nostrum.com:
Removed some unnecessary randomness in tests introduced in last commit.
- Legacy-Id: 12678
Note: SVN reference [12675] has been migrated to Git commit 4f2f691de6
This commit is contained in:
commit
1dba613197
|
@ -287,7 +287,7 @@ class ReviewTests(TestCase):
|
|||
reviewer_settings.save()
|
||||
|
||||
# Need one more person in review team one so we can test incrementing skip_count without immediately decrementing it
|
||||
another_reviewer = PersonFactory.create()
|
||||
another_reviewer = PersonFactory.create(name = u"Extra TestReviewer") # needs to be lexically greater than the exsting one
|
||||
another_reviewer.role_set.create(name_id='reviewer', email=another_reviewer.email(), group=review_req.team)
|
||||
|
||||
UnavailablePeriod.objects.create(
|
||||
|
|
|
@ -208,7 +208,7 @@ class ReviewTests(TestCase):
|
|||
)
|
||||
|
||||
# Need one more person in review team one so we can test incrementing skip_count without immediately decrementing it
|
||||
another_reviewer = PersonFactory.create()
|
||||
another_reviewer = PersonFactory.create(name = u"Extra TestReviewer") # needs to be lexically greater than the exsting one
|
||||
another_reviewer.role_set.create(name_id='reviewer', email=another_reviewer.email(), group=review_req1.team)
|
||||
|
||||
# get
|
||||
|
|
Loading…
Reference in a new issue