123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685 |
- # -*- coding: utf-8 -*-
- """
- (c) 2016-2018 - Copyright Red Hat Inc
- Authors:
- Pierre-Yves Chibon <pingou@pingoured.fr>
- """
- from __future__ import unicode_literals, absolute_import
- import datetime
- import json
- import unittest
- import shutil
- import sys
- import tempfile
- import os
- import pygit2
- from mock import patch
- sys.path.insert(
- 0, os.path.join(os.path.dirname(os.path.abspath(__file__)), "..")
- )
- import pagure.lib.query
- import tests
- from pagure.lib.repo import PagureRepo
- class PagureFlaskRoadmaptests(tests.Modeltests):
- """Tests for the pagure's roadmap"""
- @patch("pagure.lib.git.update_git")
- @patch("pagure.lib.notify.send_email")
- def test_ticket_with_no_roadmap(self, p_send_email, p_ugt):
- """Test creating a ticket without roadmap."""
- p_send_email.return_value = True
- p_ugt.return_value = True
- tests.create_projects(self.session)
- tests.create_projects_git(os.path.join(self.path, "repos"), bare=True)
- user = tests.FakeUser()
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- # Get the CSRF token
- output = self.app.get("/test/new_issue")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertTrue(
- '<h4 class="font-weight-bold mb-4">New Issue</h4>\n'
- in output_text
- )
- csrf_token = output_text.split(
- 'name="csrf_token" type="hidden" value="'
- )[1].split('">')[0]
- data = {
- "title": "Test issue",
- "issue_content": "We really should improve on this issue",
- "status": "Open",
- "csrf_token": csrf_token,
- }
- # Create the issue
- output = self.app.post(
- "/test/new_issue", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>Issue #1: Test issue - test - Pagure</title>",
- output_text,
- )
- self.assertIn(
- '<a class="btn btn-outline-secondary btn-sm border-0" '
- 'href="/test/issue/1/edit" title="Edit this issue">',
- output_text,
- )
- @patch("pagure.lib.git.update_git")
- @patch("pagure.lib.notify.send_email")
- def test_ticket_with_roadmap(self, p_send_email, p_ugt):
- """Test creating a ticket with roadmap."""
- p_send_email.return_value = True
- p_ugt.return_value = True
- tests.create_projects(self.session)
- tests.create_projects_git(os.path.join(self.path, "repos"), bare=True)
- # Set some milestone
- repo = pagure.lib.query.get_authorized_project(self.session, "test")
- repo.milestone = {"v1.0": "", "v2.0": "Tomorrow!"}
- self.session.add(repo)
- self.session.commit()
- user = tests.FakeUser()
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- # Get the CSRF token
- output = self.app.get("/test/new_issue")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertTrue(
- '<h4 class="font-weight-bold mb-4">New Issue</h4>\n'
- in output_text
- )
- csrf_token = output_text.split(
- 'name="csrf_token" type="hidden" value="'
- )[1].split('">')[0]
- data = {
- "title": "Test issue",
- "issue_content": "We really should improve on this issue",
- "status": "Open",
- "csrf_token": csrf_token,
- }
- # Create the issue
- output = self.app.post(
- "/test/new_issue", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>Issue #1: Test issue - test - Pagure</title>",
- output_text,
- )
- self.assertIn(
- '<a class="btn btn-outline-secondary btn-sm border-0" '
- 'href="/test/issue/1/edit" title="Edit this issue">',
- output_text,
- )
- # Mark the ticket for the roadmap
- data = {"tag": "roadmap", "csrf_token": csrf_token}
- output = self.app.post(
- "/test/issue/1/update", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>Issue #1: Test issue - test - Pagure</title>",
- output_text,
- )
- self.assertIn(
- '<a class="btn btn-outline-secondary btn-sm border-0" '
- 'href="/test/issue/1/edit" title="Edit this issue">',
- output_text,
- )
- def test_update_milestones(self):
- """Test updating milestones of a repo."""
- tests.create_projects(self.session)
- tests.create_projects_git(os.path.join(self.path, "repos"), bare=True)
- # Set some milestones
- repo = pagure.lib.query.get_authorized_project(self.session, "test")
- self.assertEqual(repo.milestones, {})
- user = tests.FakeUser()
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- # Get the CSRF token
- output = self.app.get("/test/settings")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>Settings - test - Pagure</title>", output_text
- )
- self.assertIn(
- '<h5 class="pl-2 font-weight-bold text-muted">'
- "Project Settings</h5>\n",
- output_text,
- )
- csrf_token = output_text.split(
- 'name="csrf_token" type="hidden" value="'
- )[1].split('">')[0]
- data = {
- "milestones": 1,
- "milestone_1_name": "1",
- "milestone_1_date": "Tomorrow",
- }
- output = self.app.post(
- "/test/update/milestones", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- # Check the redirect
- self.assertIn(
- "<title>Settings - test - Pagure</title>", output_text
- )
- self.assertIn(
- '<h5 class="pl-2 font-weight-bold text-muted">'
- "Project Settings</h5>\n",
- output_text,
- )
- # Check the result of the action -- None, no CSRF
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test"
- )
- self.assertEqual(repo.milestones, {})
- data = {
- "milestones": 1,
- "milestone_1_name": "1",
- "milestone_date": "",
- "csrf_token": csrf_token,
- }
- output = self.app.post(
- "/test/update/milestones", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- # Check the redirect
- self.assertIn(
- "<title>Settings - test - Pagure</title>", output_text
- )
- self.assertIn(
- '<h5 class="pl-2 font-weight-bold text-muted">'
- "Project Settings</h5>\n",
- output_text,
- )
- self.assertIn("Milestones updated", output_text)
- # Check the result of the action -- Milestones recorded
- self.session.commit()
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test"
- )
- self.assertEqual(
- repo.milestones, {"1": {"active": False, "date": None}}
- )
- data = {
- "milestones": [1, 2],
- "milestone_1_name": "v1.0",
- "milestone_2_name": "v2.0",
- "milestone_1_date": "Tomorrow",
- "milestone_2_date": "",
- "csrf_token": csrf_token,
- }
- output = self.app.post(
- "/test/update/milestones", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- # Check the redirect
- self.assertIn(
- "<title>Settings - test - Pagure</title>", output_text
- )
- self.assertIn(
- '<h5 class="pl-2 font-weight-bold text-muted">'
- "Project Settings</h5>\n",
- output_text,
- )
- self.assertIn("Milestones updated", output_text)
- # Check the result of the action -- Milestones recorded
- self.session.commit()
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test"
- )
- self.assertEqual(
- repo.milestones,
- {
- "v1.0": {"active": False, "date": "Tomorrow"},
- "v2.0": {"active": False, "date": None},
- },
- )
- # Check error - less milestones than dates
- data = {
- "milestones": [1, 2],
- "milestone_1_name": "v1.0",
- "milestone_2_name": "v2.0",
- "milestone_1_date": "Tomorrow",
- "milestone_2_date": "Next week",
- "milestone_3_date": "Next Year",
- "csrf_token": csrf_token,
- }
- output = self.app.post(
- "/test/update/milestones", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- # Check the redirect
- self.assertIn(
- "<title>Settings - test - Pagure</title>", output_text
- )
- self.assertIn(
- '<h5 class="pl-2 font-weight-bold text-muted">'
- "Project Settings</h5>\n",
- output_text,
- )
- # Check the result of the action -- Milestones un-changed
- self.session.commit()
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test"
- )
- self.assertEqual(
- repo.milestones,
- {
- "v1.0": {"active": False, "date": "Tomorrow"},
- "v2.0": {"active": False, "date": "Next week"},
- },
- )
- # Check error - Twice the same milestone
- data = {
- "milestones": [1, 2, 3],
- "milestone_1_name": "v1.0",
- "milestone_2_name": "v2.0",
- "milestone_3_name": "v2.0",
- "milestone_1_date": "Tomorrow",
- "milestone_2_date": "Next week",
- "milestone_3_date": "Next Year",
- "csrf_token": csrf_token,
- }
- output = self.app.post(
- "/test/update/milestones", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- # Check the redirect
- self.assertIn(
- "<title>Settings - test - Pagure</title>", output_text
- )
- self.assertIn(
- '<h5 class="pl-2 font-weight-bold text-muted">'
- "Project Settings</h5>\n",
- output_text,
- )
- self.assertIn(
- "Milestone v2.0 is present multiple times",
- "Milestone v2.0 is present multiple times",
- output_text,
- )
- # Check the result of the action -- Milestones un-changed
- self.session.commit()
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test"
- )
- self.assertEqual(
- repo.milestones,
- {
- "v1.0": {"active": False, "date": "Tomorrow"},
- "v2.0": {"active": False, "date": "Next week"},
- },
- )
- # Check error - Twice the same date
- data = {
- "milestones": [1, 2, 3],
- "milestone_1_name": "v1.0",
- "milestone_2_name": "v2.0",
- "milestone_3_name": "v3.0",
- "milestone_1_date": "Tomorrow",
- "milestone_2_date": "Next week",
- "milestone_3_date": "Next week",
- "csrf_token": csrf_token,
- }
- output = self.app.post(
- "/test/update/milestones", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- # Check the redirect
- self.assertIn(
- "<title>Settings - test - Pagure</title>", output_text
- )
- self.assertIn(
- '<h5 class="pl-2 font-weight-bold text-muted">'
- "Project Settings</h5>\n",
- output_text,
- )
- self.assertIn("Milestones updated", output_text)
- # Check the result of the action -- Milestones updated
- self.session.commit()
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test"
- )
- self.assertEqual(
- repo.milestones,
- {
- "v1.0": {"active": False, "date": "Tomorrow"},
- "v2.0": {"active": False, "date": "Next week"},
- "v3.0": {"active": False, "date": "Next week"},
- },
- )
- # Check for an invalid project
- output = self.app.post("/foo/update/milestones", data=data)
- self.assertEqual(output.status_code, 404)
- # Check the behavior if the project disabled the issue tracker
- settings = repo.settings
- settings["issue_tracker"] = False
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- output = self.app.post("/test/update/milestones", data=data)
- self.assertEqual(output.status_code, 404)
- # Check for a non-admin user
- settings = repo.settings
- settings["issue_tracker"] = True
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- user.username = "ralph"
- with tests.user_set(self.app.application, user):
- output = self.app.post("/test/update/milestones", data=data)
- self.assertEqual(output.status_code, 403)
- @patch("pagure.lib.git.update_git")
- @patch("pagure.lib.notify.send_email")
- def test_milestones_without_dates(self, p_send_email, p_ugt):
- """Test creating two milestones with no dates."""
- tests.create_projects(self.session)
- tests.create_projects_git(os.path.join(self.path, "repos"), bare=True)
- user = tests.FakeUser()
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- # Get the CSRF token
- output = self.app.get("/test/settings")
- output_text = output.get_data(as_text=True)
- csrf_token = output_text.split(
- 'name="csrf_token" type="hidden" value="'
- )[1].split('">')[0]
- data = {
- "milestones": [1, 2],
- "milestone_1_name": "v1.0",
- "milestone_2_name": "v2.0",
- "milestone_1_date": "",
- "milestone_2_date": "",
- "csrf_token": csrf_token,
- }
- output = self.app.post(
- "/test/update/milestones", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- # Check the redirect
- self.assertIn(
- "<title>Settings - test - Pagure</title>", output_text
- )
- self.assertIn(
- '<h5 class="pl-2 font-weight-bold text-muted">'
- "Project Settings</h5>\n",
- output_text,
- )
- self.assertIn("Milestones updated", output_text)
- # Check the result of the action -- Milestones recorded
- self.session.commit()
- repo = pagure.lib.query.get_authorized_project(
- self.session, "test"
- )
- self.assertEqual(
- repo.milestones,
- {
- "v1.0": {"active": False, "date": None},
- "v2.0": {"active": False, "date": None},
- },
- )
- @patch("pagure.lib.git.update_git")
- @patch("pagure.lib.notify.send_email")
- def test_roadmap_ui(self, p_send_email, p_ugt):
- """Test viewing the roadmap of a repo."""
- p_send_email.return_value = True
- p_ugt.return_value = True
- self.test_update_milestones()
- user = tests.FakeUser()
- user.username = "pingou"
- with tests.user_set(self.app.application, user):
- # Get the CSRF token
- output = self.app.get("/test/new_issue")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertTrue(
- '<h4 class="font-weight-bold mb-4">New Issue</h4>\n'
- in output_text
- )
- csrf_token = output_text.split(
- 'name="csrf_token" type="hidden" value="'
- )[1].split('">')[0]
- # Create an unplanned milestone
- data = {
- "milestones": [1, 2, 3],
- "milestone_1_name": "v1.0",
- "milestone_2_name": "v2.0",
- "milestone_3_name": "unplanned",
- "milestone_1_date": "Tomorrow",
- "milestone_2_date": "",
- "milestone_3_date": "",
- "milestone_1_active": True,
- "milestone_2_active": True,
- "milestone_3_active": True,
- "csrf_token": csrf_token,
- }
- output = self.app.post(
- "/test/update/milestones", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- # Check the redirect
- self.assertIn(
- "<title>Settings - test - Pagure</title>", output_text
- )
- self.assertIn(
- '<h5 class="pl-2 font-weight-bold text-muted">'
- "Project Settings</h5>\n",
- output_text,
- )
- self.assertIn("Milestones updated", output_text)
- # Check the result of the action -- Milestones recorded
- self.session.commit()
- repo = pagure.lib.query._get_project(self.session, "test")
- self.assertEqual(
- repo.milestones,
- {
- "unplanned": {"active": True, "date": None},
- "v1.0": {"active": True, "date": "Tomorrow"},
- "v2.0": {"active": True, "date": None},
- },
- )
- # Create the issues
- for cnt in range(6):
- cnt += 1
- data = {
- "title": "Test issue %s" % cnt,
- "issue_content": "We really should improve on this "
- "issue %s" % cnt,
- "csrf_token": csrf_token,
- }
- output = self.app.post(
- "/test/new_issue", data=data, follow_redirects=True
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>Issue #{0}: Test issue {0} - test - "
- "Pagure</title>".format(cnt),
- output_text,
- )
- self.assertIn(
- '<a class="btn btn-outline-secondary btn-sm border-0" '
- 'href="/test/issue/%s/edit" title="Edit this issue">'
- % cnt,
- output_text,
- )
- # Mark the ticket for the roadmap
- mstone = "v%s.0" % cnt
- if cnt >= 3:
- if (cnt % 3) == 0:
- mstone = "unplanned"
- else:
- mstone = "v%s.0" % (cnt % 3)
- data = {"milestone": mstone, "csrf_token": csrf_token}
- output = self.app.post(
- "/test/issue/%s/update" % cnt,
- data=data,
- follow_redirects=True,
- )
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- "<title>Issue #{0}: Test issue {0} - test - "
- "Pagure</title>".format(cnt),
- output_text,
- )
- self.assertIn(
- '<a class="btn btn-outline-secondary btn-sm border-0" '
- 'href="/test/issue/%s/edit" title="Edit this issue">'
- % cnt,
- output_text,
- )
- self.assertIn(
- "Issue set to the milestone: %s" % mstone, output_text
- )
- self.assertIn(
- '<div class="ml-2" id="milestone_plain">', output_text
- )
- self.assertIn(
- '<a href="/test/roadmap/%s/">' % mstone, output_text
- )
- repo = pagure.lib.query.get_authorized_project(self.session, "test")
- # Mark ticket #1 as Fixed
- for iid in [1, 4]:
- ticket = pagure.lib.query.search_issues(
- self.session, repo, issueid=iid
- )
- ticket.status = "Closed"
- ticket.close_status = "Fixed"
- self.session.add(ticket)
- self.session.commit()
- # test the roadmap view
- output = self.app.get("/test/roadmap")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- '<span class="fa fa-fw fa-map-signs"></span>\n'
- ' <span class="font'
- '-weight-bold">v1.0</span>',
- output_text,
- )
- self.assertIn(
- '<span class="fa fa-fw fa-map-signs"></span>\n'
- ' <span class="font'
- '-weight-bold">unplanned</span>',
- output_text,
- )
- self.assertIn(
- 'title="100% Completed | 2 Closed Issues | 0 Open Issues"\n',
- output_text,
- )
- self.assertIn(
- 'title="0% Completed | 0 Closed Issues | 2 Open Issues"\n',
- output_text,
- )
- self.assertIn(
- 'title="0% Completed | 0 Closed Issues | 2 Open Issues"\n',
- output_text,
- )
- # test the roadmap view for a specific milestone
- output = self.app.get("/test/roadmap/v2.0/")
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- '<span class="fa fa-fw fa-exclamation-circle"></span> 2 Open\n',
- output_text,
- )
- self.assertIn(
- '<span class="fa fa-fw fa-exclamation-circle"></span> 0 Closed\n',
- output_text,
- )
- self.assertIn('<a class="notblue" href="/test/issue/2">', output_text)
- self.assertEqual(
- output_text.count('<a class="notblue" href="/test/issue/2">'), 1
- )
- # test the roadmap view for errors
- output = self.app.get("/foo/roadmap")
- self.assertEqual(output.status_code, 404)
- repo = pagure.lib.query.get_authorized_project(self.session, "test")
- settings = repo.settings
- settings["issue_tracker"] = False
- repo.settings = settings
- self.session.add(repo)
- self.session.commit()
- output = self.app.get("/test/roadmap", data=data)
- self.assertEqual(output.status_code, 404)
- if __name__ == "__main__":
- unittest.main(verbosity=2)
|