Blame tests/test_pagure_flask_ui_remote_pr.py

Pierre-Yves Chibon 9f16a2
# -*- coding: utf-8 -*-
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
"""
Pierre-Yves Chibon 9f16a2
 (c) 2018 - Copyright Red Hat Inc
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
 Authors:
Pierre-Yves Chibon 9f16a2
   Pierre-Yves Chibon <pingou@pingoured.fr></pingou@pingoured.fr>
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
"""
Pierre-Yves Chibon 9f16a2
Aurélien Bompard 626417
from __future__ import unicode_literals
Aurélien Bompard 626417
Pierre-Yves Chibon 9f16a2
__requires__ = ['SQLAlchemy >= 0.8']
Pierre-Yves Chibon 9f16a2
import pkg_resources
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
import json
Pierre-Yves Chibon 9f16a2
import unittest
Pierre-Yves Chibon 9f16a2
import shutil
Pierre-Yves Chibon 9f16a2
import sys
Pierre-Yves Chibon 9f16a2
import tempfile
Pierre-Yves Chibon 9f16a2
import time
Pierre-Yves Chibon 9f16a2
import os
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
import pygit2
Pierre-Yves Chibon 9f16a2
from mock import patch, MagicMock
Pierre-Yves Chibon 9f16a2
from bs4 import BeautifulSoup
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
sys.path.insert(0, os.path.join(os.path.dirname(
Pierre-Yves Chibon 9f16a2
    os.path.abspath(__file__)), '..'))
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
import pagure
Pierre-Yves Chibon 9f16a2
import pagure.lib
Pierre-Yves Chibon 9f16a2
import tests
Pierre-Yves Chibon 9f16a2
from pagure.lib.repo import PagureRepo
Aurélien Bompard 626417
from pagure.lib.git import _make_signature
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
class PagureRemotePRtests(tests.Modeltests):
Pierre-Yves Chibon 9f16a2
    """ Tests for remote PRs in pagure """
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
    def setUp(self):
Pierre-Yves Chibon 9f16a2
        """ Set up the environment. """
Pierre-Yves Chibon 9f16a2
        super(PagureRemotePRtests, self).setUp()
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
        self.newpath = tempfile.mkdtemp(prefix='pagure-fork-test')
Pierre-Yves Chibon 9f16a2
        self.old_value = pagure.config.config['REMOTE_GIT_FOLDER']
Pierre-Yves Chibon 9f16a2
        pagure.config.config['REMOTE_GIT_FOLDER'] = os.path.join(
Pierre-Yves Chibon 9f16a2
            self.path, 'remotes')
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
    def tearDown(self):
Pierre-Yves Chibon 9f16a2
        """ Clear things up. """
Pierre-Yves Chibon 9f16a2
        super(PagureRemotePRtests, self).tearDown()
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
        pagure.config.config['REMOTE_GIT_FOLDER'] = self.old_value
Pierre-Yves Chibon 9f16a2
        shutil.rmtree(self.newpath)
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
    def set_up_git_repo(self, new_project=None, branch_from='feature'):
Pierre-Yves Chibon 9f16a2
        """ Set up the git repo and create the corresponding PullRequest
Pierre-Yves Chibon 9f16a2
        object.
Pierre-Yves Chibon 9f16a2
        """
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
        # Create a git repo to play with
Pierre-Yves Chibon 9f16a2
        gitrepo = os.path.join(self.path, 'repos', 'test.git')
Pierre-Yves Chibon 9f16a2
        repo = pygit2.init_repository(gitrepo, bare=True)
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
        repopath = os.path.join(self.newpath, 'test')
Pierre-Yves Chibon 9f16a2
        clone_repo = pygit2.clone_repository(gitrepo, repopath)
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
        # Create a file in that git repo
Pierre-Yves Chibon 9f16a2
        with open(os.path.join(repopath, 'sources'), 'w') as stream:
Pierre-Yves Chibon 9f16a2
            stream.write('foo\n bar')
Pierre-Yves Chibon 9f16a2
        clone_repo.index.add('sources')
Pierre-Yves Chibon 9f16a2
        clone_repo.index.write()
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
        try:
Pierre-Yves Chibon 9f16a2
            com = repo.revparse_single('HEAD')
Pierre-Yves Chibon 9f16a2
            prev_commit = [com.oid.hex]
Pierre-Yves Chibon 9f16a2
        except:
Pierre-Yves Chibon 9f16a2
            prev_commit = []
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
        # Commits the files added
