# -*- coding: utf-8 -*-

"""

 (c) 2015-2016 - Copyright Red Hat Inc

 Authors:

   Pierre-Yves Chibon <pingou@pingoured.fr></pingou@pingoured.fr>

"""

__requires__ = ['SQLAlchemy >= 0.8']

import pkg_resources

from unittest.case import SkipTest

import json

import unittest

import shutil

import sys

import os

try:

    import pyclamd

except:

    pyclamd = None

import tempfile

import pygit2

from mock import patch

sys.path.insert(0, os.path.join(os.path.dirname(

    os.path.abspath(__file__)), '..'))

import pagure.lib

import tests

class PagureFlaskIssuestests(tests.Modeltests):

    """ Tests for flask issues controller of pagure """

    def setUp(self):

        """ Set up the environnment, ran before every tests. """

        super(PagureFlaskIssuestests, self).setUp()

        pagure.APP.config['TESTING'] = True

        # TODO: Figure a way to enable this test on jenkins

        if not os.environ.get('BUILD_ID'):

            pagure.APP.config['VIRUS_SCAN_ATTACHMENTS'] = True

        pagure.SESSION = self.session

        pagure.ui.SESSION = self.session

        pagure.ui.app.SESSION = self.session

        pagure.ui.issues.SESSION = self.session

        pagure.ui.repo.SESSION = self.session

        pagure.ui.filters.SESSION = self.session

        pagure.APP.config['GIT_FOLDER'] = tests.HERE

        pagure.APP.config['FORK_FOLDER'] = os.path.join(

            tests.HERE, 'forks')

        pagure.APP.config['TICKETS_FOLDER'] = os.path.join(

            tests.HERE, 'tickets')

        pagure.APP.config['DOCS_FOLDER'] = os.path.join(

            tests.HERE, 'docs')

        self.app = pagure.APP.test_client()

    @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(pagure.APP, 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(tests.HERE), bare=True)

            output = self.app.get('/test/new_issue')

            self.assertEqual(output.status_code, 200)

            self.assertTrue(

                '
\n        New issue'

                in output.data)

            csrf_token = output.data.split(

                'name="csrf_token" type="hidden" value="')[1].split('">')[0]

            data = {

            }

            # Insufficient input

            output = self.app.post('/test/new_issue', data=data)

            self.assertEqual(output.status_code, 200)

            self.assertTrue(

                '
\n        New issue'

                in output.data)

            self.assertEqual(output.data.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)

            self.assertTrue(

                '
\n        New issue'

                in output.data)

            self.assertEqual(output.data.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)

            self.assertTrue(

                '
\n        New issue'

                in output.data)

            self.assertEqual(output.data.count(

                '\n                      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(

                '
No such user found in the database: username
',

                output.data)

        # User not logged in

        output = self.app.get('/test/new_issue')

        self.assertEqual(output.status_code, 302)

        user.username = 'pingou'

        with tests.user_set(pagure.APP, user):

            output = self.app.post(

                '/test/new_issue', data=data, follow_redirects=True)

            self.assertEqual(output.status_code, 200)

            self.assertIn(

                '<title>Issue #1: Test issue - test - Pagure</title>',

                output.data)

            self.assertIn(

                '

                'href="/test/issue/1/edit" title="Edit this issue">',

                output.data)

        # Project w/o issue tracker

        repo = pagure.lib.get_project(self.session, 'test')

        repo.settings = {'issue_tracker': False}

        self.session.add(repo)

        self.session.commit()

        user.username = 'pingou'

        with tests.user_set(pagure.APP, 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(tests.HERE), bare=True)

        tests.create_projects_git(

            os.path.join(tests.HERE, 'tickets'), bare=True)

        user = tests.FakeUser()

        user.username = 'pingou'

        with tests.user_set(pagure.APP, user):

            output = self.app.get('/test/new_issue')

            self.assertEqual(output.status_code, 200)

            self.assertTrue(

                '
\n        New issue'

                in output.data)

            csrf_token = output.data.split(

                'name="csrf_token" type="hidden" value="')[1].split('">')[0]

            with open(os.path.join(tests.HERE, 'placebo.png'), 'r') as stream:

                data = {

                    'title': 'Test issue',

                    'issue_content': 'We really should improve on this issue\n'

                                     '',

                    '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)

            self.assertIn(

                '<title>Issue #1: Test issue - test - Pagure</title>',

                output.data)

            self.assertIn(

                '

                'href="/test/issue/1/edit" title="Edit this issue">',

                output.data)

        # Project w/o issue tracker

        repo = pagure.lib.get_project(self.session, 'test')

        repo.settings = {'issue_tracker': False}

        self.session.add(repo)

        self.session.commit()

        user.username = 'pingou'

        with tests.user_set(pagure.APP, user):

            with open(os.path.join(tests.HERE, 'placebo.png'), 'r') 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': csrf_token,

                }

                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_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(tests.HERE), bare=True)

        output = self.app.get('/test/issues')

        self.assertEqual(output.status_code, 200)

        self.assertIn(

            'div class="projectinfo m-t-1 m-b-1">\ntest project #1        '

            '', output.data)

        self.assertTrue(

            '
\n      0 Open Issues' in output.data)

        # Create issues to play with

        repo = pagure.lib.get_project(self.session, 'test')

        msg = pagure.lib.new_issue(

            session=self.session,

            repo=repo,

            title='Test issue',

            content='We should work on this',

            user='pingou',

            ticketfolder=None

        )

        self.session.commit()

        self.assertEqual(msg.title, 'Test issue')

        msg = pagure.lib.new_issue(

            session=self.session,

            repo=repo,

            title='Test invalid issue',

            content='This really is not related',

            user='pingou',

            status='Invalid',

            ticketfolder=None

        )

        self.session.commit()

        self.assertEqual(msg.title, 'Test invalid issue')

        # Whole list

        output = self.app.get('/test/issues')

        self.assertEqual(output.status_code, 200)

        self.assertIn('<title>Issues - test - Pagure</title>', output.data)

        self.assertTrue(

            '
\n      1 Open Issues' in output.data)

        # Status = closed (all but open)

        output = self.app.get('/test/issues?status=cloSED')

        self.assertEqual(output.status_code, 200)

        self.assertIn('<title>Issues - test - Pagure</title>', output.data)

        self.assertTrue(

            '
\n      1 Closed Issues' in output.data)

        # Status = fixed

        output = self.app.get('/test/issues?status=fixed')

        self.assertEqual(output.status_code, 200)

        self.assertIn('<title>Issues - test - Pagure</title>', output.data)

        self.assertTrue(

            '
\n      0 Closed Issues' in output.data)

        # Status = Invalid

        output = self.app.get('/test/issues?status=Invalid')

        self.assertEqual(output.status_code, 200)

        self.assertIn('<title>Issues - test - Pagure</title>', output.data)

        self.assertTrue(

            '
\n      1 Closed Issues' in output.data)

        # All tickets

        output = self.app.get('/test/issues?status=all')

        self.assertEqual(output.status_code, 200)

        self.assertIn('<title>Issues - test - Pagure</title>', output.data)

        self.assertTrue(

            '
\n      2 Issues' in output.data)

        # New issue button is shown

        user = tests.FakeUser()

        with tests.user_set(pagure.APP, user):

            output = self.app.get('/test')

            self.assertEqual(output.status_code, 200)

            self.assertIn(

                'class="btn btn-success btn-sm">New Issue',

                output.data)

        # Project w/o issue tracker

        repo = pagure.lib.get_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(pagure.APP, user):

            output = self.app.get('/test')

            self.assertEqual(output.status_code, 200)

            self.assertNotIn(

                'class="btn btn-success btn-sm">New Issue',

                output.data)

    @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(tests.HERE), bare=True)

        output = self.app.get('/test/issue/1')

        self.assertEqual(output.status_code, 404)

        # Create issues to play with

        repo = pagure.lib.get_project(self.session, 'test')

        msg = pagure.lib.new_issue(

            session=self.session,

            repo=repo,

            title='Test issue',

            content='We should work on this',

            user='pingou',

            ticketfolder=None

        )

        self.session.commit()

        self.assertEqual(msg.title, 'Test issue')

        output = self.app.get('/test/issue/1')

        self.assertEqual(output.status_code, 200)

        # Not authentified = No edit

        self.assertNotIn(

            '

            'title="Edit this issue">',

            output.data)

        self.assertTrue(

            ''

            'Login\n            to comment on this ticket.'

            in output.data)

        user = tests.FakeUser()

        with tests.user_set(pagure.APP, user):

            output = self.app.get('/test/issue/1')

            self.assertEqual(output.status_code, 200)

            # Not author nor admin = No edit

            self.assertNotIn(

                '

                'href="/test/issue/1/edit" title="Edit this issue">',

                output.data)

            self.assertNotIn(

                '

                output.data)

            self.assertNotIn('title="Delete this ticket">', output.data)

            self.assertFalse(

                'Login to comment on this ticket.'

                in output.data)

        user.username = 'pingou'

        with tests.user_set(pagure.APP, user):

            output = self.app.get('/test/issue/1')

            self.assertEqual(output.status_code, 200)

            self.assertIn(

                '

                'href="/test/issue/1/edit" title="Edit this issue">',

                output.data)

            self.assertIn(

                '

                output.data)

            self.assertIn('title="Delete this ticket">', output.data)

            csrf_token = output.data.split(

                'name="csrf_token" type="hidden" value="')[1].split('">')[0]

        # Create private issue

        repo = pagure.lib.get_project(self.session, 'test')

        msg = pagure.lib.new_issue(

            session=self.session,

            repo=repo,

            title='Test issue',

            content='We should work on this',

            user='pingou',

            ticketfolder=None,

            private=True,

        )

        self.session.commit()

        self.assertEqual(msg.title, 'Test issue')

        # Not logged in

        output = self.app.get('/test/issue/2')

        self.assertEqual(output.status_code, 403)

        # Wrong user

        user = tests.FakeUser()

        with tests.user_set(pagure.APP, user):

            output = self.app.get('/test/issue/2')

            self.assertEqual(output.status_code, 403)

        # reporter

        user.username = 'pingou'

        with tests.user_set(pagure.APP, user):

            output = self.app.get('/test/issue/2')

            self.assertEqual(output.status_code, 200)

            self.assertIn(

                '<title>Issue #2: Test issue - test - Pagure</title>',

                output.data)

            self.assertIn(

                '

                'title="Private issue">', output.data)

            self.assertIn(

                '

                'href="/test/issue/2/edit" title="Edit this issue">',

                output.data)

        # Project w/o issue tracker

        repo = pagure.lib.get_project(self.session, 'test')

        repo.settings = {'issue_tracker': False}

        self.session.add(repo)

        self.session.commit()

        output = self.app.get('/test/issue/1')

        self.assertEqual(output.status_code, 404)

    @patch('pagure.lib.git.update_git')

    @patch('pagure.lib.notify.send_email')

    def test_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(tests.HERE), bare=True)

        output = self.app.get('/test/issue/1/update')

        self.assertEqual(output.status_code, 302)

        # Create issues to play with

        repo = pagure.lib.get_project(self.session, 'test')

        msg = pagure.lib.new_issue(

            session=self.session,

            repo=repo,

            title='Test issue',

            content='We should work on this',

            user='pingou',

            ticketfolder=None

        )

        self.session.commit()

        self.assertEqual(msg.title, 'Test issue')

        user = tests.FakeUser()

        user.username = 'pingou'

        with tests.user_set(pagure.APP, 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.data)

            self.assertIn(

                '

                'href="/test/issue/1/edit" title="Edit this issue">',

                output.data)

            csrf_token = output.data.split(

                'name="csrf_token" type="hidden" value="')[1].split('">')[0]

            data = {

                '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)

            self.assertIn(

                '<title>Issue #1: Test issue - test - Pagure</title>',

                output.data)

            self.assertIn(

                '

                'href="/test/issue/1/edit" title="Edit this issue">',

                output.data)

            self.assertFalse(

                '<option selected="" value="Fixed">Fixed</option>'

                in output.data)

            # Right status, wrong csrf

            data['status'] = 'Fixed'

            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.data)

            self.assertIn(

                '

                'href="/test/issue/1/edit" title="Edit this issue">',

                output.data)

            self.assertFalse(

                '<option selected="" value="Fixed">Fixed</option>'

                in output.data)

            # 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)

            self.assertIn(

                '<title>Issue #1: Test issue - test - Pagure</title>',

                output.data)

            self.assertIn(

                '

                'href="/test/issue/1/edit" title="Edit this issue">',

                output.data)

            self.assertIn(

                '\n                      Successfully edited issue #1',

                output.data)

            self.assertTrue(

                '<option selected="" value="Fixed">Fixed</option>'

                in output.data)

            self.assertIn(

                '<small>
 '
</small>

                '@pingou changed the status to Fixed'

                '
'.format(

                    app_url=pagure.APP.config['APP_URL'].rstrip('/')),

                output.data)

            # Add new comment

            data = {

                'csrf_token': csrf_token,

                '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)

            self.assertIn(

                '<title>Issue #1: Test issue - test - Pagure</title>',

                output.data)

            self.assertIn(

                '

                'href="/test/issue/1/edit" title="Edit this issue">',

                output.data)

            self.assertIn(

                '\n                      Comment added',

                output.data)

            self.assertNotIn(

                '\n                      No changes to edit',

                output.data)

            self.assertTrue(

                '
Woohoo a second comment !
' in output.data)

            self.assertEqual(output.data.count('comment_body">'), 2)

            self.assertTrue(

                '<option selected="" value="Fixed">Fixed</option>'

                in output.data)

            # Add new tag

            data = {

                'csrf_token': csrf_token,

                'status': 'Fixed',

                'tag': 'tag2',

            }

            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.data)

            self.assertIn(

                '

                'href="/test/issue/1/edit" title="Edit this issue">',

                output.data)

            self.assertTrue(

                '
Woohoo a second comment !
' in output.data)

            self.assertEqual(output.data.count('comment_body">'), 2)

            self.assertTrue(

                '<option selected="" value="Fixed">Fixed</option>'

                in output.data)

            # Assign issue to an non-existent user

            data = {

                'csrf_token': csrf_token,

                'status': 'Fixed',

                'assignee': 'ralph',

            }

            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.data)

            self.assertIn(

                '

                'href="/test/issue/1/edit" title="Edit this issue">',

                output.data)

            self.assertIn(

                '\n                      No user "ralph" found',

                output.data)

            self.assertTrue(

                '
Woohoo a second comment !
' in output.data)

            self.assertEqual(output.data.count('comment_body">'), 2)

            self.assertTrue(

                '<option selected="" value="Fixed">Fixed</option>'

                in output.data)

            # Assign issue properly

            data = {

                'csrf_token': csrf_token,

                'status': 'Fixed',

                'assignee': 'pingou',

            }

            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.data)

            self.assertIn(

                '

                'href="/test/issue/1/edit" title="Edit this issue">',

                output.data)

            self.assertIn(

                '\n                      Issue assigned',

                output.data)

            self.assertTrue(

                '' in output.data)

            self.assertTrue(

                '
Woohoo a second comment !
' in output.data)

            self.assertEqual(output.data.count('comment_body">'), 2)

            self.assertTrue(

                '<option selected="" value="Fixed">Fixed</option>'

                in output.data)

        # Create another issue with a dependency

        repo = pagure.lib.get_project(self.session, 'test')

        msg = pagure.lib.new_issue(

            session=self.session,

            repo=repo,

            title='Test issue',

            content='We should work on this',

            user='pingou',

            ticketfolder=None

        )

        self.session.commit()

        self.assertEqual(msg.title, 'Test issue')

        # Reset the status of the first issue

        parent_issue = pagure.lib.search_issues(

            self.session, repo, issueid=2)

        parent_issue.status = 'Open'

        # Add the dependency relationship

        self.session.add(parent_issue)

        issue = pagure.lib.search_issues(self.session, repo, issueid=2)

        issue.parents.append(parent_issue)

        self.session.add(issue)

        self.session.commit()

        with tests.user_set(pagure.APP, 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)

            self.assertIn(

                '<title>Issue #2: Test issue - test - Pagure</title>',

                output.data)

            self.assertIn(

                '

                'href="/test/issue/2/edit" title="Edit this issue">',

                output.data)

            self.assertIn(

                '\n                      You cannot close a ticket '

                'that has ticket depending that are still open.',

                output.data)

            self.assertTrue(

                '<option selected="" value="Open">Open</option>'

                in output.data)

        # Create private issue

        repo = pagure.lib.get_project(self.session, 'test')

        msg = pagure.lib.new_issue(

            session=self.session,

            repo=repo,

            title='Test issue',

            content='We should work on this',

            user='pingou',

            ticketfolder=None,

            private=True,

        )

        self.session.commit()

        self.assertEqual(msg.title, 'Test issue')

        # Wrong user

        user = tests.FakeUser()

        with tests.user_set(pagure.APP, user):

            output = self.app.post(

                '/test/issue/3/update', data=data, follow_redirects=True)

            self.assertEqual(output.status_code, 403)

        # Project w/o issue tracker

        repo = pagure.lib.get_project(self.session, 'test')

        repo.settings = {'issue_tracker': False}

        self.session.add(repo)

        self.session.commit()

        with tests.user_set(pagure.APP, user):

            output = self.app.get('/test/issue/1/update')

            self.assertEqual(output.status_code, 302)

            # 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(tests.HERE), bare=True)

        # Create issues to play with

        repo = pagure.lib.get_project(self.session, 'test')

        msg = pagure.lib.new_issue(

            session=self.session,

            repo=repo,

            title='Test issue',

            content='We should work on this',

            user='pingou',

            ticketfolder=None

        )

        self.session.commit()

        self.assertEqual(msg.title, 'Test issue')

        user = tests.FakeUser()

        user.username = 'pingou'

        with tests.user_set(pagure.APP, 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.data)

            self.assertIn(

                '

                'href="/test/issue/1/edit" title="Edit this issue">',

                output.data)

            csrf_token = output.data.split(

                'name="csrf_token" type="hidden" value="')[1].split('">')[0]

            # 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)

            self.assertIn(

                '<title>Issue #1: Test issue - test - Pagure</title>',

                output.data)

            self.assertIn(

                '

                'href="/test/issue/1/edit" title="Edit this issue">',

                output.data)

            self.assertIn(

                '\n                      Comment added',

                output.data)

            self.assertTrue(

                '
Woohoo a second comment !
' in output.data)

            self.assertEqual(output.data.count('comment_body">'), 2)

        repo = pagure.lib.get_project(self.session, 'test')

        issue = pagure.lib.search_issues(self.session, repo, issueid=1)

        self.assertEqual(len(issue.comments), 1)

        data = {

            'csrf_token': csrf_token,

            'drop_comment': 1,

        }

        user = tests.FakeUser()

        with tests.user_set(pagure.APP, 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(pagure.APP, user):

            # Drop the new comment

            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.data)

            self.assertIn(

                '

                'href="/test/issue/1/edit" title="Edit this issue">',

                output.data)

            self.assertIn(

                '\n                      Comment removed',

                output.data)

            # Drop non-existant comment

            output = self.app.post(

                '/test/issue/1/update', data=data, follow_redirects=True)

            self.assertEqual(output.status_code, 404)

        repo = pagure.lib.get_project(self.session, 'test')

        issue = pagure.lib.search_issues(self.session, repo, issueid=1)

        self.assertEqual(len(issue.comments), 0)

    @patch('pagure.lib.git.update_git')

    @patch('pagure.lib.notify.send_email')

    def test_update_issue_depend(self, p_send_email, p_ugt):

        """ Test adding dependency via the update_issue endpoint. """