diff --git a/tests/test_pagure_flask_ui_plugins_pagure_ci.py b/tests/test_pagure_flask_ui_plugins_pagure_ci.py
index 7783e76..1305b3f 100644
--- a/tests/test_pagure_flask_ui_plugins_pagure_ci.py
+++ b/tests/test_pagure_flask_ui_plugins_pagure_ci.py
@@ -56,18 +56,12 @@ class PagureFlaskPluginPagureCItests(tests.Modeltests):
'
\n'
'test project #1
', output.data)
self.assertTrue('Pagure CI settings
' in output.data)
- self.assertTrue(
- ' | '
- in output.data)
- self.assertTrue(
- ' | '
- in output.data)
- self.assertTrue(
- ' | '
- in output.data)
- self.assertTrue(
- ''
- in output.data)
+ self.assertIn(
+ ' | ' , output.data)
+ self.assertIn(
+ '',
+ output.data)
csrf_token = output.data.split(
'name="csrf_token" type="hidden" value="')[1].split('">')[0]
@@ -80,56 +74,55 @@ class PagureFlaskPluginPagureCItests(tests.Modeltests):
'\n'
'test project #1
', output.data)
self.assertTrue('Pagure CI settings
' in output.data)
- self.assertTrue(
- ' | '
- in output.data)
- self.assertTrue(
- ' | '
- in output.data)
- self.assertTrue(
- ' | '
- in output.data)
- self.assertTrue(
- ' | '
- in output.data)
- self.assertTrue(
- ''
- in output.data)
+ self.assertIn(
+ ' | ' , output.data)
+ self.assertIn(
+ '',
+ output.data)
# Activate hook
data = {
- 'csrf_token': csrf_token,
'active': 'y',
- 'pagure_name': 'test',
- 'jenkins_name': 'jenkins_test',
- 'jenkins_url': 'https://jenkins.fedoraproject.org',
- 'jenkins_token': 'BEEFCAFE'
+ 'ci_url': 'https://jenkins.fedoraproject.org',
+ 'ci_type': 'jenkins',
}
- # No git found
+ # CSRF Token missing
output = self.app.post(
'/test/settings/Pagure CI', data=data, follow_redirects=True)
+ self.assertEqual(output.status_code, 200)
+ self.assertIn(
+ '\n'
+ 'test project #1
', output.data)
+ self.assertTrue('Pagure CI settings
' in output.data)
+ self.assertIn(
+ ' | ' , output.data)
+ self.assertIn(
+ '', output.data)
+
+ data['csrf_token'] = csrf_token
+
+ # No git found
+ output = self.app.post('/test/settings/Pagure', data=data)
self.assertEqual(output.status_code, 404)
tests.create_projects_git(tests.HERE)
- data = {'csrf_token': csrf_token}
- # With the git repo
+ # Activate hook
output = self.app.post(
'/test/settings/Pagure CI', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
-
self.assertIn(
'\n'
'test project #1
', output.data)
- self.assertTrue('Pagure CI settings
' in output.data)
- self.assertFalse(
- '\n Hook activated' in output.data)
- self.assertTrue(
- ' | '
- '\nThis field is required. | '
- in output.data)
- self.assertTrue(
- '' in output.data)
+ self.assertIn(
+ 'Settings - test - Pagure', output.data)
+ self.assertIn('Settings for test
', output.data)
+ self.assertIn(
+ '\n Hook Pagure CI activated',
+ output.data)
output = self.app.get('/test/settings/Pagure CI')
self.assertEqual(output.status_code, 200)
@@ -137,99 +130,60 @@ class PagureFlaskPluginPagureCItests(tests.Modeltests):
'\n'
'test project #1
', output.data)
self.assertTrue('Pagure CI settings
' in output.data)
- self.assertTrue(
- ' | '
- in output.data)
- self.assertTrue(
- ' | '
- in output.data)
- self.assertTrue(
- ' | '
- in output.data)
- self.assertTrue(
- ' | '
- in output.data)
- self.assertTrue(
- ''
- in output.data)
-
- # Missing the required
- data = {'csrf_token': csrf_token, 'active': 'y'}
-
- output = self.app.post(
- '/test/settings/Pagure CI', data=data, follow_redirects=True)
- self.assertEqual(output.status_code, 200)
self.assertIn(
- '\n'
- 'test project #1
', output.data)
- self.assertTrue('Pagure CI settings
' in output.data)
- self.assertFalse(
- '\n Hook activated' in output.data)
+ ' | ' , output.data)
self.assertTrue(
- ' | '
- '\nThis field is required. | '
+ ''
in output.data)
- self.assertTrue(
- '' in output.data)
- # Activate hook
+ # De-activate the hook
data = {
'csrf_token': csrf_token,
- 'active': 'y',
- 'pagure_name': 'test',
- 'jenkins_name': 'jenkins_test',
- 'jenkins_url': 'https://jenkins.fedoraproject.org',
- 'jenkins_token': 'BEEFCAFE'
}
-
output = self.app.post(
'/test/settings/Pagure CI', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
self.assertIn(
+ '\n Hook Pagure CI inactived',
+ output.data)
+ self.assertIn(
'\n Settings for test
',
output.data)
- self.assertTrue(
- '\n Hook Pagure CI activated' in output.data)
output = self.app.get('/test/settings/Pagure CI')
+ self.assertEqual(output.status_code, 200)
self.assertIn(
'\n'
'test project #1
', output.data)
self.assertTrue('Pagure CI settings
' in output.data)
- self.assertTrue(
- ' | '
- in output.data)
- self.assertTrue(
- ' | '
- in output.data)
- self.assertTrue(
- ' | '
- in output.data)
- self.assertTrue(
- ' | '
- in output.data)
- self.assertTrue(
- ''
- in output.data)
+ self.assertIn(
+ ' | ' , output.data)
+ self.assertIn(
+ '', output.data)
+
+ # Missing the required ci_url
+ data = {'csrf_token': csrf_token, 'active': 'y'}
- # De-Activate hook
- data = {
- 'csrf_token': csrf_token,
- 'pagure_name': 'test',
- 'jenkins_name': 'jenkins_test',
- 'jenkins_url': 'https://jenkins.fedoraproject.org',
- 'jenkins_token': 'BEEFCAFE'
- }
output = self.app.post(
'/test/settings/Pagure CI', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
- self.assertTrue(
- '\n Hook Pagure CI inactived' in output.data)
-
self.assertIn(
- '