Comment 4 for bug 1333144

Revision history for this message
Julie Pichon (jpichon) wrote :

The problem appears due to 1.1.6 and bug 1308637. However as far as I can tell that patch is not causing the problem itself but only surfacing an issue that already existed within the Horizon test. At this point when we redirect the user to /auth/logout/ in a unit test the user is not properly logged out and the django session is not terminated. (It works fine testing in a real environment.)

To fix the test we need to mock the logout method properly, though it doesn't seem totally straightforward because of the way the redirection is handled. I think the test is trying to do too much and would fit better as an integration test.

My suggestion for now would be to disable the test to get the horizon gate going again. I'll propose a second patch later today to change it so it has a reduced scope.

This also affects Icehouse.