FAIL: test_admin_site_with_openid_login_non_authenticated_user

[Chris Johnston]

With trunk:

======================================================================
FAIL: test_admin_site_with_openid_login_non_authenticated_user (django_openid_auth.tests.test_admin.SiteAdminTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/vagrant/test-test/django_openid_auth/tests/test_admin.py", line 84, in test_admin_site_with_openid_login_non_authenticated_user
    response['Location'])
AssertionError: 'http://testserver/openid/login/?next=/admin/' != 'http://testserver/openid/login?next=/admin/'

----------------------------------------------------------------------

[Chris Johnston (cjohnston)]

To reproduce:

=== modified file 'example_consumer/settings.py'
--- example_consumer/settings.py 2013-03-14 01:03:36 +0000
+++ example_consumer/settings.py 2013-03-14 01:05:35 +0000
@@ -126,7 +126,7 @@
 OPENID_SSO_SERVER_URL = 'https://login.launchpad.net/'

 # Tell django.contrib.auth to use the OpenID signin URLs.
-LOGIN_URL = '/openid/login/'
+LOGIN_URL = '/openid/login'
 LOGIN_REDIRECT_URL = '/'

 # Should django_auth_openid be used to sign into the admin interface?

[Chris Johnston (cjohnston)]

So the problem with the way that the test currently is in trunk is that if you use anything other than the default LOGIN_URL = '/openid/login/' you will have this test failing. With this MP, it allows you to use whatever LOGIN_URL you like.