# -*- coding: utf-8 -*-
"""
(c) 2015-2018 - Copyright Red Hat Inc
Authors:
Pierre-Yves Chibon <pingou@pingoured.fr>
"""
from __future__ import unicode_literals, absolute_import
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.lib.query
import tests
class PagureFlaskIssuestests(tests.Modeltests):
""" Tests for flask issues controller of pagure """
@patch.dict(
"pagure.config.config", {"ENABLE_TICKETS_NAMESPACE": ["foobar"]}
)
@patch("pagure.lib.git.update_git", MagicMock(return_value=True))
@patch("pagure.lib.notify.send_email", MagicMock(return_value=True))
def test_new_issue_wrong_namespace(self):
""" Test the new_issue endpoint. """
tests.create_projects(self.session)
tests.create_projects_git(os.path.join(self.path, "repos"), bare=True)
user = tests.FakeUser(username="pingou")
with tests.user_set(self.app.application, user):
csrf_token = self.get_csrf()
data = {
"title": "Test issue",
"issue_content": "We really should improve on this issue",
"status": "Open",
"csrf_token": csrf_token,
}
# Things work fine when the project has no namespace
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,
)
# Things do not work when the project has a namespace not allowed
output = self.app.post(
"/somenamespace/test3/new_issue",
data=data,
follow_redirects=True,
)
self.assertEqual(output.status_code, 404)
output_text = output.get_data(as_text=True)
self.assertIn(
"<title>Page not found :'( - Pagure</title>", output_text
)
self.assertIn(
" <p>No issue tracker found for this project</p>", output_text
)
@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.query.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/files/'
"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.query.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/files/'
"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/files/'
"8a06845923010b27bfd8e7e75acff7badc40d1021b4"
"994e01f5e11ca40bc3abe",
output_text,
)
self.assertIn(
'href="/test/issue/raw/files/'
"6498a2de405546200b6144da56fc25d0a3976ae688d"
"bfccaca609c8b4480523e",
output_text,
)
# Check that the files are accessible
_, full_name = output_text.split("/test/issue/raw/files/", 1)
full_name1, full_name2 = full_name.split(
"/test/issue/raw/files/", 1
)
full_name1 = full_name1.split(".png", 1)[0]
full_name2 = full_name2.split(".png", 1)[0]
for full_name in [full_name1, full_name2]:
req = self.app.get("/test/issue/raw/files/%s.png" % full_name)
self.assertEqual(req.status_code, 200)
@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/files/'
"8a06845923010b27bfd8e7e75acff7badc40d1021b4"
"994e01f5e11ca40bc3abe",
output_text,
)
self.assertIn(
'href="/somenamespace/test3/issue/raw/files/'
"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.query.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.query.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.dict(
"pagure.config.config", {"ENABLE_TICKETS_NAMESPACE": ["foobar"]}
)
@patch("pagure.lib.git.update_git", MagicMock(return_value=True))
@patch("pagure.lib.notify.send_email", MagicMock(return_value=True))
def test_view_issues_wrong_namespace(self):
""" Test the view_issues endpoint. """
tests.create_projects(self.session)
tests.create_projects_git(os.path.join(self.path, "repos"), bare=True)
user = tests.FakeUser(username="pingou")
with tests.user_set(self.app.application, user):
# Things work fine when the project has no namespace
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)
# Things do not work when the project has a namespace not allowed
output = self.app.get("/somenamespace/test3/issues")
self.assertEqual(output.status_code, 404)
output_text = output.get_data(as_text=True)
self.assertIn(
"<title>Page not found :'( - Pagure</title>", output_text
)
self.assertIn(
" <p>No issue tracker found for this project</p>", output_text
)
@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.query.get_authorized_project(self.session, "test")
# Create some custom fields to play with
msg = pagure.lib.query.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.query.get_custom_key(
session=self.session, project=repo, keyname="test1"
)
# Create issues to play with
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="tést íssüé",
content="We should work on this",
user="pingou",
)
self.session.commit()
self.assertEqual(msg.title, "tést íssüé")
msg = pagure.lib.query.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.query.new_issue(
session=self.session,
repo=repo,
title="Tést íssüé with milestone",
content="Testing search",
user="pingou",
milestone="1.1",
)
self.session.commit()
self.assertEqual(msg.title, "Tést íssüé with milestone")
# Add a comment to that ticket
pagure.lib.query.add_issue_comment(
session=self.session,
issue=msg,
comment="How about nóã!",
user="foo",
)
self.session.commit()
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test invalid issue",
content="This really is not related",
user="pingou",
status="Closed",
close_status="Invalid",
)
self.session.commit()
self.assertEqual(msg.title, "Test invalid issue")
msg = pagure.lib.query.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'
' data-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'
' data-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)
# Content search - description
output = self.app.get(
"/test/issues?status=all&search_pattern=content:work"
)
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)
# Content search - comment
output = self.app.get(
"/test/issues?status=all&search_pattern=content:nóã"
)
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
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.query.new_issue(
session=self.session,
repo=repo,
title="Big problÈm!",
content="I need help ASAP",
user="foo",
)
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.query.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.query.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.query.get_authorized_project(self.session, "test")
# Create 2 issues to play with
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue ☃",
content="We should work on this ❤",
user="pingou",
)
self.session.commit()
self.assertEqual(msg.title, "Test issue ☃")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue with milestone",
content="Testing search",
user="pingou",
milestone="1.1",
)
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.query.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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
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 pointer" 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 class="pointer" 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 pointer" id="closeticket"\n'
' title="Delete this ticket">\n',
output_text,
)
csrf_token = self.get_csrf(output=output)
# Create private issue
repo = pagure.lib.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
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.query.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", MagicMock(return_value=True))
@patch("pagure.lib.notify.send_email", MagicMock(return_value=True))
def test_view_issue_author(self):
""" Test the view_issue endpoint when you're the author. """
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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="foo",
)
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 & no Close
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.assertNotIn(
'<form action="/test/issue/1/update" method="post" class="hidden"',
output_text,
)
self.assertNotIn(
'<input type="hidden" id="statusform_status" name="status" '
'value=""/>\n',
output_text,
)
self.assertNotIn(
'<input type="hidden" id="statusform_close_status" '
'name="close_status" value=""/>',
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(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)
# Author = Ability to close ticket
self.assertIn(
'<input type="hidden" id="statusform_status" name="status" '
'value=""/>',
output_text,
)
self.assertIn(
'<input type="hidden" id="statusform_close_status" '
'name="close_status" value=""/>',
output_text,
)
# Author = edit
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(
'<a href="/login/">Login</a> to comment on this ticket.'
in output_text
)
# author admin = take
self.assertIn("function take_issue(){", output_text)
self.assertIn("function drop_issue(){", output_text)
self.assertIn(
'<a class="pointer" id="take-btn"\n', output_text
)
@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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
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.query.get_authorized_project(self.session, "test")
# Add user 'foo' with ticket access on repo
msg = pagure.lib.query.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 pointer" 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 class="pointer" 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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
self.session.commit()
self.assertEqual(msg.title, "Test issue")
# Add user 'foo' with ticket access on repo
repo = pagure.lib.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.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 pointer" 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 class="pointer" 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 pointer" 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 class="pointer" 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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
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.query.search_issues(self.session, repo, issueid=1)
message = pagure.lib.query.edit_issue(
self.session,
issue=issue,
milestone=stone,
private=False,
user="pingou",
)
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.query.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.query.get_authorized_project(self.session, "test")
# Add custom fields to the project
msg = pagure.lib.query.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.query.new_issue(
session=self.session,
repo=repo,
title="Big problÈm!",
content="We should work on this",
user="pingou",
)
self.session.commit()
self.assertEqual(msg.title, "Big problÈm!")
# Assign a value to the custom key on that ticket
cfield = pagure.lib.query.get_custom_key(
session=self.session, project=repo, keyname="test1"
)
msg = pagure.lib.query.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.dict(
"pagure.config.config", {"ENABLE_TICKETS_NAMESPACE": ["foobar"]}
)
@patch("pagure.lib.git.update_git", MagicMock(return_value=True))
@patch("pagure.lib.notify.send_email", MagicMock(return_value=True))
def test_update_issue_wrong_namespace(self):
""" Test the update_issue endpoint. """
tests.create_projects(self.session)
tests.create_projects_git(os.path.join(self.path, "repos"), bare=True)
# Create normal issue on test
repo = pagure.lib.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue #1",
content="We should work on this",
user="pingou",
private=False,
)
self.session.commit()
self.assertEqual(msg.title, "Test issue #1")
# Create normal issue on test3
repo = pagure.lib.query.get_authorized_project(
self.session, "test3", namespace="somenamespace"
)
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue #1",
content="We should work on this",
user="pingou",
private=False,
)
self.session.commit()
self.assertEqual(msg.title, "Test issue #1")
user = tests.FakeUser(username="pingou")
with tests.user_set(self.app.application, user):
# Add new comment
data = {
"csrf_token": self.get_csrf(),
"status": "Closed",
"close_status": "Fixed",
"comment": "Woohoo a second comment!",
}
# Things work fine when the project has no namespace
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 #1 - test - Pagure</title>",
output_text,
)
# Things do not work when the project has a namespace not allowed
output = self.app.post(
"/somenamespace/test3/issue/1/update",
data=data,
follow_redirects=True,
)
self.assertEqual(output.status_code, 404)
output_text = output.get_data(as_text=True)
self.assertIn(
"<title>Page not found :'( - Pagure</title>", output_text
)
self.assertIn(
" <p>No issue tracker found for this project</p>", output_text
)
@patch("pagure.lib.git.update_git")
@patch("pagure.lib.notify.send_email")
def test_update_issue_add_tags(self, p_send_email, p_ugt):
""" Test 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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
self.session.commit()
self.assertEqual(msg.title, "Test issue")
# Before update, list tags
tags = pagure.lib.query.get_tags_of_project(self.session, repo)
self.assertEqual([tag.tag for tag in tags], [])
self.assertEqual(repo.issues[0].tags_text, [])
user = tests.FakeUser()
user.username = "pingou"
with tests.user_set(self.app.application, user):
csrf_token = self.get_csrf()
# Add two tags to the project
data = {
"tag": "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)
data = {
"tag": "red",
"tag_description": "lorem ipsum",
"tag_color": "#rrr",
"csrf_token": csrf_token,
}
output = self.app.post(
"/test/update/tags", data=data, follow_redirects=True
)
self.assertEqual(output.status_code, 200)
# Add two tags to the issue
data = {"csrf_token": csrf_token, "tag": "red,green"}
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(
"</i> Issue tagged with: green, red</div>", output_text
)
# After update, list tags
tags = pagure.lib.query.get_tags_of_project(self.session, repo)
self.assertEqual([tag.tag for tag in tags], ["green", "red"])
self.assertEqual(repo.issues[0].tags_text, [])
@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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
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="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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
self.session.commit()
self.assertEqual(msg.title, "Test issue")
# Reset the status of the first issue
parent_issue = pagure.lib.query.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.query.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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
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.query.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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
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.query.get_authorized_project(self.session, "test")
issue = pagure.lib.query.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.query.get_authorized_project(self.session, "test")
issue = pagure.lib.query.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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
self.session.commit()
self.assertEqual(msg.title, "Test issue")
repo = pagure.lib.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue #2",
content="We should work on this again",
user="foo",
)
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.query.get_authorized_project(self.session, "test")
issue = pagure.lib.query.search_issues(self.session, repo, issueid=1)
self.assertEqual(issue.depending_text, [2])
self.assertEqual(issue.blocking_text, [])
@patch("pagure.lib.git.update_git", MagicMock(return_value=True))
@patch("pagure.lib.notify.send_email", MagicMock(return_value=True))
def test_update_issue_block_closed(self):
""" Test how blocked issue shows in the UI when the blocking ticket
is open and closed. """
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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
self.session.commit()
self.assertEqual(msg.title, "Test issue")
repo = pagure.lib.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue #2",
content="We should work on this again",
user="foo",
)
self.session.commit()
self.assertEqual(msg.title, "Test issue #2")
user = tests.FakeUser(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 - Open
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,
)
self.assertIn(
'<span class="fa fa-fw text-success fa-exclamation-circle pt-1"></span>',
output_text,
)
self.assertIn(
'<span class="text-success font-weight-bold">#2</span>',
output_text,
)
# Close ticket #1
data = {
"csrf_token": csrf_token,
"status": "Closed",
"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,
)
# Now looking at how the dependent ticket looks like:
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 #2 - 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(
'<span class="fa fa-fw text-danger fa-exclamation-circle pt-1"></span>',
output_text,
)
self.assertIn(
'<span class="text-danger font-weight-bold">#1</span>',
output_text,
)
self.session.commit()
repo = pagure.lib.query.get_authorized_project(self.session, "test")
issue = pagure.lib.query.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_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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
self.session.commit()
self.assertEqual(msg.title, "Test issue")
repo = pagure.lib.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue #2",
content="We should work on this again",
user="foo",
)
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.query.get_authorized_project(
self.session, "test"
)
issue = pagure.lib.query.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.query.get_authorized_project(self.session, "test")
issue = pagure.lib.query.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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
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.query.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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
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.query.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.query.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.query.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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
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.query.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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
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", MagicMock(return_value=True))
@patch("pagure.lib.notify.send_email", MagicMock(return_value=True))
def test_edit_tag_issue_disabled(self):
""" Test the edit_tag endpoint when issues are disabled. """
tests.create_projects(self.session)
tests.create_projects_git(os.path.join(self.path, "repos"))
# disable issue tracker
repo = pagure.lib.query.get_authorized_project(self.session, "test")
repo.settings = {"issue_tracker": False}
self.session.add(repo)
self.session.commit()
# Create issues to play with
repo = pagure.lib.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
self.session.commit()
self.assertEqual(msg.title, "Test issue")
# Add a tag to the issue
issue = pagure.lib.query.search_issues(self.session, repo, issueid=1)
msg = pagure.lib.query.add_tag_obj(
session=self.session, obj=issue, tags="tag1", user="pingou"
)
self.session.commit()
self.assertEqual(msg, "Issue tagged with: tag1")
# Before edit, list tags
tags = pagure.lib.query.get_tags_of_project(self.session, repo)
self.assertEqual([tag.tag for tag in tags], ["tag1"])
# Edit tag
user = tests.FakeUser(username="pingou")
with tests.user_set(self.app.application, user):
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)
)
# After edit, list tags
self.session.commit()
tags = pagure.lib.query.get_tags_of_project(self.session, repo)
self.assertEqual([tag.tag for tag in tags], ["tag1"])
@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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
self.session.commit()
self.assertEqual(msg.title, "Test issue")
# Add a tag to the issue
issue = pagure.lib.query.search_issues(self.session, repo, issueid=1)
msg = pagure.lib.query.add_tag_obj(
session=self.session, obj=issue, tags="tag1", user="pingou"
)
self.session.commit()
self.assertEqual(msg, "Issue tagged with: tag1")
# Before edit, list tags
tags = pagure.lib.query.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.query.get_tags_of_project(self.session, repo)
self.assertEqual([tag.tag for tag in tags], ["tag2"])
@patch("pagure.lib.git.update_git", MagicMock(return_value=True))
@patch("pagure.lib.notify.send_email", MagicMock(return_value=True))
def test_remove_tag_issue_disabled(self):
""" Test the remove_tag endpoint. """
tests.create_projects(self.session)
tests.create_projects_git(os.path.join(self.path, "repos"))
# disable issue tracker
repo = pagure.lib.query.get_authorized_project(self.session, "test")
repo.settings = {"issue_tracker": False}
self.session.add(repo)
self.session.commit()
# Create issues to play with
repo = pagure.lib.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
self.session.commit()
self.assertEqual(msg.title, "Test issue")
# Add a tag to the issue
issue = pagure.lib.query.search_issues(self.session, repo, issueid=1)
msg = pagure.lib.query.add_tag_obj(
session=self.session, obj=issue, tags="tag1", user="pingou"
)
self.session.commit()
self.assertEqual(msg, "Issue tagged with: tag1")
# Before edit, list tags
tags = pagure.lib.query.get_tags_of_project(self.session, repo)
self.assertEqual([tag.tag for tag in tags], ["tag1"])
# Edit tag
user = tests.FakeUser(username="pingou")
with tests.user_set(self.app.application, user):
data = {"tag": "tag1", "csrf_token": self.get_csrf()}
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_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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
self.session.commit()
self.assertEqual(msg.title, "Test issue")
# Add a tag to the issue
issue = pagure.lib.query.search_issues(self.session, repo, issueid=1)
msg = pagure.lib.query.add_tag_obj(
session=self.session, obj=issue, tags="tag1", user="pingou"
)
self.session.commit()
self.assertEqual(msg, "Issue tagged with: tag1")
# Before edit, list tags
tags = pagure.lib.query.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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
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.query.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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
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.query.get_authorized_project(self.session, "test")
issue = pagure.lib.query.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.query.get_authorized_project(self.session, "test")
issue = pagure.lib.query.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.assertIn(
'<textarea class="form-control width-100per" id="update_comment"',
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.query.get_authorized_project(self.session, "test")
issue = pagure.lib.query.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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="foo",
)
self.session.commit()
self.assertEqual(msg.title, "Test issue")
issue = pagure.lib.query.search_issues(self.session, repo, issueid=1)
self.assertEqual(len(issue.comments), 1)
self.assertEqual(issue.status, "Open")
issue = pagure.lib.query.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 pointer 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 pointer 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.query.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.query.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.query._get_project(self.session, "test")
pagure.lib.query.update_read_only_mode(
self.session, repo, read_only=False
)
pingou = pagure.lib.query.get_user(self.session, "pingou")
pagure.lib.query.add_sshkey_to_project_or_user(
session=self.session,
user=pingou,
ssh_key="ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAAAgQDAzBMSIlvPRaEiLOTVInErkRIw9CzQQcnslDekAn1jFnGf+SNa1acvbTiATbCX71AA03giKrPxPH79dxcC7aDXerc6zRcKjJs6MAL9PrCjnbyxCKXRNNZU5U9X/DLaaL1b3caB+WD6OoorhS3LTEtKPX8xyjOzhf3OQSzNjhJp5Q==",
pushaccess=True,
creator=pingou,
)
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.query.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", MagicMock(return_value=True))
@patch("pagure.lib.notify.send_email", MagicMock(return_value=True))
def test_update_tags_issue_disabled(self):
""" Test the update_tags endpoint. """
tests.create_projects(self.session)
tests.create_projects_git(os.path.join(self.path, "repos"))
# disable issue tracker
repo = pagure.lib.query.get_authorized_project(self.session, "test")
repo.settings = {"issue_tracker": False}
self.session.add(repo)
self.session.commit()
# Create issues to play with
repo = pagure.lib.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
self.session.commit()
self.assertEqual(msg.title, "Test issue")
# Before update, list tags
tags = pagure.lib.query.get_tags_of_project(self.session, repo)
self.assertEqual([tag.tag for tag in tags], [])
user = tests.FakeUser(username="pingou")
with tests.user_set(self.app.application, user):
csrf_token = self.get_csrf()
# 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" '
'data-bg-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" '
'data-bg-color="#ff0000">red1</span>\n'
" "
'<span class="text-muted">lorem ipsum</span>',
output_text,
)
# After update, list tags
tags = pagure.lib.query.get_tags_of_project(self.session, repo)
self.assertEqual(sorted([tag.tag for tag in tags]), ["green", "red1"])
@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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
self.session.commit()
self.assertEqual(msg.title, "Test issue")
# Before update, list tags
tags = pagure.lib.query.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 one or more invalid characters",
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" '
'data-bg-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" '
'data-bg-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" '
'data-bg-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" '
'data-bg-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" '
'data-bg-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" '
'data-bg-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" '
'data-bg-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" '
'data-bg-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.query.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_update_tags_with_colon(self, p_send_email, p_ugt):
""" Test the update_tags endpoint with a tag containing a colon. """
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"))
# Create issues to play with
repo = pagure.lib.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
self.session.commit()
self.assertEqual(msg.title, "Test issue")
# Before update, list tags
tags = pagure.lib.query.get_tags_of_project(self.session, repo)
self.assertEqual([tag.tag for tag in tags], [])
user = tests.FakeUser()
user.username = "pingou"
with tests.user_set(self.app.application, user):
csrf_token = self.get_csrf()
# Tag with a colon ':'
data = {
"tag": ["is: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(
'<h5 class="pl-2 font-weight-bold text-muted">Project '
"Settings</h5>",
output_text,
)
self.assertIn(
'<span class="badge badge-info" '
'data-bg-color="#000">is:red2</span>\n'
" "
'<span class="text-muted"></span>',
output_text,
)
self.assertIn(
'<input type="hidden" value="is:red2" name="tag" />',
output_text,
)
# After update, list tags
tags = pagure.lib.query.get_tags_of_project(self.session, repo)
self.assertEqual(sorted([tag.tag for tag in tags]), ["is:red2"])
@patch("pagure.lib.git.update_git")
@patch("pagure.lib.notify.send_email")
def test_update_tags_with_coma(self, p_send_email, p_ugt):
""" Test the update_tags endpoint with a tag containing a coma. """
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"))
# Create issues to play with
repo = pagure.lib.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
self.session.commit()
self.assertEqual(msg.title, "Test issue")
# Before update, list tags
tags = pagure.lib.query.get_tags_of_project(self.session, repo)
self.assertEqual([tag.tag for tag in tags], [])
user = tests.FakeUser()
user.username = "pingou"
with tests.user_set(self.app.application, user):
csrf_token = self.get_csrf()
# Tag with a colon ':'
data = {
"tag": ["green,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(
'<h5 class="pl-2 font-weight-bold text-muted">Project '
"Settings</h5>",
output_text,
)
self.assertIn(
"</i> Tag: green,red2 contains one or more invalid "
"characters</div>\n",
output_text,
)
# After update, list tags
tags = pagure.lib.query.get_tags_of_project(self.session, repo)
self.assertEqual(sorted([tag.tag for tag in tags]), [])
@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.query.new_issue(
issue_id=1,
session=self.session,
repo=item,
title="test issue",
content="content test issue",
user="pingou",
)
self.session.commit()
self.assertEqual(iss.id, 1)
self.assertEqual(iss.title, "test issue")
self.assertEqual(iss.project.fullname, "ns/test3")
iss = pagure.lib.query.new_issue(
issue_id=2,
session=self.session,
repo=item,
title="test issue2",
content="content test issue2",
user="pingou",
)
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_comment_and_close(self, p_send_email, p_ugt):
""" Test if the comment and close button shows up on a ticket opened
by the user
"""
# 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 1 issue
iss = pagure.lib.query.new_issue(
issue_id=1,
session=self.session,
repo=item,
title="test issue2",
content="content test issue2",
user="foo",
)
self.session.commit()
self.assertEqual(iss.id, 1)
self.assertEqual(iss.title, "test issue2")
self.assertEqual(iss.project.fullname, "ns/test3")
user = tests.FakeUser(username="foo")
with tests.user_set(self.app.application, user):
output = self.app.get("/ns/test3/issue/1")
self.assertEqual(output.status_code, 200)
self.assertIn(
'<a class="btn btn-outline-primary comment_and_close_action pointer" '
'data-value="">\n'
" Comment & Close\n",
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.query.new_issue(
issue_id=1,
session=self.session,
repo=item,
title="test issue",
content="content test issue",
user="pingou",
)
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.query.new_issue(
issue_id=2,
session=self.session,
repo=item,
title="test issue2",
content="content test issue2",
user="pingou",
)
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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="We should work on this",
user="pingou",
)
self.session.commit()
self.assertEqual(msg.title, "Test issue")
user = tests.FakeUser()
user.username = "foo"
msg = pagure.lib.query.add_user_to_project(
self.session, repo, "foo", "pingou"
)
self.session.commit()
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
)
self.assertIn(
" Closed: Fixed\n"
" </span> just now\n"
" </span>\n"
" by\n"
' <span title="foo bar (foo)">foo.</span>\n',
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.query.get_authorized_project(self.session, "test")
msg = pagure.lib.query.new_issue(
session=self.session,
repo=repo,
title="Test issue",
content="Fix me",
user="pingou",
private=private,
)
pagure.lib.query.add_issue_comment(
session=self.session,
issue=msg,
comment="How about no",
user="naysayer",
)
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)