Removed some unnecessary randomness in tests introduced in last commit. Commit ready for merge.

 - Legacy-Id: 12675

Author: rjsparks

ietf/doc/tests_review.py

@@ -287,7 +287,7 @@ def test_assign_reviewer(self):
         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(

ietf/group/tests_review.py

@@ -208,7 +208,7 @@ def test_manage_review_requests(self):
         )
 
         # 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