google-code-export / google-app-engine-django

Automatically exported from code.google.com/p/google-app-engine-django
Apache License 2.0
1 stars 0 forks source link

Integration test fails when using authentication middleware #164

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Set up a Django app with custom authentication middleware (Django auth,
not Google Accounts)
2. Copy in appengine_django so that it runs on GAE
3. Run python manage.py test

What is the expected output? What do you see instead?

All appengine_django tests should pass but
integration_test.py:IntegrationTest.testBasic() fails since the auth
middleware is redirecting to a login page and its status is 302 instead of
200 like the current test expects.

I have attached a patch that fixes this problem by asserting the home page
does not return a 500 or 404 error.  This should accomplish the same basic
integration test.

What version of the product are you using? On what operating system?

r102 with SDK 1.3.1

Please provide any additional information below.

Original issue reported on code.google.com by kumar.mcmillan on 20 Feb 2010 at 4:39

Attachments:

GoogleCodeExporter commented 9 years ago
The patch looks acceptable for merging thanks.

Please sign the CLA as described in the readme, we need this done before we can 
accept external patches!

Thanks.

Original comment by m...@google.com on 8 Jun 2010 at 11:45

GoogleCodeExporter commented 9 years ago
ok, sure, I just signed it using the email for my Google account.

Original comment by kumar.mcmillan on 8 Jun 2010 at 6:42

GoogleCodeExporter commented 9 years ago
Hello.  It looks like this patch has been reviewed and accepted, nice! :)  Is 
someone able to apply the patch to trunk?

Original comment by kumar.mcmillan on 14 Nov 2010 at 4:40

GoogleCodeExporter commented 9 years ago
This issue was closed by revision r111.

Original comment by m...@google.com on 18 Nov 2010 at 6:44