diff --git a/pagure/templates/settings.html b/pagure/templates/settings.html index 2e58bf3..1bc8fb7 100644 --- a/pagure/templates/settings.html +++ b/pagure/templates/settings.html @@ -574,7 +574,6 @@ {% endif %} - {% if config.get('ENABLE_TICKETS', True) and repo.settings.get('issue_tracker', True) %}
diff --git a/tests/test_pagure_flask_ui_repo.py b/tests/test_pagure_flask_ui_repo.py index 9f50006..bca04f5 100644 --- a/tests/test_pagure_flask_ui_repo.py +++ b/tests/test_pagure_flask_ui_repo.py @@ -1138,6 +1138,110 @@ class PagureFlaskRepotests(tests.Modeltests): '', output.data) + @patch('pagure.ui.repo.admin_session_timedout') + def test_fields_in_view_settings(self, ast): + """ Test the default fields in view_settings endpoint. """ + ast.return_value = False + + # No Git repo + output = self.app.get('/foo/settings') + self.assertEqual(output.status_code, 404) + + user = tests.FakeUser() + with tests.user_set(pagure.APP, user): + output = self.app.get('/foo/settings') + self.assertEqual(output.status_code, 404) + + item = pagure.lib.model.Project( + user_id=1, # pingou + name='test', + description='test project #1', + hook_token='aaabbbccc', + ) + self.session.add(item) + self.session.commit() + tests.create_projects_git(self.path) + + output = self.app.get('/test/settings') + self.assertEqual(output.status_code, 403) + + # User not logged in + output = self.app.get('/test/settings') + self.assertEqual(output.status_code, 302) + + user.username = 'pingou' + with tests.user_set(pagure.APP, user): + ast.return_value = True + output = self.app.get('/test/settings') + self.assertEqual(output.status_code, 302) + + ast.return_value = False + output = self.app.get('/test/settings') + self.assertEqual(output.status_code, 200) + self.assertIn( + 'Settings - test - Pagure', output.data) + self.assertIn('

Settings for test

', output.data) + # Check that the priorities have their empty fields + self.assertIn( + '''
+
+
+ +
+
+ +
+
+
''', output.data) + + # Check that the milestones have their empty fields + self.assertIn( + '''
+
+
+ +
+
+ +
+
''', output.data) + + # Check that the close_status have its empty field + self.assertIn( + '''
+
+
+ +
+
''', output.data) + + # Check that the custom fields have their empty fields + self.assertIn( + '''
+
+
+ +
+
+ +
+
+ +
+
''', output.data) + def test_view_forks(self): """ Test the view_forks endpoint. """