123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699700701702703704705706707708709710711712713714715716717718719720721722723724725726727728729730731732733734735736737738739740741742743744745746747748749750751752753754755756757758759760761762763764765766767768769770771772773774775776777778779780781782783784785786 |
- # -*- coding: utf-8 -*-
- """
- (c) 2015 - Copyright Red Hat Inc
- Authors:
- Pierre-Yves Chibon <pingou@pingoured.fr>
- """
- __requires__ = ['SQLAlchemy >= 0.8']
- import pkg_resources
- import datetime
- import unittest
- import shutil
- import sys
- import os
- import json
- from mock import patch
- sys.path.insert(0, os.path.join(os.path.dirname(
- os.path.abspath(__file__)), '..'))
- import pagure.lib
- import tests
- class PagureFlaskApiForktests(tests.Modeltests):
- """ Tests for the flask API of pagure for issue """
- def setUp(self):
- """ Set up the environnment, ran before every tests. """
- super(PagureFlaskApiForktests, self).setUp()
- pagure.APP.config['TESTING'] = True
- pagure.SESSION = self.session
- pagure.api.SESSION = self.session
- pagure.api.fork.SESSION = self.session
- pagure.lib.SESSION = self.session
- pagure.APP.config['REQUESTS_FOLDER'] = None
- self.app = pagure.APP.test_client()
- @patch('pagure.lib.notify.send_email')
- def test_api_pull_request_views(self, send_email):
- """ Test the api_pull_request_views method of the flask api. """
- send_email.return_value = True
- tests.create_projects(self.session)
- tests.create_tokens(self.session)
- tests.create_tokens_acl(self.session)
- # Create a pull-request
- repo = pagure.lib.get_project(self.session, 'test')
- forked_repo = pagure.lib.get_project(self.session, 'test')
- req = pagure.lib.new_pull_request(
- session=self.session,
- repo_from=forked_repo,
- branch_from='master',
- repo_to=repo,
- branch_to='master',
- title='test pull-request',
- user='pingou',
- requestfolder=None,
- )
- self.session.commit()
- self.assertEqual(req.id, 1)
- self.assertEqual(req.title, 'test pull-request')
- # Invalid repo
- output = self.app.get('/api/0/foo/pull-requests')
- self.assertEqual(output.status_code, 404)
- data = json.loads(output.data)
- self.assertDictEqual(
- data,
- {
- "error": "Project not found",
- "error_code": "ENOPROJECT",
- }
- )
- # List pull-requests
- output = self.app.get('/api/0/test/pull-requests')
- self.assertEqual(output.status_code, 200)
- data = json.loads(output.data)
- data['requests'][0]['date_created'] = '1431414800'
- data['requests'][0]['updated_on'] = '1431414800'
- data['requests'][0]['project']['date_created'] = '1431414800'
- data['requests'][0]['repo_from']['date_created'] = '1431414800'
- data['requests'][0]['uid'] = '1431414800'
- self.assertDictEqual(
- data,
- {
- "args": {
- "assignee": None,
- "author": None,
- "status": True
- },
- "total_requests": 1,
- "requests": [
- {
- "assignee": None,
- "branch": "master",
- "branch_from": "master",
- "closed_at": None,
- "closed_by": None,
- "comments": [],
- "commit_start": None,
- "commit_stop": None,
- "date_created": "1431414800",
- "id": 1,
- "initial_comment": None,
- "project": {
- "date_created": "1431414800",
- "description": "test project #1",
- "id": 1,
- "name": "test",
- "parent": None,
- "priorities": {},
- "tags": [],
- "user": {
- "fullname": "PY C",
- "name": "pingou"
- }
- },
- "remote_git": None,
- "repo_from": {
- "date_created": "1431414800",
- "description": "test project #1",
- "id": 1,
- "name": "test",
- "parent": None,
- "priorities": {},
- "tags": [],
- "user": {
- "fullname": "PY C",
- "name": "pingou"
- }
- },
- "status": 'Open',
- "title": "test pull-request",
- "uid": "1431414800",
- "updated_on": "1431414800",
- "user": {
- "fullname": "PY C",
- "name": "pingou"
- }
- }
- ]
- }
- )
- headers = {'Authorization': 'token aaabbbcccddd'}
- # Access Pull-Request authenticated
- output = self.app.get('/api/0/test/pull-requests', headers=headers)
- self.assertEqual(output.status_code, 200)
- data2 = json.loads(output.data)
- data2['requests'][0]['date_created'] = '1431414800'
- data2['requests'][0]['updated_on'] = '1431414800'
- data2['requests'][0]['project']['date_created'] = '1431414800'
- data2['requests'][0]['repo_from']['date_created'] = '1431414800'
- data2['requests'][0]['uid'] = '1431414800'
- self.assertDictEqual(data, data2)
- @patch('pagure.lib.notify.send_email')
- def test_api_pull_request_view(self, send_email):
- """ Test the api_pull_request_view method of the flask api. """
- send_email.return_value = True
- tests.create_projects(self.session)
- tests.create_tokens(self.session)
- tests.create_tokens_acl(self.session)
- # Create a pull-request
- repo = pagure.lib.get_project(self.session, 'test')
- forked_repo = pagure.lib.get_project(self.session, 'test')
- req = pagure.lib.new_pull_request(
- session=self.session,
- repo_from=forked_repo,
- branch_from='master',
- repo_to=repo,
- branch_to='master',
- title='test pull-request',
- user='pingou',
- requestfolder=None,
- )
- self.session.commit()
- self.assertEqual(req.id, 1)
- self.assertEqual(req.title, 'test pull-request')
- # Invalid repo
- output = self.app.get('/api/0/foo/pull-request/1')
- self.assertEqual(output.status_code, 404)
- data = json.loads(output.data)
- self.assertDictEqual(
- data,
- {
- "error": "Project not found",
- "error_code": "ENOPROJECT",
- }
- )
- # Invalid issue for this repo
- output = self.app.get('/api/0/test2/pull-request/1')
- self.assertEqual(output.status_code, 404)
- data = json.loads(output.data)
- self.assertDictEqual(
- data,
- {
- "error": "Pull-Request not found",
- "error_code": "ENOREQ",
- }
- )
- # Valid issue
- output = self.app.get('/api/0/test/pull-request/1')
- self.assertEqual(output.status_code, 200)
- data = json.loads(output.data)
- data['date_created'] = '1431414800'
- data['updated_on'] = '1431414800'
- data['project']['date_created'] = '1431414800'
- data['repo_from']['date_created'] = '1431414800'
- data['uid'] = '1431414800'
- self.assertDictEqual(
- data,
- {
- "assignee": None,
- "branch": "master",
- "branch_from": "master",
- "closed_at": None,
- "closed_by": None,
- "comments": [],
- "commit_start": None,
- "commit_stop": None,
- "date_created": "1431414800",
- "id": 1,
- "initial_comment": None,
- "project": {
- "date_created": "1431414800",
- "description": "test project #1",
- "id": 1,
- "name": "test",
- "parent": None,
- "priorities": {},
- "tags": [],
- "user": {
- "fullname": "PY C",
- "name": "pingou"
- }
- },
- "remote_git": None,
- "repo_from": {
- "date_created": "1431414800",
- "description": "test project #1",
- "id": 1,
- "name": "test",
- "parent": None,
- "priorities": {},
- "tags": [],
- "user": {
- "fullname": "PY C",
- "name": "pingou"
- }
- },
- "status": 'Open',
- "title": "test pull-request",
- "uid": "1431414800",
- "updated_on": "1431414800",
- "user": {
- "fullname": "PY C",
- "name": "pingou"
- }
- }
- )
- headers = {'Authorization': 'token aaabbbcccddd'}
- # Access Pull-Request authenticated
- output = self.app.get('/api/0/test/pull-request/1', headers=headers)
- self.assertEqual(output.status_code, 200)
- data2 = json.loads(output.data)
- data2['date_created'] = '1431414800'
- data2['project']['date_created'] = '1431414800'
- data2['repo_from']['date_created'] = '1431414800'
- data2['uid'] = '1431414800'
- data2['date_created'] = '1431414800'
- data2['updated_on'] = '1431414800'
- self.assertDictEqual(data, data2)
- @patch('pagure.lib.notify.send_email')
- def test_api_pull_request_close(self, send_email):
- """ Test the api_pull_request_close method of the flask api. """
- send_email.return_value = True
- tests.create_projects(self.session)
- tests.create_tokens(self.session)
- tests.create_tokens_acl(self.session)
- # Create the pull-request to close
- repo = pagure.lib.get_project(self.session, 'test')
- forked_repo = pagure.lib.get_project(self.session, 'test')
- req = pagure.lib.new_pull_request(
- session=self.session,
- repo_from=forked_repo,
- branch_from='master',
- repo_to=repo,
- branch_to='master',
- title='test pull-request',
- user='pingou',
- requestfolder=None,
- )
- self.session.commit()
- self.assertEqual(req.id, 1)
- self.assertEqual(req.title, 'test pull-request')
- headers = {'Authorization': 'token aaabbbcccddd'}
- # Invalid project
- output = self.app.post(
- '/api/0/foo/pull-request/1/close', headers=headers)
- self.assertEqual(output.status_code, 404)
- data = json.loads(output.data)
- self.assertDictEqual(
- data,
- {
- "error": "Project not found",
- "error_code": "ENOPROJECT",
- }
- )
- # Valid token, wrong project
- output = self.app.post(
- '/api/0/test2/pull-request/1/close', headers=headers)
- self.assertEqual(output.status_code, 401)
- data = json.loads(output.data)
- self.assertDictEqual(
- data,
- {
- "error": "Invalid or expired token. Please visit " \
- "https://pagure.org/ to get or renew your API token.",
- "error_code": "EINVALIDTOK",
- }
- )
- # Invalid PR
- output = self.app.post(
- '/api/0/test/pull-request/2/close', headers=headers)
- self.assertEqual(output.status_code, 404)
- data = json.loads(output.data)
- self.assertDictEqual(
- data,
- {'error': 'Pull-Request not found', 'error_code': "ENOREQ"}
- )
- # Create a token for foo for this project
- item = pagure.lib.model.Token(
- id='foobar_token',
- user_id=2,
- project_id=1,
- expiration=datetime.datetime.utcnow() + datetime.timedelta(
- days=30)
- )
- self.session.add(item)
- self.session.commit()
- # Allow the token to close PR
- item = pagure.lib.model.TokenAcl(
- token_id='foobar_token',
- acl_id=7,
- )
- self.session.add(item)
- self.session.commit()
- headers = {'Authorization': 'token foobar_token'}
- # User not admin
- output = self.app.post(
- '/api/0/test/pull-request/1/close', headers=headers)
- self.assertEqual(output.status_code, 403)
- data = json.loads(output.data)
- self.assertDictEqual(
- data,
- {
- 'error': 'You are not allowed to merge/close pull-request '
- 'for this project',
- 'error_code': "ENOPRCLOSE",
- }
- )
- headers = {'Authorization': 'token aaabbbcccddd'}
- # Close PR
- output = self.app.post(
- '/api/0/test/pull-request/1/close', headers=headers)
- self.assertEqual(output.status_code, 200)
- data = json.loads(output.data)
- self.assertDictEqual(
- data,
- {"message": "Pull-request closed!"}
- )
- @patch('pagure.lib.notify.send_email')
- @patch('pagure.lib.git.merge_pull_request')
- def test_api_pull_request_merge(self, mpr, send_email):
- """ Test the api_pull_request_merge method of the flask api. """
- mpr.return_value = 'Changes merged!'
- send_email.return_value = True
- tests.create_projects(self.session)
- tests.create_tokens(self.session)
- tests.create_tokens_acl(self.session)
- # Create the pull-request to close
- repo = pagure.lib.get_project(self.session, 'test')
- forked_repo = pagure.lib.get_project(self.session, 'test')
- req = pagure.lib.new_pull_request(
- session=self.session,
- repo_from=forked_repo,
- branch_from='master',
- repo_to=repo,
- branch_to='master',
- title='test pull-request',
- user='pingou',
- requestfolder=None,
- )
- self.session.commit()
- self.assertEqual(req.id, 1)
- self.assertEqual(req.title, 'test pull-request')
- headers = {'Authorization': 'token aaabbbcccddd'}
- # Invalid project
- output = self.app.post(
- '/api/0/foo/pull-request/1/merge', headers=headers)
- self.assertEqual(output.status_code, 404)
- data = json.loads(output.data)
- self.assertDictEqual(
- data,
- {
- "error": "Project not found",
- "error_code": "ENOPROJECT",
- }
- )
- # Valid token, wrong project
- output = self.app.post(
- '/api/0/test2/pull-request/1/merge', headers=headers)
- self.assertEqual(output.status_code, 401)
- data = json.loads(output.data)
- self.assertDictEqual(
- data,
- {
- "error": "Invalid or expired token. Please visit " \
- "https://pagure.org/ to get or renew your API token.",
- "error_code": "EINVALIDTOK",
- }
- )
- # Invalid PR
- output = self.app.post(
- '/api/0/test/pull-request/2/merge', headers=headers)
- self.assertEqual(output.status_code, 404)
- data = json.loads(output.data)
- self.assertDictEqual(
- data,
- {'error': 'Pull-Request not found', 'error_code': "ENOREQ"}
- )
- # Create a token for foo for this project
- item = pagure.lib.model.Token(
- id='foobar_token',
- user_id=2,
- project_id=1,
- expiration=datetime.datetime.utcnow() + datetime.timedelta(
- days=30)
- )
- self.session.add(item)
- self.session.commit()
- # Allow the token to merge PR
- item = pagure.lib.model.TokenAcl(
- token_id='foobar_token',
- acl_id=10,
- )
- self.session.add(item)
- self.session.commit()
- headers = {'Authorization': 'token foobar_token'}
- # User not admin
- output = self.app.post(
- '/api/0/test/pull-request/1/merge', headers=headers)
- self.assertEqual(output.status_code, 403)
- data = json.loads(output.data)
- self.assertDictEqual(
- data,
- {
- 'error': 'You are not allowed to merge/close pull-request '
- 'for this project',
- 'error_code': "ENOPRCLOSE",
- }
- )
- headers = {'Authorization': 'token aaabbbcccddd'}
- # Merge PR
- output = self.app.post(
- '/api/0/test/pull-request/1/merge', headers=headers)
- self.assertEqual(output.status_code, 200)
- data = json.loads(output.data)
- self.assertDictEqual(
- data,
- {"message": "Changes merged!"}
- )
- @patch('pagure.lib.notify.send_email')
- def test_api_pull_request_add_comment(self, mockemail):
- """ Test the api_pull_request_add_comment method of the flask api. """
- mockemail.return_value = True
- tests.create_projects(self.session)
- tests.create_tokens(self.session)
- tests.create_tokens_acl(self.session)
- headers = {'Authorization': 'token aaabbbcccddd'}
- # Invalid project
- output = self.app.post(
- '/api/0/foo/pull-request/1/comment', headers=headers)
- self.assertEqual(output.status_code, 404)
- data = json.loads(output.data)
- self.assertDictEqual(
- data,
- {
- "error": "Project not found",
- "error_code": "ENOPROJECT",
- }
- )
- # Valid token, wrong project
- output = self.app.post(
- '/api/0/test2/pull-request/1/comment', headers=headers)
- self.assertEqual(output.status_code, 401)
- data = json.loads(output.data)
- self.assertDictEqual(
- data,
- {
- "error": "Invalid or expired token. Please visit " \
- "https://pagure.org/ to get or renew your API token.",
- "error_code": "EINVALIDTOK",
- }
- )
- # No input
- output = self.app.post(
- '/api/0/test/pull-request/1/comment', headers=headers)
- self.assertEqual(output.status_code, 404)
- data = json.loads(output.data)
- self.assertDictEqual(
- data,
- {
- "error": "Pull-Request not found",
- "error_code": "ENOREQ",
- }
- )
- # Create a pull-request
- repo = pagure.lib.get_project(self.session, 'test')
- forked_repo = pagure.lib.get_project(self.session, 'test')
- req = pagure.lib.new_pull_request(
- session=self.session,
- repo_from=forked_repo,
- branch_from='master',
- repo_to=repo,
- branch_to='master',
- title='test pull-request',
- user='pingou',
- requestfolder=None,
- )
- self.session.commit()
- self.assertEqual(req.id, 1)
- self.assertEqual(req.title, 'test pull-request')
- # Check comments before
- request = pagure.lib.search_pull_requests(
- self.session, project_id=1, requestid=1)
- self.assertEqual(len(request.comments), 0)
- data = {
- 'title': 'test issue',
- }
- # Incomplete request
- output = self.app.post(
- '/api/0/test/pull-request/1/comment', data=data, headers=headers)
- self.assertEqual(output.status_code, 400)
- data = json.loads(output.data)
- self.assertDictEqual(
- data,
- {
- "error": "Invalid or incomplete input submited",
- "error_code": "EINVALIDREQ",
- }
- )
- # No change
- request = pagure.lib.search_pull_requests(
- self.session, project_id=1, requestid=1)
- self.assertEqual(len(request.comments), 0)
- data = {
- 'comment': 'This is a very interesting question',
- }
- # Valid request
- output = self.app.post(
- '/api/0/test/pull-request/1/comment', data=data, headers=headers)
- self.assertEqual(output.status_code, 200)
- data = json.loads(output.data)
- self.assertDictEqual(
- data,
- {'message': 'Comment added'}
- )
- # One comment added
- request = pagure.lib.search_pull_requests(
- self.session, project_id=1, requestid=1)
- self.assertEqual(len(request.comments), 1)
- @patch('pagure.lib.notify.send_email')
- def test_api_pull_request_add_flag(self, mockemail):
- """ Test the api_pull_request_add_flag method of the flask api. """
- mockemail.return_value = True
- tests.create_projects(self.session)
- tests.create_tokens(self.session)
- tests.create_tokens_acl(self.session)
- headers = {'Authorization': 'token aaabbbcccddd'}
- # Invalid project
- output = self.app.post(
- '/api/0/foo/pull-request/1/flag', headers=headers)
- self.assertEqual(output.status_code, 404)
- data = json.loads(output.data)
- self.assertDictEqual(
- data,
- {
- "error": "Project not found",
- "error_code": "ENOPROJECT",
- }
- )
- # Valid token, wrong project
- output = self.app.post(
- '/api/0/test2/pull-request/1/flag', headers=headers)
- self.assertEqual(output.status_code, 401)
- data = json.loads(output.data)
- self.assertDictEqual(
- data,
- {
- "error": "Invalid or expired token. Please visit " \
- "https://pagure.org/ to get or renew your API token.",
- "error_code": "EINVALIDTOK",
- }
- )
- # No input
- output = self.app.post(
- '/api/0/test/pull-request/1/flag', headers=headers)
- self.assertEqual(output.status_code, 404)
- data = json.loads(output.data)
- self.assertDictEqual(
- data,
- {
- "error": "Pull-Request not found",
- "error_code": "ENOREQ",
- }
- )
- # Create a pull-request
- repo = pagure.lib.get_project(self.session, 'test')
- forked_repo = pagure.lib.get_project(self.session, 'test')
- req = pagure.lib.new_pull_request(
- session=self.session,
- repo_from=forked_repo,
- branch_from='master',
- repo_to=repo,
- branch_to='master',
- title='test pull-request',
- user='pingou',
- requestfolder=None,
- )
- self.session.commit()
- self.assertEqual(req.id, 1)
- self.assertEqual(req.title, 'test pull-request')
- # Check comments before
- request = pagure.lib.search_pull_requests(
- self.session, project_id=1, requestid=1)
- self.assertEqual(len(request.flags), 0)
- data = {
- 'username': 'Jenkins',
- 'percent': 100,
- 'url': 'http://jenkins.cloud.fedoraproject.org/',
- 'uid': 'jenkins_build_pagure_100+seed',
- }
- # Incomplete request
- output = self.app.post(
- '/api/0/test/pull-request/1/flag', data=data, headers=headers)
- self.assertEqual(output.status_code, 400)
- data = json.loads(output.data)
- self.assertDictEqual(
- data,
- {
- "error": "Invalid or incomplete input submited",
- "error_code": "EINVALIDREQ",
- }
- )
- # No change
- request = pagure.lib.search_pull_requests(
- self.session, project_id=1, requestid=1)
- self.assertEqual(len(request.flags), 0)
- data = {
- 'username': 'Jenkins',
- 'percent': 0,
- 'comment': 'Tests failed',
- 'url': 'http://jenkins.cloud.fedoraproject.org/',
- 'uid': 'jenkins_build_pagure_100+seed',
- }
- # Valid request
- output = self.app.post(
- '/api/0/test/pull-request/1/flag', data=data, headers=headers)
- self.assertEqual(output.status_code, 200)
- data = json.loads(output.data)
- self.assertDictEqual(
- data,
- {'message': 'Flag added'}
- )
- # One flag added
- request = pagure.lib.search_pull_requests(
- self.session, project_id=1, requestid=1)
- self.assertEqual(len(request.flags), 1)
- self.assertEqual(request.flags[0].comment, 'Tests failed')
- self.assertEqual(request.flags[0].percent, 0)
- # Update flag
- data = {
- 'username': 'Jenkins',
- 'percent': 100,
- 'comment': 'Tests passed',
- 'url': 'http://jenkins.cloud.fedoraproject.org/',
- 'uid': 'jenkins_build_pagure_100+seed',
- }
- output = self.app.post(
- '/api/0/test/pull-request/1/flag', data=data, headers=headers)
- self.assertEqual(output.status_code, 200)
- data = json.loads(output.data)
- self.assertDictEqual(
- data,
- {'message': 'Flag updated'}
- )
- # One flag added
- request = pagure.lib.search_pull_requests(
- self.session, project_id=1, requestid=1)
- self.assertEqual(len(request.flags), 1)
- self.assertEqual(request.flags[0].comment, 'Tests passed')
- self.assertEqual(request.flags[0].percent, 100)
- if __name__ == '__main__':
- SUITE = unittest.TestLoader().loadTestsFromTestCase(
- PagureFlaskApiForktests)
- unittest.TextTestRunner(verbosity=2).run(SUITE)
|