|
Pierre-Yves Chibon |
7c5e53 |
# -*- coding: utf-8 -*-
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
"""
|
|
Pierre-Yves Chibon |
7c5e53 |
(c) 2018 - Copyright Red Hat Inc
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
Authors:
|
|
Pierre-Yves Chibon |
7c5e53 |
Pierre-Yves Chibon <pingou@pingoured.fr></pingou@pingoured.fr>
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
"""
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
67d1cc |
from __future__ import unicode_literals, absolute_import
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
import unittest
|
|
Pierre-Yves Chibon |
7c5e53 |
import sys
|
|
Pierre-Yves Chibon |
7c5e53 |
import os
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
import mock
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
sys.path.insert(0, os.path.join(os.path.dirname(
|
|
Pierre-Yves Chibon |
7c5e53 |
os.path.abspath(__file__)), '..'))
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
import pagure.hooks.pagure_hook
|
|
Pierre-Yves Chibon |
7c5e53 |
import tests
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
class PagureHooksPagureHooktests(tests.SimplePagureTest):
|
|
Pierre-Yves Chibon |
7c5e53 |
""" Tests for pagure.hooks.pagure_hook """
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
def setUp(self):
|
|
Pierre-Yves Chibon |
7c5e53 |
""" Set up the environnment, ran before every tests. """
|
|
Pierre-Yves Chibon |
7c5e53 |
super(PagureHooksPagureHooktests, self).setUp()
|
|
Pierre-Yves Chibon |
7c5e53 |
tests.create_projects(self.session)
|
|
Pierre-Yves Chibon |
7c5e53 |
tests.create_projects_git(os.path.join(self.path, 'repos'), bare=True)
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
# Add one issue to each projects otherwise we won't be able to find
|
|
Pierre-Yves Chibon |
7c5e53 |
project = pagure.lib.query.get_authorized_project(self.session, 'test')
|
|
Pierre-Yves Chibon |
7c5e53 |
msg = pagure.lib.query.new_issue(
|
|
Pierre-Yves Chibon |
7c5e53 |
session=self.session,
|
|
Pierre-Yves Chibon |
7c5e53 |
repo=project,
|
|
Pierre-Yves Chibon |
7c5e53 |
title='Test issue',
|
|
Pierre-Yves Chibon |
7c5e53 |
content='We should work on this',
|
|
Pierre-Yves Chibon |
7c5e53 |
user='pingou',
|
|
Pierre-Yves Chibon |
7c5e53 |
)
|
|
Pierre-Yves Chibon |
7c5e53 |
self.session.commit()
|
|
Pierre-Yves Chibon |
7c5e53 |
self.assertEqual(msg.title, 'Test issue')
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
project = pagure.lib.query.get_authorized_project(
|
|
Pierre-Yves Chibon |
7c5e53 |
self.session, 'test2')
|
|
Pierre-Yves Chibon |
7c5e53 |
msg = pagure.lib.query.new_issue(
|
|
Pierre-Yves Chibon |
7c5e53 |
session=self.session,
|
|
Pierre-Yves Chibon |
7c5e53 |
repo=project,
|
|
Pierre-Yves Chibon |
7c5e53 |
title='Test issue on test2',
|
|
Pierre-Yves Chibon |
7c5e53 |
content='We should work on this, really',
|
|
Pierre-Yves Chibon |
7c5e53 |
user='foo',
|
|
Pierre-Yves Chibon |
7c5e53 |
)
|
|
Pierre-Yves Chibon |
7c5e53 |
self.session.commit()
|
|
Pierre-Yves Chibon |
7c5e53 |
self.assertEqual(msg.title, 'Test issue on test2')
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
# Create a fork of test for foo with its own ticket
|
|
Pierre-Yves Chibon |
7c5e53 |
item = pagure.lib.model.Project(
|
|
Pierre-Yves Chibon |
7c5e53 |
user_id=2, # foo
|
|
Pierre-Yves Chibon |
7c5e53 |
name='test',
|
|
Pierre-Yves Chibon |
7c5e53 |
is_fork=True,
|
|
Pierre-Yves Chibon |
7c5e53 |
parent_id=1,
|
|
Pierre-Yves Chibon |
7c5e53 |
description='test project #1',
|
|
Pierre-Yves Chibon |
7c5e53 |
hook_token='aaabbbccc_foo',
|
|
Pierre-Yves Chibon |
7c5e53 |
)
|
|
Pierre-Yves Chibon |
7c5e53 |
item.close_status = [
|
|
Pierre-Yves Chibon |
7c5e53 |
'Invalid', 'Insufficient data', 'Fixed', 'Duplicate']
|
|
Pierre-Yves Chibon |
7c5e53 |
self.session.add(item)
|
|
Pierre-Yves Chibon |
7c5e53 |
self.session.commit()
|
|
Pierre-Yves Chibon |
7c5e53 |
project = pagure.lib.query.get_authorized_project(
|
|
Pierre-Yves Chibon |
7c5e53 |
self.session, 'test', user="foo")
|
|
Pierre-Yves Chibon |
7c5e53 |
msg = pagure.lib.query.new_issue(
|
|
Pierre-Yves Chibon |
7c5e53 |
session=self.session,
|
|
Pierre-Yves Chibon |
7c5e53 |
repo=project,
|
|
Pierre-Yves Chibon |
7c5e53 |
title='Test issue on fork/foo/test',
|
|
Pierre-Yves Chibon |
7c5e53 |
content='We should work on this, really',
|
|
Pierre-Yves Chibon |
7c5e53 |
user='foo',
|
|
Pierre-Yves Chibon |
7c5e53 |
)
|
|
Pierre-Yves Chibon |
7c5e53 |
self.session.commit()
|
|
Pierre-Yves Chibon |
7c5e53 |
self.assertEqual(msg.title, 'Test issue on fork/foo/test')
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
self.folder = os.path.join(self.path, 'repos', 'test.git')
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
# Add a README to the git repo - First commit
|
|
Pierre-Yves Chibon |
7c5e53 |
tests.add_readme_git_repo(self.folder)
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
@mock.patch("pagure.hooks.pagure_hook.fixes_relation")
|
|
Pierre-Yves Chibon |
7c5e53 |
def test_generate_revision_change_log_short_url(self, fixes_relation):
|
|
Pierre-Yves Chibon |
7c5e53 |
""" Test generate_revision_change_log when the comment contains
|
|
Pierre-Yves Chibon |
7c5e53 |
a short link to the same project.
|
|
Pierre-Yves Chibon |
7c5e53 |
"""
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
# Add a commit with an url in the commit message
|
|
Pierre-Yves Chibon |
7c5e53 |
tests.add_content_to_git(
|
|
Pierre-Yves Chibon |
7c5e53 |
self.folder, branch='master', filename='sources', content='foo',
|
|
Pierre-Yves Chibon |
7c5e53 |
message='Test commit message\n\nFixes #1'
|
|
Pierre-Yves Chibon |
7c5e53 |
)
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
project = pagure.lib.query.get_authorized_project(self.session, 'test')
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
pagure.hooks.pagure_hook.generate_revision_change_log(
|
|
Pierre-Yves Chibon |
7c5e53 |
session=self.session,
|
|
Pierre-Yves Chibon |
7c5e53 |
project=project,
|
|
Pierre-Yves Chibon |
7c5e53 |
username=None,
|
|
Pierre-Yves Chibon |
7c5e53 |
repodir=self.folder,
|
|
Pierre-Yves Chibon |
7c5e53 |
new_commits_list=['HEAD']
|
|
Pierre-Yves Chibon |
7c5e53 |
)
|
|
Pierre-Yves Chibon |
7c5e53 |
fixes_relation.assert_called_once_with(
|
|
Pierre-Yves Chibon |
7c5e53 |
mock.ANY, None, mock.ANY, project.issues[0],
|
|
Pierre-Yves Chibon |
7c5e53 |
'http://localhost.localdomain/')
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
@mock.patch("pagure.hooks.pagure_hook.fixes_relation")
|
|
Pierre-Yves Chibon |
7c5e53 |
def test_generate_revision_change_log_full_url(self, fixes_relation):
|
|
Pierre-Yves Chibon |
7c5e53 |
""" Test generate_revision_change_log when the comment contains
|
|
Pierre-Yves Chibon |
7c5e53 |
a full link to another project.
|
|
Pierre-Yves Chibon |
7c5e53 |
"""
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
# Add a commit with an url in the commit message
|
|
Pierre-Yves Chibon |
7c5e53 |
tests.add_content_to_git(
|
|
Pierre-Yves Chibon |
7c5e53 |
self.folder, branch='master', filename='sources', content='foo',
|
|
Pierre-Yves Chibon |
7c5e53 |
message='Test commit message\n\n'
|
|
Pierre-Yves Chibon |
7c5e53 |
'Fixes http://localhost.localdomain/test2/issue/1'
|
|
Pierre-Yves Chibon |
7c5e53 |
)
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
project = pagure.lib.query.get_authorized_project(
|
|
Pierre-Yves Chibon |
7c5e53 |
self.session, 'test')
|
|
Pierre-Yves Chibon |
7c5e53 |
project2 = pagure.lib.query.get_authorized_project(
|
|
Pierre-Yves Chibon |
7c5e53 |
self.session, 'test2')
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
pagure.hooks.pagure_hook.generate_revision_change_log(
|
|
Pierre-Yves Chibon |
7c5e53 |
session=self.session,
|
|
Pierre-Yves Chibon |
7c5e53 |
project=project,
|
|
Pierre-Yves Chibon |
7c5e53 |
username=None,
|
|
Pierre-Yves Chibon |
7c5e53 |
repodir=self.folder,
|
|
Pierre-Yves Chibon |
7c5e53 |
new_commits_list=['HEAD']
|
|
Pierre-Yves Chibon |
7c5e53 |
)
|
|
Pierre-Yves Chibon |
7c5e53 |
fixes_relation.assert_called_once_with(
|
|
Pierre-Yves Chibon |
7c5e53 |
mock.ANY, None, mock.ANY, project2.issues[0],
|
|
Pierre-Yves Chibon |
7c5e53 |
'http://localhost.localdomain/')
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
@mock.patch("pagure.hooks.pagure_hook.fixes_relation")
|
|
Pierre-Yves Chibon |
7c5e53 |
def test_generate_revision_change_log_full_url_fork(self, fixes_relation):
|
|
Pierre-Yves Chibon |
7c5e53 |
""" Test generate_revision_change_log when the comment contains
|
|
Pierre-Yves Chibon |
7c5e53 |
a full link to a fork.
|
|
Pierre-Yves Chibon |
7c5e53 |
"""
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
# Add a commit with an url in the commit message
|
|
Pierre-Yves Chibon |
7c5e53 |
tests.add_content_to_git(
|
|
Pierre-Yves Chibon |
7c5e53 |
self.folder, branch='master', filename='sources', content='foo',
|
|
Pierre-Yves Chibon |
7c5e53 |
message='Test commit message\n\n'
|
|
Pierre-Yves Chibon |
7c5e53 |
'Fixes http://localhost.localdomain/fork/foo/test/issue/1'
|
|
Pierre-Yves Chibon |
7c5e53 |
)
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
project = pagure.lib.query.get_authorized_project(
|
|
Pierre-Yves Chibon |
7c5e53 |
self.session, 'test')
|
|
Pierre-Yves Chibon |
7c5e53 |
project_fork = pagure.lib.query.get_authorized_project(
|
|
Pierre-Yves Chibon |
7c5e53 |
self.session, 'test', user="foo")
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
pagure.hooks.pagure_hook.generate_revision_change_log(
|
|
Pierre-Yves Chibon |
7c5e53 |
session=self.session,
|
|
Pierre-Yves Chibon |
7c5e53 |
project=project,
|
|
Pierre-Yves Chibon |
7c5e53 |
username=None,
|
|
Pierre-Yves Chibon |
7c5e53 |
repodir=self.folder,
|
|
Pierre-Yves Chibon |
7c5e53 |
new_commits_list=['HEAD']
|
|
Pierre-Yves Chibon |
7c5e53 |
)
|
|
Pierre-Yves Chibon |
7c5e53 |
fixes_relation.assert_called_once_with(
|
|
Pierre-Yves Chibon |
7c5e53 |
mock.ANY, None, mock.ANY, project_fork.issues[0],
|
|
Pierre-Yves Chibon |
7c5e53 |
'http://localhost.localdomain/')
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
|
|
Pierre-Yves Chibon |
7c5e53 |
if __name__ == '__main__':
|
|
Pierre-Yves Chibon |
7c5e53 |
unittest.main(verbosity=2)
|