pagure/tests/test_pagure_flask_ui_issues.py

# -*- 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 &amp; 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 &amp; 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 &amp; 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 &amp; 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 &amp; 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 &amp; 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 &amp; 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 &amp; 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 &amp; 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"><div class="markdown">'
                '<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></div></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 &#34;ralph&#34; 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'
                "                          &nbsp;"
                '<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'
                "                          &nbsp;"
                '<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'
                "                          &nbsp;"
                '<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'
                "                          &nbsp;"
                '<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'
                "                          &nbsp;"
                '<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'
                "                          &nbsp;"
                '<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'
                "                          &nbsp;"
                '<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'
                "                          &nbsp;"
                '<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'
                "                          &nbsp;"
                '<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'
                "                          &nbsp;"
                '<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'
                "                          &nbsp;"
                '<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">'
            '<div class="markdown"><p>foo bar <a href="/ns/test3/issue/1" '
            'title="[Open] test issue">#1</a> see?</p></div></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 &amp; 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">'
            '<div class="markdown"><p>foo bar <a href="/fork/pingou/ns/test3/issue/1" '
            'title="[Open] test issue">#1</a> see?</p></div></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)