# -*- coding: utf-8 -*-
"""
(c) 2015-2018 - Copyright Red Hat Inc
Authors:
Pierre-Yves Chibon <pingou@pingoured.fr>
"""
from __future__ import unicode_literals
__requires__ = ['SQLAlchemy >= 0.8']
import pkg_resources
from unittest.case import SkipTest
import json
import unittest
import shutil
import sys
import os
try:
import pyclamd
except ImportError:
pyclamd = None
import six
import tempfile
import re
from datetime import datetime, timedelta
from six.moves.urllib.parse import urlparse, parse_qs
import pygit2
from bs4 import BeautifulSoup
from mock import patch, MagicMock
sys.path.insert(0, os.path.join(os.path.dirname(
os.path.abspath(__file__)), '..'))
import pagure
import pagure.lib
import tests
class PagureFlaskIssuestests(tests.Modeltests):
""" Tests for flask issues controller of pagure """
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_new_issue(self, p_send_email, p_ugt):
""" Test the new_issue endpoint. """
p_send_email.return_value = True
p_ugt.return_value = True
# No Git repo
output = self.app.get('/foo/new_issue')
self.assertEqual(output.status_code, 404)
user = tests.FakeUser()
with tests.user_set(self.app.application, user):
output = self.app.get('/foo/new_issue')
self.assertEqual(output.status_code, 404)
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
output = self.app.get('/test/new_issue')
self.assertEqual(output.status_code, 200)
self.assertIn(
'New Issue',
output.get_data(as_text=True))
csrf_token = self.get_csrf(output=output)
data = {
}
# Insufficient input
output = self.app.post('/test/new_issue', data=data)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<h4 class="font-weight-bold mb-4">New Issue</h4>\n',
output_text)
self.assertEqual(output_text.count(
'This field is required.'), 2)
data['title'] = 'Test issue'
output = self.app.post('/test/new_issue', data=data)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<h4 class="font-weight-bold mb-4">New Issue</h4>\n',
output_text)
self.assertEqual(output_text.count(
'This field is required.'), 1)
data['issue_content'] = 'We really should improve on this issue'
data['status'] = 'Open'
output = self.app.post('/test/new_issue', data=data)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<h4 class="font-weight-bold mb-4">New Issue</h4>\n',
output_text)
self.assertEqual(output_text.count(
'This field is required.'),
0)
# Invalid user
data['csrf_token'] = csrf_token
output = self.app.post('/test/new_issue', data=data)
self.assertEqual(output.status_code, 404)
self.assertIn(
'<p>No such user found in the database: username</p>',
output.get_data(as_text=True))
# User not logged in
output = self.app.get('/test/new_issue')
self.assertEqual(output.status_code, 302)
user.username = 'pingou'
with tests.user_set(self.app.application, user):
output = self.app.post(
'/test/new_issue', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
# Project w/o issue tracker
repo = pagure.lib.get_authorized_project(self.session, 'test')
repo.settings = {'issue_tracker': False}
self.session.add(repo)
self.session.commit()
user.username = 'pingou'
with tests.user_set(self.app.application, user):
output = self.app.post(
'/test/new_issue', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 404)
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_new_issue_w_file(self, p_send_email, p_ugt):
""" Test the new_issue endpoint with a file. """
p_send_email.return_value = True
p_ugt.return_value = True
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
tests.create_projects_git(
os.path.join(self.path, 'tickets'), bare=True)
user = tests.FakeUser()
user.username = 'pingou'
with tests.user_set(self.app.application, user):
output = self.app.get('/test/new_issue')
self.assertEqual(output.status_code, 200)
self.assertIn(
'<h4 class="font-weight-bold mb-4">New Issue</h4>\n',
output.get_data(as_text=True))
csrf_token = self.get_csrf()
with open(os.path.join(tests.HERE, 'placebo.png'), 'rb') as stream:
data = {
'title': 'Test issue',
'issue_content': 'We really should improve on this issue\n'
'<!!image>',
'status': 'Open',
'filestream': stream,
'enctype': 'multipart/form-data',
'csrf_token': csrf_token,
}
output = self.app.post(
'/test/new_issue', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
# Check the image was uploaded
self.assertIn(
'href="/test/issue/raw/'
'8a06845923010b27bfd8e7e75acff7badc40d1021b4'
'994e01f5e11ca40bc3abe',
output_text)
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_new_issue_w_file_no_issue_tracker(self, p_send_email, p_ugt):
""" Test the new_issue endpoint with a file. """
p_send_email.return_value = True
p_ugt.return_value = True
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
tests.create_projects_git(
os.path.join(self.path, 'tickets'), bare=True)
# Project w/o issue tracker
repo = pagure.lib.get_authorized_project(self.session, 'test')
repo.settings = {'issue_tracker': False}
self.session.add(repo)
self.session.commit()
user = tests.FakeUser()
user.username = 'pingou'
with tests.user_set(self.app.application, user):
with open(os.path.join(tests.HERE, 'placebo.png'), 'rb') as stream:
data = {
'title': 'Test issue',
'issue_content': 'We really should improve on this issue',
'status': 'Open',
'filestream': stream,
'enctype': 'multipart/form-data',
'csrf_token': self.get_csrf(),
}
output = self.app.post(
'/test/new_issue', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 404)
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_new_issue_w_file_namespace(self, p_send_email, p_ugt):
""" Test the new_issue endpoint with a file. """
p_send_email.return_value = True
p_ugt.return_value = True
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
tests.create_projects_git(
os.path.join(self.path, 'tickets'), bare=True)
# Project with a namespace
user = tests.FakeUser()
user.username = 'pingou'
with tests.user_set(self.app.application, user):
output = self.app.get('/somenamespace/test3/new_issue')
self.assertEqual(output.status_code, 200)
self.assertTrue(
'<h4 class="font-weight-bold mb-4">New Issue</h4>\n'
in output.get_data(as_text=True))
csrf_token = self.get_csrf()
with open(os.path.join(tests.HERE, 'placebo.png'), 'rb') as stream:
data = {
'title': 'Test issue3',
'issue_content': 'We really should improve on this issue\n'
'<!!image>',
'status': 'Open',
'filestream': stream,
'enctype': 'multipart/form-data',
'csrf_token': csrf_token,
}
output = self.app.post(
'/somenamespace/test3/new_issue', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue3 - test3 - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0" '
'href="/somenamespace/test3/issue/1/edit" title="Edit this issue">\n',
output_text)
# Check the image was uploaded
self.assertIn(
'href="/somenamespace/test3/issue/raw/'
'8a06845923010b27bfd8e7e75acff7badc40d1021b4'
'994e01f5e11ca40bc3abe',
output_text)
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_new_issue_w_files(self, p_send_email, p_ugt):
""" Test the new_issue endpoint with two files. """
p_send_email.return_value = True
p_ugt.return_value = True
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
tests.create_projects_git(
os.path.join(self.path, 'tickets'), bare=True)
user = tests.FakeUser()
user.username = 'pingou'
with tests.user_set(self.app.application, user):
output = self.app.get('/test/new_issue')
self.assertEqual(output.status_code, 200)
self.assertTrue(
'<h4 class="font-weight-bold mb-4">New Issue</h4>\n'
in output.get_data(as_text=True))
csrf_token = self.get_csrf()
with open(
os.path.join(tests.HERE, 'placebo.png'), 'rb'
) as stream:
with open(
os.path.join(tests.HERE, 'pagure.png'), 'rb'
) as stream2:
data = {
'title': 'Test issue',
'issue_content': 'We really should improve on this issue\n'
'<!!image>\n<!!image>',
'status': 'Open',
'filestream': [stream, stream2],
'enctype': 'multipart/form-data',
'csrf_token': csrf_token,
}
output = self.app.post(
'/test/new_issue', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
# Check the image was uploaded
self.assertIn(
'href="/test/issue/raw/'
'8a06845923010b27bfd8e7e75acff7badc40d1021b4'
'994e01f5e11ca40bc3abe',
output_text)
self.assertIn(
'href="/test/issue/raw/'
'6498a2de405546200b6144da56fc25d0a3976ae688d'
'bfccaca609c8b4480523e',
output_text)
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_new_issue_w_files_namespace(self, p_send_email, p_ugt):
""" Test the new_issue endpoint with two files. """
p_send_email.return_value = True
p_ugt.return_value = True
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
tests.create_projects_git(
os.path.join(self.path, 'tickets'), bare=True)
# Project with a namespace
user = tests.FakeUser()
user.username = 'pingou'
with tests.user_set(self.app.application, user):
output = self.app.get('/somenamespace/test3/new_issue')
self.assertEqual(output.status_code, 200)
self.assertTrue(
'<h4 class="font-weight-bold mb-4">New Issue</h4>\n'
in output.get_data(as_text=True))
csrf_token = self.get_csrf()
with open(
os.path.join(tests.HERE, 'placebo.png'), 'rb'
) as stream:
with open(
os.path.join(tests.HERE, 'pagure.png'), 'rb'
) as stream2:
data = {
'title': 'Test issue3',
'issue_content': 'We really should improve on this issue\n'
'<!!image>\n<!!image>',
'status': 'Open',
'filestream': [stream, stream2],
'enctype': 'multipart/form-data',
'csrf_token': csrf_token,
}
output = self.app.post(
'/somenamespace/test3/new_issue',
data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue3 - test3 - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0" '
'href="/somenamespace/test3/issue/1/edit" title="Edit this issue">\n',
output_text)
# Check the image was uploaded
self.assertIn(
'href="/somenamespace/test3/issue/raw/'
'8a06845923010b27bfd8e7e75acff7badc40d1021b4'
'994e01f5e11ca40bc3abe',
output_text)
self.assertIn(
'href="/somenamespace/test3/issue/raw/'
'6498a2de405546200b6144da56fc25d0a3976ae688d'
'bfccaca609c8b4480523e',
output_text)
def test_new_issue_metadata_user(self):
""" Test the new_issue endpoint when the user has access to the
project. """
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
tests.create_projects_git(
os.path.join(self.path, 'tickets'), bare=True)
user = tests.FakeUser()
user.username = 'pingou'
with tests.user_set(self.app.application, user):
output = self.app.get('/test/new_issue')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<h4 class="font-weight-bold mb-4">New Issue</h4>\n',
output_text)
self.assertIn(
'<strong>Tags</strong>',
output_text)
self.assertIn(
'<strong>Assignee</strong>',
output_text)
def test_new_issue_metadata_not_user(self):
""" Test the new_issue endpoint when the user does not have access
to the project. """
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
tests.create_projects_git(
os.path.join(self.path, 'tickets'), bare=True)
user = tests.FakeUser()
user.username = 'foo'
with tests.user_set(self.app.application, user):
output = self.app.get('/test/new_issue')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<h4 class="font-weight-bold mb-4">New Issue</h4>\n',
output_text)
self.assertNotIn(
'<strong>Tags</strong>',
output_text)
self.assertNotIn(
'<strong>Assignee</strong>',
output_text)
@patch('pagure.lib.git.update_git', MagicMock(return_value=True))
@patch('pagure.lib.notify.send_email', MagicMock(return_value=True))
def test_new_issue_with_metadata(self):
""" Test the new_issue endpoint when the user has access to the
project. """
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
tests.create_projects_git(
os.path.join(self.path, 'tickets'), bare=True)
# Set some milestone
repo = pagure.lib.get_authorized_project(self.session, 'test')
repo.milestones = {'v1.0': '', 'v2.0': 'Tomorrow!'}
self.session.add(repo)
self.session.commit()
user = tests.FakeUser()
user.username = 'pingou'
with tests.user_set(self.app.application, user):
output = self.app.get('/test/new_issue')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<h4 class="font-weight-bold mb-4">New Issue</h4>\n',
output_text)
self.assertIn(
'<strong>Tags</strong>',
output_text)
self.assertIn(
'<strong>Assignee</strong>',
output_text)
csrf_token = self.get_csrf(output=output)
data = {
'title': 'Test issue3',
'issue_content': 'We really should improve on this issue\n',
'status': 'Open',
'assignee': 'foo',
'milestone': 'v2.0',
'tag': 'tag2',
'csrf_token': csrf_token,
}
output = self.app.post(
'/test/new_issue', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue3 - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0" '
'href="/test/issue/1/edit" title="Edit this issue">\n',
output_text)
# Check the metadata
self.assertIn(
'title="comma separated list of tags"\n '
'value="tag2" />', output_text)
self.assertIn(
'placeholder="username"\n value="foo" />\n',
output_text)
self.assertIn(
'href="/test/roadmap/v2.0/"',
output_text)
@patch('pagure.lib.git.update_git', MagicMock(return_value=True))
@patch('pagure.lib.notify.send_email', MagicMock(return_value=True))
def test_new_issue_with_metadata_not_user(self):
""" Test the new_issue endpoint when the user does not have access
to the project but still tries to.
"""
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
tests.create_projects_git(
os.path.join(self.path, 'tickets'), bare=True)
# Set some milestone
repo = pagure.lib.get_authorized_project(self.session, 'test')
repo.milestones = {'v1.0': '', 'v2.0': 'Tomorrow!'}
self.session.add(repo)
self.session.commit()
user = tests.FakeUser()
user.username = 'foo'
with tests.user_set(self.app.application, user):
output = self.app.get('/test/new_issue')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<h4 class="font-weight-bold mb-4">New Issue</h4>\n',
output_text)
self.assertNotIn(
'<strong>Tags</strong>',
output_text)
self.assertNotIn(
'<strong>Assignee</strong>',
output_text)
csrf_token = self.get_csrf(output=output)
data = {
'title': 'Test issue3',
'issue_content': 'We really should improve on this issue\n',
'status': 'Open',
'assignee': 'foo',
'milestone': 'v2.0',
'tag': 'tag2',
'csrf_token': csrf_token,
}
output = self.app.post(
'/test/new_issue', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue3 - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0" '
'href="/test/issue/1/edit" title="Edit this issue">\n',
output_text)
# Check the metadata
self.assertNotIn(
'title="comma separated list of tags"\n '
'value="tag2" />', output_text)
self.assertNotIn(
'placeholder="username"\n value="foo" />\n',
output_text)
self.assertNotIn(
'<div id="milestone_plain">\n <span>'
'\n <a href="/test/roadmap/v2.0/">'
'\n v2.0\n', output_text)
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_view_issues_roadmap_view(self, p_send_email, p_ugt):
""" Test the view_issues endpoint when view is set as roadmap view """
p_send_email.return_value = True
p_ugt.return_value = True
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
# Change settings to show roadmap on issue page
repo = pagure.lib.get_authorized_project(self.session, 'test')
old_settings = repo.settings
old_settings['roadmap_on_issues_page'] = True
repo.settings = old_settings
self.session.add(repo)
self.session.commit()
# check on landing page of project for the URL that on the Issues tab
output = self.app.get('/test/')
self.assertEqual(output.status_code, 200)
self.assertIn('<a class="nav-link" href="/test/roadmap"', output.get_data(as_text=True))
# Revert and check
old_settings = repo.settings
old_settings['roadmap_on_issues_page'] = False
repo.settings = old_settings
self.session.add(repo)
self.session.commit()
output = self.app.get('/test/')
self.assertEqual(output.status_code, 200)
self.assertIn('<a class="nav-link" href="/test/issues"', output.get_data(as_text=True))
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_view_issues(self, p_send_email, p_ugt):
""" Test the view_issues endpoint. """
p_send_email.return_value = True
p_ugt.return_value = True
output = self.app.get('/foo/issues')
self.assertEqual(output.status_code, 404)
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
output = self.app.get('/test/issues')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<i class="fa fa-calendar-o fa-rotate-270 text-muted"></i></h3>',
output_text)
self.assertIn(
'<a href="/test"><strong>test</strong></a>', output_text)
self.assertIn(
'<span class="fa fa-fw fa-exclamation-circle"></span> 0 Open Issues\n', output_text)
repo = pagure.lib.get_authorized_project(self.session, 'test')
# Create some custom fields to play with
msg = pagure.lib.set_custom_key_fields(
session=self.session,
project=repo,
fields=['test1'],
types=['text'],
data=[None],
notify=[None]
)
self.session.commit()
self.assertEqual(msg, 'List of custom fields updated')
cfield = pagure.lib.get_custom_key(
session=self.session,
project=repo,
keyname='test1')
# Create issues to play with
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='tést íssüé',
content='We should work on this',
user='pingou',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'tést íssüé')
msg = pagure.lib.set_custom_key_value(
session=self.session,
issue=msg,
key=cfield,
value='firstissue')
self.session.commit()
self.assertEqual(msg, 'Custom field test1 adjusted to firstissue')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Tést íssüé with milestone',
content='Testing search',
user='pingou',
milestone='1.1',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Tést íssüé with milestone')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test invalid issue',
content='This really is not related',
user='pingou',
status='Closed',
close_status='Invalid',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test invalid issue')
msg = pagure.lib.set_custom_key_value(
session=self.session,
issue=msg,
key=cfield,
value='second issue')
self.session.commit()
self.assertEqual(msg, 'Custom field test1 adjusted to second issue')
# Whole list
output = self.app.get('/test/issues')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn('<title>Issues - test - Pagure</title>',
output_text)
self.assertIn(
'<span class="fa fa-fw fa-exclamation-circle"></span> 2 Open Issues\n', output_text)
self.assertIn(
'title="2 Open Issues | 1 Closed Issues', output_text)
self.assertIn(
'bg-success" role="progressbar"\n'
' style="width:67%"\n', output_text)
# Status = closed (all but open)
output = self.app.get('/test/issues?status=cloSED')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn('<title>Issues - test - Pagure</title>',
output_text)
self.assertIn(
'<span class="fa fa-fw fa-exclamation-circle"></span> 1 Closed Issues\n', output_text)
self.assertIn(
'title="2 Open Issues | 1 Closed Issues"', output_text)
self.assertIn(
'bg-danger" role="progressbar"\n'
' style="width:33%"\n', output_text)
# Status = fixed
output = self.app.get('/test/issues?status=fixed')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn('<title>Issues - test - Pagure</title>',
output_text)
self.assertIn(
'<span class="fa fa-fw fa-exclamation-circle"></span> 0 Closed:Fixed Issues\n', output_text)
# Status = Invalid
output = self.app.get('/test/issues?status=Invalid')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn('<title>Issues - test - Pagure</title>', output_text)
self.assertTrue(
'<span class="fa fa-fw fa-exclamation-circle"></span> 1 Closed:Invalid Issues\n' in output_text)
# All tickets
output = self.app.get('/test/issues?status=all')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn('<title>Issues - test - Pagure</title>', output_text)
self.assertTrue(
'<span class="fa fa-fw fa-exclamation-circle"></span> 3 Open & Closed Issues\n' in output_text)
# Unicode search pattern
output = self.app.get(
'/test/issues?status=all&search_pattern=گروه')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn('<title>Issues - test - Pagure</title>', output_text)
self.assertIn('0 Open & Closed Issues', output_text)
# Custom key searching
output = self.app.get(
'/test/issues?status=all&search_pattern=test1:firstissue')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issues - test - Pagure</title>',
output_text)
self.assertIn('1 Open & Closed Issues', output_text)
# Custom key searching with space
output = self.app.get(
'/test/issues?status=all&search_pattern=test1:"second issue"')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issues - test - Pagure</title>',
output_text)
self.assertIn('1 Open & Closed Issues', output_text)
# All tickets - different pagination
before = pagure.config.config['ITEM_PER_PAGE']
pagure.config.config['ITEM_PER_PAGE'] = 1
output = self.app.get('/test/issues?status=all')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issues - test - Pagure</title>',
output_text)
self.assertIn(
'<span class="fa fa-fw fa-exclamation-circle"></span> 3 Open & Closed Issues\n',
output_text)
self.assertIn(
'page 1 of 3', output_text)
# All tickets - filtered for 1 - checking the pagination
output = self.app.get(
'/test/issues?status=all&search_pattern=invalid')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn('<title>Issues - test - Pagure</title>', output_text)
self.assertIn('<span class="fa fa-fw fa-exclamation-circle"></span> 1 Open & Closed Issues\n', output_text)
self.assertNotIn(
'page 1', output_text)
pagure.config.config['ITEM_PER_PAGE'] = before
# Search for issues with no milestone MARK
output = self.app.get(
'/test/issues?milestone=none')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issues - test - Pagure</title>',
output_text)
self.assertIn('<span class="fa fa-fw fa-exclamation-circle"></span> 1 Open Issues\n', output_text)
# Search for issues with no milestone and milestone 1.1
output = self.app.get(
'/test/issues?milestone=none&milestone=1.1')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issues - test - Pagure</title>',
output_text)
self.assertIn('<span class="fa fa-fw fa-exclamation-circle"></span> 2 Open Issues\n', output_text)
# Add another issue to test sorting
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Big problÈm!',
content='I need help ASAP',
user='foo',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Big problÈm!')
# Sort by last_updated
output = self.app.get('/test/issues?order_key=last_updated')
self.assertEqual(output.status_code, 200)
tr_elements = re.findall(r'<div class="issuerow list-group-item list-group-item-action">(.*?)</div><!-- end issuerow -->',
output.get_data(as_text=True), re.M | re.S)
# Make sure that issue four is first since it was modified last
self.assertIn('href="/test/issue/4"', tr_elements[0])
# Make sure that issue two is second since it was modified second
self.assertIn('href="/test/issue/2"', tr_elements[1])
# Make sure that issue one is last since it was modified first
self.assertIn('href="/test/issue/1"', tr_elements[2])
# Modify the date of the first issue and try again
issue_one = pagure.lib.search_issues(self.session, repo, 1)
issue_one.last_updated = datetime.utcnow() + timedelta(seconds=2)
self.session.add(issue_one)
self.session.commit()
output = self.app.get('/test/issues?order_key=last_updated')
self.assertEqual(output.status_code, 200)
tr_elements = re.findall(r'<div class="issuerow list-group-item list-group-item-action">(.*?)</div><!-- end issuerow -->',
output.get_data(as_text=True), re.M | re.S)
# Make sure that issue one is first since it was modified last
self.assertIn('href="/test/issue/1"', tr_elements[0])
# Make sure that issue four is second since it was modified before
# last
self.assertIn('href="/test/issue/4"', tr_elements[1])
# Make sure that issue two is last since it was modified before issue
# one and four
self.assertIn('href="/test/issue/2"', tr_elements[2])
# Now query so that the results are ascending
output = self.app.get('/test/issues?order_key=last_updated&order=asc')
tr_elements = re.findall(r'<div class="issuerow list-group-item list-group-item-action">(.*?)</div><!-- end issuerow -->',
output.get_data(as_text=True), re.M | re.S)
self.assertIn('href="/test/issue/2"', tr_elements[0])
self.assertIn('href="/test/issue/4"', tr_elements[1])
self.assertIn('href="/test/issue/1"', tr_elements[2])
# Sort by title descending
output = self.app.get('/test/issues?order_key=title')
self.assertEqual(output.status_code, 200)
tr_elements = re.findall(r'<div class="issuerow list-group-item list-group-item-action">(.*?)</div><!-- end issuerow -->',
output.get_data(as_text=True), re.M | re.S)
self.assertIn('href="/test/issue/2"', tr_elements[0])
self.assertIn('href="/test/issue/1"', tr_elements[1])
self.assertIn('href="/test/issue/4"', tr_elements[2])
# Sort by title ascending
output = self.app.get('/test/issues?order_key=title&order=asc')
self.assertEqual(output.status_code, 200)
tr_elements = re.findall(r'<div class="issuerow list-group-item list-group-item-action">(.*?)</div><!-- end issuerow -->',
output.get_data(as_text=True), re.M | re.S)
self.assertIn('href="/test/issue/4"', tr_elements[0])
self.assertIn('href="/test/issue/1"', tr_elements[1])
self.assertIn('href="/test/issue/2"', tr_elements[2])
# Sort by user (reporter/author) descending
output = self.app.get('/test/issues?order_key=user&order=desc')
self.assertEqual(output.status_code, 200)
tr_elements = re.findall(r'<div class="issuerow list-group-item list-group-item-action">(.*?)</div><!-- end issuerow -->',
output.get_data(as_text=True), re.M | re.S)
# We check that they are unassigned, otherwise our previous check is
# not specific enough as it can catch an assignee of "pingou"
self.assertNotIn('fa-user-plus', tr_elements[0])
self.assertIn('pingou', tr_elements[1])
self.assertNotIn('fa-user-plus', tr_elements[1])
self.assertIn('foo', tr_elements[2])
self.assertNotIn('fa-user-plus', tr_elements[2])
# Sort by user (reporter/author) ascending
output = self.app.get('/test/issues?order_key=user&order=asc')
self.assertEqual(output.status_code, 200)
tr_elements = re.findall(r'<div class="issuerow list-group-item list-group-item-action">(.*?)</div><!-- end issuerow -->',
output.get_data(as_text=True), re.M | re.S)
# Check for the name after the avatar
self.assertIn('foo', tr_elements[0])
# We check that they are unassigned, otherwise our previous check is
# not specific enough as it can catch an assignee of "foo"
self.assertNotIn('fa-user-plus', tr_elements[0])
self.assertIn('pingou', tr_elements[1])
self.assertNotIn('fa-user-plus', tr_elements[1])
self.assertIn('pingou', tr_elements[2])
self.assertNotIn('fa-user-plus', tr_elements[2])
# Set some assignees
issues = self.session.query(pagure.lib.model.Issue).filter_by(
status='Open').order_by(pagure.lib.model.Issue.id).all()
issues[0].assignee_id = 1
issues[1].assignee_id = 2
issues[2].assignee_id = 1
self.session.commit()
# This detects the assignee but keying on if a certain link is present
def _check_assignee_link(html, expected_links):
soup = BeautifulSoup(html, "html.parser")
for index, expected_link in enumerate(expected_links):
link = soup.find_all("tr")[index + 1].find(
"a", title="Filter issues by assignee")
self.assertIsNotNone(link, "Link %s was not found" % expected_link)
self.assertURLEqual(link["href"], expected_link)
# Sort by assignee descending
output = self.app.get('/test/issues?order_key=assignee&order=desc')
self.assertEqual(output.status_code, 200)
#tr_elements = re.findall(r'<div class="issuerow list-group-item list-group-item-action">(.*?)</div><!-- end issuerow -->',
# output.get_data(as_text=True), re.M | re.S)
#arrowed_th = ('Assignee</a>\n <span class="oi" data-glyph='
# '"arrow-thick-bottom"></span>')
# First table row is the header
#self.assertIn(arrowed_th, tr_elements[0])
#_check_assignee_link(output.get_data(as_text=True), [
# '/test/issues?status=Open&assignee=pingou',
# '/test/issues?status=Open&assignee=pingou',
# '/test/issues?status=Open&assignee=foo',
#])
# Sort by assignee ascending
#output = self.app.get('/test/issues?order_key=assignee&order=asc')
#self.assertEqual(output.status_code, 200)
#tr_elements = re.findall(r'<tr>(.*?)</tr>', output.get_data(as_text=True), re.M | re.S)
#arrowed_th = ('Assignee</a>\n <span class="oi" data-glyph='
# '"arrow-thick-top"></span>')
# First table row is the header
#self.assertIn(arrowed_th, tr_elements[0])
#_check_assignee_link(output.get_data(as_text=True), [
# '/test/issues?status=Open&assignee=foo',
# '/test/issues?status=Open&assignee=pingou',
# '/test/issues?status=Open&assignee=pingou',
#])
# New issue button is shown
user = tests.FakeUser()
with tests.user_set(self.app.application, user):
output = self.app.get('/test')
self.assertEqual(output.status_code, 200)
self.assertIn(
'fa-exclamation-circle fa-fw"></i> New issue</a>',
output.get_data(as_text=True))
output = self.app.get('/test')
self.assertEqual(output.status_code, 200)
self.assertNotIn(
'fa-exclamation-circle fa-fw"></i> New issue</a>',
output.get_data(as_text=True))
# Project w/o issue tracker
repo = pagure.lib.get_authorized_project(self.session, 'test')
repo.settings = {'issue_tracker': False}
self.session.add(repo)
self.session.commit()
output = self.app.get('/test/issues')
self.assertEqual(output.status_code, 404)
# New issue button is hidden
user = tests.FakeUser()
with tests.user_set(self.app.application, user):
output = self.app.get('/test')
self.assertEqual(output.status_code, 200)
self.assertNotIn(
'fa-exclamation-circle fa-fw"></i> New issue</a>',
output.get_data(as_text=True))
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_search_issues_unicode(self, p_send_email, p_ugt):
""" Test the view_issues endpoint filtering for an unicode char. """
p_send_email.return_value = True
p_ugt.return_value = True
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
repo = pagure.lib.get_authorized_project(self.session, 'test')
# Create 2 issues to play with
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue ☃',
content='We should work on this ❤',
user='pingou',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue ☃')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue with milestone',
content='Testing search',
user='pingou',
milestone='1.1',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue with milestone')
# Whole list
output = self.app.get('/test/issues')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn('<title>Issues - test - Pagure</title>', output_text)
self.assertTrue(
'<span class="fa fa-fw fa-exclamation-circle"></span> 2 Open Issues\n' in output_text)
# Unicode search pattern
output = self.app.get(
'/test/issues?status=all&search_pattern=☃')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn('<title>Issues - test - Pagure</title>', output_text)
self.assertIn('<span class="fa fa-fw fa-exclamation-circle"></span> 1 Open & Closed Issues\n', output_text)
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_view_issue_inconsistent_milestone(self, p_send_email, p_ugt):
""" Test the view_issue endpoint when the milestone keys are
inconsistent with the milestones of the project. """
p_send_email.return_value = True
p_ugt.return_value = True
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
# Add milestones to the project
repo = pagure.lib.get_authorized_project(self.session, 'test')
milestones = {
'v1.0': {'date': None, 'active': True},
'v2.0': {'date': 'in the future', 'active': True},
}
repo.milestones = milestones
repo.milestones_keys = ['', 'v1.0', 'v2.0']
# Create issues to play with
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='We should work on this',
user='pingou',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue')
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
# Not authentified = No edit
self.assertNotIn(
'<a class="btn btn-outline-secondary btn-sm border-0" ''href="/test/issue/1/edit" title="Edit this issue">\n',
output_text)
self.assertIn(
'<a href="/login/?next=http%3A%2F%2Flocalhost%2Ftest%2Fissue%2F1">'
'Login</a>\n to comment on this ticket.',
output_text)
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_view_issue(self, p_send_email, p_ugt):
""" Test the view_issue endpoint. """
p_send_email.return_value = True
p_ugt.return_value = True
output = self.app.get('/foo/issue/1')
self.assertEqual(output.status_code, 404)
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 404)
# Create issues to play with
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='We should work on this',
user='pingou',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue')
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
# Not authentified = No edit
self.assertNotIn(
'<a class="btn btn-outline-secondary btn-sm border-0" '
'href="/test/issue/1/edit" title="Edit this issue">\n',
output_text)
self.assertIn(
'<a href="/login/?next=http%3A%2F%2Flocalhost%2Ftest%2Fissue%2F1">'
'Login</a>\n to comment on this ticket.',
output_text)
user = tests.FakeUser()
with tests.user_set(self.app.application, user):
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
# Not author nor admin = No edit
self.assertNotIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
self.assertNotIn(
'<a class="dropdown-item text-danger" href="javascript:void(0)" id="closeticket"\n'
' title="Delete this ticket">\n',
output_text)
self.assertFalse(
'<a href="/login/">Login</a> to comment on this ticket.'
in output_text)
# Not author nor admin = No take
self.assertNotIn('function take_issue(){', output_text)
self.assertNotIn('function drop_issue(){', output_text)
self.assertNotIn(
'<a href="javascript:void(0)" id="take-btn"\n',
output_text)
user.username = 'pingou'
with tests.user_set(self.app.application, user):
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
self.assertIn(
'<a class="dropdown-item text-danger" href="javascript:void(0)" id="closeticket"\n'
' title="Delete this ticket">\n',
output_text)
csrf_token = self.get_csrf(output=output)
# Create private issue
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='We should work on this',
user='pingou',
ticketfolder=None,
private=True,
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue')
# Not logged in
output = self.app.get('/test/issue/2')
self.assertEqual(output.status_code, 404)
# Wrong user
user = tests.FakeUser()
with tests.user_set(self.app.application, user):
output = self.app.get('/test/issue/2')
self.assertEqual(output.status_code, 404)
# reporter
user.username = 'pingou'
with tests.user_set(self.app.application, user):
output = self.app.get('/test/issue/2')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #2: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<span title="Private ticket" class="text-danger fa fa-fw '
'fa-lock"></span>', output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/2/edit" title="Edit this issue">',
output_text)
# Project w/o issue tracker
repo = pagure.lib.get_authorized_project(self.session, 'test')
repo.settings = {'issue_tracker': False}
self.session.add(repo)
self.session.commit()
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 404)
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_view_issue_user_ticket(self, p_send_email, p_ugt):
""" Test the view_issue endpoint. """
p_send_email.return_value = True
p_ugt.return_value = True
output = self.app.get('/foo/issue/1')
self.assertEqual(output.status_code, 404)
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 404)
# Create issues to play with
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='We should work on this',
user='pingou',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue')
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
# Not authentified = No edit
self.assertNotIn(
'<a class="btn btn-outline-secondary btn-sm border-0" '
'href="/test/issue/1/edit" title="Edit this issue">\n',
output_text)
self.assertTrue(
'<a href="/login/?next=http%3A%2F%2Flocalhost%2Ftest%2Fissue%2F1">'
'Login</a>\n to comment on this ticket.'
in output_text)
# Create issues to play with
repo = pagure.lib.get_authorized_project(self.session, 'test')
# Add user 'foo' with ticket access on repo
msg = pagure.lib.add_user_to_project(
self.session,
repo,
new_user='foo',
user='pingou',
access='ticket',
)
self.assertEqual(msg, 'User added')
self.session.commit()
user = tests.FakeUser(username='foo')
with tests.user_set(self.app.application, user):
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
# Not author nor admin = No edit
self.assertNotIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
self.assertNotIn(
'<a class="dropdown-item text-danger" href="javascript:void(0)" id="closeticket"\n'
' title="Delete this ticket">\n',
output_text)
self.assertFalse(
'<a href="/login/">Login</a> to comment on this ticket.'
in output_text)
# user has ticket = take ok
self.assertIn('function take_issue(){', output_text)
self.assertIn('function drop_issue(){', output_text)
self.assertIn(
'<a href="javascript:void(0)" id="take-btn"\n',
output_text)
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_view_issue_custom_field_user_ticket(self, p_send_email, p_ugt):
""" Test the view_issue endpoint. """
p_send_email.return_value = True
p_ugt.return_value = True
output = self.app.get('/foo/issue/1')
self.assertEqual(output.status_code, 404)
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 404)
# Create issues to play with
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='We should work on this',
user='pingou',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue')
# Add user 'foo' with ticket access on repo
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.add_user_to_project(
self.session,
repo,
new_user='foo',
user='pingou',
access='ticket',
)
self.assertEqual(msg, 'User added')
self.session.commit()
# Set some custom fields
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.set_custom_key_fields(
self.session,
repo,
['bugzilla', 'upstream', 'reviewstatus'],
['link', 'boolean', 'list'],
['unused data for non-list type', '', 'ack, nack , needs review'],
[None, None, None])
self.session.commit()
self.assertEqual(msg, 'List of custom fields updated')
# User with no rights
user = tests.FakeUser()
with tests.user_set(self.app.application, user):
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertNotIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
self.assertNotIn(
'<a class="dropdown-item text-danger" href="javascript:void(0)" id="closeticket"\n'
' title="Delete this ticket">\n',
output_text)
# user no ACLs = no take action/button
self.assertNotIn('function take_issue(){', output_text)
self.assertNotIn('function drop_issue(){', output_text)
self.assertNotIn(
'<a href="javascript:void(0)" id="take-btn"\n',
output_text)
# user no ACLs = no metadata form
self.assertNotIn(
'<input class="form-control" '
'name="bugzilla" id="bugzilla"/>', output_text)
self.assertNotIn(
'<select class="form-control" name="reviewstatus" '
'id="reviewstatus>', output_text)
self.assertNotIn(
'<input type="checkbox" '
'class="form-control" name="upstream" id="upstream"/>',
output_text)
user = tests.FakeUser(username='foo')
with tests.user_set(self.app.application, user):
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertNotIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
self.assertNotIn(
'<a class="dropdown-item text-danger" href="javascript:void(0)" id="closeticket"\n'
' title="Delete this ticket">\n',
output_text)
self.assertNotIn(
'<a href="/login/">Login</a> to comment on this ticket.',
output_text)
# user has ticket = take ok
self.assertIn('function take_issue(){', output_text)
self.assertIn('function drop_issue(){', output_text)
self.assertIn(
'<a href="javascript:void(0)" id="take-btn"\n',
output_text)
# user has ticket == Sees the metadata
self.assertIn(
'<input class="form-control" '
'name="bugzilla" id="bugzilla"/>', output_text)
self.assertIn(
'<select class="form-control"\n'
' name="reviewstatus"\n'
' id="reviewstatus">\n',
output_text)
self.assertIn(
'<input type="checkbox" '
'class="form-control" name="upstream" id="upstream"/>',
output_text)
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_view_issue_non_ascii_milestone(self, p_send_email, p_ugt):
""" Test the view_issue endpoint with non-ascii milestone. """
p_send_email.return_value = True
p_ugt.return_value = True
output = self.app.get('/foo/issue/1')
self.assertEqual(output.status_code, 404)
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 404)
stone = 'käpy'
# Create issues to play with
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='We should work on this',
user='pingou',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue')
# Add a non-ascii milestone to the issue but project has no milestone
issue = pagure.lib.search_issues(self.session, repo, issueid=1)
message = pagure.lib.edit_issue(
self.session,
issue=issue,
milestone=stone,
private=False,
user='pingou',
ticketfolder=None
)
self.assertEqual(
message,
[
'Issue set to the milestone: k\xe4py'
]
)
self.session.commit()
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertNotIn(stone, output_text)
# Add a non-ascii milestone to the project
repo = pagure.lib.get_authorized_project(self.session, 'test')
repo.milestones = {'käpy': None}
self.session.add(repo)
self.session.commit()
# View the issue
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(stone, output_text)
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_view_issue_list_no_data(self, p_send_email, p_ugt):
""" Test the view_issue endpoint when the issue has a custom field
of type list with no data attached. """
p_send_email.return_value = True
p_ugt.return_value = True
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
repo = pagure.lib.get_authorized_project(self.session, 'test')
# Add custom fields to the project
msg = pagure.lib.set_custom_key_fields(
session=self.session,
project=repo,
fields=['test1'],
types=['list'],
data=[None],
notify=[None]
)
self.session.commit()
self.assertEqual(msg, 'List of custom fields updated')
# Create issues to play with
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Big problÈm!',
content='We should work on this',
user='pingou',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Big problÈm!')
# Assign a value to the custom key on that ticket
cfield = pagure.lib.get_custom_key(
session=self.session,
project=repo,
keyname='test1')
msg = pagure.lib.set_custom_key_value(
session=self.session,
issue=msg,
key=cfield,
value='item')
self.session.commit()
self.assertEqual(msg, 'Custom field test1 adjusted to item')
user = tests.FakeUser()
user.username = 'pingou'
with tests.user_set(self.app.application, user):
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 200)
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_update_issue(self, p_send_email, p_ugt):
""" Test the update_issue endpoint. """
p_send_email.return_value = True
p_ugt.return_value = True
# No Git repo
output = self.app.get('/foo/issue/1/update')
self.assertEqual(output.status_code, 404)
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
output = self.app.get('/test/issue/1/update')
self.assertEqual(output.status_code, 302)
# Create issues to play with
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='We should work on this',
user='pingou',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue')
user = tests.FakeUser()
user.username = 'pingou'
with tests.user_set(self.app.application, user):
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
self.assertEqual(output_text.count('title="PY C (pingou)"'), 1)
csrf_token = self.get_csrf(output=output)
data = {
'status': 'Closed',
'close_status': 'fixed'
}
# Invalid repo
output = self.app.post('/bar/issue/1/update', data=data)
self.assertEqual(output.status_code, 404)
# Non-existing issue
output = self.app.post('/test/issue/100/update', data=data)
self.assertEqual(output.status_code, 404)
output = self.app.post(
'/test/issue/1/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
self.assertFalse(
'<option selected value="Fixed">Fixed</option>'
in output_text)
# Right status, wrong csrf
data['close_status'] = 'Fixed'
output = self.app.post(
'/test/issue/1/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
self.assertFalse(
'<option selected value="Fixed">Fixed</option>'
in output_text)
# working status update
data['csrf_token'] = csrf_token
output = self.app.post(
'/test/issue/1/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
self.assertIn(
''
'Issue close_status updated to: Fixed',
output_text)
self.assertIn(
''
'Issue status updated to: Closed (was: Open)',
output_text)
self.assertTrue(
'<option selected value="Fixed">Fixed</option>'
in output_text)
# FIXME: There is likely something going wrong in the html
# below
self.assertIn(
'<span class="text-semimuted font-size-09 autogenerated-comment pl-4">'
'<p><strong>Metadata Update from <a href="http://localhost.localdomain/user/pingou">'
'</a><a href="http://localhost.localdomain/user/pingou">@pingou</a></strong>:'
'<br>\n- Issue close_status updated to: Fixed<br>\n- Issue status updated to:'
' Closed (was: Open)</p></span>\n',
output_text)
# Add new comment
data = {
'csrf_token': csrf_token,
'status': 'Closed',
'close_status': 'Fixed',
'comment': 'Woohoo a second comment!',
}
output = self.app.post(
'/test/issue/1/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
self.assertIn(
'Comment added',
output_text)
self.assertNotIn(
'No changes to edit',
output_text)
self.assertIn(
'<p>Woohoo a second comment!</p>',
output_text)
self.assertEqual(
output_text.count('comment_body">'), 2)
self.assertTrue(
'<option selected value="Fixed">Fixed</option>'
in output_text)
# 3: one for the original comment, one for the new comment, one for the metadata update
self.assertEqual(
output_text.count('title="PY C (pingou)"'),
3)
# Add new tag
data = {
'csrf_token': csrf_token,
'status': 'Closed',
'close_status': 'Fixed',
'tag': 'tag2',
}
output = self.app.post(
'/test/issue/1/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
self.assertIn(
'<p>Woohoo a second comment!</p>',
output_text)
self.assertEqual(
output_text.count('comment_body">'), 2)
self.assertTrue(
'<option selected value="Fixed">Fixed</option>'
in output_text)
# Assign issue to an non-existent user
data = {
'csrf_token': csrf_token,
'status': 'Closed',
'close_status': 'Fixed',
'assignee': 'ralph',
}
output = self.app.post(
'/test/issue/1/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
self.assertIn(
'No user "ralph" found',
output_text)
self.assertIn(
'<p>Woohoo a second comment!</p>',
output_text)
self.assertEqual(
output_text.count('comment_body">'), 2)
self.assertTrue(
'<option selected value="Fixed">Fixed</option>'
in output_text)
# Assign issue properly
data = {
'csrf_token': csrf_token,
'status': 'Closed',
'close_status': 'Fixed',
'assignee': 'pingou',
}
output = self.app.post(
'/test/issue/1/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
self.assertIn(
'Issue assigned to pingou',
output_text)
self.assertIn(
'<a href="/test/issues?assignee=pingou" title="PY C (pingou)"',
output_text)
self.assertIn(
'<p>Woohoo a second comment!</p>',
output_text)
self.assertEqual(
output_text.count('comment_body">'), 2)
self.assertTrue(
'<option selected value="Fixed">Fixed</option>'
in output_text)
# Create another issue with a dependency
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='We should work on this',
user='pingou',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue')
# Reset the status of the first issue
parent_issue = pagure.lib.search_issues(
self.session, repo, issueid=1)
parent_issue.status = 'Open'
self.session.add(parent_issue)
# Add the dependency relationship
self.session.add(parent_issue)
issue = pagure.lib.search_issues(self.session, repo, issueid=2)
issue.parents.append(parent_issue)
self.session.add(issue)
self.session.commit()
with tests.user_set(self.app.application, user):
data['csrf_token'] = csrf_token
output = self.app.post(
'/test/issue/2/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #2: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/2/edit" title="Edit this issue">',
output_text)
self.assertIn(
'You cannot close a ticket '
'that has ticket depending that are still open.',
output_text)
self.assertTrue(
'<option selected value="Open">Open</option>'
in output_text)
# Create private issue
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='We should work on this',
user='pingou',
ticketfolder=None,
private=True,
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue')
# Wrong user
user = tests.FakeUser()
with tests.user_set(self.app.application, user):
output = self.app.post(
'/test/issue/3/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 403)
# Project w/o issue tracker
repo = pagure.lib.get_authorized_project(self.session, 'test')
repo.settings = {'issue_tracker': False}
self.session.add(repo)
self.session.commit()
with tests.user_set(self.app.application, user):
# Repo not set-up for issue tracker
output = self.app.post('/test/issue/1/update', data=data)
self.assertEqual(output.status_code, 404)
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_update_issue_drop_comment(self, p_send_email, p_ugt):
""" Test droping comment via the update_issue endpoint. """
p_send_email.return_value = True
p_ugt.return_value = True
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
# Create issues to play with
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='We should work on this',
user='pingou',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue')
user = tests.FakeUser()
user.username = 'pingou'
with tests.user_set(self.app.application, user):
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
csrf_token = self.get_csrf(output=output)
# Add new comment
data = {
'csrf_token': csrf_token,
'comment': 'Woohoo a second comment!',
}
output = self.app.post(
'/test/issue/1/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
self.assertIn(
'Comment added',
output_text)
self.assertIn(
'<p>Woohoo a second comment!</p>',
output_text)
self.assertEqual(
output_text.count('comment_body">'), 2)
repo = pagure.lib.get_authorized_project(self.session, 'test')
issue = pagure.lib.search_issues(self.session, repo, issueid=1)
self.assertEqual(len(issue.comments), 1)
data = {
'csrf_token': csrf_token,
'drop_comment': 1,
}
user = tests.FakeUser()
with tests.user_set(self.app.application, user):
# Wrong issue id
output = self.app.post(
'/test/issue/3/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 404)
# Wrong user
output = self.app.post(
'/test/issue/1/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 403)
user = tests.FakeUser()
user.username = 'pingou'
with tests.user_set(self.app.application, user):
# Drop the new comment
output = self.app.post(
'/test/issue/1/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
self.assertIn(
'Comment removed',
output_text)
# Drop non-existant comment
output = self.app.post(
'/test/issue/1/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 404)
self.session.commit()
repo = pagure.lib.get_authorized_project(self.session, 'test')
issue = pagure.lib.search_issues(self.session, repo, issueid=1)
self.assertEqual(len(issue.comments), 0)
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_update_issue_depend(self, p_send_email, p_ugt):
""" Test adding dependency via the update_issue endpoint. """
p_send_email.return_value = True
p_ugt.return_value = True
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
# Create issues to play with
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='We should work on this',
user='pingou',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue')
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue #2',
content='We should work on this again',
user='foo',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue #2')
user = tests.FakeUser()
user.username = 'pingou'
with tests.user_set(self.app.application, user):
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
csrf_token = self.get_csrf(output=output)
# Add a dependent ticket
data = {
'csrf_token': csrf_token,
'depending': '2',
}
output = self.app.post(
'/test/issue/1/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
# Add an invalid dependent ticket
data = {
'csrf_token': csrf_token,
'depending': '2,abc',
}
output = self.app.post(
'/test/issue/1/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
self.assertNotIn(
''
'Successfully edited issue #1',
output_text)
repo = pagure.lib.get_authorized_project(self.session, 'test')
issue = pagure.lib.search_issues(self.session, repo, issueid=1)
self.assertEqual(issue.depending_text, [2])
self.assertEqual(issue.blocking_text, [])
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_update_issue_block(self, p_send_email, p_ugt):
""" Test adding blocked issue via the update_issue endpoint. """
p_send_email.return_value = True
p_ugt.return_value = True
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
# Create issues to play with
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='We should work on this',
user='pingou',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue')
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue #2',
content='We should work on this again',
user='foo',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue #2')
# User is not an admin of the project
user = tests.FakeUser(username='foo')
with tests.user_set(self.app.application, user):
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 200)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output.get_data(as_text=True))
csrf_token = self.get_csrf(output=output)
# Add a dependent ticket
data = {
'csrf_token': csrf_token,
'blocking': '2',
}
output = self.app.post(
'/test/issue/1/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output.get_data(as_text=True))
repo = pagure.lib.get_authorized_project(self.session, 'test')
issue = pagure.lib.search_issues(self.session, repo, issueid=1)
self.assertEqual(issue.depending_text, [])
self.assertEqual(issue.blocking_text, [])
user = tests.FakeUser()
user.username = 'pingou'
with tests.user_set(self.app.application, user):
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
csrf_token = self.get_csrf(output=output)
# Add a dependent ticket
data = {
'csrf_token': csrf_token,
'blocking': '2',
}
output = self.app.post(
'/test/issue/1/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
# Add an invalid dependent ticket
data = {
'csrf_token': csrf_token,
'blocking': '2,abc',
}
output = self.app.post(
'/test/issue/1/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
self.assertNotIn(
''
'Successfully edited issue #1',
output_text)
self.session.commit()
repo = pagure.lib.get_authorized_project(self.session, 'test')
issue = pagure.lib.search_issues(self.session, repo, issueid=1)
self.assertEqual(issue.depending_text, [])
self.assertEqual(issue.blocking_text, [2])
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_upload_issue(self, p_send_email, p_ugt):
""" Test the upload_issue endpoint. """
p_send_email.return_value = True
p_ugt.return_value = True
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
tests.create_projects_git(
os.path.join(self.path, 'repos', 'tickets'), bare=True)
# Create issues to play with
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='We should work on this',
user='pingou',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue')
user = tests.FakeUser()
user.username = 'pingou'
with tests.user_set(self.app.application, user):
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
csrf_token = self.get_csrf(output=output)
output = self.app.post('/foo/issue/1/upload')
self.assertEqual(output.status_code, 404)
output = self.app.post('/test/issue/100/upload')
self.assertEqual(output.status_code, 404)
# Invalid upload
data = {
'enctype': 'multipart/form-data',
}
output = self.app.post(
'/test/issue/1/upload', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
json_data = json.loads(output.get_data(as_text=True))
exp = {'output': 'notok'}
self.assertDictEqual(json_data, exp)
# Attach a file to a ticket
with open(os.path.join(tests.HERE, 'placebo.png'), 'rb') as stream:
data = {
'csrf_token': csrf_token,
'filestream': stream,
'enctype': 'multipart/form-data',
}
output = self.app.post(
'/test/issue/1/upload', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
json_data = json.loads(output.get_data(as_text=True))
folder = os.path.dirname(
os.path.abspath(__file__))[1:].replace('/', '_')
exp = {
'filelocations': [
'/test/issue/raw/files/8a06845923010b27bfd8'
'e7e75acff7badc40d1021b4994e01f5e11ca40bc3a'
'be-%s_placebo.png' % folder
],
'filenames': ['%s_placebo.png' % folder],
'output': 'ok'
}
self.assertDictEqual(json_data, exp)
# Project w/o issue tracker
repo = pagure.lib.get_authorized_project(self.session, 'test')
repo.settings = {'issue_tracker': False}
self.session.add(repo)
self.session.commit()
with tests.user_set(self.app.application, user):
output = self.app.post('/test/issue/1/upload')
self.assertEqual(output.status_code, 404)
@patch.dict('pagure.config.config', {'PR_ONLY': True})
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_upload_issue_virus(self, p_send_email, p_ugt):
""" Test the upload_issue endpoint. """
if not pyclamd:
raise SkipTest()
p_send_email.return_value = True
p_ugt.return_value = True
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
tests.create_projects_git(
os.path.join(self.path, 'tickets'), bare=True)
# Create issues to play with
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='We should work on this',
user='pingou',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue')
user = tests.FakeUser()
user.username = 'pingou'
with tests.user_set(self.app.application, user):
csrf_token = self.get_csrf()
# TODO: Figure a way to enable this test on jenkins
# Try to attach a virus
if not os.environ.get('BUILD_ID'):
with tempfile.NamedTemporaryFile() as eicarfile:
eicarfile.write(pyclamd.ClamdUnixSocket().EICAR())
eicarfile.flush()
with open(eicarfile.name, 'rb') as stream:
data = {
'csrf_token': csrf_token,
'filestream': stream,
'enctype': 'multipart/form-data',
}
output = self.app.post(
'/test/issue/1/upload', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
json_data = json.loads(output.get_data(as_text=True))
exp = {
'output': 'notok',
}
self.assertDictEqual(json_data, exp)
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_upload_issue_two_files(self, p_send_email, p_ugt):
""" Test the upload_issue endpoint with two files. """
p_send_email.return_value = True
p_ugt.return_value = True
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
tests.create_projects_git(
os.path.join(self.path, 'tickets'), bare=True)
# Create issues to play with
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='We should work on this',
user='pingou',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue')
user = tests.FakeUser()
user.username = 'pingou'
with tests.user_set(self.app.application, user):
csrf_token = self.get_csrf()
# Attach two files to a ticket
with open(os.path.join(tests.HERE, 'placebo.png'), 'rb') as stream:
with open(os.path.join(tests.HERE, 'placebo.png'), 'rb') as stream2:
data = {
'csrf_token': csrf_token,
'filestream': [stream, stream2],
'enctype': 'multipart/form-data',
}
output = self.app.post(
'/test/issue/1/upload', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
json_data = json.loads(output.get_data(as_text=True))
folder = os.path.dirname(
os.path.abspath(__file__))[1:].replace('/', '_')
exp = {
'output': 'ok',
'filelocations': [
'/test/issue/raw/files/8a06845923010b27bfd8'
'e7e75acff7badc40d1021b4994e01f5e11ca40bc3a'
'be-%s_placebo.png' % folder,
'/test/issue/raw/files/8a06845923010b27bfd8'
'e7e75acff7badc40d1021b4994e01f5e11ca40bc3a'
'be-%s_placebo.png' % folder,
],
'filenames': [
'%s_placebo.png' % folder,
'%s_placebo.png' % folder
],
}
self.assertDictEqual(json_data, exp)
def test_view_issue_raw_file_empty(self):
""" Test the view_issue_raw_file endpoint. """
# Create the project and git repos
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'tickets'), bare=True)
# Create issues to play with
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='We should work on this',
user='pingou',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue')
url = '/issue/raw/8a06845923010b27bfd8'\
'e7e75acff7badc40d1021b4994e01f5e11ca40bc3a'\
'be-home_pierrey_repos_gitrepo_pagure_tests'\
'_placebo.png'
output = self.app.get('/foo' + url)
self.assertEqual(output.status_code, 404)
output = self.app.get('/test' + url)
self.assertEqual(output.status_code, 404)
# Project w/o issue tracker
repo = pagure.lib.get_authorized_project(self.session, 'test')
repo.settings = {'issue_tracker': False}
self.session.add(repo)
self.session.commit()
output = self.app.get('/test' + url)
self.assertEqual(output.status_code, 404)
def test_view_issue_raw_file(self):
""" Test the view_issue_raw_file endpoint. """
# Create the issue and upload to it
self.test_upload_issue()
# Project w/ issue tracker
repo = pagure.lib.get_authorized_project(self.session, 'test')
repo.settings = {'issue_tracker': True}
self.session.add(repo)
self.session.commit()
url = '/issue/raw/8a06845923010b27bfd8'\
'e7e75acff7badc40d1021b4994e01f5e11ca40bc3a'\
'be-%s_placebo.png' % os.path.dirname(
os.path.abspath(__file__))[1:].replace('/', '_')
output = self.app.get('/foo' + url)
self.assertEqual(output.status_code, 404)
output = self.app.get('/test/issue/raw/test.png')
self.assertEqual(output.status_code, 404)
# Access file by name
output = self.app.get('/test' + url)
self.assertEqual(output.status_code, 200)
# Project w/o issue tracker
repo = pagure.lib.get_authorized_project(self.session, 'test')
repo.settings = {'issue_tracker': False}
self.session.add(repo)
self.session.commit()
output = self.app.get('/test' + url)
self.assertEqual(output.status_code, 404)
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_edit_issue(self, p_send_email, p_ugt):
""" Test the edit_issue endpoint. """
p_send_email.return_value = True
p_ugt.return_value = True
# No Git repo
output = self.app.get('/foo/issue/1/edit')
self.assertEqual(output.status_code, 404)
user = tests.FakeUser()
with tests.user_set(self.app.application, user):
output = self.app.get('/foo/issue/1/edit')
self.assertEqual(output.status_code, 404)
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
output = self.app.get('/test/issue/1/edit')
self.assertEqual(output.status_code, 404)
# User not logged in
output = self.app.get('/foo/issue/1/edit')
self.assertEqual(output.status_code, 404)
user.username = 'pingou'
with tests.user_set(self.app.application, user):
output = self.app.get('/test/issue/1/edit')
self.assertEqual(output.status_code, 404)
# Create issues to play with
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='We should work on this',
user='pingou',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue')
user = tests.FakeUser()
with tests.user_set(self.app.application, user):
output = self.app.get('/test/issue/1/edit')
self.assertEqual(output.status_code, 403)
user.username = 'pingou'
with tests.user_set(self.app.application, user):
output = self.app.get('/test/issue/1/edit')
self.assertEqual(output.status_code, 200)
self.assertTrue(
'Edit Issue' in output.get_data(as_text=True))
csrf_token = self.get_csrf(output=output)
data = {
'issue_content': 'We should work on this!'
}
output = self.app.post('/test/issue/1/edit', data=data)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertTrue(
'Edit Issue' in output_text)
self.assertEqual(output_text.count(
'This field is required.'), 1)
data['status'] = 'Open'
data['title'] = 'Test issue #1'
output = self.app.post('/test/issue/1/edit', data=data)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertTrue(
'Edit Issue' in output_text)
self.assertEqual(output_text.count(
'This field is required.'), 0)
self.assertEqual(output_text.count(
'Not a valid choice'), 0)
data['csrf_token'] = csrf_token
output = self.app.post(
'/test/issue/1/edit', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
' <span class="fa fa-fw text-success fa-exclamation-circle pt-1"></span>\n'
' <span class="text-success font-weight-bold">#1</span>\n ',
output_text)
self.assertEqual(output_text.count(
'<option selected value="Open">Open</option>'), 1)
self.assertEqual(output_text.count('comment_body">'), 1)
self.assertEqual(output_text.count(
'<p>We should work on this!</p>'), 1)
# Project w/o issue tracker
repo = pagure.lib.get_authorized_project(self.session, 'test')
repo.settings = {'issue_tracker': False}
self.session.add(repo)
self.session.commit()
user.username = 'pingou'
with tests.user_set(self.app.application, user):
output = self.app.post('/test/issue/1/edit', data=data)
self.assertEqual(output.status_code, 404)
@patch('pagure.lib.git.update_git', MagicMock(return_value=True))
@patch('pagure.lib.notify.send_email', MagicMock(return_value=True))
def test_edit_issue_no_change(self):
""" Test the edit_issue endpoint. """
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
# Create an issue to play with
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='We should work on this',
user='pingou',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue')
user = tests.FakeUser(username = 'pingou')
with tests.user_set(self.app.application, user):
output = self.app.get('/test/issue/1/edit')
self.assertEqual(output.status_code, 200)
self.assertTrue(
'Edit Issue' in output.get_data(as_text=True))
csrf_token = self.get_csrf(output=output)
# Change nothing in the issue
data = {
'issue_content': 'We should work on this',
'status': 'Open',
'title': 'Test issue',
'csrf_token': csrf_token
}
output = self.app.post(
'/test/issue/1/edit', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
' <span class="fa fa-fw text-success fa-exclamation-circle pt-1"></span>\n'
' <span class="text-success font-weight-bold">#1</span>\n ',
output_text)
self.assertEqual(output_text.count(
'<option selected value="Open">Open</option>'), 1)
self.assertEqual(output_text.count('comment_body">'), 1)
self.assertEqual(output_text.count(
'<p>We should work on this</p>'), 1)
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_edit_tag(self, p_send_email, p_ugt):
""" Test the edit_tag endpoint. """
p_send_email.return_value = True
p_ugt.return_value = True
# No Git repo
output = self.app.get('/foo/tag/foo/edit')
self.assertEqual(output.status_code, 404)
user = tests.FakeUser()
with tests.user_set(self.app.application, user):
output = self.app.get('/foo/tag/foo/edit')
self.assertEqual(output.status_code, 404)
tests.create_projects(self.session)
tests.create_projects_git(os.path.join(self.path, 'repos'))
output = self.app.get('/test/tag/foo/edit')
self.assertEqual(output.status_code, 403)
# User not logged in
output = self.app.get('/test/tag/foo/edit')
self.assertEqual(output.status_code, 302)
# Create issues to play with
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='We should work on this',
user='pingou',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue')
# Add a tag to the issue
issue = pagure.lib.search_issues(self.session, repo, issueid=1)
msg = pagure.lib.add_tag_obj(
session=self.session,
obj=issue,
tags='tag1',
user='pingou',
gitfolder=None)
self.session.commit()
self.assertEqual(msg, 'Issue tagged with: tag1')
# Before edit, list tags
tags = pagure.lib.get_tags_of_project(self.session, repo)
self.assertEqual([tag.tag for tag in tags], ['tag1'])
# Edit tag
user.username = 'pingou'
with tests.user_set(self.app.application, user):
#Edit a tag that doesn't exit
output = self.app.get('/test/tag/does_not_exist/edit')
self.assertEqual(output.status_code, 404)
output = self.app.get('/test/tag/tag1/edit')
self.assertEqual(output.status_code, 200)
self.assertTrue('<strong>Edit tag: tag1</strong>' in output.get_data(as_text=True))
csrf_token = self.get_csrf(output=output)
data = {'tag': 'tag2',
'tag_description': 'lorem ipsum',
'tag_color': 'DeepSkyBlue'}
output = self.app.post('/test/tag/tag1/edit', data=data)
self.assertEqual(output.status_code, 200)
self.assertTrue('<strong>Edit tag: tag1</strong>' in output.get_data(as_text=True))
data['csrf_token'] = csrf_token
output = self.app.post(
'/test/tag/tag1/edit', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'Settings - test - Pagure', output_text)
self.assertIn(
''
'Edited tag: tag1()[DeepSkyBlue] to tag2(lorem ipsum)[DeepSkyBlue]',
output_text)
# update tag with empty description
data['tag_description'] = ''
output = self.app.post(
'/test/tag/tag2/edit', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'Settings - test - Pagure', output_text)
self.assertIn(
''
'Edited tag: tag2(lorem ipsum)[DeepSkyBlue] to tag2()[DeepSkyBlue]',
output_text)
# After edit, list tags
self.session.commit()
tags = pagure.lib.get_tags_of_project(self.session, repo)
self.assertEqual([tag.tag for tag in tags], ['tag2'])
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_remove_tag(self, p_send_email, p_ugt):
""" Test the remove_tag endpoint. """
p_send_email.return_value = True
p_ugt.return_value = True
# No Git repo
output = self.app.post('/foo/droptag/')
self.assertEqual(output.status_code, 404)
user = tests.FakeUser()
with tests.user_set(self.app.application, user):
output = self.app.post('/foo/droptag/')
self.assertEqual(output.status_code, 404)
tests.create_projects(self.session)
tests.create_projects_git(os.path.join(self.path, 'repos'))
output = self.app.post('/test/droptag/')
self.assertEqual(output.status_code, 403)
# User not logged in
output = self.app.post('/test/droptag/')
self.assertEqual(output.status_code, 302)
# Create issues to play with
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='We should work on this',
user='pingou',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue')
# Add a tag to the issue
issue = pagure.lib.search_issues(self.session, repo, issueid=1)
msg = pagure.lib.add_tag_obj(
session=self.session,
obj=issue,
tags='tag1',
user='pingou',
gitfolder=None)
self.session.commit()
self.assertEqual(msg, 'Issue tagged with: tag1')
# Before edit, list tags
tags = pagure.lib.get_tags_of_project(self.session, repo)
self.assertEqual([tag.tag for tag in tags], ['tag1'])
# Edit tag
user.username = 'pingou'
with tests.user_set(self.app.application, user):
output = self.app.post(
'/test/droptag/', data={}, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertTrue(
'<title>Settings - test - Pagure</title>' in output_text)
self.assertIn(
'<h5 class="pl-2 font-weight-bold text-muted">Project '
'Settings</h5>', output_text)
csrf_token = self.get_csrf(output=output)
data = {'tag': 'tag1'}
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.assertIn(
'<h5 class="pl-2 font-weight-bold text-muted">Project '
'Settings</h5>', 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.assertIn(
'<h5 class="pl-2 font-weight-bold text-muted">Project '
'Settings</h5>', output_text)
self.assertIn(
''
'Tag: tag1 has been deleted', output_text)
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_delete_issue(self, p_send_email, p_ugt):
""" Test the delete_issue endpoint. """
p_send_email.return_value = True
p_ugt.return_value = True
tests.create_projects(self.session)
tests.create_projects_git(os.path.join(self.path, 'repos'))
tests.create_projects_git(os.path.join(self.path, 'tickets'))
# Create issues to play with
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='We should work on this',
user='pingou',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue')
user = tests.FakeUser()
with tests.user_set(self.app.application, user):
output = self.app.post(
'/foo/issue/1/drop', follow_redirects=True)
self.assertEqual(output.status_code, 404)
output = self.app.post(
'/test/issue/100/drop', follow_redirects=True)
self.assertEqual(output.status_code, 404)
output = self.app.post(
'/test/issue/1/drop', follow_redirects=True)
self.assertEqual(output.status_code, 403)
user.username = 'pingou'
with tests.user_set(self.app.application, user):
output = self.app.post(
'/test/issue/1/drop', follow_redirects=True)
self.assertEqual(output.status_code, 200)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output.get_data(as_text=True))
csrf_token = self.get_csrf(output=output)
data = {
}
# No CSRF token
output = self.app.post(
'/test/issue/1/drop', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output.get_data(as_text=True))
data['csrf_token'] = csrf_token
output = self.app.post(
'/test/issue/1/drop', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issues - test - Pagure</title>', output_text)
self.assertIn(
'Issue deleted',
output_text)
# Project w/o issue tracker
repo = pagure.lib.get_authorized_project(self.session, 'test')
repo.settings = {'issue_tracker': False}
self.session.add(repo)
self.session.commit()
user.username = 'pingou'
with tests.user_set(self.app.application, user):
output = self.app.post('/test/issue/1/drop', data=data)
self.assertEqual(output.status_code, 404)
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_update_issue_edit_comment(self, p_send_email, p_ugt):
""" Test the issues edit comment endpoint """
p_send_email.return_value = True
p_ugt.return_value = True
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
# Create issues to play with
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='We should work on this',
user='pingou',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue')
user = tests.FakeUser()
user.username = 'pingou'
with tests.user_set(self.app.application, user):
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">\n',
output_text)
csrf_token = self.get_csrf(output=output)
# Add new comment
data = {
'csrf_token': csrf_token,
'comment': 'Woohoo a second comment!',
}
output = self.app.post(
'/test/issue/1/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">\n',
output_text)
self.assertIn(
'Comment added',
output_text)
self.assertIn(
'<p>Woohoo a second comment!</p>',
output_text)
self.assertEqual(
output_text.count('comment_body">'), 2)
repo = pagure.lib.get_authorized_project(self.session, 'test')
issue = pagure.lib.search_issues(self.session, repo, issueid=1)
self.assertEqual(len(issue.comments), 1)
self.assertEqual(
issue.comments[0].comment,
'Woohoo a second comment!')
data = {
'csrf_token': csrf_token,
'edit_comment': 1,
'update_comment': 'Updated comment',
}
user = tests.FakeUser()
with tests.user_set(self.app.application, user):
# Wrong issue id
output = self.app.post(
'/test/issue/3/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 404)
# Wrong user
output = self.app.post(
'/test/issue/1/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 403)
user = tests.FakeUser()
user.username = 'pingou'
with tests.user_set(self.app.application, user):
# Edit comment
output = self.app.post(
'/test/issue/1/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
self.assertIn(
'Comment updated',
output_text)
self.session.commit()
repo = pagure.lib.get_authorized_project(self.session, 'test')
issue = pagure.lib.search_issues(self.session, repo, issueid=1)
self.assertEqual(len(issue.comments), 1)
self.assertEqual(issue.comments[0].comment, 'Updated comment')
with tests.user_set(self.app.application, user):
output = self.app.get('/test/issue/1/comment/1/edit')
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>test - Pagure</title>', output_text)
self.assertTrue('<div id="edit">' in output_text)
self.assertTrue('<section class="edit_comment">' in output_text)
self.assertTrue(
'<textarea class="form-control" id="update_comment"'
in output_text)
csrf_token = self.get_csrf(output=output)
data['csrf_token'] = csrf_token
data['update_comment'] = 'Second update'
# Edit the comment with the other endpoint
output = self.app.post(
'/test/issue/1/comment/1/edit',
data=data,
follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
self.assertIn(
'Comment updated',
output_text)
self.session.commit()
repo = pagure.lib.get_authorized_project(self.session, 'test')
issue = pagure.lib.search_issues(self.session, repo, issueid=1)
self.assertEqual(len(issue.comments), 1)
self.assertEqual(issue.comments[0].comment, 'Second update')
# Create another issue from someone else
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='We should work on this',
user='foo',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue')
issue = pagure.lib.search_issues(self.session, repo, issueid=1)
self.assertEqual(len(issue.comments), 1)
self.assertEqual(issue.status, 'Open')
issue = pagure.lib.search_issues(self.session, repo, issueid=2)
self.assertEqual(len(issue.comments), 0)
self.assertEqual(issue.status, 'Open')
user = tests.FakeUser(username='foo')
with tests.user_set(self.app.application, user):
data = {
'csrf_token': csrf_token,
'comment': 'Nevermind figured it out',
'status': 'Closed',
'close_status': 'Invalid'
}
# Add a comment and close the ticket #1
output = self.app.post(
'/test/issue/1/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertNotIn(
''
'Successfully edited issue #1\n',
output_text
)
self.assertIn(
'Comment added',
output_text
)
self.assertNotIn(
'<a class="btn btn-outline-primary border-0 btn-sm issue-metadata-display'
' editmetadatatoggle" href="javascript:void(0)" style="display: inline-block;">'
'<i class="fa fa-fw fa-pencil">',
output_text
)
data = {
'csrf_token': csrf_token,
'comment': 'Nevermind figured it out',
'status': 'Closed',
'close_status': 'Invalid'
}
# Add a comment and close the ticket #2
output = self.app.post(
'/test/issue/2/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
''
'Issue close_status updated to: Invalid',
output_text
)
self.assertIn(
'Comment added',
output_text
)
self.assertIn(
''
'Issue status updated to: Closed (was: Open)',
output_text
)
self.assertIn(
'<a class="btn btn-outline-primary border-0 btn-sm issue-metadata-display'
' editmetadatatoggle" href="javascript:void(0)" style="display: inline-block;">'
'<i class="fa fa-fw fa-pencil">',
output_text
)
# Ticket #1 has one more comment and is still open
self.session.commit()
issue = pagure.lib.search_issues(self.session, repo, issueid=1)
self.assertEqual(len(issue.comments), 2)
self.assertEqual(issue.status, 'Open')
# Ticket #2 has one less comment and is closed
issue = pagure.lib.search_issues(self.session, repo, issueid=2)
self.assertEqual(len(issue.comments), 2)
self.assertEqual(
issue.comments[0].comment,
'Nevermind figured it out')
self.assertEqual(
issue.comments[1].comment,
'**Metadata Update from @foo**:\n'
'- Issue close_status updated to: Invalid\n'
'- Issue status updated to: Closed (was: Open)')
self.assertEqual(issue.status, 'Closed')
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_git_urls(self, p_send_email, p_ugt):
""" Check that the url to the git repo for issues is present/absent when
it should.
"""
p_send_email.return_value = True
p_ugt.return_value = True
self.test_view_issues()
user = tests.FakeUser()
user.username = 'pingou'
repo = pagure.lib._get_project(self.session, 'test')
pagure.lib.update_read_only_mode(self.session, repo, read_only=False)
pagure.lib.get_user(self.session, 'pingou').public_ssh_key = 'foo'
self.session.commit()
with tests.user_set(self.app.application, user):
# Check that the git issue URL is present
output = self.app.get('/test')
self.assertNotIn(
'<h5><strong>Issues GIT URLs</strong></h5>',
output.get_data(as_text=True))
# Project w/o issue tracker
repo = pagure.lib.get_authorized_project(self.session, 'test')
repo.settings = {'issue_tracker': True}
self.session.add(repo)
self.session.commit()
# Check that the git issue URL is gone
output = self.app.get('/test')
output_text = output.get_data(as_text=True)
self.assertIn(
'<h5><strong>Issues</strong></h5>', output_text)
self.assertIn(
'value="ssh://git@localhost.localdomain/tickets/test.git',
output_text)
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_update_tags(self, p_send_email, p_ugt):
""" Test the update_tags endpoint. """
p_send_email.return_value = True
p_ugt.return_value = True
# No Git repo
output = self.app.post('/foo/update/tags')
self.assertEqual(output.status_code, 404)
user = tests.FakeUser()
with tests.user_set(self.app.application, user):
output = self.app.post('/foo/update/tags')
self.assertEqual(output.status_code, 404)
tests.create_projects(self.session)
tests.create_projects_git(os.path.join(self.path, 'repos'))
output = self.app.post('/test/update/tags')
self.assertEqual(output.status_code, 403)
# User not logged in
output = self.app.post('/test/update/tags')
self.assertEqual(output.status_code, 302)
# Create issues to play with
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='We should work on this',
user='pingou',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue')
# Before update, list tags
tags = pagure.lib.get_tags_of_project(self.session, repo)
self.assertEqual([tag.tag for tag in tags], [])
user.username = 'pingou'
with tests.user_set(self.app.application, user):
# No CSRF
data = {
'tag': 'red',
'tag_description': 'lorem ipsum',
'tag_color': '#ff0000'
}
output = self.app.post(
'/test/update/tags', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Settings - test - Pagure</title>', output_text)
self.assertIn(
'<h5 class="pl-2 font-weight-bold text-muted">Project '
'Settings</h5>', output_text)
csrf_token = self.get_csrf(output=output)
# Invalid color
data = {
'tag': 'red',
'tag_description': 'lorem ipsum',
'tag_color': 'red',
'csrf_token': csrf_token,
}
output = self.app.post(
'/test/update/tags', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Settings - test - Pagure</title>', output_text)
self.assertIn(
''
'Color: red does not match the expected pattern',
output_text)
self.assertIn(
'<h5 class="pl-2 font-weight-bold text-muted">Project '
'Settings</h5>', output_text)
# Invalid tag name
data = {
'tag': 'red/green',
'tag_description': 'lorem ipsum',
'tag_color': '#fff',
'csrf_token': csrf_token,
}
output = self.app.post(
'/test/update/tags', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Settings - test - Pagure</title>', output_text)
self.assertIn(
''
'Tag: red/green contains an invalid character: "/"',
output_text)
self.assertIn(
'<h5 class="pl-2 font-weight-bold text-muted">Project '
'Settings</h5>', output_text)
# Inconsistent length tags (missing tag field)
data = {
'tag': 'red',
'tag_description': ['lorem ipsum', 'foo bar'],
'tag_color': ['#ff0000', '#003cff'],
'csrf_token': csrf_token,
}
output = self.app.post(
'/test/update/tags', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Settings - test - Pagure</title>', output_text)
self.assertIn(
'Error: Incomplete request. '
'One or more tag fields missing.', output_text)
self.assertIn(
'<h5 class="pl-2 font-weight-bold text-muted">Project '
'Settings</h5>', output_text)
# Inconsistent length color
data = {
'tag': ['red', 'blue'],
'tag_description': ['lorem ipsum', 'foo bar'],
'tag_color': 'red',
'csrf_token': csrf_token,
}
output = self.app.post(
'/test/update/tags', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Settings - test - Pagure</title>', output_text)
self.assertIn(
''
'Color: red does not match the expected pattern',
output_text)
self.assertIn(
'Error: Incomplete request. '
'One or more tag color fields missing.', output_text)
self.assertIn(
'<h5 class="pl-2 font-weight-bold text-muted">Project '
'Settings</h5>', output_text)
# Inconsistent length description
data = {
'tag': ['red', 'blue'],
'tag_description': 'lorem ipsum',
'tag_color': ['#ff0000', '#003cff'],
'csrf_token': csrf_token,
}
output = self.app.post(
'/test/update/tags', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Settings - test - Pagure</title>', output_text)
self.assertIn(
'Error: Incomplete request. '
'One or more tag description fields missing.', output_text)
self.assertIn(
'<h5 class="pl-2 font-weight-bold text-muted">Project '
'Settings</h5>', output_text)
# consistent length, but empty description
data = {
'tag': ['red', 'blue'],
'tag_description': ['lorem ipsum', ''],
'tag_color': ['#ff0000', '#003cff'],
'csrf_token': csrf_token,
}
output = self.app.post(
'/test/update/tags', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Settings - test - Pagure</title>', output_text)
self.assertIn(
'<span class="badge badge-info" '
'style="background-color:#003cff">blue</span>\n'
' '
'<span class="text-muted"></span>', output_text)
self.assertIn(
'<input type="hidden" value="blue" name="tag" />',
output_text)
self.assertIn(
'<span class="badge badge-info" '
'style="background-color:#ff0000">red</span>\n'
' '
'<span class="text-muted">lorem ipsum</span>', output_text)
self.assertIn(
'<input type="hidden" value="red" name="tag" />',
output_text)
# Valid query
data = {
'tag': ['red1', 'green'],
'tag_description': ['lorem ipsum', 'sample description'],
'tag_color': ['#ff0000', '#00ff00'],
'csrf_token': csrf_token,
}
output = self.app.post(
'/test/update/tags', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Settings - test - Pagure</title>', output_text)
self.assertIn(
'<span class="badge badge-info" '
'style="background-color:#00ff00">green</span>\n'
' '
'<span class="text-muted">sample description</span>', output_text)
self.assertIn(
'<input type="hidden" value="green" name="tag" />',
output_text)
self.assertIn(
'<span class="badge badge-info" '
'style="background-color:#ff0000">red1</span>\n'
' '
'<span class="text-muted">lorem ipsum</span>', output_text)
self.assertIn(
'<input type="hidden" value="red" name="tag" />',
output_text)
# Valid query - Two tags of the same color
data = {
'tag': ['red2', 'red3'],
'tag_color': ['#ff0000', '#ff0000'],
'tag_description': ['', ''],
'csrf_token': csrf_token,
}
output = self.app.post(
'/test/update/tags', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Settings - test - Pagure</title>', output_text)
self.assertIn(
'<span class="badge badge-info" '
'style="background-color:#ff0000">red2</span>\n'
' '
'<span class="text-muted"></span>', output_text)
self.assertIn(
'<input type="hidden" value="green" name="tag" />',
output_text)
self.assertIn(
'<span class="badge badge-info" '
'style="background-color:#ff0000">red3</span>\n'
' '
'<span class="text-muted"></span>', output_text)
self.assertIn(
'<input type="hidden" value="red" name="tag" />',
output_text)
# Invalid query - Tag already known
data = {
'tag': ['red2'],
'tag_color': ['#000'],
'tag_description': [''],
'csrf_token': csrf_token,
}
output = self.app.post(
'/test/update/tags', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Settings - test - Pagure</title>', output_text)
self.assertIn(
'<span class="badge badge-info" '
'style="background-color:#ff0000">red2</span>\n'
' '
'<span class="text-muted"></span>', output_text)
self.assertIn(
'<input type="hidden" value="green" name="tag" />',
output_text)
self.assertIn(
'<span class="badge badge-info" '
'style="background-color:#ff0000">red3</span>\n'
' '
'<span class="text-muted"></span>', output_text)
self.assertIn(
'<input type="hidden" value="red" name="tag" />',
output_text)
self.assertIn(
'Duplicated tag: red2',
output_text)
# After update, list tags
tags = pagure.lib.get_tags_of_project(self.session, repo)
self.assertEqual(
sorted([tag.tag for tag in tags]),
['blue', 'green', 'red', 'red1', 'red2', 'red3'])
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_view_issue_namespace_comment(self, p_send_email, p_ugt):
""" Test comment on the view_issue endpoint on namespaced project.
"""
# Create the project ns/test
item = pagure.lib.model.Project(
user_id=1, # pingou
name='test3',
namespace='ns',
description='test project #3',
hook_token='aaabbbcccdd',
)
self.session.add(item)
self.session.commit()
self.assertEqual(item.fullname, 'ns/test3')
pygit2.init_repository(
os.path.join(self.path, 'repos', 'ns', 'test3.git'),
bare=True)
# Create 2 issues
iss = pagure.lib.new_issue(
issue_id=1,
session=self.session,
repo=item,
title='test issue',
content='content test issue',
user='pingou',
ticketfolder=None,
)
self.session.commit()
self.assertEqual(iss.id, 1)
self.assertEqual(iss.title, 'test issue')
self.assertEqual(iss.project.fullname, 'ns/test3')
iss = pagure.lib.new_issue(
issue_id=2,
session=self.session,
repo=item,
title='test issue2',
content='content test issue2',
user='pingou',
ticketfolder=None,
)
self.session.commit()
self.assertEqual(iss.id, 2)
self.assertEqual(iss.title, 'test issue2')
self.assertEqual(iss.project.fullname, 'ns/test3')
# Add a comment on the second issue pointing to the first one
issue_comment = pagure.lib.model.IssueComment(
issue_uid=iss.uid,
comment='foo bar #1 see?',
user_id=1, # pingou
notification=False,
)
self.session.add(issue_comment)
self.session.commit()
output = self.app.get('/ns/test3/issue/2')
self.assertEqual(output.status_code, 200)
self.assertIn(
'<span class="comment_text comment_body">'
'<p>foo bar <a href="/ns/test3/issue/1" '
'title="[Open] test issue">#1</a> see?</p></span>', output.get_data(as_text=True))
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_view_issue_forked_namespace_comment(self, p_send_email, p_ugt):
""" Test comment on the view_issue endpoint on namespaced project.
"""
# Create the project ns/test
item = pagure.lib.model.Project(
user_id=1, # pingou
name='test3',
namespace='ns',
description='test project #3',
hook_token='aaabbbcccdd',
)
self.session.add(item)
self.session.commit()
self.assertEqual(item.fullname, 'ns/test3')
# Fork the project ns/test
item = pagure.lib.model.Project(
user_id=1, # pingou
parent_id=1, # ns/test
is_fork=True,
name='test3',
namespace='ns',
description='test project #3',
hook_token='aaabbbcccddff',
)
self.session.add(item)
self.session.commit()
self.assertEqual(item.fullname, 'forks/pingou/ns/test3')
pygit2.init_repository(
os.path.join(
self.path, 'repos', 'forks', 'pingou', 'ns', 'test3.git'),
bare=True)
# Create 2 issues
iss = pagure.lib.new_issue(
issue_id=1,
session=self.session,
repo=item,
title='test issue',
content='content test issue',
user='pingou',
ticketfolder=None,
)
self.session.commit()
self.assertEqual(iss.id, 1)
self.assertEqual(iss.title, 'test issue')
self.assertEqual(iss.project.fullname, 'forks/pingou/ns/test3')
iss = pagure.lib.new_issue(
issue_id=2,
session=self.session,
repo=item,
title='test issue2',
content='content test issue2',
user='pingou',
ticketfolder=None,
)
self.session.commit()
self.assertEqual(iss.id, 2)
self.assertEqual(iss.title, 'test issue2')
self.assertEqual(iss.project.fullname, 'forks/pingou/ns/test3')
# Add a comment on the second issue pointing to the first one
issue_comment = pagure.lib.model.IssueComment(
issue_uid=iss.uid,
comment='foo bar #1 see?',
user_id=1, # pingou
notification=False,
)
self.session.add(issue_comment)
self.session.commit()
output = self.app.get('/fork/pingou/ns/test3/issue/2')
self.assertEqual(output.status_code, 200)
self.assertIn(
'<span class="comment_text comment_body">'
'<p>foo bar <a href="/fork/pingou/ns/test3/issue/1" '
'title="[Open] test issue">#1</a> see?</p></span>', output.get_data(as_text=True))
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_view_issue_closed(self, p_send_email, p_ugt):
""" Test viewing a closed issue. """
p_send_email.return_value = True
p_ugt.return_value = True
tests.create_projects(self.session)
tests.create_projects_git(
os.path.join(self.path, 'repos'), bare=True)
# Create issues to play with
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='We should work on this',
user='pingou',
ticketfolder=None
)
self.session.commit()
self.assertEqual(msg.title, 'Test issue')
user = tests.FakeUser()
user.username = 'pingou'
with tests.user_set(self.app.application, user):
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
csrf_token = self.get_csrf(output=output)
# Add new comment
data = {
'csrf_token': csrf_token,
'status': 'Closed',
'close_status': 'Fixed',
'comment': 'Woohoo a second comment!',
}
output = self.app.post(
'/test/issue/1/update', data=data, follow_redirects=True)
self.assertEqual(output.status_code, 200)
output_text = output.get_data(as_text=True)
self.assertIn(
'<title>Issue #1: Test issue - test - Pagure</title>',
output_text)
self.assertIn(
'<a class="btn btn-outline-secondary btn-sm border-0"'
' href="/test/issue/1/edit" title="Edit this issue">',
output_text)
self.assertIn(
'Comment added',
output_text)
self.assertTrue(
'<p>Woohoo a second comment!</p>' in output_text)
self.assertEqual(output_text.count('comment_body">'), 2)
self.assertTrue(
'<option selected value="Fixed">Fixed</option>'
in output_text)
def _set_up_for_reaction_test(self, private=False):
tests.create_projects(self.session)
tests.create_projects_git(os.path.join(self.path, 'repos'), bare=True)
self.session.add(pagure.lib.model.User(
user='naysayer',
fullname='John Doe',
password=b'password',
default_email='jdoe@example.com',
))
self.session.commit()
repo = pagure.lib.get_authorized_project(self.session, 'test')
msg = pagure.lib.new_issue(
session=self.session,
repo=repo,
title='Test issue',
content='Fix me',
user='pingou',
ticketfolder=None,
private=private,
)
pagure.lib.add_issue_comment(
session=self.session,
issue=msg,
comment='How about no',
user='naysayer',
ticketfolder=None,
)
self.session.commit()
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_add_reaction(self, p_send_email, p_ugt):
''' Test adding a reaction to an issue comment.'''
p_send_email.return_value = True
p_ugt.return_value = True
self._set_up_for_reaction_test()
user = tests.FakeUser()
user.username = 'pingou'
with tests.user_set(self.app.application, user):
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 200)
data = {
'csrf_token': self.get_csrf(output=output),
'reaction': 'Thumbs down',
}
output = self.app.post(
'/test/issue/1/comment/1/react',
data=data,
follow_redirects=True,
)
self.assertEqual(output.status_code, 200)
# Load the page and check reaction is added.
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 200)
self.assertIn(
'Thumbs down sent by pingou',
output.get_data(as_text=True)
)
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_add_reaction_unauthenticated(self, p_send_email, p_ugt):
'''
Test adding a reaction to an issue comment without authentication.
'''
p_send_email.return_value = True
p_ugt.return_value = True
self._set_up_for_reaction_test()
output = self.app.get('/test/issue/1')
self.assertEqual(output.status_code, 200)
data = {
'csrf_token': self.get_csrf(output=output),
'reaction': 'Thumbs down',
}
output = self.app.post(
'/test/issue/1/comment/1/react',
data=data,
follow_redirects=False,
)
# Redirect to login page
self.assertEqual(output.status_code, 302)
self.assertIn('/login/', output.headers['Location'])
@patch('pagure.lib.git.update_git')
@patch('pagure.lib.notify.send_email')
def test_add_reaction_private_issue(self, p_send_email, p_ugt):
'''Test adding a reaction to a private issue comment.'''
p_send_email.return_value = True
p_ugt.return_value = True
self._set_up_for_reaction_test(private=True)
user = tests.FakeUser()
user.username = 'naysayer'
with tests.user_set(self.app.application, user):
# Steal CSRF token from new issue page
output = self.app.get('/test/new_issue')
data = {
'csrf_token': self.get_csrf(output=output),
'reaction': 'Thumbs down',
}
output = self.app.post(
'/test/issue/1/comment/1/react',
data=data,
follow_redirects=True,
)
self.assertEqual(output.status_code, 404)
if __name__ == '__main__':
unittest.main(verbosity=2)