diff --git a/pagure/ui/issues.py b/pagure/ui/issues.py index ff44673..f4fce99 100644 --- a/pagure/ui/issues.py +++ b/pagure/ui/issues.py @@ -1517,6 +1517,7 @@ def edit_comment_issue( form=form, comment=comment, is_js=is_js, + confirmationform=pagure.forms.ConfirmationForm(), ) diff --git a/tests/test_pagure_flask_ui_issues.py b/tests/test_pagure_flask_ui_issues.py index bc3065a..01fe904 100644 --- a/tests/test_pagure_flask_ui_issues.py +++ b/tests/test_pagure_flask_ui_issues.py @@ -673,8 +673,10 @@ class PagureFlaskIssuestests(tests.Modeltests): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - 'div class="projectinfo m-t-1 m-b-1">\ntest project #1 ' - '', output_text) + '', + output_text) + self.assertIn( + 'test', output_text) self.assertIn( '

\n 0 Open Issues', output_text) @@ -881,7 +883,8 @@ class PagureFlaskIssuestests(tests.Modeltests): '

\n 1 Issues (of 3)', output_text) self.assertIn( - '
  • page 1 of 3
  • ', output_text) + '
  • page 1 of 3
  • ', output_text) # All tickets - filtered for 1 - checking the pagination output = self.app.get( @@ -891,7 +894,8 @@ class PagureFlaskIssuestests(tests.Modeltests): self.assertIn('Issues - test - Pagure', output_text) self.assertIn('

    \n 1 Issues (of 1)', output_text) self.assertIn( - '
  • page 1 of 1
  • ', output_text) + '
  • page 1 of 1
  • ', output_text) pagure.config.config['ITEM_PER_PAGE'] = before # Search for issues with no milestone MARK @@ -902,7 +906,7 @@ class PagureFlaskIssuestests(tests.Modeltests): self.assertIn( 'Issues - test - Pagure', output_text) - self.assertIn('1 Open Issues (of 1)', output_text) + self.assertIn('

    \n 1 Open Issues (of 1)', output_text) # Search for issues with no milestone and milestone 1.1 output = self.app.get( @@ -912,7 +916,7 @@ class PagureFlaskIssuestests(tests.Modeltests): self.assertIn( 'Issues - test - Pagure', output_text) - self.assertIn('2 Open Issues (of 2)', output_text) + self.assertIn('

    \n 2 Open Issues (of 2)', output_text) # Add another issue to test sorting msg = pagure.lib.new_issue( @@ -1079,8 +1083,13 @@ class PagureFlaskIssuestests(tests.Modeltests): output = self.app.get('/test') self.assertEqual(output.status_code, 200) self.assertIn( - 'class="btn btn-success btn-sm">New Issue', + 'fa-exclamation-circle fa-fw"> New issue', output.get_data(as_text=True)) + output = self.app.get('/test') + self.assertEqual(output.status_code, 200) + self.assertNotIn( + 'fa-exclamation-circle fa-fw"> New issue', + output.get_data(as_text=True)) # Project w/o issue tracker repo = pagure.lib.get_authorized_project(self.session, 'test') @@ -1097,7 +1106,7 @@ class PagureFlaskIssuestests(tests.Modeltests): output = self.app.get('/test') self.assertEqual(output.status_code, 200) self.assertNotIn( - 'class="btn btn-success btn-sm">New Issue', + 'fa-exclamation-circle fa-fw"> New issue', output.get_data(as_text=True)) @patch('pagure.lib.git.update_git') @@ -2215,13 +2224,13 @@ class PagureFlaskIssuestests(tests.Modeltests): output_text = output.get_data(as_text=True) self.assertEqual( output_text.count( - ''), + ''), 1) self.assertEqual( output_text.count( - ''), + ''), 1) @patch('pagure.lib.git.update_git') @@ -2975,7 +2984,9 @@ class PagureFlaskIssuestests(tests.Modeltests): output_text = output.get_data(as_text=True) self.assertTrue( 'Settings - test - Pagure' in output_text) - self.assertTrue("

    Settings for test

    " in output_text) + self.assertIn( + '
    Project ' + 'Settings
    ', output_text) csrf_token = self.get_csrf(output=output) @@ -2984,14 +2995,19 @@ class PagureFlaskIssuestests(tests.Modeltests): output = self.app.post( '/test/droptag/', data=data, follow_redirects=True) self.assertEqual(output.status_code, 200) - self.assertTrue("

    Settings for test

    " in output.get_data(as_text=True)) + output_text = output.get_data(as_text=True) + self.assertIn( + '
    Project ' + 'Settings
    ', output_text) data['csrf_token'] = csrf_token output = self.app.post( '/test/droptag/', data=data, follow_redirects=True) self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) - self.assertTrue("

    Settings for test

    " in output_text) + self.assertIn( + '
    Project ' + 'Settings
    ', output_text) self.assertIn( '\n ' 'Tag: tag1 has been deleted', output_text) @@ -3359,8 +3375,11 @@ class PagureFlaskIssuestests(tests.Modeltests): # Check that the git issue URL is gone output = self.app.get('/test') + output_text = output.get_data(as_text=True) self.assertIn( - '
    Issues GIT URLs
    ', output.get_data(as_text=True)) + '
    Issues
    ', output_text) + self.assertIn( + 'value="ssh://git@pagure.org/tickets/test.git', output_text) @patch('pagure.lib.git.update_git') @patch('pagure.lib.notify.send_email') @@ -3420,8 +3439,9 @@ class PagureFlaskIssuestests(tests.Modeltests): self.assertIn( 'Settings - test - Pagure', output_text) self.assertIn( - ' ', output_text) + '
    Project ' + 'Settings
    ', output_text) + csrf_token = self.get_csrf(output=output) @@ -3443,8 +3463,8 @@ class PagureFlaskIssuestests(tests.Modeltests): 'Color: red does not match the expected pattern', output_text) self.assertIn( - ' ', output_text) + '
    Project ' + 'Settings
    ', output_text) # Invalid tag name data = { @@ -3464,8 +3484,8 @@ class PagureFlaskIssuestests(tests.Modeltests): 'Tag: red/green contains an invalid character: "/"', output_text) self.assertIn( - ' ', output_text) + '
    Project ' + 'Settings
    ', output_text) # Inconsistent length tags (missing tag field) data = { @@ -3484,8 +3504,8 @@ class PagureFlaskIssuestests(tests.Modeltests): '\n Error: Incomplete request. ' 'One or more tag fields missing.', output_text) self.assertIn( - ' ', output_text) + '
    Project ' + 'Settings
    ', output_text) # Inconsistent length color data = { @@ -3508,8 +3528,8 @@ class PagureFlaskIssuestests(tests.Modeltests): '\n Error: Incomplete request. ' 'One or more tag color fields missing.', output_text) self.assertIn( - ' ', output_text) + '
    Project ' + 'Settings
    ', output_text) # Inconsistent length description data = { @@ -3528,8 +3548,8 @@ class PagureFlaskIssuestests(tests.Modeltests): '\n Error: Incomplete request. ' 'One or more tag description fields missing.', output_text) self.assertIn( - ' ', output_text) + '
    Project ' + 'Settings
    ', output_text) # consistent length, but empty description data = { @@ -3545,18 +3565,18 @@ class PagureFlaskIssuestests(tests.Modeltests): self.assertIn( 'Settings - test - Pagure', output_text) self.assertIn( - 'blue\n' - '  ', - output_text) + 'blue\n' + '  ' + '', output_text) self.assertIn( '', output_text) self.assertIn( - 'red\n' - '  lorem ipsum' - '', output_text) + 'red\n' + '  ' + 'lorem ipsum', output_text) self.assertIn( '', output_text) @@ -3575,18 +3595,18 @@ class PagureFlaskIssuestests(tests.Modeltests): self.assertIn( 'Settings - test - Pagure', output_text) self.assertIn( - 'green\n' - '  sample description' - '', output_text) + 'green\n' + '  ' + 'sample description', output_text) self.assertIn( '', output_text) self.assertIn( - 'red\n' - '  lorem ipsum' - '', output_text) + 'red\n' + '  ' + 'lorem ipsum', output_text) self.assertIn( '', output_text) @@ -3605,18 +3625,18 @@ class PagureFlaskIssuestests(tests.Modeltests): self.assertIn( 'Settings - test - Pagure', output_text) self.assertIn( - 'red2\n' - '  ', - output_text) + 'red2\n' + '  ' + '', output_text) self.assertIn( '', output_text) self.assertIn( - 'red3\n' - '  ', - output_text) + 'red3\n' + '  ' + '', output_text) self.assertIn( '', output_text) diff --git a/tests/test_pagure_flask_ui_issues_private.py b/tests/test_pagure_flask_ui_issues_private.py index 1c955f8..988d5f2 100644 --- a/tests/test_pagure_flask_ui_issues_private.py +++ b/tests/test_pagure_flask_ui_issues_private.py @@ -82,11 +82,11 @@ class PagureFlaskIssuesPrivatetests(tests.Modeltests): output = self.app.get('/test/issues') self.assertEqual(output.status_code, 200) + output_text = output.get_data(as_text=True) self.assertIn( - 'div class="projectinfo m-t-1 m-b-1">\ntest project #1' - ' ', output.get_data(as_text=True)) + 'Issues - test - Pagure', output_text) self.assertIn( - '

    \n 1 Open Issues', output.get_data(as_text=True)) + '

    \n 1 Open Issues (of 1)\n', output_text) def test_issue_list_admin(self): """ Test the list of issues when user is an admin of the project. @@ -96,11 +96,11 @@ class PagureFlaskIssuesPrivatetests(tests.Modeltests): with tests.user_set(self.app.application, user): output = self.app.get('/test/issues') self.assertEqual(output.status_code, 200) + output_text = output.get_data(as_text=True) self.assertIn( - 'div class="projectinfo m-t-1 m-b-1">\ntest project #1' - ' ', output.get_data(as_text=True)) + 'Issues - test - Pagure', output_text) self.assertIn( - '

    \n 2 Open Issues', output.get_data(as_text=True)) + '

    \n 2 Open Issues (of 2)\n', output_text) def test_issue_list_author(self): """ Test the list of issues when user is an admin of the project. @@ -110,11 +110,11 @@ class PagureFlaskIssuesPrivatetests(tests.Modeltests): with tests.user_set(self.app.application, user): output = self.app.get('/test/issues') self.assertEqual(output.status_code, 200) + output_text = output.get_data(as_text=True) self.assertIn( - 'div class="projectinfo m-t-1 m-b-1">\ntest project #1' - ' ', output.get_data(as_text=True)) + 'Issues - test - Pagure', output_text) self.assertIn( - '

    \n 2 Open Issues', output.get_data(as_text=True)) + '

    \n 2 Open Issues (of 2)\n', output_text) def test_issue_list_authenticated(self): """ Test the list of issues when user is authenticated but has no @@ -125,11 +125,11 @@ class PagureFlaskIssuesPrivatetests(tests.Modeltests): with tests.user_set(self.app.application, user): output = self.app.get('/test/issues') self.assertEqual(output.status_code, 200) + output_text = output.get_data(as_text=True) self.assertIn( - 'div class="projectinfo m-t-1 m-b-1">\ntest project #1' - ' ', output.get_data(as_text=True)) + 'Issues - test - Pagure', output_text) self.assertIn( - '

    \n 1 Open Issues', output.get_data(as_text=True)) + '

    \n 1 Open Issues', output_text) def test_issue_list_authenticated_ticket(self): """ Test the list of issues when user is authenticated but has @@ -150,11 +150,11 @@ class PagureFlaskIssuesPrivatetests(tests.Modeltests): with tests.user_set(self.app.application, user): output = self.app.get('/test/issues') self.assertEqual(output.status_code, 200) + output_text = output.get_data(as_text=True) self.assertIn( - 'div class="projectinfo m-t-1 m-b-1">\ntest project #1' - ' ', output.get_data(as_text=True)) + 'Issues - test - Pagure', output_text) self.assertIn( - '

    \n 1 Open Issues', output.get_data(as_text=True)) + '

    \n 1 Open Issues', output_text) def test_issue_list_authenticated_commit(self): """ Test the list of issues when user is authenticated but has @@ -175,11 +175,11 @@ class PagureFlaskIssuesPrivatetests(tests.Modeltests): with tests.user_set(self.app.application, user): output = self.app.get('/test/issues') self.assertEqual(output.status_code, 200) + output_text = output.get_data(as_text=True) self.assertIn( - 'div class="projectinfo m-t-1 m-b-1">\ntest project #1' - ' ', output.get_data(as_text=True)) + 'Issues - test - Pagure', output_text) self.assertIn( - '

    \n 2 Open Issues', output.get_data(as_text=True)) + '

    \n 2 Open Issues', output_text) def test_issue_list_authenticated_assigned(self): """ Test the list of issues when user is authenticated and is @@ -196,11 +196,11 @@ class PagureFlaskIssuesPrivatetests(tests.Modeltests): with tests.user_set(self.app.application, user): output = self.app.get('/test/issues') self.assertEqual(output.status_code, 200) + output_text = output.get_data(as_text=True) self.assertIn( - 'div class="projectinfo m-t-1 m-b-1">\ntest project #1' - ' ', output.get_data(as_text=True)) + 'Issues - test - Pagure', output_text) self.assertIn( - '

    \n 2 Open Issues', output.get_data(as_text=True)) + '

    \n 2 Open Issues (of 2)', output_text) def test_view_issue_anonymous(self): """ Test accessing a private ticket when user is logged out. """ @@ -217,12 +217,13 @@ class PagureFlaskIssuesPrivatetests(tests.Modeltests): with tests.user_set(self.app.application, user): output = self.app.get('/test/issue/1') self.assertEqual(output.status_code, 200) - self.assertIn( - 'div class="projectinfo m-t-1 m-b-1">\ntest project #1' - ' ', output.get_data(as_text=True)) + output_text = output.get_data(as_text=True) self.assertIn( 'Issue #1: Test issue #1 - test - Pagure', - output.get_data(as_text=True)) + output_text) + self.assertIn( + '#1\n', + output_text) def test_view_issue_author(self): """ Test accessing a private ticket when user opened the ticket. @@ -232,12 +233,13 @@ class PagureFlaskIssuesPrivatetests(tests.Modeltests): with tests.user_set(self.app.application, user): output = self.app.get('/test/issue/1') self.assertEqual(output.status_code, 200) - self.assertIn( - 'div class="projectinfo m-t-1 m-b-1">\ntest project #1' - ' ', output.get_data(as_text=True)) + output_text = output.get_data(as_text=True) self.assertIn( 'Issue #1: Test issue #1 - test - Pagure', - output.get_data(as_text=True)) + output_text) + self.assertIn( + '#1\n', + output_text) def test_view_issue_authenticated(self): """ Test accessing a private ticket when user is authenticated but @@ -288,12 +290,13 @@ class PagureFlaskIssuesPrivatetests(tests.Modeltests): with tests.user_set(self.app.application, user): output = self.app.get('/test/issue/1') self.assertEqual(output.status_code, 200) - self.assertIn( - 'div class="projectinfo m-t-1 m-b-1">\ntest project #1' - ' ', output.get_data(as_text=True)) + output_text = output.get_data(as_text=True) self.assertIn( 'Issue #1: Test issue #1 - test - Pagure', - output.get_data(as_text=True)) + output_text) + self.assertIn( + '#1\n', + output_text) def test_view_issue_authenticated_assigned(self): """ Test accessing a private ticket when user is authenticated and @@ -310,12 +313,13 @@ class PagureFlaskIssuesPrivatetests(tests.Modeltests): with tests.user_set(self.app.application, user): output = self.app.get('/test/issue/1') self.assertEqual(output.status_code, 200) - self.assertIn( - 'div class="projectinfo m-t-1 m-b-1">\ntest project #1' - ' ', output.get_data(as_text=True)) + output_text = output.get_data(as_text=True) self.assertIn( 'Issue #1: Test issue #1 - test - Pagure', - output.get_data(as_text=True)) + output_text) + self.assertIn( + '#1\n', + output_text) if __name__ == '__main__': diff --git a/tests/test_pagure_flask_ui_no_master_branch.py b/tests/test_pagure_flask_ui_no_master_branch.py index 69aab0f..45a49c5 100644 --- a/tests/test_pagure_flask_ui_no_master_branch.py +++ b/tests/test_pagure_flask_ui_no_master_branch.py @@ -126,6 +126,7 @@ class PagureFlaskNoMasterBranchtests(tests.SimplePagureTest): output_text.count('', output_text) self.assertIn( "The test project's README file is empty or unavailable.", output_text) + ''' @patch('pagure.lib.notify.send_email') def test_view_commits(self, send_email): @@ -168,7 +171,7 @@ class PagureFlaskNoMasterBranchtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n
    ', + '
    \n\n\n
    ', output_text) self.assertNotIn( '
    ', output_text) @@ -182,9 +185,10 @@ class PagureFlaskNoMasterBranchtests(tests.SimplePagureTest): self.assertIn('Add sources file for testing', output_text) self.assertIn('Add .gitignore file for testing', output_text) self.assertNotIn( - '
    \n
    ', output_text) + '
    \n\n\n
    ', + output_text) self.assertEqual( - output_text.count('class="list-group-item"'), 2) + output_text.count('class="list-group-item "'), 2) @patch('pagure.lib.notify.send_email') def test_view_file(self, send_email): @@ -210,14 +214,14 @@ class PagureFlaskNoMasterBranchtests(tests.SimplePagureTest): self.assertIn( '''
    ''', output_text) diff --git a/tests/test_pagure_flask_ui_old_commit.py b/tests/test_pagure_flask_ui_old_commit.py index 5d179ba..c20ef87 100644 --- a/tests/test_pagure_flask_ui_old_commit.py +++ b/tests/test_pagure_flask_ui_old_commit.py @@ -1,7 +1,7 @@ # -*- coding: utf-8 -*- """ - (c) 2016-2017 - Copyright Red Hat Inc + (c) 2016-2018 - Copyright Red Hat Inc Authors: Pierre-Yves Chibon diff --git a/tests/test_pagure_flask_ui_plugins_fedmsg.py b/tests/test_pagure_flask_ui_plugins_fedmsg.py index 7166e9e..48ce25f 100644 --- a/tests/test_pagure_flask_ui_plugins_fedmsg.py +++ b/tests/test_pagure_flask_ui_plugins_fedmsg.py @@ -1,7 +1,7 @@ # -*- coding: utf-8 -*- """ - (c) 2015-2016 - Copyright Red Hat Inc + (c) 2015-2018 - Copyright Red Hat Inc Authors: Pierre-Yves Chibon @@ -50,9 +50,7 @@ class PagureFlaskPluginFedmsgtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Fedmsg settings

    ', output_text) + 'Settings Fedmsg - test - Pagure', output_text) self.assertIn( '', output_text) @@ -65,9 +63,7 @@ class PagureFlaskPluginFedmsgtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Fedmsg settings

    ', output_text) + 'Settings Fedmsg - test - Pagure', output_text) self.assertIn( '', output_text) @@ -96,17 +92,16 @@ class PagureFlaskPluginFedmsgtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook Fedmsg deactivated', output_text) + output = self.app.get('/test/settings/Fedmsg', data=data) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Fedmsg settings

    ', output_text) + 'Settings Fedmsg - test - Pagure', output_text) self.assertIn( '', output_text) @@ -140,18 +135,17 @@ class PagureFlaskPluginFedmsgtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook Fedmsg activated', output_text) + output = self.app.get('/test/settings/Fedmsg', data=data) self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Fedmsg settings

    ', output_text) + 'Settings Fedmsg - test - Pagure', output_text) self.assertIn( '', output_text) @@ -180,18 +174,17 @@ class PagureFlaskPluginFedmsgtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook Fedmsg deactivated', output_text) + output = self.app.get('/test/settings/Fedmsg', data=data) self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Fedmsg settings

    ', output_text) + 'Settings Fedmsg - test - Pagure', output_text) self.assertIn( '', output.get_data(as_text=True)) diff --git a/tests/test_pagure_flask_ui_plugins_irc.py b/tests/test_pagure_flask_ui_plugins_irc.py index 2a5c1f6..5d391e0 100644 --- a/tests/test_pagure_flask_ui_plugins_irc.py +++ b/tests/test_pagure_flask_ui_plugins_irc.py @@ -1,7 +1,7 @@ # -*- coding: utf-8 -*- """ - (c) 2015-2016 - Copyright Red Hat Inc + (c) 2015-2018 - Copyright Red Hat Inc Authors: Pierre-Yves Chibon @@ -37,9 +37,7 @@ class PagureFlaskPluginIRCtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    IRC settings

    ', output_text) + 'Settings IRC - test - Pagure', output_text) self.assertIn( '', output_text) @@ -53,9 +51,7 @@ class PagureFlaskPluginIRCtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    IRC settings

    ', output_text) + 'Settings IRC - test - Pagure', output_text) self.assertIn( '', output_text) @@ -71,8 +67,8 @@ class PagureFlaskPluginIRCtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook IRC deactivated', output_text) @@ -80,9 +76,7 @@ class PagureFlaskPluginIRCtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    IRC settings

    ', output_text) + 'Settings IRC - test - Pagure', output_text) self.assertIn( '', output_text) @@ -104,8 +98,8 @@ class PagureFlaskPluginIRCtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook IRC activated', output_text) @@ -113,9 +107,7 @@ class PagureFlaskPluginIRCtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    IRC settings

    ', output_text) + 'Settings IRC - test - Pagure', output_text) self.assertIn( '', output_text) @@ -130,8 +122,8 @@ class PagureFlaskPluginIRCtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook IRC deactivated', output_text) @@ -139,9 +131,7 @@ class PagureFlaskPluginIRCtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    IRC settings

    ', output_text) + 'Settings IRC - test - Pagure', output_text) self.assertIn( '', output_text) diff --git a/tests/test_pagure_flask_ui_plugins_mail.py b/tests/test_pagure_flask_ui_plugins_mail.py index e24b5c6..3fd255f 100644 --- a/tests/test_pagure_flask_ui_plugins_mail.py +++ b/tests/test_pagure_flask_ui_plugins_mail.py @@ -1,7 +1,7 @@ # -*- coding: utf-8 -*- """ - (c) 2015-2016 - Copyright Red Hat Inc + (c) 2015-2018 - Copyright Red Hat Inc Authors: Pierre-Yves Chibon @@ -39,9 +39,7 @@ class PagureFlaskPluginMailtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Mail settings

    ', output_text) + 'Settings Mail - test - Pagure', output_text) self.assertIn( '', output_text) @@ -58,9 +56,7 @@ class PagureFlaskPluginMailtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Mail settings

    ', output_text) + 'Settings Mail - test - Pagure', output_text) self.assertIn( '', output_text) self.assertIn( @@ -75,8 +71,8 @@ class PagureFlaskPluginMailtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook Mail deactivated', output_text) @@ -84,9 +80,7 @@ class PagureFlaskPluginMailtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Mail settings

    ', output_text) + 'Settings Mail - test - Pagure', output_text) self.assertIn( '', output_text) self.assertIn( @@ -104,9 +98,7 @@ class PagureFlaskPluginMailtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Mail settings

    ', output_text) + 'Settings Mail - test - Pagure', output_text) self.assertNotIn( '\n Hook activated', output_text) if self.get_wtforms_version() >= (2, 2): @@ -138,17 +130,15 @@ class PagureFlaskPluginMailtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook Mail activated', output_text) output = self.app.get('/test/settings/Mail') output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Mail settings

    ', output_text) + 'Settings Mail - test - Pagure', output_text) self.assertIn( '', output_text) self.assertIn( @@ -165,17 +155,15 @@ class PagureFlaskPluginMailtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook Mail deactivated', output_text) output = self.app.get('/test/settings/Mail') output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Mail settings

    ', output_text) + 'Settings Mail - test - Pagure', output_text) self.assertIn( '', output_text) self.assertIn( diff --git a/tests/test_pagure_flask_ui_plugins_noff.py b/tests/test_pagure_flask_ui_plugins_noff.py index 1233c4e..813b438 100644 --- a/tests/test_pagure_flask_ui_plugins_noff.py +++ b/tests/test_pagure_flask_ui_plugins_noff.py @@ -1,7 +1,7 @@ # -*- coding: utf-8 -*- """ - (c) 2016 - Copyright Red Hat Inc + (c) 2016-2018 - Copyright Red Hat Inc Authors: Pierre-Yves Chibon @@ -40,19 +40,11 @@ class PagureFlaskPluginNoFFtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) + 'Settings Block non fast-forward pushes - test - ' + 'Pagure', output_text) self.assertIn( - '

    Block non fast-forward pushes settings

    ', - output_text) - if self.get_wtforms_version() >= (2, 2): - self.assertIn( - '', output_text) - else: - self.assertIn( - '', output_text) + '', output_text) self.assertTrue( '' in output_text) @@ -67,19 +59,11 @@ class PagureFlaskPluginNoFFtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) + 'Settings Block non fast-forward pushes - test - ' + 'Pagure', output_text) self.assertIn( - '

    Block non fast-forward pushes settings

    ', - output_text) - if self.get_wtforms_version() >= (2, 2): - self.assertIn( - '', output_text) - else: - self.assertIn( - '', output_text) + '', output_text) self.assertTrue( '' in output_text) @@ -93,8 +77,8 @@ class PagureFlaskPluginNoFFtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertTrue( '\n Hook Block non ' 'fast-forward pushes deactivated' in output_text) @@ -104,19 +88,11 @@ class PagureFlaskPluginNoFFtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) + 'Settings Block non fast-forward pushes - test - ' + 'Pagure', output_text) self.assertIn( - '

    Block non fast-forward pushes settings

    ', - output_text) - if self.get_wtforms_version() >= (2, 2): - self.assertIn( - '', output_text) - else: - self.assertIn( - '', output_text) + '', output_text) self.assertTrue( '' in output_text) @@ -133,11 +109,8 @@ class PagureFlaskPluginNoFFtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn( - '

    Block non fast-forward pushes settings

    ', - output_text) + 'Settings Block non fast-forward pushes - test - ' + 'Pagure', output_text) self.assertNotIn( '\n Hook activated', output_text) @@ -170,8 +143,8 @@ class PagureFlaskPluginNoFFtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook Block non ' 'fast-forward pushes activated', output_text) @@ -180,19 +153,11 @@ class PagureFlaskPluginNoFFtests(tests.SimplePagureTest): '/test/settings/Block non fast-forward pushes') output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) + 'Settings Block non fast-forward pushes - test - ' + 'Pagure', output_text) self.assertIn( - '

    Block non fast-forward pushes settings

    ', - output_text) - if self.get_wtforms_version() >= (2, 2): - self.assertIn( - '', output_text) - else: - self.assertIn( - '', output_text) + '', output_text) self.assertIn( '', output_text) @@ -209,8 +174,8 @@ class PagureFlaskPluginNoFFtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook Block non ' 'fast-forward pushes deactivated', output_text) @@ -219,19 +184,11 @@ class PagureFlaskPluginNoFFtests(tests.SimplePagureTest): '/test/settings/Block non fast-forward pushes') output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) + 'Settings Block non fast-forward pushes - test - ' + 'Pagure', output_text) self.assertIn( - '

    Block non fast-forward pushes settings

    ', - output_text) - if self.get_wtforms_version() >= (2, 2): - self.assertIn( - '', output_text) - else: - self.assertIn( - '', output_text) + '', output_text) self.assertIn( '', output_text) diff --git a/tests/test_pagure_flask_ui_plugins_pagure_ci.py b/tests/test_pagure_flask_ui_plugins_pagure_ci.py index 0dfd04c..8a78f4f 100644 --- a/tests/test_pagure_flask_ui_plugins_pagure_ci.py +++ b/tests/test_pagure_flask_ui_plugins_pagure_ci.py @@ -27,9 +27,8 @@ class PagureFlaskPluginPagureCItests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Pagure CI settings

    ', output_text) + 'Settings Pagure CI - test - Pagure', + output_text) self.assertIn( '', output_text) @@ -52,9 +51,8 @@ class PagureFlaskPluginPagureCItests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Pagure CI settings

    ', output_text) + 'Settings Pagure CI - test - Pagure', + output_text) self.assertIn( '', output_text) @@ -81,9 +79,8 @@ class PagureFlaskPluginPagureCItests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Pagure CI settings

    ', output_text) + 'Settings Pagure CI - test - Pagure', + output_text) self.assertIn( '', output_text) @@ -105,11 +102,13 @@ class PagureFlaskPluginPagureCItests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( 'Settings - test - Pagure', output_text) - self.assertIn('

    Settings for test

    ', output_text) + self.assertIn( + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook Pagure CI activated', output_text) @@ -118,9 +117,8 @@ class PagureFlaskPluginPagureCItests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Pagure CI settings

    ', output_text) + 'Settings Pagure CI - test - Pagure', + output_text) self.assertIn( '', output_text) @@ -143,19 +141,18 @@ class PagureFlaskPluginPagureCItests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '\n Hook Pagure CI deactivated', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( - '
    \n

    Settings for test

    ', + '\n Hook Pagure CI deactivated', output_text) output = self.app.get('/test/settings/Pagure CI') self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Pagure CI settings

    ', output_text) + 'Settings Pagure CI - test - Pagure', + output_text) self.assertIn( '', output_text) @@ -174,9 +171,8 @@ class PagureFlaskPluginPagureCItests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Pagure CI settings

    ', output_text) + 'Settings Pagure CI - test - Pagure', + output_text) self.assertFalse( '\n Hook activated' in output_text) @@ -218,9 +214,8 @@ class PagureFlaskPluginPagureCItests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'namespaced test project
    ', output_text) - self.assertIn('

    Pagure CI settings

    ', output_text) + 'Settings Pagure CI - somenamespace/test3 - ' + 'Pagure', output_text) self.assertIn( '', output_text) @@ -249,14 +244,14 @@ class PagureFlaskPluginPagureCItests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'namespaced test project
    ', output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( 'Settings - somenamespace/test3 - Pagure', output_text) self.assertIn( - '

    Settings for somenamespace/test3

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook Pagure CI activated', output_text) @@ -265,9 +260,8 @@ class PagureFlaskPluginPagureCItests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'namespaced test project
    ', output_text) - self.assertIn('

    Pagure CI settings

    ', output_text) + 'Settings Pagure CI - somenamespace/test3 - ' + 'Pagure', output_text) self.assertIn( '', output_text) diff --git a/tests/test_pagure_flask_ui_plugins_pagure_hook.py b/tests/test_pagure_flask_ui_plugins_pagure_hook.py index 9cab566..9f87eb0 100644 --- a/tests/test_pagure_flask_ui_plugins_pagure_hook.py +++ b/tests/test_pagure_flask_ui_plugins_pagure_hook.py @@ -1,7 +1,7 @@ # -*- coding: utf-8 -*- """ - (c) 2015-2016 - Copyright Red Hat Inc + (c) 2015-2018 - Copyright Red Hat Inc Authors: Pierre-Yves Chibon @@ -50,9 +50,8 @@ class PagureFlaskPluginPagureHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Pagure settings

    ', output_text) + 'Settings Pagure - test - Pagure', + output_text) self.assertIn( '', output_text) @@ -69,9 +68,8 @@ class PagureFlaskPluginPagureHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Pagure settings

    ', output_text) + 'Settings Pagure - test - Pagure', + output_text) self.assertIn( '', output_text) @@ -103,8 +101,8 @@ class PagureFlaskPluginPagureHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook Pagure deactivated', output_text) @@ -113,9 +111,8 @@ class PagureFlaskPluginPagureHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Pagure settings

    ', output_text) + 'Settings Pagure - test - Pagure', + output_text) self.assertIn( '', output_text) @@ -151,8 +148,8 @@ class PagureFlaskPluginPagureHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output.get_data(as_text=True)) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook Pagure activated', output_text) @@ -161,9 +158,8 @@ class PagureFlaskPluginPagureHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Pagure settings

    ', output_text) + 'Settings Pagure - test - Pagure', + output_text) self.assertIn( '', output_text) @@ -194,8 +190,8 @@ class PagureFlaskPluginPagureHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook Pagure deactivated', output_text) @@ -204,9 +200,8 @@ class PagureFlaskPluginPagureHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Pagure settings

    ', output_text) + 'Settings Pagure - test - Pagure', + output_text) self.assertIn( '', output_text) @@ -276,8 +271,8 @@ class PagureFlaskPluginPagureHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook Pagure activated', output_text) 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 e7bcdb8..adc9746 100644 --- a/tests/test_pagure_flask_ui_plugins_pagure_request_hook.py +++ b/tests/test_pagure_flask_ui_plugins_pagure_request_hook.py @@ -1,7 +1,7 @@ # -*- coding: utf-8 -*- """ - (c) 2015-2016 - Copyright Red Hat Inc + (c) 2015-2018 - Copyright Red Hat Inc Authors: Pierre-Yves Chibon @@ -39,9 +39,8 @@ class PagureFlaskPluginPagureRequestHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Pagure requests settings

    ', output_text) + 'Settings Pagure request - test - Pagure', + output_text) self.assertIn( '', output_text) @@ -55,9 +54,8 @@ class PagureFlaskPluginPagureRequestHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Pagure requests settings

    ', output_text) + 'Settings Pagure request - test - Pagure', + output_text) self.assertIn( '', output_text) @@ -73,8 +71,8 @@ class PagureFlaskPluginPagureRequestHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook Pagure requests deactivated', output_text) @@ -83,9 +81,8 @@ class PagureFlaskPluginPagureRequestHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Pagure requests settings

    ', output_text) + 'Settings Pagure request - test - Pagure', + output_text) self.assertIn( '', output_text) @@ -106,8 +103,8 @@ class PagureFlaskPluginPagureRequestHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook Pagure requests activated', output_text) @@ -116,9 +113,8 @@ class PagureFlaskPluginPagureRequestHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Pagure requests settings

    ', output_text) + 'Settings Pagure request - test - Pagure', + output_text) self.assertIn( '', output_text) @@ -135,8 +131,8 @@ class PagureFlaskPluginPagureRequestHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook Pagure requests deactivated', output_text) @@ -145,9 +141,8 @@ class PagureFlaskPluginPagureRequestHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Pagure requests settings

    ', output_text) + 'Settings Pagure request - test - Pagure', + output_text) self.assertIn( '', output_text) diff --git a/tests/test_pagure_flask_ui_plugins_pagure_ticket_hook.py b/tests/test_pagure_flask_ui_plugins_pagure_ticket_hook.py index 82b40a1..c156ee9 100644 --- a/tests/test_pagure_flask_ui_plugins_pagure_ticket_hook.py +++ b/tests/test_pagure_flask_ui_plugins_pagure_ticket_hook.py @@ -1,7 +1,7 @@ # -*- coding: utf-8 -*- """ - (c) 2015-2016 - Copyright Red Hat Inc + (c) 2015-2018 - Copyright Red Hat Inc Authors: Pierre-Yves Chibon @@ -40,9 +40,8 @@ class PagureFlaskPluginPagureTicketHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Pagure tickets settings

    ', output_text) + 'Settings Pagure tickets - test - Pagure', + output_text) self.assertIn( '', output_text) @@ -56,9 +55,8 @@ class PagureFlaskPluginPagureTicketHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Pagure tickets settings

    ', output_text) + 'Settings Pagure tickets - test - Pagure', + output_text) self.assertIn( '', output_text) @@ -74,8 +72,8 @@ class PagureFlaskPluginPagureTicketHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook Pagure tickets deactivated', output_text) @@ -84,9 +82,8 @@ class PagureFlaskPluginPagureTicketHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Pagure tickets settings

    ', output_text) + 'Settings Pagure tickets - test - Pagure', + output_text) self.assertIn( '', output_text) @@ -107,8 +104,8 @@ class PagureFlaskPluginPagureTicketHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook Pagure tickets activated', output_text) @@ -117,9 +114,8 @@ class PagureFlaskPluginPagureTicketHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Pagure tickets settings

    ', output_text) + 'Settings Pagure tickets - test - Pagure', + output_text) self.assertIn( '', output_text) @@ -136,8 +132,8 @@ class PagureFlaskPluginPagureTicketHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook Pagure tickets deactivated', output_text) @@ -146,9 +142,8 @@ class PagureFlaskPluginPagureTicketHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Pagure tickets settings

    ', output_text) + 'Settings Pagure tickets - test - Pagure', + output_text) self.assertIn( '', output_text) diff --git a/tests/test_pagure_flask_ui_plugins_rtd_hook.py b/tests/test_pagure_flask_ui_plugins_rtd_hook.py index d57ccd8..6fe98f9 100644 --- a/tests/test_pagure_flask_ui_plugins_rtd_hook.py +++ b/tests/test_pagure_flask_ui_plugins_rtd_hook.py @@ -1,7 +1,7 @@ # -*- coding: utf-8 -*- """ - (c) 2016 - Copyright Red Hat Inc + (c) 2016-2018 - Copyright Red Hat Inc Authors: Pierre-Yves Chibon @@ -39,9 +39,8 @@ class PagureFlaskPluginRtdHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Read the Doc settings

    ', output_text) + 'Settings Read the Doc - test - Pagure', + output_text) self.assertIn( '', output_text) @@ -55,9 +54,8 @@ class PagureFlaskPluginRtdHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Read the Doc settings

    ', output_text) + 'Settings Read the Doc - test - Pagure', + output_text) self.assertIn( '', output_text) @@ -73,8 +71,8 @@ class PagureFlaskPluginRtdHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook Read the Doc deactivated', output_text) @@ -83,9 +81,8 @@ class PagureFlaskPluginRtdHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Read the Doc settings

    ', output_text) + 'Settings Read the Doc - test - Pagure', + output_text) self.assertIn( '', output_text) @@ -107,8 +104,8 @@ class PagureFlaskPluginRtdHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook Read the Doc activated', output_text) @@ -117,9 +114,8 @@ class PagureFlaskPluginRtdHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Read the Doc settings

    ', output_text) + 'Settings Read the Doc - test - Pagure', + output_text) self.assertIn( '', output_text) @@ -136,8 +132,8 @@ class PagureFlaskPluginRtdHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertIn( '\n Hook Read the Doc deactivated', output_text) @@ -146,9 +142,8 @@ class PagureFlaskPluginRtdHooktests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn('

    Read the Doc settings

    ', output_text) + 'Settings Read the Doc - test - Pagure', + output_text) self.assertIn( '', output_text) diff --git a/tests/test_pagure_flask_ui_plugins_unsigned.py b/tests/test_pagure_flask_ui_plugins_unsigned.py index 56d8c66..c444c3e 100644 --- a/tests/test_pagure_flask_ui_plugins_unsigned.py +++ b/tests/test_pagure_flask_ui_plugins_unsigned.py @@ -1,7 +1,7 @@ # -*- coding: utf-8 -*- """ - (c) 2016 - Copyright Red Hat Inc + (c) 2016-2018 - Copyright Red Hat Inc Authors: Pierre-Yves Chibon @@ -40,11 +40,8 @@ class PagureFlaskPluginUnsignedtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn( - '

    Block Un-Signed commits settings

    ', - output_text) + 'Settings Block Un-Signed commits - test - ' + 'Pagure', output_text) self.assertTrue( '' in output_text) @@ -59,11 +56,8 @@ class PagureFlaskPluginUnsignedtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn( - '

    Block Un-Signed commits settings

    ', - output_text) + 'Settings Block Un-Signed commits - test - ' + 'Pagure', output_text) self.assertTrue( '' in output_text) @@ -77,8 +71,8 @@ class PagureFlaskPluginUnsignedtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertTrue( '\n Hook Block Un-Signed ' 'commits deactivated' in output_text) @@ -88,11 +82,8 @@ class PagureFlaskPluginUnsignedtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn( - '

    Block Un-Signed commits settings

    ', - output_text) + 'Settings Block Un-Signed commits - test - ' + 'Pagure', output_text) self.assertTrue( '' in output_text) @@ -110,11 +101,8 @@ class PagureFlaskPluginUnsignedtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertNotIn( '\n Hook activated', output_text) @@ -131,8 +119,8 @@ class PagureFlaskPluginUnsignedtests(tests.SimplePagureTest): self.assertEqual(output.status_code, 200) output_text = output.get_data(as_text=True) self.assertIn( - '
    \n

    Settings for test

    ', - output_text) + '
    ' + 'Project Settings
    \n', output_text) self.assertTrue( '\n Hook Block Un-Signed ' 'commits deactivated' in output_text) @@ -141,11 +129,8 @@ class PagureFlaskPluginUnsignedtests(tests.SimplePagureTest): '/test/settings/Block Un-Signed commits') output_text = output.get_data(as_text=True) self.assertIn( - '
    \n' - 'test project #1
    ', output_text) - self.assertIn( - '

    Block Un-Signed commits settings

    ', - output_text) + 'Settings Block Un-Signed commits - test - ' + 'Pagure', output_text) self.assertIn( '', output_text) diff --git a/tests/test_pagure_flask_ui_pr_no_sources.py b/tests/test_pagure_flask_ui_pr_no_sources.py index b8e349d..8f42e06 100644 --- a/tests/test_pagure_flask_ui_pr_no_sources.py +++ b/tests/test_pagure_flask_ui_pr_no_sources.py @@ -1,7 +1,7 @@ # -*- coding: utf-8 -*- """ - (c) 2015 - Copyright Red Hat Inc + (c) 2015-2018 - Copyright Red Hat Inc Authors: Pierre-Yves Chibon