diff options
-rw-r--r-- | patchwork/tests/api/test_person.py | 2 | ||||
-rw-r--r-- | patchwork/tests/utils.py | 11 |
2 files changed, 10 insertions, 3 deletions
diff --git a/patchwork/tests/api/test_person.py b/patchwork/tests/api/test_person.py index 6bd3cb6..2139574 100644 --- 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 5ebc933..4ac9afe 100644 --- 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) |