diff --git a/tests/test_progit_lib.py b/tests/test_progit_lib.py index 0f00f10..50df3ef 100644 --- a/tests/test_progit_lib.py +++ b/tests/test_progit_lib.py @@ -146,6 +146,8 @@ class PagureLibtests(tests.Modeltests): # Before issues = pagure.lib.search_issues(self.session, repo) self.assertEqual(len(issues), 0) + self.assertEqual(len(repo.open_tickets), 0) + self.assertEqual(len(repo.open_tickets_public), 0) # See where it fails self.assertRaises( @@ -193,6 +195,8 @@ class PagureLibtests(tests.Modeltests): ) self.session.commit() self.assertEqual(msg.title, 'Test issue') + self.assertEqual(len(repo.open_tickets), 1) + self.assertEqual(len(repo.open_tickets_public), 1) msg = pagure.lib.new_issue( session=self.session, @@ -205,6 +209,8 @@ class PagureLibtests(tests.Modeltests): ) self.session.commit() self.assertEqual(msg.title, 'Test issue #2') + self.assertEqual(len(repo.open_tickets), 2) + self.assertEqual(len(repo.open_tickets_public), 2) # After issues = pagure.lib.search_issues(self.session, repo) @@ -218,9 +224,13 @@ class PagureLibtests(tests.Modeltests): p_ugt.return_value = True self.test_new_issue() + repo = pagure.lib.get_project(self.session, 'test') issue = pagure.lib.search_issues(self.session, repo, issueid=2) + self.assertEqual(len(repo.open_tickets), 2) + self.assertEqual(len(repo.open_tickets_public), 2) + # Edit the issue msg = pagure.lib.edit_issue( session=self.session, @@ -250,11 +260,14 @@ class PagureLibtests(tests.Modeltests): title='Foo issue #2', content='We should work on this period', status='Invalid', - private=True + private=True, ) self.session.commit() self.assertEqual(msg, 'Successfully edited issue #2') + self.assertEqual(len(repo.open_tickets), 1) + self.assertEqual(len(repo.open_tickets_public), 1) + @patch('pagure.lib.git.update_git') @patch('pagure.lib.notify.send_email') def test_add_issue_dependency(self, p_send_email, p_ugt): @@ -1274,6 +1287,8 @@ class PagureLibtests(tests.Modeltests): # Add an extra user to project `foo` repo = pagure.lib.get_project(self.session, 'test') + self.assertEqual(len(repo.open_requests), 0) + msg = pagure.lib.add_user_to_project( session=self.session, project=repo, @@ -1300,6 +1315,7 @@ class PagureLibtests(tests.Modeltests): self.session.commit() self.assertEqual(req.id, 1) self.assertEqual(req.title, 'test pull-request') + self.assertEqual(len(repo.open_requests), 1) @patch('pagure.lib.notify.send_email') def test_add_pull_request_comment(self, mockemail): @@ -1435,6 +1451,8 @@ class PagureLibtests(tests.Modeltests): self.test_new_pull_request() + repo = pagure.lib.get_project(self.session, 'test') + self.assertEqual(len(repo.open_requests), 1) request = pagure.lib.search_pull_requests(self.session, requestid=1) pagure.lib.close_pull_request( @@ -1445,6 +1463,8 @@ class PagureLibtests(tests.Modeltests): merged=True, ) self.session.commit() + repo = pagure.lib.get_project(self.session, 'test') + self.assertEqual(len(repo.open_requests), 0) prs = pagure.lib.search_pull_requests( session=self.session, @@ -1605,6 +1625,9 @@ class PagureLibtests(tests.Modeltests): repo = pagure.lib.get_project(self.session, 'test') issue = pagure.lib.search_issues(self.session, repo, issueid=1) + self.assertEqual(len(repo.open_tickets), 2) + self.assertEqual(len(repo.open_tickets_public), 2) + # Create issues to play with msg = pagure.lib.new_issue( session=self.session, @@ -1618,6 +1641,9 @@ class PagureLibtests(tests.Modeltests): self.session.commit() self.assertEqual(msg.title, 'Test issue #3') + self.assertEqual(len(repo.open_tickets), 3) + self.assertEqual(len(repo.open_tickets_public), 2) + # before self.assertEqual(issue.tags_text, []) self.assertEqual(issue.depends_text, [])