tests: Configure User.first_name, User.last_name

'User.name' is not a valid attribute. This also highlights a small issue
with the existing tests.

Signed-off-by: Stephen Finucane <stephen@that.guru>

diff --git a/patchwork/tests/api/test_person.py b/patchwork/tests/api/test_person.py
index 6bd3cb67cdb1f2c21688d70489538cb0067f1f19..2139574b524a26a055c70bc48eec706401ea3cfc 100644 (file)
--- a/patchwork/tests/api/test_person.py
+++ b/patchwork/tests/api/test_person.py
@@ -32,7 +32,7 @@ class TestPersonAPI(utils.APITestCase):
             self.assertEqual(person_obj.name, person_json['name'])
             self.assertEqual(person_obj.email, person_json['email'])
         else:
-            self.assertEqual(person_obj.user.username, person_json['name'])
+            self.assertEqual(person_obj.user.profile.name, person_json['name'])
             self.assertEqual(person_obj.user.email, person_json['email'])
             # nested fields
             self.assertEqual(person_obj.user.id,

diff --git a/patchwork/tests/utils.py b/patchwork/tests/utils.py
index 5ebc933b86738e264c632b61ebb39535c1a6f0a3..4ac9afe0de01c3085831cb4bf8ec68aee1c1d1af 100644 (file)
--- a/patchwork/tests/utils.py
+++ b/patchwork/tests/utils.py
@@ -89,15 +89,22 @@ def create_user(link_person=True, **kwargs):
 
     values = {
         'username': 'test_user_%d' % num,
-        'name': 'test_user_%d' % num,
         'email': 'test_user_%d@example.com' % num,
+        'first_name': 'Tester',
+        'last_name': 'Num%d' % num,
     }
     values.update(kwargs)
 
+    # this one must be done rather specifically
     user = User.objects.create_user(values['username'], values['email'],
-                                    values['name'])
+                                    values['username'],
+                                    first_name=values['first_name'],
+                                    last_name=values['last_name'])
 
     if link_person:
+        # unfortunately we don't split on these
+        values['name'] = ' '.join([values.pop('first_name'),
+                                   values.pop('last_name')])
         values.pop('username')
         create_person(user=user, **values)