##// END OF EJS Templates
webhook: quote URL variables to prevent url errors with special chars like # in pr title.
webhook: quote URL variables to prevent url errors with special chars like # in pr title.

File last commit:

r3477:976a0af2 default
r3477:976a0af2 default
Show More
test_webhook.py
135 lines | 5.1 KiB | text/x-python | PythonLexer
webhooks: added variables into the call URL. Fixes #4211
r938 # -*- coding: utf-8 -*-
docs: updated copyrights to 2019
r3363 # Copyright (C) 2010-2019 RhodeCode GmbH
webhooks: added variables into the call URL. Fixes #4211
r938 #
# This program is free software: you can redistribute it and/or modify
# it under the terms of the GNU Affero General Public License, version 3
# (only), as published by the Free Software Foundation.
#
# This program is distributed in the hope that it will be useful,
# but WITHOUT ANY WARRANTY; without even the implied warranty of
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
# GNU General Public License for more details.
#
# You should have received a copy of the GNU Affero General Public License
# along with this program. If not, see <http://www.gnu.org/licenses/>.
#
# This program is dual-licensed. If you wish to learn more about the
# RhodeCode Enterprise Edition, including its added features, Support services,
# and proprietary license terms, please see https://rhodecode.com/licenses/
import pytest
from rhodecode import events
from rhodecode.lib.utils2 import AttributeDict
webhook: abstract webhook handler to base for easier re-usage.
r2585 from rhodecode.integrations.types.webhook import WebhookDataHandler
webhooks: added variables into the call URL. Fixes #4211
r938
@pytest.fixture
def base_data():
return {
webhook: abstract webhook handler to base for easier re-usage.
r2585 'name': 'event',
webhooks: added variables into the call URL. Fixes #4211
r938 'repo': {
'repo_name': 'foo',
'repo_type': 'hg',
'repo_id': '12',
integrations: expose actor user_id, and username in webhook integration templates args....
r1709 'url': 'http://repo.url/foo',
webhook: enable support of extra repo variables as replacement in template url.
r1761 'extra_fields': {},
integrations: expose actor user_id, and username in webhook integration templates args....
r1709 },
'actor': {
'username': 'actor_name',
'user_id': 1
webhooks: added variables into the call URL. Fixes #4211
r938 }
}
def test_webhook_parse_url_invalid_event():
template_url = 'http://server.com/${repo_name}/build'
webhook: abstract webhook handler to base for easier re-usage.
r2585 handler = WebhookDataHandler(
template_url, {'exmaple-header': 'header-values'})
event = events.RepoDeleteEvent('')
webhooks: added variables into the call URL. Fixes #4211
r938 with pytest.raises(ValueError) as err:
webhook: abstract webhook handler to base for easier re-usage.
r2585 handler(event, {})
err = str(err.value)
assert err.startswith(
'event type `%s` not in supported list' % event.__class__)
webhooks: added variables into the call URL. Fixes #4211
r938
@pytest.mark.parametrize('template,expected_urls', [
webhook: abstract webhook handler to base for easier re-usage.
r2585 ('http://server.com/${repo_name}/build',
['http://server.com/foo/build']),
('http://server.com/${repo_name}/${repo_type}',
['http://server.com/foo/hg']),
('http://${server}.com/${repo_name}/${repo_id}',
['http://${server}.com/foo/12']),
('http://server.com/${branch}/build',
['http://server.com/${branch}/build']),
webhooks: added variables into the call URL. Fixes #4211
r938 ])
def test_webook_parse_url_for_create_event(base_data, template, expected_urls):
integrations: webhook, allow to set a custom header. Fixes #5384
r2086 headers = {'exmaple-header': 'header-values'}
webhook: abstract webhook handler to base for easier re-usage.
r2585 handler = WebhookDataHandler(template, headers)
webhooks: added variables into the call URL. Fixes #4211
r938 urls = handler(events.RepoCreateEvent(''), base_data)
integrations: webhook, allow to set a custom header. Fixes #5384
r2086 assert urls == [
webhook: abstract webhook handler to base for easier re-usage.
r2585 (url, headers, base_data) for url in expected_urls]
webhooks: added variables into the call URL. Fixes #4211
r938
@pytest.mark.parametrize('template,expected_urls', [
webhook: abstract webhook handler to base for easier re-usage.
r2585 ('http://server.com/${repo_name}/${pull_request_id}',
['http://server.com/foo/999']),
('http://server.com/${repo_name}/${pull_request_url}',
webhook: quote URL variables to prevent url errors with special chars like # in pr title.
r3477 ['http://server.com/foo/http%3A//pr-url.com']),
('http://server.com/${repo_name}/${pull_request_url}/?TITLE=${pull_request_title}',
['http://server.com/foo/http%3A//pr-url.com/?TITLE=example-pr-title%20Ticket%20%23123']),
('http://server.com/${repo_name}/?SHADOW_URL=${pull_request_shadow_url}',
['http://server.com/foo/?SHADOW_URL=http%3A//pr-url.com/repository']),
webhooks: added variables into the call URL. Fixes #4211
r938 ])
webhook: quote URL variables to prevent url errors with special chars like # in pr title.
r3477 def test_webook_parse_url_for_pull_request_event(base_data, template, expected_urls):
integrations: webhook, allow to set a custom header. Fixes #5384
r2086
webhooks: added variables into the call URL. Fixes #4211
r938 base_data['pullrequest'] = {
'pull_request_id': 999,
'url': 'http://pr-url.com',
webhook: quote URL variables to prevent url errors with special chars like # in pr title.
r3477 'title': 'example-pr-title Ticket #123',
pull-request-events: expose pr title and uid for commits inside....
r2588 'commits_uid': 'abcdefg1234',
webhook: abstract webhook handler to base for easier re-usage.
r2585 'shadow_url': 'http://pr-url.com/repository'
webhooks: added variables into the call URL. Fixes #4211
r938 }
integrations: webhook, allow to set a custom header. Fixes #5384
r2086 headers = {'exmaple-header': 'header-values'}
webhook: abstract webhook handler to base for easier re-usage.
r2585 handler = WebhookDataHandler(template, headers)
webhooks: added variables into the call URL. Fixes #4211
r938 urls = handler(events.PullRequestCreateEvent(
AttributeDict({'target_repo': 'foo'})), base_data)
integrations: webhook, allow to set a custom header. Fixes #5384
r2086 assert urls == [
webhook: abstract webhook handler to base for easier re-usage.
r2585 (url, headers, base_data) for url in expected_urls]
webhooks: added variables into the call URL. Fixes #4211
r938
@pytest.mark.parametrize('template,expected_urls', [
webhook: abstract webhook handler to base for easier re-usage.
r2585 ('http://server.com/${branch}/build',
['http://server.com/stable/build',
'http://server.com/dev/build']),
('http://server.com/${branch}/${commit_id}',
['http://server.com/stable/stable-xxx',
'http://server.com/stable/stable-yyy',
'http://server.com/dev/dev-xxx',
'http://server.com/dev/dev-yyy']),
webhook: handle ${commit_id} variable independent from ${branch}....
r3041 ('http://server.com/${branch_head}',
['http://server.com/stable-yyy',
'http://server.com/dev-yyy']),
('http://server.com/${commit_id}',
['http://server.com/stable-xxx',
'http://server.com/stable-yyy',
'http://server.com/dev-xxx',
'http://server.com/dev-yyy']),
webhooks: added variables into the call URL. Fixes #4211
r938 ])
integrations: webhook, allow to set a custom header. Fixes #5384
r2086 def test_webook_parse_url_for_push_event(
pylons: remove pylons as dependency...
r2351 baseapp, repo_push_event, base_data, template, expected_urls):
webhooks: added variables into the call URL. Fixes #4211
r938 base_data['push'] = {
'branches': [{'name': 'stable'}, {'name': 'dev'}],
'commits': [{'branch': 'stable', 'raw_id': 'stable-xxx'},
{'branch': 'stable', 'raw_id': 'stable-yyy'},
{'branch': 'dev', 'raw_id': 'dev-xxx'},
{'branch': 'dev', 'raw_id': 'dev-yyy'}]
}
integrations: webhook, allow to set a custom header. Fixes #5384
r2086 headers = {'exmaple-header': 'header-values'}
webhook: abstract webhook handler to base for easier re-usage.
r2585 handler = WebhookDataHandler(template, headers)
webhooks: added variables into the call URL. Fixes #4211
r938 urls = handler(repo_push_event, base_data)
integrations: webhook, allow to set a custom header. Fixes #5384
r2086 assert urls == [
webhook: abstract webhook handler to base for easier re-usage.
r2585 (url, headers, base_data) for url in expected_urls]