Hotfix: Renaming cookie broke unit tests
authorElrond <elrond+mediagoblin.org@samba-tng.org>
Sat, 15 Oct 2011 22:25:59 +0000 (00:25 +0200)
committerElrond <elrond+mediagoblin.org@samba-tng.org>
Sat, 15 Oct 2011 22:25:59 +0000 (00:25 +0200)
The name of the cookie was hardcoded in the unit tests and
while renaming the cookie this part was missed.

So now read the cookie_name from the config in the test.
This also tests for the config item existing!

mediagoblin/tests/test_csrf_middleware.py

index cf03fe5874a47043119cd9e38324263c59e5754d..691f10b994701796c7e458120aabf29b4653284f 100644 (file)
@@ -26,12 +26,14 @@ from mediagoblin import mg_globals
 @setup_fresh_app
 def test_csrf_cookie_set(test_app):
 
+    cookie_name = mg_globals.app_config['csrf_cookie_name']
+    
     # get login page
     response = test_app.get('/auth/login/')
 
     # assert that the mediagoblin nonce cookie has been set
     assert 'Set-Cookie' in response.headers
-    assert 'mediagoblin_nonce' in response.cookies_set
+    assert cookie_name in response.cookies_set
 
     # assert that we're also sending a vary header
     assert response.headers.get('Vary', False) == 'Cookie'