diff --git a/tests/test_pagure_flask_ui_repo.py b/tests/test_pagure_flask_ui_repo.py
index 816c0c3..9f50006 100644
--- a/tests/test_pagure_flask_ui_repo.py
+++ b/tests/test_pagure_flask_ui_repo.py
@@ -264,53 +264,47 @@ class PagureFlaskRepotests(tests.Modeltests):
'user': 'ralph',
}
- # Missing access
+ # Missing access and no CSRF
output = self.app.post('/test/adduser', data=data)
self.assertEqual(output.status_code, 200)
self.assertIn(
'
Add user - test - Pagure', output.data)
self.assertTrue('Add user to the' in output.data)
- data['csrf_token'] = csrf_token
+ # No CSRF
output = self.app.post('/test/adduser', data=data)
self.assertEqual(output.status_code, 200)
self.assertIn(
'Add user - test - Pagure', output.data)
- self.assertIn('Add user to the', output.data)
- data = {
- 'user': 'ralph',
- 'access': 'commit',
- }
-
- # No CSRF
+ # Missing access
+ data['csrf_token'] = csrf_token
output = self.app.post('/test/adduser', data=data)
self.assertEqual(output.status_code, 200)
self.assertIn(
'Add user - test - Pagure', output.data)
+ self.assertIn('Add user to the', output.data)
# Unknown user
- data['csrf_token'] = csrf_token
+ data['access'] = 'commit'
output = self.app.post('/test/adduser', data=data)
self.assertEqual(output.status_code, 200)
self.assertIn(
'Add user - test - Pagure', output.data)
self.assertIn('Add user to the', output.data)
self.assertIn(
- '\n No user "ralph" '
- 'found', output.data)
+ '\n No user "ralph" found\n',
+ output.data)
+ # All correct
data['user'] = 'foo'
output = self.app.post(
'/test/adduser', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
- self.assertIn(
- 'Settings - test - Pagure', output.data)
self.assertIn('Settings for test
', output.data)
self.assertIn(
'\n User added', output.data)
-
@patch('pagure.ui.repo.admin_session_timedout')
def test_add_group_project_when_user_mngt_off(self, ast):
""" Test the add_group_project endpoint when user management is