heckyel/mediagoblin
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Hotfix: Renaming cookie broke unit tests

Browse Source 
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!
This commit is contained in:
Elrond 2011-10-16 00:25:59 +02:00
parent 4715091970
commit adf7945081
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
mediagoblin/tests/test_csrf_middleware.py
View File

@ -26,12 +26,14 @@ from mediagoblin import mg_globals
@setup_fresh_app @setup_fresh_app
def test_csrf_cookie_set(test_app): def test_csrf_cookie_set(test_app):
cookie_name = mg_globals.app_config['csrf_cookie_name']
# get login page # get login page
response = test_app.get('/auth/login/') response = test_app.get('/auth/login/')
# assert that the mediagoblin nonce cookie has been set # assert that the mediagoblin nonce cookie has been set
assert 'Set-Cookie' in response.headers 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 that we're also sending a vary header
assert response.headers.get('Vary', False) == 'Cookie' assert response.headers.get('Vary', False) == 'Cookie'