Pierre-Yves Chibon 9f16a2
        tree = clone_repo.index.write_tree()
Aurélien Bompard 626417
        author = _make_signature(
Pierre-Yves Chibon 9f16a2
            'Alice Author', 'alice@authors.tld')
Aurélien Bompard 626417
        committer = _make_signature(
Pierre-Yves Chibon 9f16a2
            'Cecil Committer', 'cecil@committers.tld')
Pierre-Yves Chibon 9f16a2
        clone_repo.create_commit(
Pierre-Yves Chibon 9f16a2
            'refs/heads/master',  # the name of the reference to update
Pierre-Yves Chibon 9f16a2
            author,
Pierre-Yves Chibon 9f16a2
            committer,
Pierre-Yves Chibon 9f16a2
            'Add sources file for testing',
Pierre-Yves Chibon 9f16a2
            # binary string representing the tree object ID
Pierre-Yves Chibon 9f16a2
            tree,
Pierre-Yves Chibon 9f16a2
            # list of binary strings representing parents of the new commit
Pierre-Yves Chibon 9f16a2
            prev_commit
Pierre-Yves Chibon 9f16a2
        )
Pierre-Yves Chibon 9f16a2
        time.sleep(1)
Pierre-Yves Chibon 9f16a2
        refname = 'refs/heads/master:refs/heads/master'
Pierre-Yves Chibon 9f16a2
        ori_remote = clone_repo.remotes[0]
Pierre-Yves Chibon 9f16a2
        PagureRepo.push(ori_remote, refname)
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
        first_commit = repo.revparse_single('HEAD')
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
        with open(os.path.join(repopath, '.gitignore'), 'w') as stream:
Pierre-Yves Chibon 9f16a2
            stream.write('*~')
Pierre-Yves Chibon 9f16a2
        clone_repo.index.add('.gitignore')
Pierre-Yves Chibon 9f16a2
        clone_repo.index.write()
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
        # Commits the files added
Pierre-Yves Chibon 9f16a2
        tree = clone_repo.index.write_tree()
Aurélien Bompard 626417
        author = _make_signature(
Pierre-Yves Chibon 9f16a2
            'Alice Äuthòr', 'alice@äuthòrs.tld')
Aurélien Bompard 626417
        committer = _make_signature(
Pierre-Yves Chibon 9f16a2
            'Cecil Cõmmîttër', 'cecil@cõmmîttërs.tld')
Pierre-Yves Chibon 9f16a2
        clone_repo.create_commit(
Pierre-Yves Chibon 9f16a2
            'refs/heads/master',
Pierre-Yves Chibon 9f16a2
            author,
Pierre-Yves Chibon 9f16a2
            committer,
Pierre-Yves Chibon 9f16a2
            'Add .gitignore file for testing',
Pierre-Yves Chibon 9f16a2
            # binary string representing the tree object ID
Pierre-Yves Chibon 9f16a2
            tree,
Pierre-Yves Chibon 9f16a2
            # list of binary strings representing parents of the new commit
Pierre-Yves Chibon 9f16a2
            [first_commit.oid.hex]
Pierre-Yves Chibon 9f16a2
        )
Pierre-Yves Chibon 9f16a2
        refname = 'refs/heads/master:refs/heads/master'
Pierre-Yves Chibon 9f16a2
        ori_remote = clone_repo.remotes[0]
Pierre-Yves Chibon 9f16a2
        PagureRepo.push(ori_remote, refname)
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
        # Set the second repo
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
        new_gitrepo = repopath
Pierre-Yves Chibon 9f16a2
        if new_project:
Pierre-Yves Chibon 9f16a2
            # Create a new git repo to play with
Pierre-Yves Chibon 9f16a2
            new_gitrepo = os.path.join(self.newpath, new_project.fullname)
Pierre-Yves Chibon 9f16a2
            if not os.path.exists(new_gitrepo):
Pierre-Yves Chibon 9f16a2
                os.makedirs(new_gitrepo)
Pierre-Yves Chibon 9f16a2
                new_repo = pygit2.clone_repository(gitrepo, new_gitrepo)
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
        repo = pygit2.Repository(new_gitrepo)
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
        # Edit the sources file again
Pierre-Yves Chibon 9f16a2
        with open(os.path.join(new_gitrepo, 'sources'), 'w') as stream:
Pierre-Yves Chibon 9f16a2
            stream.write('foo\n bar\nbaz\n boose')
