diff --git a/pagure/lib/__init__.py b/pagure/lib/__init__.py index 5008787..17759db 100644 --- a/pagure/lib/__init__.py +++ b/pagure/lib/__init__.py @@ -1112,9 +1112,7 @@ def add_pull_request_comment(session, request, commit, tree_id, filename, # Send notification for the event-source server if REDIS and not request.project.private: - comment_text = pr_comment.comment - if not notification: - comment_text = text2markdown(pr_comment.comment) + comment_text = text2markdown(pr_comment.comment) REDIS.publish('pagure.%s' % request.uid, json.dumps({ 'request_id': request.id, diff --git a/pagure/pfmarkdown.py b/pagure/pfmarkdown.py index 9566d19..ac6954d 100644 --- a/pagure/pfmarkdown.py +++ b/pagure/pfmarkdown.py @@ -316,7 +316,8 @@ def makeExtension(*arg, **kwargs): def _issue_exists(user, namespace, repo, idx): """ Utility method checking if a given issue exists. """ - repo_obj = pagure.get_authorized_project(pagure.SESSION, repo=repo, user=user, namespace=namespace) + repo_obj = pagure.get_authorized_project(pagure.SESSION, project_name=repo, + user=user, namespace=namespace) if not repo_obj: return False @@ -331,7 +332,8 @@ def _issue_exists(user, namespace, repo, idx): def _pr_exists(user, namespace, repo, idx): """ Utility method checking if a given PR exists. """ - repo_obj = pagure.get_authorized_project(pagure.SESSION, repo=repo, user=user, namespace=namespace) + repo_obj = pagure.get_authorized_project(pagure.SESSION, project_name=repo, + user=user, namespace=namespace) if not repo_obj: return False diff --git a/tests/test_pagure_lib.py b/tests/test_pagure_lib.py index 5600014..fbe35f9 100644 --- a/tests/test_pagure_lib.py +++ b/tests/test_pagure_lib.py @@ -723,7 +723,6 @@ class PagureLibtests(tests.Modeltests): ticketfolder=None, ) self.session.commit() - print msgs self.assertEqual( msgs, ['Edited tag: tag1()[DeepSkyBlue] to tag2(lorem ipsum)[black]'] diff --git a/tests/test_pagure_lib_model.py b/tests/test_pagure_lib_model.py index f2c1b79..ffdc3e4 100644 --- a/tests/test_pagure_lib_model.py +++ b/tests/test_pagure_lib_model.py @@ -147,7 +147,7 @@ class PagureLibModeltests(tests.Modeltests): def test_tagissuecolor__repr__(self): """ Test the TagIssue.__repr__ function of pagure.lib.model. """ self.test_issue__repr__() - repo = pagure.lib.get_project(self.session, 'test') + repo = pagure.get_authorized_project(self.session, 'test') issues = pagure.lib.search_issues(self.session, repo) self.assertEqual(len(issues), 1)