diff --git a/pagure/templates/_formhelper.html b/pagure/templates/_formhelper.html index cf7fed6..b171be5 100644 --- a/pagure/templates/_formhelper.html +++ b/pagure/templates/_formhelper.html @@ -71,17 +71,23 @@ {% endmacro %} -{% macro render_field_in_row(field, after="", readonly=False) %} -
\nhttp://localhost.localdomain/api/0/ci/jenkins/test/',
@@ -160,7 +160,7 @@ class PagureFlaskPluginPagureCItests(tests.SimplePagureTest):
'', output_text)
self.assertIn(
- '', output_text)
# Missing the required ci_url
@@ -190,17 +190,19 @@ class PagureFlaskPluginPagureCItests(tests.SimplePagureTest):
output_text)
else:
self.assertIn(
- ''
- ' \nThis field is required. ',
+ '\n '
+ '\n \n \n '
+ 'This field is required.',
output_text)
self.assertIn(
- ''
- ' \nThis field is required. ',
+ '\n '
+ '\n \n \n '
+ 'This field is required.',
output_text)
self.assertIn(
- '', output_text)
def test_plugin_pagure_ci_namespaced(self):
@@ -224,7 +226,7 @@ class PagureFlaskPluginPagureCItests(tests.SimplePagureTest):
'', output_text)
self.assertIn(
- '', output_text)
csrf_token = output_text.split(
@@ -270,7 +272,7 @@ class PagureFlaskPluginPagureCItests(tests.SimplePagureTest):
'', output_text)
self.assertIn(
- '', output_text)
self.assertIn(
'\nhttp://localhost.localdomain/api/0/ci/jenkins/somenamespace/test3/',
diff --git a/tests/test_pagure_flask_ui_plugins_pagure_hook.py b/tests/test_pagure_flask_ui_plugins_pagure_hook.py
index be39fff..efd341d 100644
--- a/tests/test_pagure_flask_ui_plugins_pagure_hook.py
+++ b/tests/test_pagure_flask_ui_plugins_pagure_hook.py
@@ -53,7 +53,7 @@ class PagureFlaskPluginPagureHooktests(tests.SimplePagureTest):
'Settings Pagure - test - Pagure ',
output_text)
self.assertIn(
- '', output_text)
def test_plugin_mail_no_data(self):
@@ -71,7 +71,7 @@ class PagureFlaskPluginPagureHooktests(tests.SimplePagureTest):
'Settings Pagure - test - Pagure ',
output_text)
self.assertIn(
- '', output_text)
self.assertFalse(os.path.exists(os.path.join(
@@ -114,7 +114,7 @@ class PagureFlaskPluginPagureHooktests(tests.SimplePagureTest):
'Settings Pagure - test - Pagure ',
output_text)
self.assertIn(
- '', output_text)
self.assertFalse(os.path.exists(os.path.join(
@@ -161,7 +161,7 @@ class PagureFlaskPluginPagureHooktests(tests.SimplePagureTest):
'Settings Pagure - test - Pagure ',
output_text)
self.assertIn(
- '', output_text)
self.assertTrue(os.path.exists(os.path.join(
@@ -203,7 +203,7 @@ class PagureFlaskPluginPagureHooktests(tests.SimplePagureTest):
'Settings Pagure - test - Pagure ',
output_text)
self.assertIn(
- '', output_text)
self.assertFalse(os.path.exists(os.path.join(
diff --git a/tests/test_pagure_flask_ui_plugins_pagure_no_new_branch.py b/tests/test_pagure_flask_ui_plugins_pagure_no_new_branch.py
index 9c594bb..be385d7 100644
--- a/tests/test_pagure_flask_ui_plugins_pagure_no_new_branch.py
+++ b/tests/test_pagure_flask_ui_plugins_pagure_no_new_branch.py
@@ -54,7 +54,7 @@ class PagureFlaskPluginPagureNoNewBranchHooktests(tests.SimplePagureTest):
'Settings Prevent creating new branches by git '
'push - test - Pagure ', output_text)
self.assertIn(
- '', output_text)
data = {}
@@ -68,7 +68,7 @@ class PagureFlaskPluginPagureNoNewBranchHooktests(tests.SimplePagureTest):
'Settings Prevent creating new branches by git push '
'- test - Pagure ', output_text)
self.assertIn(
- '', output_text)
def test_plugin_pagure_ticket_deactivate(self):
@@ -97,7 +97,7 @@ class PagureFlaskPluginPagureNoNewBranchHooktests(tests.SimplePagureTest):
'Settings Prevent creating new branches by git push '
'- test - Pagure ', output_text)
self.assertIn(
- '', output_text)
self.assertFalse(os.path.exists(os.path.join(
@@ -134,7 +134,7 @@ class PagureFlaskPluginPagureNoNewBranchHooktests(tests.SimplePagureTest):
'Settings Prevent creating new branches by git push '
'- test - Pagure ', output_text)
self.assertIn(
- '', output_text)
self.assertTrue(os.path.exists(os.path.join(
@@ -163,7 +163,7 @@ class PagureFlaskPluginPagureNoNewBranchHooktests(tests.SimplePagureTest):
'Settings Prevent creating new branches by git push '
'- test - Pagure ', output_text)
self.assertIn(
- '', output_text)
self.assertFalse(os.path.exists(os.path.join(
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 360488e..ba5291f 100644
--- a/tests/test_pagure_flask_ui_plugins_pagure_request_hook.py
+++ b/tests/test_pagure_flask_ui_plugins_pagure_request_hook.py
@@ -42,7 +42,7 @@ class PagureFlaskPluginPagureRequestHooktests(tests.SimplePagureTest):
'Settings Pagure requests - test - Pagure ',
output_text)
self.assertIn(
- '', output_text)
csrf_token = output.get_data(as_text=True).split(
@@ -57,7 +57,7 @@ class PagureFlaskPluginPagureRequestHooktests(tests.SimplePagureTest):
'Settings Pagure requests - test - Pagure ',
output_text)
self.assertIn(
- '', output_text)
data['csrf_token'] = csrf_token
@@ -84,7 +84,7 @@ class PagureFlaskPluginPagureRequestHooktests(tests.SimplePagureTest):
'Settings Pagure requests - test - Pagure ',
output_text)
self.assertIn(
- '', output_text)
self.assertFalse(os.path.exists(os.path.join(
@@ -116,7 +116,7 @@ class PagureFlaskPluginPagureRequestHooktests(tests.SimplePagureTest):
'Settings Pagure requests - test - Pagure ',
output_text)
self.assertIn(
- '', output_text)
self.assertTrue(os.path.exists(os.path.join(
@@ -144,7 +144,7 @@ class PagureFlaskPluginPagureRequestHooktests(tests.SimplePagureTest):
'Settings Pagure requests - test - Pagure ',
output_text)
self.assertIn(
- '', output_text)
self.assertFalse(os.path.exists(os.path.join(
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 e69cb8b..c773466 100644
--- a/tests/test_pagure_flask_ui_plugins_pagure_ticket_hook.py
+++ b/tests/test_pagure_flask_ui_plugins_pagure_ticket_hook.py
@@ -43,7 +43,7 @@ class PagureFlaskPluginPagureTicketHooktests(tests.SimplePagureTest):
'Settings Pagure tickets - test - Pagure ',
output_text)
self.assertIn(
- '', output_text)
csrf_token = output_text.split(
@@ -58,7 +58,7 @@ class PagureFlaskPluginPagureTicketHooktests(tests.SimplePagureTest):
'Settings Pagure tickets - test - Pagure ',
output_text)
self.assertIn(
- '', output_text)
data['csrf_token'] = csrf_token
@@ -85,7 +85,7 @@ class PagureFlaskPluginPagureTicketHooktests(tests.SimplePagureTest):
'Settings Pagure tickets - test - Pagure ',
output_text)
self.assertIn(
- '', output_text)
self.assertFalse(os.path.exists(os.path.join(
@@ -117,7 +117,7 @@ class PagureFlaskPluginPagureTicketHooktests(tests.SimplePagureTest):
'Settings Pagure tickets - test - Pagure ',
output_text)
self.assertIn(
- '', output_text)
self.assertTrue(os.path.exists(os.path.join(
@@ -145,7 +145,7 @@ class PagureFlaskPluginPagureTicketHooktests(tests.SimplePagureTest):
'Settings Pagure tickets - test - Pagure ',
output_text)
self.assertIn(
- '', output_text)
self.assertFalse(os.path.exists(os.path.join(
diff --git a/tests/test_pagure_flask_ui_plugins_rtd_hook.py b/tests/test_pagure_flask_ui_plugins_rtd_hook.py
index 068ec5c..91a40e5 100644
--- a/tests/test_pagure_flask_ui_plugins_rtd_hook.py
+++ b/tests/test_pagure_flask_ui_plugins_rtd_hook.py
@@ -42,7 +42,7 @@ class PagureFlaskPluginRtdHooktests(tests.SimplePagureTest):
'Settings Read the Doc - test - Pagure ',
output_text)
self.assertIn(
- '', output_text)
csrf_token = output_text.split(
@@ -57,7 +57,7 @@ class PagureFlaskPluginRtdHooktests(tests.SimplePagureTest):
'Settings Read the Doc - test - Pagure ',
output_text)
self.assertIn(
- '', output_text)
data['csrf_token'] = csrf_token
@@ -84,7 +84,7 @@ class PagureFlaskPluginRtdHooktests(tests.SimplePagureTest):
'Settings Read the Doc - test - Pagure ',
output_text)
self.assertIn(
- '', output_text)
self.assertFalse(os.path.exists(os.path.join(
@@ -117,7 +117,7 @@ class PagureFlaskPluginRtdHooktests(tests.SimplePagureTest):
'Settings Read the Doc - test - Pagure ',
output_text)
self.assertIn(
- '', output_text)
# De-Activate hook
@@ -141,7 +141,7 @@ class PagureFlaskPluginRtdHooktests(tests.SimplePagureTest):
'Settings Read the Doc - test - Pagure ',
output_text)
self.assertIn(
- '', output_text)
self.assertFalse(os.path.exists(os.path.join(
diff --git a/tests/test_pagure_flask_ui_plugins_unsigned.py b/tests/test_pagure_flask_ui_plugins_unsigned.py
index 747f0b9..d786ec8 100644
--- a/tests/test_pagure_flask_ui_plugins_unsigned.py
+++ b/tests/test_pagure_flask_ui_plugins_unsigned.py
@@ -43,7 +43,7 @@ class PagureFlaskPluginUnsignedtests(tests.SimplePagureTest):
'Settings Block Un-Signed commits - test - '
'Pagure ', output_text)
self.assertTrue(
- '' in output_text)
csrf_token = output_text.split(
@@ -59,7 +59,7 @@ class PagureFlaskPluginUnsignedtests(tests.SimplePagureTest):
'Settings Block Un-Signed commits - test - '
'Pagure ', output_text)
self.assertTrue(
- '' in output_text)
data['csrf_token'] = csrf_token
@@ -85,7 +85,7 @@ class PagureFlaskPluginUnsignedtests(tests.SimplePagureTest):
'Settings Block Un-Signed commits - test - '
'Pagure ', output_text)
self.assertTrue(
- '' in output_text)
self.assertFalse(os.path.exists(os.path.join(
@@ -132,7 +132,7 @@ class PagureFlaskPluginUnsignedtests(tests.SimplePagureTest):
'Settings Block Un-Signed commits - test - '
'Pagure ', output_text)
self.assertIn(
- '', output_text)
self.assertFalse(os.path.exists(os.path.join(