This isn't needed, so remove it.

Signed-off-by: Stephen Finucane <[email protected]>
---
 patchwork/tests/utils.py |   16 ++++++++--------
 1 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/patchwork/tests/utils.py b/patchwork/tests/utils.py
index 37478b6..8bd0288 100644
--- a/patchwork/tests/utils.py
+++ b/patchwork/tests/utils.py
@@ -63,29 +63,29 @@ error_strings = {
     'email': 'Enter a valid email address.',
 }
 
-_user_idx = 1
-
 
 def create_user():
-    global _user_idx
-    userid = 'test%d' % _user_idx
-    email = '%[email protected]' % userid
-    _user_idx += 1
+    _user_id = User.objects.count()
+
+    username = 'test%d' % _user_id
+    email = '%[email protected]' % username
 
-    user = User.objects.create_user(userid, email, userid)
+    user = User.objects.create_user(username, email, username)
     user.save()
 
-    person = Person(email=email, name=userid, user=user)
+    person = Person(email=email, name=username, user=user)
     person.save()
 
     return user
 
 
 def create_maintainer(project):
+    """Create a 'User' and set as maintainer for provided project."""
     user = create_user()
     profile = user.profile
     profile.maintainer_projects.add(project)
     profile.save()
+
     return user
 
 
-- 
1.7.4.1

_______________________________________________
Patchwork mailing list
[email protected]
https://lists.ozlabs.org/listinfo/patchwork

Reply via email to