\n', output_text)
self.assertIn(
- '\n Hook Mail deactivated', output.get_data(as_text=True))
+ '\n Hook Mail deactivated',
+ output_text)
def test_RequiredIf(self):
""" Test the behavior of the RequiredIf validator. """
diff --git a/tests/test_pagure_flask_ui_plugins_pagure_request_hook.py b/tests/test_pagure_flask_ui_plugins_pagure_request_hook.py
index adc9746..3ba2be4 100644
--- a/tests/test_pagure_flask_ui_plugins_pagure_request_hook.py
+++ b/tests/test_pagure_flask_ui_plugins_pagure_request_hook.py
@@ -39,7 +39,7 @@ class PagureFlaskPluginPagureRequestHooktests(tests.SimplePagureTest):
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
- 'Settings Pagure request - test - Pagure',
+ 'Settings Pagure requests - test - Pagure',
output_text)
self.assertIn(
'Settings Pagure request - test - Pagure',
+ 'Settings Pagure requests - test - Pagure',
output_text)
self.assertIn(
'Settings Pagure request - test - Pagure',
+ 'Settings Pagure requests - test - Pagure',
output_text)
self.assertIn(
'Settings Pagure request - test - Pagure',
+ 'Settings Pagure requests - test - Pagure',
output_text)
self.assertIn(
'Settings Pagure request - test - Pagure',
+ 'Settings Pagure requests - test - Pagure',
output_text)
self.assertIn(
'
@@ -155,7 +155,9 @@ class PagureFlaskPrioritiestests(tests.Modeltests):
output_text = output.get_data(as_text=True)
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
csrf_token = output_text.split(
'name="csrf_token" type="hidden" value="')[1].split('">')[0]
@@ -171,7 +173,9 @@ class PagureFlaskPrioritiestests(tests.Modeltests):
# Check the redirect
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
# Check the result of the action -- None, no CSRF
repo = pagure.lib.get_authorized_project(self.session, 'test')
self.assertEqual(repo.priorities, {})
@@ -188,7 +192,9 @@ class PagureFlaskPrioritiestests(tests.Modeltests):
# Check the redirect
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
# Check the result of the action -- Priority recorded
self.session.commit()
repo = pagure.lib.get_authorized_project(self.session, 'test')
@@ -206,7 +212,9 @@ class PagureFlaskPrioritiestests(tests.Modeltests):
# Check the redirect
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
# Check the ordering
self.assertTrue(
output_text.find('High') < output_text.find('Normal'))
@@ -233,7 +241,9 @@ class PagureFlaskPrioritiestests(tests.Modeltests):
# Check the redirect
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
self.assertIn(
'\n'
' Priorities weights and titles are '
@@ -259,7 +269,9 @@ class PagureFlaskPrioritiestests(tests.Modeltests):
# Check the redirect
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
self.assertIn(
'\n'
' Priorities weights must be numbers',
@@ -285,7 +297,9 @@ class PagureFlaskPrioritiestests(tests.Modeltests):
# Check the redirect
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
self.assertIn(
'\n'
' Priority weight 2 is present 2 times',
@@ -311,7 +325,9 @@ class PagureFlaskPrioritiestests(tests.Modeltests):
# Check the redirect
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
self.assertIn(
'\n'
' Priority Normal is present 2 times',
@@ -377,7 +393,9 @@ class PagureFlaskPrioritiestests(tests.Modeltests):
output_text = output.get_data(as_text=True)
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
\n', output_text)
# Check the result of the action -- Priority recorded
self.session.commit()
@@ -442,7 +462,9 @@ class PagureFlaskPrioritiestests(tests.Modeltests):
output_text = output.get_data(as_text=True)
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
# Check that the issue list renders fine
output = self.app.get('/test/issues')
@@ -485,7 +507,9 @@ class PagureFlaskPrioritiestests(tests.Modeltests):
output_text = output.get_data(as_text=True)
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
\n', output_text)
# Check the result of the action -- Priority recorded
self.session.commit()
@@ -551,7 +577,9 @@ class PagureFlaskPrioritiestests(tests.Modeltests):
output_text = output.get_data(as_text=True)
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
# Check that the issue list renders fine
output = self.app.get('/test/issues')
@@ -592,7 +620,9 @@ class PagureFlaskPrioritiestests(tests.Modeltests):
output_text = output.get_data(as_text=True)
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
\n', output_text)
# Check the result of the action -- Priority recorded
self.session.commit()
@@ -722,7 +754,9 @@ class PagureFlaskPrioritiestests(tests.Modeltests):
output_text = output.get_data(as_text=True)
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
\n', output_text)
# Check the result of the action -- Priority recorded
self.session.commit()
@@ -830,7 +866,9 @@ class PagureFlaskPrioritiestests(tests.Modeltests):
output_text = output.get_data(as_text=True)
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
\n', output_text)
# Check the result of the action -- Priority recorded
self.session.commit()
@@ -946,7 +986,9 @@ class PagureFlaskPrioritiestests(tests.Modeltests):
# Check the redirect
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
# Check the ordering
self.assertTrue(
output_text.find('High') < output_text.find('Normal'))
@@ -970,7 +1012,9 @@ class PagureFlaskPrioritiestests(tests.Modeltests):
# Check the redirect
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
# Check the result of the action -- default_priority no change
self.session.commit()
repo = pagure.lib.get_authorized_project(self.session, 'test')
@@ -986,7 +1030,9 @@ class PagureFlaskPrioritiestests(tests.Modeltests):
# Check the redirect
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
self.assertIn(
'\n Default priority set '
'to High', output_text)
@@ -1005,7 +1051,9 @@ class PagureFlaskPrioritiestests(tests.Modeltests):
# Check the redirect
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
# Check the result of the action -- default_priority no change
self.session.commit()
repo = pagure.lib.get_authorized_project(self.session, 'test')
@@ -1021,7 +1069,9 @@ class PagureFlaskPrioritiestests(tests.Modeltests):
# Check the redirect
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
self.assertIn(
'\n Default priority reset',
output_text)
@@ -1092,7 +1142,9 @@ class PagureFlaskPrioritiestests(tests.Modeltests):
# Check the redirect
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
# Check the ordering
self.assertTrue(
output_text.find('High') < output_text.find('Normal'))
@@ -1116,7 +1168,9 @@ class PagureFlaskPrioritiestests(tests.Modeltests):
# Check the redirect
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
self.assertIn(
'\n Default priority set '
'to High', output_text)
@@ -1138,7 +1192,9 @@ class PagureFlaskPrioritiestests(tests.Modeltests):
# Check the redirect
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
self.assertIn(
'\n Priorities updated',
output_text)
diff --git a/tests/test_pagure_flask_ui_quick_reply.py b/tests/test_pagure_flask_ui_quick_reply.py
index e45b19f..0fc4efd 100644
--- a/tests/test_pagure_flask_ui_quick_reply.py
+++ b/tests/test_pagure_flask_ui_quick_reply.py
@@ -1,7 +1,7 @@
# -*- coding: utf-8 -*-
"""
- (c) 2015-2016 - Copyright Red Hat Inc
+ (c) 2015-2018 - Copyright Red Hat Inc
Authors:
Lubomír Sedlář
@@ -74,11 +74,15 @@ class PagureFlaskQuickReplytest(tests.Modeltests):
and that a given notice was printed.
"""
self.assertEqual(output.status_code, 200)
+ output_text = output.get_data(as_text=True)
self.assertIn(
- 'Settings - %s - Pagure' % project, output.get_data(as_text=True))
- self.assertIn('
',
output_text)
self.assertIn(
'\n User removed', output_text)
@@ -1241,9 +1242,6 @@ class PagureFlaskRepotests(tests.Modeltests):
self.assertIn(
'Overview - test - Pagure', output_text)
self.assertIn(
- '
\n'
- 'test project #1
', output_text)
- self.assertIn(
'\n Edited successfully '
'settings of repo: test', output_text)
@@ -1273,9 +1271,6 @@ class PagureFlaskRepotests(tests.Modeltests):
self.assertIn(
'Overview - test - Pagure', output_text)
self.assertIn(
- '
\n'
- 'test project #1
', output_text)
- self.assertIn(
'\n Edited successfully '
'settings of repo: test', output_text)
@@ -1325,9 +1320,6 @@ class PagureFlaskRepotests(tests.Modeltests):
self.assertIn(
'Overview - test - Pagure', output_text)
self.assertIn(
- '
\n'
- 'test project #1
', output_text)
- self.assertIn(
'\n Edited successfully '
'settings of repo: test', output_text)
@@ -1402,18 +1394,16 @@ class PagureFlaskRepotests(tests.Modeltests):
self.assertIn('
Project Settings
', output_text)
# Check that the priorities have their empty fields
self.assertIn(
- '''
-
-
-
-
-
-
-
-
-
''', output_text)
+ '''
+
+
+
+
+
+
+
''', output_text)
# Check that the milestones have their empty fields
self.assertIn(
@@ -1428,12 +1418,10 @@ class PagureFlaskRepotests(tests.Modeltests):
value="" class="form-control"/>
-
-
+
+
@@ -1442,39 +1430,36 @@ class PagureFlaskRepotests(tests.Modeltests):
# Check that the close_status have its empty field
self.assertIn(
- '''
-
-
-
-
-
''', output_text)
+ '''
+
+
+
+
''', output_text)
# Check that the custom fields have their empty fields
self.assertIn(
- '''
-
-
-
-
-
-
-
-
-
-
-
-
-
-
''', output_text)
+ '''
+
+
+
+
+
+
+
+
+
+
+
+
+
''', output_text)
def test_view_forks(self):
""" Test the view_forks endpoint. """
@@ -1520,10 +1505,9 @@ class PagureFlaskRepotests(tests.Modeltests):
'href="#moregiturls"', output_text)
self.assertIn('
' in output.data)
self.assertFalse('Forked from' in output.data)
self.assertFalse('README.txt' in output.data)
self.assertFalse('README.dummy' in output.data)
self.assertIn(
- '
\n'
- 'test project #1
', output_text)
+ 'Overview - test - Pagure', output_text)
self.perfMaxWalks(3, 8) # Target: (1, 3)
self.perfReset()
@@ -1657,8 +1640,7 @@ class PagureFlaskRepotests(tests.Modeltests):
self.assertFalse('README.txt' in output.data)
self.assertFalse('README.dummy' in output.data)
self.assertIn(
- '
', output_text)
+ '', output_text)
# We can't create the project `forks/test` the normal way
self.assertRaises(
@@ -215,9 +215,8 @@ class PagureFlaskSlashInNametests(tests.SimplePagureTest):
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
- '
', output.get_data(as_text=True))
def test_view_file_in_branch_in_fork(self):
""" Test the view_file in a specific branch of a fork. """
diff --git a/tests/test_pagure_flask_ui_roadmap.py b/tests/test_pagure_flask_ui_roadmap.py
index b352cf9..cb5d84a 100644
--- a/tests/test_pagure_flask_ui_roadmap.py
+++ b/tests/test_pagure_flask_ui_roadmap.py
@@ -168,7 +168,9 @@ class PagureFlaskRoadmaptests(tests.Modeltests):
output_text = output.get_data(as_text=True)
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
csrf_token = output_text.split(
'name="csrf_token" type="hidden" value="')[1].split('">')[0]
@@ -184,7 +186,9 @@ class PagureFlaskRoadmaptests(tests.Modeltests):
# Check the redirect
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
# Check the result of the action -- None, no CSRF
repo = pagure.lib.get_authorized_project(self.session, 'test')
self.assertEqual(repo.milestones, {})
@@ -201,7 +205,9 @@ class PagureFlaskRoadmaptests(tests.Modeltests):
# Check the redirect
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
self.assertIn('Milestones updated', output_text)
# Check the result of the action -- Milestones recorded
self.session.commit()
@@ -221,7 +227,9 @@ class PagureFlaskRoadmaptests(tests.Modeltests):
# Check the redirect
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
self.assertIn('Milestones updated', output_text)
# Check the result of the action -- Milestones recorded
self.session.commit()
@@ -249,7 +257,9 @@ class PagureFlaskRoadmaptests(tests.Modeltests):
# Check the redirect
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
# Check the result of the action -- Milestones un-changed
self.session.commit()
repo = pagure.lib.get_authorized_project(self.session, 'test')
@@ -276,7 +286,9 @@ class PagureFlaskRoadmaptests(tests.Modeltests):
# Check the redirect
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
self.assertIn(
'\n'
' Milestone v2.0 is present 2 times',
@@ -307,7 +319,9 @@ class PagureFlaskRoadmaptests(tests.Modeltests):
# Check the redirect
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
self.assertIn(
'\n'
' Milestones updated',
@@ -381,7 +395,9 @@ class PagureFlaskRoadmaptests(tests.Modeltests):
# Check the redirect
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
self.assertIn('Milestones updated', output_text)
# Check the result of the action -- Milestones recorded
self.session.commit()
@@ -436,7 +452,9 @@ class PagureFlaskRoadmaptests(tests.Modeltests):
# Check the redirect
self.assertIn(
'Settings - test - Pagure', output_text)
- self.assertIn('
Settings for test
', output_text)
+ self.assertIn(
+ '
'
+ 'Project Settings
\n', output_text)
self.assertIn('Milestones updated', output_text)
# Check the result of the action -- Milestones recorded
self.session.commit()
@@ -522,28 +540,40 @@ class PagureFlaskRoadmaptests(tests.Modeltests):
output = self.app.get('/test/roadmap')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
- self.assertIn('
v2.0', output_text)
- self.assertIn('
unplanned', output_text)
+ self.assertIn(
+ 'title="Filter issues by milestone">\n v2.0',
+ output_text)
+ self.assertIn(
+ 'title="Filter issues by milestone">\n unplanned',
+ output_text)
self.assertEqual(
- output_text.count('#'), 4)
+ output_text.count('#'), 4)
# test the roadmap view for all milestones
output = self.app.get('/test/roadmap?all_stones=True&status=All')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
- self.assertIn('
v1.0', output_text)
- self.assertIn('
v2.0', output_text)
- self.assertIn('
unplanned', output_text)
+ self.assertIn(
+ 'title="Filter issues by milestone">\n v1.0',
+ output_text)
+ self.assertIn(
+ 'title="Filter issues by milestone">\n v2.0',
+ output_text)
+ self.assertIn(
+ 'title="Filter issues by milestone">\n unplanned',
+ output_text)
self.assertEqual(
- output_text.count('#'), 6)
+ output_text.count('#'), 6)
# test the roadmap view for a specific milestone
output = self.app.get('/test/roadmap?milestone=v2.0')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
- self.assertIn('
v2.0', output_text)
+ self.assertIn(
+ 'title="Filter issues by milestone">\n v2.0',
+ output_text)
self.assertEqual(
- output_text.count('#'), 2)
+ output_text.count('#'), 2)
# test the roadmap view for a specific milestone - open
output = self.app.get('/test/roadmap?milestone=v1.0')
@@ -551,16 +581,18 @@ class PagureFlaskRoadmaptests(tests.Modeltests):
output_text = output.get_data(as_text=True)
self.assertIn('No issues found', output_text)
self.assertEqual(
- output_text.count('#'), 0)
+ output_text.count('#'), 0)
# test the roadmap view for a specific milestone - closed
output = self.app.get(
'/test/roadmap?milestone=v1.0&status=All&all_stones=True')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
- self.assertIn('
v1.0', output_text)
+ self.assertIn(
+ 'title="Filter issues by milestone">\n v1.0',
+ output_text)
self.assertEqual(
- output_text.count('#'), 2)
+ output_text.count('#'), 2)
# test the roadmap view for a specific tag
output = self.app.get('/test/roadmap?milestone=v2.0&tag=unknown')
@@ -568,7 +600,7 @@ class PagureFlaskRoadmaptests(tests.Modeltests):
output_text = output.get_data(as_text=True)
self.assertIn('No issues found', output_text)
self.assertEqual(
- output_text.count('#'), 0)
+ output_text.count('#'), 0)
# test the roadmap view for errors
output = self.app.get('/foo/roadmap')
diff --git a/tests/test_pagure_flask_ui_slash_branch_name.py b/tests/test_pagure_flask_ui_slash_branch_name.py
index fe2c78f..c9884a2 100644
--- a/tests/test_pagure_flask_ui_slash_branch_name.py
+++ b/tests/test_pagure_flask_ui_slash_branch_name.py
@@ -120,11 +120,12 @@ class PagureFlaskSlashInBranchtests(tests.SimplePagureTest):
# With git repo
output = self.app.get('/test')
self.assertEqual(output.status_code, 200)
+ output_text = output.get_data(as_text=True)
self.assertIn(
- '
\n '
- '
Source GIT URLs
',
- output.get_data(as_text=True))
+ '', output_text)
+ '''
@patch('pagure.lib.notify.send_email')
def test_view_repo_branch(self, send_email):
""" Test the view_repo_branch endpoint when the git repo has no
@@ -144,8 +145,9 @@ class PagureFlaskSlashInBranchtests(tests.SimplePagureTest):
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
- '