|
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(
|
|
Ryan Lerch |
2aa95c |
'#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(
|
|
Pierre-Yves Chibon |
277586 |
'
|
|
Pierre-Yves Chibon |
9f16a2 |
self.assertIn(
|
|
Pierre-Yves Chibon |
277586 |
''
|
|
Pierre-Yves Chibon |
277586 |
'\n <small>.gitignore</small>', output_text)
|
|
Pierre-Yves Chibon |
277586 |
self.assertIn(
|
|
Pierre-Yves Chibon |
277586 |
'
|
|
Pierre-Yves Chibon |
277586 |
self.assertIn(
|
|
Pierre-Yves Chibon |
277586 |
''
|
|
Pierre-Yves Chibon |
277586 |
'\n <small>sources</small>', 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 |
59f36c |
@patch('pagure.lib.notify.send_email', MagicMock(return_value=True))
|
|
Pierre-Yves Chibon |
59f36c |
def test_new_remote_pr_empty_target(self):
|
|
Pierre-Yves Chibon |
59f36c |
""" Test creating a new remote PR authenticated against an empty
|
|
Pierre-Yves Chibon |
59f36c |
git repo. """
|
|
Pierre-Yves Chibon |
59f36c |
|
|
Pierre-Yves Chibon |
59f36c |
tests.create_projects(self.session)
|
|
Pierre-Yves Chibon |
59f36c |
tests.create_projects_git(
|
|
Pierre-Yves Chibon |
59f36c |
os.path.join(self.path, 'requests'), bare=True)
|
|
Pierre-Yves Chibon |
59f36c |
|
|
Pierre-Yves Chibon |
59f36c |
# Create empty target git repo
|
|
Pierre-Yves Chibon |
59f36c |
gitrepo = os.path.join(self.path, 'repos', 'test.git')
|
|
Pierre-Yves Chibon |
59f36c |
pygit2.init_repository(gitrepo, bare=True)
|
|
Pierre-Yves Chibon |
59f36c |
|
|
Pierre-Yves Chibon |
59f36c |
# Create git repo we'll pull from
|
|
Pierre-Yves Chibon |
59f36c |
gitrepo = os.path.join(self.path, 'repos', 'test_origin.git')
|
|
Pierre-Yves Chibon |
59f36c |
repo = pygit2.init_repository(gitrepo)
|
|
Pierre-Yves Chibon |
59f36c |
|
|
Pierre-Yves Chibon |
59f36c |
# Create a file in that git repo
|
|
Pierre-Yves Chibon |
59f36c |
with open(os.path.join(gitrepo, 'sources'), 'w') as stream:
|
|
Pierre-Yves Chibon |
59f36c |
stream.write('foo\n bar')
|
|
Pierre-Yves Chibon |
59f36c |
repo.index.add('sources')
|
|
Pierre-Yves Chibon |
59f36c |
repo.index.write()
|
|
Pierre-Yves Chibon |
59f36c |
|
|
Pierre-Yves Chibon |
59f36c |
prev_commit = []
|
|
Pierre-Yves Chibon |
59f36c |
|
|
Pierre-Yves Chibon |
59f36c |
# Commits the files added
|
|
Pierre-Yves Chibon |
59f36c |
tree = repo.index.write_tree()
|
|
Pierre-Yves Chibon |
59f36c |
author = _make_signature(
|
|
Pierre-Yves Chibon |
59f36c |
'Alice Author', 'alice@authors.tld')
|
|
Pierre-Yves Chibon |
59f36c |
committer = _make_signature(
|
|
Pierre-Yves Chibon |
59f36c |
'Cecil Committer', 'cecil@committers.tld')
|
|
Pierre-Yves Chibon |
59f36c |
repo.create_commit(
|
|
Pierre-Yves Chibon |
59f36c |
'refs/heads/feature', # the name of the reference to update
|
|
Pierre-Yves Chibon |
59f36c |
author,
|
|
Pierre-Yves Chibon |
59f36c |
committer,
|
|
Pierre-Yves Chibon |
59f36c |
'Add sources file for testing',
|
|
Pierre-Yves Chibon |
59f36c |
# binary string representing the tree object ID
|
|
Pierre-Yves Chibon |
59f36c |
tree,
|
|
Pierre-Yves Chibon |
59f36c |
# list of binary strings representing parents of the new commit
|
|
Pierre-Yves Chibon |
59f36c |
prev_commit
|
|
Pierre-Yves Chibon |
59f36c |
)
|
|
Pierre-Yves Chibon |
59f36c |
|
|
Pierre-Yves Chibon |
59f36c |
# Before
|
|
Pierre-Yves Chibon |
59f36c |
self.session = pagure.lib.create_session(self.dbpath)
|
|
Pierre-Yves Chibon |
59f36c |
project = pagure.lib.get_authorized_project(self.session, 'test')
|
|
Pierre-Yves Chibon |
59f36c |
self.assertEqual(len(project.requests), 0)
|
|
Pierre-Yves Chibon |
59f36c |
|
|
Pierre-Yves Chibon |
59f36c |
# Try creating a remote PR
|
|
Pierre-Yves Chibon |
59f36c |
user = tests.FakeUser(username='foo')
|
|
Pierre-Yves Chibon |
59f36c |
with tests.user_set(self.app.application, user):
|
|
Pierre-Yves Chibon |
59f36c |
output = self.app.get('/test/diff/remote')
|
|
Pierre-Yves Chibon |
59f36c |
self.assertEqual(output.status_code, 200)
|
|
Pierre-Yves Chibon |
59f36c |
self.assertIn(
|
|
Pierre-Yves Chibon |
59f36c |
'New remote pull-request',
|
|
Pierre-Yves Chibon |
59f36c |
output.get_data(as_text=True))
|
|
Pierre-Yves Chibon |
59f36c |
|
|
Pierre-Yves Chibon |
59f36c |
csrf_token = self.get_csrf(output=output)
|
|
Pierre-Yves Chibon |
59f36c |
data = {
|
|
Pierre-Yves Chibon |
59f36c |
'csrf_token': csrf_token,
|
|
Pierre-Yves Chibon |
59f36c |
'title': 'Remote PR title',
|
|
Pierre-Yves Chibon |
59f36c |
'branch_from': 'feature',
|
|
Pierre-Yves Chibon |
59f36c |
'branch_to': 'master',
|
|
Pierre-Yves Chibon |
59f36c |
'git_repo': gitrepo,
|
|
Pierre-Yves Chibon |
59f36c |
}
|
|
Pierre-Yves Chibon |
59f36c |
output = self.app.post('/test/diff/remote', data=data)
|
|
Pierre-Yves Chibon |
59f36c |
self.assertEqual(output.status_code, 200)
|
|
Pierre-Yves Chibon |
59f36c |
output_text = output.get_data(as_text=True)
|
|
Pierre-Yves Chibon |
59f36c |
self.assertIn('Create pull request', output_text)
|
|
Pierre-Yves Chibon |
59f36c |
self.assertIn(
|
|
Pierre-Yves Chibon |
59f36c |
'', output_text)
|
|
Pierre-Yves Chibon |
59f36c |
self.assertNotIn(
|
|
Pierre-Yves Chibon |
59f36c |
'', output_text)
|
|
Pierre-Yves Chibon |
59f36c |
|
|
Pierre-Yves Chibon |
59f36c |
# Not saved yet
|
|
Pierre-Yves Chibon |
59f36c |
self.session = pagure.lib.create_session(self.dbpath)
|
|
Pierre-Yves Chibon |
59f36c |
project = pagure.lib.get_authorized_project(self.session, 'test')
|
|
Pierre-Yves Chibon |
59f36c |
self.assertEqual(len(project.requests), 0)
|
|
Pierre-Yves Chibon |
59f36c |
|
|
Pierre-Yves Chibon |
59f36c |
data = {
|
|
Pierre-Yves Chibon |
59f36c |
'csrf_token': csrf_token,
|
|
Pierre-Yves Chibon |
59f36c |
'title': 'Remote PR title',
|
|
Pierre-Yves Chibon |
59f36c |
'branch_from': 'feature',
|
|
Pierre-Yves Chibon |
59f36c |
'branch_to': 'master',
|
|
Pierre-Yves Chibon |
59f36c |
'git_repo': gitrepo,
|
|
Pierre-Yves Chibon |
59f36c |
'confirm': 1,
|
|
Pierre-Yves Chibon |
59f36c |
}
|
|
Pierre-Yves Chibon |
59f36c |
output = self.app.post(
|
|
Pierre-Yves Chibon |
59f36c |
'/test/diff/remote', data=data, follow_redirects=True)
|
|
Pierre-Yves Chibon |
59f36c |
self.assertEqual(output.status_code, 200)
|
|
Pierre-Yves Chibon |
59f36c |
output_text = output.get_data(as_text=True)
|
|
Pierre-Yves Chibon |
59f36c |
self.assertIn(
|
|
Pierre-Yves Chibon |
59f36c |
'<title>PR#1: Remote PR title - test\n - Pagure</title>',
|
|
Pierre-Yves Chibon |
59f36c |
output_text)
|
|
Pierre-Yves Chibon |
59f36c |
self.assertIn(
|
|
Pierre-Yves Chibon |
59f36c |
'', output_text)
|
|
Pierre-Yves Chibon |
59f36c |
self.assertNotIn(
|
|
Pierre-Yves Chibon |
59f36c |
'', output_text)
|
|
Pierre-Yves Chibon |
59f36c |
|
|
Pierre-Yves Chibon |
59f36c |
# Show the filename in the diff view
|
|
Pierre-Yves Chibon |
59f36c |
self.assertIn(
|
|
Pierre-Yves Chibon |
59f36c |
'''
|
|
Pierre-Yves Chibon |
59f36c |
sources
|
|
Pierre-Yves Chibon |
59f36c |
<small></small>
|
|
Pierre-Yves Chibon |
59f36c |
this is a remote pull-request, so we cannot provide you''',
|
|
Pierre-Yves Chibon |
59f36c |
output_text)
|
|
Pierre-Yves Chibon |
59f36c |
# Show the filename in the Changes summary
|
|
Pierre-Yves Chibon |
59f36c |
self.assertIn(
|
|
Pierre-Yves Chibon |
59f36c |
'
|
|
Pierre-Yves Chibon |
59f36c |
self.assertIn(
|
|
Pierre-Yves Chibon |
59f36c |
''
|
|
Pierre-Yves Chibon |
59f36c |
'\n <small>sources</small>', output_text)
|
|
Pierre-Yves Chibon |
59f36c |
|
|
Pierre-Yves Chibon |
59f36c |
# Remote PR Created
|
|
Pierre-Yves Chibon |
59f36c |
self.session = pagure.lib.create_session(self.dbpath)
|
|
Pierre-Yves Chibon |
59f36c |
project = pagure.lib.get_authorized_project(self.session, 'test')
|
|
Pierre-Yves Chibon |
59f36c |
self.assertEqual(len(project.requests), 1)
|
|
Pierre-Yves Chibon |
59f36c |
|
|
Pierre-Yves Chibon |
59f36c |
# Check the merge state of the PR
|
|
Pierre-Yves Chibon |
59f36c |
data = {
|
|
Pierre-Yves Chibon |
59f36c |
'csrf_token': csrf_token,
|
|
Pierre-Yves Chibon |
59f36c |
'requestid': project.requests[0].uid,
|
|
Pierre-Yves Chibon |
59f36c |
}
|
|
Pierre-Yves Chibon |
59f36c |
output = self.app.post('/pv/pull-request/merge', data=data)
|
|
Pierre-Yves Chibon |
59f36c |
self.assertEqual(output.status_code, 200)
|
|
Pierre-Yves Chibon |
59f36c |
output_text = output.get_data(as_text=True)
|
|
Pierre-Yves Chibon |
59f36c |
data = json.loads(output_text)
|
|
Pierre-Yves Chibon |
59f36c |
self.assertEqual(
|
|
Pierre-Yves Chibon |
59f36c |
data,
|
|
Pierre-Yves Chibon |
59f36c |
{
|
|
Pierre-Yves Chibon |
59f36c |
"code": "FFORWARD",
|
|
Pierre-Yves Chibon |
59f36c |
"message": "The pull-request can be merged and fast-forwarded",
|
|
Pierre-Yves Chibon |
59f36c |
"short_code": "Ok"
|
|
Pierre-Yves Chibon |
59f36c |
}
|
|
Pierre-Yves Chibon |
59f36c |
)
|
|
Pierre-Yves Chibon |
59f36c |
|
|
Pierre-Yves Chibon |
59f36c |
user = tests.FakeUser(username='pingou')
|
|
Pierre-Yves Chibon |
59f36c |
with tests.user_set(self.app.application, user):
|
|
Pierre-Yves Chibon |
59f36c |
# Merge the PR
|
|
Pierre-Yves Chibon |
59f36c |
data = {
|
|
Pierre-Yves Chibon |
59f36c |
'csrf_token': csrf_token,
|
|
Pierre-Yves Chibon |
59f36c |
}
|
|
Pierre-Yves Chibon |
59f36c |
output = self.app.post(
|
|
Pierre-Yves Chibon |
59f36c |
'/test/pull-request/1/merge', data=data, follow_redirects=True)
|
|
Pierre-Yves Chibon |
59f36c |
output_text = output.get_data(as_text=True)
|
|
Pierre-Yves Chibon |
59f36c |
self.assertEqual(output.status_code, 200)
|
|
Pierre-Yves Chibon |
59f36c |
self.assertIn(
|
|
Pierre-Yves Chibon |
59f36c |
'<title>Overview - test - Pagure</title>', output_text
|
|
Pierre-Yves Chibon |
59f36c |
)
|
|
Pierre-Yves Chibon |
59f36c |
|
|
Pierre-Yves Chibon |
59f36c |
|
|
Pierre-Yves Chibon |
9f16a2 |
if __name__ == '__main__':
|
|
Pierre-Yves Chibon |
9f16a2 |
unittest.main(verbosity=2)
|