Pierre-Yves Chibon 9f16a2
        repo.index.add('sources')
Pierre-Yves Chibon 9f16a2
        repo.index.write()
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
        # Commits the files added
Pierre-Yves Chibon 9f16a2
        tree = repo.index.write_tree()
Aurélien Bompard 626417
        author = _make_signature(
Pierre-Yves Chibon 9f16a2
            'Alice Author', 'alice@authors.tld')
Aurélien Bompard 626417
        committer = _make_signature(
Pierre-Yves Chibon 9f16a2
            'Cecil Committer', 'cecil@committers.tld')
Pierre-Yves Chibon 9f16a2
        repo.create_commit(
Pierre-Yves Chibon 9f16a2
            'refs/heads/%s' % branch_from,
Pierre-Yves Chibon 9f16a2
            author,
Pierre-Yves Chibon 9f16a2
            committer,
Pierre-Yves Chibon 9f16a2
            'A commit on branch %s' % branch_from,
Pierre-Yves Chibon 9f16a2
            tree,
Pierre-Yves Chibon 9f16a2
            [first_commit.oid.hex]
Pierre-Yves Chibon 9f16a2
        )
Pierre-Yves Chibon 9f16a2
        refname = 'refs/heads/%s' % (branch_from)
Pierre-Yves Chibon 9f16a2
        ori_remote = repo.remotes[0]
Pierre-Yves Chibon 9f16a2
        PagureRepo.push(ori_remote, refname)
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
    @patch('pagure.lib.notify.send_email',  MagicMock(return_value=True))
Pierre-Yves Chibon 9f16a2
    def test_new_remote_pr_unauth(self):
Pierre-Yves Chibon 9f16a2
        """ Test creating a new remote PR un-authenticated. """
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
        tests.create_projects(self.session)
Pierre-Yves Chibon 9f16a2
        tests.create_projects_git(
Pierre-Yves Chibon 9f16a2
            os.path.join(self.path, 'requests'), bare=True)
Pierre-Yves Chibon 9f16a2
        self.set_up_git_repo()
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
        # Before
Pierre-Yves Chibon 9f16a2
        project = pagure.lib.get_authorized_project(self.session, 'test')
Pierre-Yves Chibon 9f16a2
        self.assertEqual(len(project.requests), 0)
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
        # Try creating a remote PR
Pierre-Yves Chibon 9f16a2
        output = self.app.get('/test/diff/remote')
Pierre-Yves Chibon 9f16a2
        self.assertEqual(output.status_code, 302)
