Test failure fix

Bug #963171 reported by Denis Darii
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
django-openid-auth
Confirmed
Undecided
Unassigned

Bug Description

Minor fix to prevent test(test_update_user_details_long_names) failure on release 0.4 and current trunk.

If you try:
$ ./manage.py test django_openid_auth.OpenIDBackendTests.test_update_user_details_long_names

the output:

Creating test database for alias 'default'...
E
======================================================================
ERROR: test_update_user_details_long_names (django_openid_auth.tests.test_auth.OpenIDBackendTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/dnx/workspace/envs/doauth/lib/python2.6/site-packages/django_openid_auth/tests/test_auth.py", line 132, in test_update_user_details_long_names
    self.backend.update_user_details(user, data, response)
  File "/home/dnx/workspace/envs/doauth/lib/python2.6/site-packages/django_openid_auth/auth.py", line 271, in update_user_details
    user.username = self._get_available_username(details['nickname'], openid_response.identity_url)
KeyError: 'nickname'

----------------------------------------------------------------------
Ran 1 test in 0.002s

FAILED (errors=1)

Tags: test
Revision history for this message
Denis Darii (dnx) wrote :
Changed in django-openid-auth:
status: New → Confirmed
Revision history for this message
Chris Johnston (cjohnston) wrote :

This patch does not work for me with django 1.4

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.