|
Pierre-Yves Chibon |
a52cca |
# -*- coding: utf-8 -*-
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
"""
|
|
Pierre-Yves Chibon |
a52cca |
(c) 2015 - Copyright Red Hat Inc
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
Authors:
|
|
Pierre-Yves Chibon |
a52cca |
Pierre-Yves Chibon <pingou@pingoured.fr></pingou@pingoured.fr>
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
"""
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
__requires__ = ['SQLAlchemy >= 0.8']
|
|
Pierre-Yves Chibon |
a52cca |
import pkg_resources
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
import json
|
|
Pierre-Yves Chibon |
a52cca |
import unittest
|
|
Pierre-Yves Chibon |
a52cca |
import shutil
|
|
Pierre-Yves Chibon |
a52cca |
import sys
|
|
Pierre-Yves Chibon |
a52cca |
import os
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
import pygit2
|
|
Pierre-Yves Chibon |
a52cca |
from mock import patch
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
sys.path.insert(0, os.path.join(os.path.dirname(
|
|
Pierre-Yves Chibon |
a52cca |
os.path.abspath(__file__)), '..'))
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
import progit.lib
|
|
Pierre-Yves Chibon |
a52cca |
import tests
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
class ProgitFlaskIssuestests(tests.Modeltests):
|
|
Pierre-Yves Chibon |
a52cca |
""" Tests for flask docs of progit """
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
def setUp(self):
|
|
Pierre-Yves Chibon |
a52cca |
""" Set up the environnment, ran before every tests. """
|
|
Pierre-Yves Chibon |
a52cca |
super(ProgitFlaskIssuestests, self).setUp()
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
progit.APP.config['TESTING'] = True
|
|
Pierre-Yves Chibon |
a52cca |
progit.SESSION = self.session
|
|
Pierre-Yves Chibon |
a52cca |
progit.ui.SESSION = self.session
|
|
Pierre-Yves Chibon |
a52cca |
progit.ui.app.SESSION = self.session
|
|
Pierre-Yves Chibon |
a52cca |
progit.ui.issues.SESSION = self.session
|
|
Pierre-Yves Chibon |
a52cca |
progit.ui.repo.SESSION = self.session
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
progit.APP.config['GIT_FOLDER'] = tests.HERE
|
|
Pierre-Yves Chibon |
a52cca |
progit.APP.config['FORK_FOLDER'] = os.path.join(
|
|
Pierre-Yves Chibon |
a52cca |
tests.HERE, 'forks')
|
|
Pierre-Yves Chibon |
a52cca |
progit.APP.config['TICKETS_FOLDER'] = os.path.join(
|
|
Pierre-Yves Chibon |
a52cca |
tests.HERE, 'tickets')
|
|
Pierre-Yves Chibon |
a52cca |
progit.APP.config['DOCS_FOLDER'] = os.path.join(
|
|
Pierre-Yves Chibon |
a52cca |
tests.HERE, 'docs')
|
|
Pierre-Yves Chibon |
a52cca |
self.app = progit.APP.test_client()
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
@patch('progit.lib.git.update_git_ticket')
|
|
Pierre-Yves Chibon |
a52cca |
@patch('progit.lib.notify.send_email')
|
|
Pierre-Yves Chibon |
a52cca |
def test_new_issue(self, p_send_email, p_ugt):
|
|
Pierre-Yves Chibon |
a52cca |
""" Test the new_issue endpoint. """
|
|
Pierre-Yves Chibon |
a52cca |
p_send_email.return_value = True
|
|
Pierre-Yves Chibon |
a52cca |
p_ugt.return_value = True
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
output = self.app.get('/foo/new_issue')
|
|
Pierre-Yves Chibon |
a52cca |
self.assertEqual(output.status_code, 302)
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
user = tests.FakeUser()
|
|
Pierre-Yves Chibon |
a52cca |
with tests.user_set(progit.APP, user):
|
|
Pierre-Yves Chibon |
a52cca |
output = self.app.get('/foo/new_issue')
|
|
Pierre-Yves Chibon |
a52cca |
self.assertEqual(output.status_code, 404)
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
tests.create_projects(self.session)
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
output = self.app.get('/test/new_issue')
|
|
Pierre-Yves Chibon |
a52cca |
self.assertEqual(output.status_code, 200)
|
|
Pierre-Yves Chibon |
a52cca |
self.assertTrue('New issue' in output.data)
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
csrf_token = output.data.split(
|
|
Pierre-Yves Chibon |
a52cca |
'name="csrf_token" type="hidden" value="')[1].split('">')[0]
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
data = {
|
|
Pierre-Yves Chibon |
a52cca |
}
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
# Insufficient input
|
|
Pierre-Yves Chibon |
a52cca |
output = self.app.post('/test/new_issue', data=data)
|
|
Pierre-Yves Chibon |
a52cca |
self.assertEqual(output.status_code, 200)
|
|
Pierre-Yves Chibon |
a52cca |
self.assertTrue('New issue' in output.data)
|
|
Pierre-Yves Chibon |
a52cca |
self.assertEqual(output.data.count(
|
|
Pierre-Yves Chibon |
a52cca |
'This field is required.'), 2)
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
data['title'] = 'Test issue'
|
|
Pierre-Yves Chibon |
a52cca |
output = self.app.post('/test/new_issue', data=data)
|
|
Pierre-Yves Chibon |
a52cca |
self.assertEqual(output.status_code, 200)
|
|
Pierre-Yves Chibon |
a52cca |
self.assertTrue('New issue' in output.data)
|
|
Pierre-Yves Chibon |
a52cca |
self.assertEqual(output.data.count(
|
|
Pierre-Yves Chibon |
a52cca |
'This field is required.'), 1)
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
data['issue_content'] = 'We really should improve on this issue'
|
|
Pierre-Yves Chibon |
a52cca |
data['status'] = 'Open'
|
|
Pierre-Yves Chibon |
a52cca |
output = self.app.post('/test/new_issue', data=data)
|
|
Pierre-Yves Chibon |
a52cca |
self.assertEqual(output.status_code, 200)
|
|
Pierre-Yves Chibon |
a52cca |
self.assertTrue('New issue' in output.data)
|
|
Pierre-Yves Chibon |
a52cca |
self.assertEqual(output.data.count(
|
|
Pierre-Yves Chibon |
a52cca |
'This field is required.'), 0)
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
# Invalid user
|
|
Pierre-Yves Chibon |
a52cca |
data['csrf_token'] = csrf_token
|
|
Pierre-Yves Chibon |
a52cca |
output = self.app.post('/test/new_issue', data=data)
|
|
Pierre-Yves Chibon |
a52cca |
self.assertEqual(output.status_code, 200)
|
|
Pierre-Yves Chibon |
a52cca |
self.assertTrue('New issue' in output.data)
|
|
Pierre-Yves Chibon |
a52cca |
self.assertEqual(output.data.count(
|
|
Pierre-Yves Chibon |
a52cca |
'This field is required.'), 0)
|
|
Pierre-Yves Chibon |
a52cca |
self.assertTrue(
|
|
Pierre-Yves Chibon |
a52cca |
'No user "username" found'
|
|
Pierre-Yves Chibon |
a52cca |
in output.data)
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
user.username = 'pingou'
|
|
Pierre-Yves Chibon |
a52cca |
with tests.user_set(progit.APP, user):
|
|
Pierre-Yves Chibon |
a52cca |
output = self.app.post(
|
|
Pierre-Yves Chibon |
a52cca |
'/test/new_issue', data=data, follow_redirects=True)
|
|
Pierre-Yves Chibon |
a52cca |
self.assertEqual(output.status_code, 200)
|
|
Pierre-Yves Chibon |
a52cca |
self.assertTrue(
|
|
Pierre-Yves Chibon |
a52cca |
'Issue created' in output.data)
|
|
Pierre-Yves Chibon |
a52cca |
self.assertTrue('test project #1 ' in output.data)
|
|
Pierre-Yves Chibon |
a52cca |
self.assertTrue('\n Issues (1)\n ' in output.data)
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
7dfb23 |
@patch('progit.lib.git.update_git_ticket')
|
|
Pierre-Yves Chibon |
7dfb23 |
@patch('progit.lib.notify.send_email')
|
|
Pierre-Yves Chibon |
7dfb23 |
def test_view_issues(self, p_send_email, p_ugt):
|
|
Pierre-Yves Chibon |
7dfb23 |
""" Test the view_issues endpoint. """
|
|
Pierre-Yves Chibon |
7dfb23 |
p_send_email.return_value = True
|
|
Pierre-Yves Chibon |
7dfb23 |
p_ugt.return_value = True
|
|
Pierre-Yves Chibon |
7dfb23 |
|
|
Pierre-Yves Chibon |
7dfb23 |
output = self.app.get('/foo/issues')
|
|
Pierre-Yves Chibon |
7dfb23 |
self.assertEqual(output.status_code, 404)
|
|
Pierre-Yves Chibon |
7dfb23 |
|
|
Pierre-Yves Chibon |
7dfb23 |
tests.create_projects(self.session)
|
|
Pierre-Yves Chibon |
7dfb23 |
|
|
Pierre-Yves Chibon |
7dfb23 |
output = self.app.get('/test/issues')
|
|
Pierre-Yves Chibon |
7dfb23 |
self.assertEqual(output.status_code, 200)
|
|
Pierre-Yves Chibon |
7dfb23 |
self.assertTrue('test project #1 ' in output.data)
|
|
Pierre-Yves Chibon |
7dfb23 |
self.assertTrue('\n Issues (0)\n ' in output.data)
|
|
Pierre-Yves Chibon |
7dfb23 |
|
|
Pierre-Yves Chibon |
7dfb23 |
# Create issues to play with
|
|
Pierre-Yves Chibon |
7dfb23 |
repo = progit.lib.get_project(self.session, 'test')
|
|
Pierre-Yves Chibon |
7dfb23 |
msg = progit.lib.new_issue(
|
|
Pierre-Yves Chibon |
7dfb23 |
session=self.session,
|
|
Pierre-Yves Chibon |
7dfb23 |
repo=repo,
|
|
Pierre-Yves Chibon |
7dfb23 |
title='Test issue',
|
|
Pierre-Yves Chibon |
7dfb23 |
content='We should work on this',
|
|
Pierre-Yves Chibon |
7dfb23 |
user='pingou',
|
|
Pierre-Yves Chibon |
7dfb23 |
ticketfolder=None
|
|
Pierre-Yves Chibon |
7dfb23 |
)
|
|
Pierre-Yves Chibon |
7dfb23 |
self.session.commit()
|
|
Pierre-Yves Chibon |
7dfb23 |
self.assertEqual(msg, 'Issue created')
|
|
Pierre-Yves Chibon |
7dfb23 |
|
|
Pierre-Yves Chibon |
7dfb23 |
# Whole list
|
|
Pierre-Yves Chibon |
7dfb23 |
output = self.app.get('/test/issues')
|
|
Pierre-Yves Chibon |
7dfb23 |
self.assertEqual(output.status_code, 200)
|
|
Pierre-Yves Chibon |
7dfb23 |
self.assertTrue('test project #1 ' in output.data)
|
|
Pierre-Yves Chibon |
7dfb23 |
self.assertTrue('\n Issues (1)\n ' in output.data)
|
|
Pierre-Yves Chibon |
7dfb23 |
|
|
Pierre-Yves Chibon |
7dfb23 |
# Status = closed
|
|
Pierre-Yves Chibon |
7dfb23 |
output = self.app.get('/test/issues?status=cloSED')
|
|
Pierre-Yves Chibon |
7dfb23 |
self.assertEqual(output.status_code, 200)
|
|
Pierre-Yves Chibon |
7dfb23 |
self.assertTrue('test project #1 ' in output.data)
|
|
Pierre-Yves Chibon |
7dfb23 |
self.assertTrue(
|
|
Pierre-Yves Chibon |
7dfb23 |
'\n Closed\n Issues (0)\n ' in output.data)
|
|
Pierre-Yves Chibon |
7dfb23 |
|
|
Pierre-Yves Chibon |
7dfb23 |
# Status = fixed
|
|
Pierre-Yves Chibon |
7dfb23 |
output = self.app.get('/test/issues?status=fixed')
|
|
Pierre-Yves Chibon |
7dfb23 |
self.assertEqual(output.status_code, 200)
|
|
Pierre-Yves Chibon |
7dfb23 |
self.assertTrue('test project #1 ' in output.data)
|
|
Pierre-Yves Chibon |
7dfb23 |
self.assertTrue(
|
|
Pierre-Yves Chibon |
7dfb23 |
'\n Closed\n Issues (0)\n ' in output.data)
|
|
Pierre-Yves Chibon |
7dfb23 |
|
|
Pierre-Yves Chibon |
7dfb23 |
# Project w/o issue tracker
|
|
Pierre-Yves Chibon |
7dfb23 |
repo = progit.lib.get_project(self.session, 'test')
|
|
Pierre-Yves Chibon |
7dfb23 |
repo.issue_tracker = False
|
|
Pierre-Yves Chibon |
7dfb23 |
self.session.add(repo)
|
|
Pierre-Yves Chibon |
7dfb23 |
self.session.commit()
|
|
Pierre-Yves Chibon |
7dfb23 |
|
|
Pierre-Yves Chibon |
7dfb23 |
output = self.app.get('/test/issues')
|
|
Pierre-Yves Chibon |
7dfb23 |
self.assertEqual(output.status_code, 404)
|
|
Pierre-Yves Chibon |
7dfb23 |
|
|
Pierre-Yves Chibon |
a52cca |
|
|
Pierre-Yves Chibon |
a52cca |
if __name__ == '__main__':
|
|
Pierre-Yves Chibon |
a52cca |
SUITE = unittest.TestLoader().loadTestsFromTestCase(ProgitFlaskIssuestests)
|
|
Pierre-Yves Chibon |
a52cca |
unittest.TextTestRunner(verbosity=2).run(SUITE)
|