Pierre-Yves Chibon 9f16a2
        self.assertIn(
Aurélien Bompard 626417
            'You should be redirected automatically to target URL: '
Aurélien Bompard 626417
            '
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
    @patch('pagure.lib.notify.send_email',  MagicMock(return_value=True))
Pierre-Yves Chibon 9f16a2
    def test_new_remote_pr_auth(self):
Pierre-Yves Chibon 9f16a2
        """ Test creating a new remote PR un-authenticated. """
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
        tests.create_projects(self.session)
Pierre-Yves Chibon 9f16a2
        tests.create_projects_git(
Pierre-Yves Chibon 9f16a2
            os.path.join(self.path, 'requests'), bare=True)
Pierre-Yves Chibon 9f16a2
        self.set_up_git_repo()
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
        # Before
Pierre-Yves Chibon 9f16a2
        self.session = pagure.lib.create_session(self.dbpath)
Pierre-Yves Chibon 9f16a2
        project = pagure.lib.get_authorized_project(self.session, 'test')
Pierre-Yves Chibon 9f16a2
        self.assertEqual(len(project.requests), 0)
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
        # Try creating a remote PR
Pierre-Yves Chibon 9f16a2
        user = tests.FakeUser(username='foo')
Pierre-Yves Chibon 9f16a2
        with tests.user_set(self.app.application, user):
Pierre-Yves Chibon 9f16a2
            output = self.app.get('/test/diff/remote')
Pierre-Yves Chibon 9f16a2
            self.assertEqual(output.status_code, 200)
Aurélien Bompard 626417
            self.assertIn(
Aurélien Bompard 626417
                '

New remote pull-request

',
Aurélien Bompard 626417
                output.get_data(as_text=True))
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
            csrf_token = self.get_csrf(output=output)
Pierre-Yves Chibon 9f16a2
            data = {
Pierre-Yves Chibon 9f16a2
                'csrf_token': csrf_token,
Pierre-Yves Chibon 9f16a2
                'title': 'Remote PR title',
Pierre-Yves Chibon 9f16a2
                'branch_from': 'feature',
Pierre-Yves Chibon 9f16a2
                'branch_to': 'master',
Pierre-Yves Chibon 9f16a2
                'git_repo': os.path.join(self.newpath, 'test'),
Pierre-Yves Chibon 9f16a2
            }
Pierre-Yves Chibon 9f16a2
            output = self.app.post('/test/diff/remote', data=data)
Pierre-Yves Chibon 9f16a2
            self.assertEqual(output.status_code, 200)
Aurélien Bompard 626417
            output_text = output.get_data(as_text=True)
Aurélien Bompard 626417
            self.assertIn('

Create pull request

', output_text)
Pierre-Yves Chibon 9f16a2
            self.assertIn(
Aurélien Bompard 626417
                '
', output_text)
Pierre-Yves Chibon 9f16a2
            self.assertNotIn(
Aurélien Bompard 626417
                '
', output_text)
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
            # Not saved yet
Pierre-Yves Chibon 9f16a2
            self.session = pagure.lib.create_session(self.dbpath)
Pierre-Yves Chibon 9f16a2
            project = pagure.lib.get_authorized_project(self.session, 'test')
Pierre-Yves Chibon 9f16a2
            self.assertEqual(len(project.requests), 0)
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
            data = {
Pierre-Yves Chibon 9f16a2
                'csrf_token': csrf_token,
Pierre-Yves Chibon 9f16a2
                'title': 'Remote PR title',
Pierre-Yves Chibon 9f16a2
                'branch_from': 'feature',
Pierre-Yves Chibon 9f16a2
                'branch_to': 'master',
Pierre-Yves Chibon 9f16a2
                'git_repo': os.path.join(self.newpath, 'test'),
Pierre-Yves Chibon 9f16a2
                'confirm': 1,
Pierre-Yves Chibon 9f16a2
            }
Pierre-Yves Chibon 9f16a2
            output = self.app.post(
Pierre-Yves Chibon 9f16a2
                '/test/diff/remote', data=data, follow_redirects=True)
Pierre-Yves Chibon 9f16a2
            self.assertEqual(output.status_code, 200)
Aurélien Bompard 626417
            output_text = output.get_data(as_text=True)
Pierre-Yves Chibon 9f16a2
            self.assertIn(
Aurélien Bompard 626417
                '

PR#1',

Aurélien Bompard 626417
                output_text)
Pierre-Yves Chibon 9f16a2
            self.assertIn(
Aurélien Bompard 626417
                '
', output_text)
Pierre-Yves Chibon 9f16a2
            self.assertNotIn(
Aurélien Bompard 626417
                '
', output_text)
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
            # Show the filename in the diff view
Pierre-Yves Chibon 9f16a2
            self.assertIn(
Aurélien Bompard 626417
                '''
Pierre-Yves Chibon 9f16a2
                                        .gitignore
Pierre-Yves Chibon 9f16a2
                  
<small></small>
Pierre-Yves Chibon 9f16a2
                  this is a remote pull-request, so we cannot provide you''',
Aurélien Bompard 626417
                output_text)
Pierre-Yves Chibon 9f16a2
            self.assertIn(
Aurélien Bompard 626417
                '''
Pierre-Yves Chibon 9f16a2
                                        sources
Pierre-Yves Chibon 9f16a2
                  
<small></small>
Pierre-Yves Chibon 9f16a2
                  this is a remote pull-request, so we cannot provide you''',
Aurélien Bompard 626417
                output_text)
Pierre-Yves Chibon 9f16a2
            # Show the filename in the Changes summary
Pierre-Yves Chibon 9f16a2
            self.assertIn(
Aurélien Bompard 626417
                '.gitignore', output_text)
Pierre-Yves Chibon 9f16a2
            self.assertIn(
Aurélien Bompard 626417
                'sources', output_text)
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
        # Remote PR Created
Pierre-Yves Chibon 9f16a2
        self.session = pagure.lib.create_session(self.dbpath)
Pierre-Yves Chibon 9f16a2
        project = pagure.lib.get_authorized_project(self.session, 'test')
Pierre-Yves Chibon 9f16a2
        self.assertEqual(len(project.requests), 1)
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
Pierre-Yves Chibon 9f16a2
if __name__ == '__main__':
Pierre-Yves Chibon 9f16a2
    unittest.main(verbosity=2)