diff --git a/kallithea/controllers/pullrequests.py b/kallithea/controllers/pullrequests.py --- a/kallithea/controllers/pullrequests.py +++ b/kallithea/controllers/pullrequests.py @@ -356,7 +356,7 @@ class PullrequestsController(BaseRepoCon # requested destination and have the exact ancestor other_ref = '%s:%s:%s' % (other_ref_type, other_ref_name, ancestor_rev) - reviewers = _form['review_members'] + reviewers = [] title = _form['pullrequest_title'] if not title: diff --git a/kallithea/model/forms.py b/kallithea/model/forms.py --- a/kallithea/model/forms.py +++ b/kallithea/model/forms.py @@ -524,7 +524,6 @@ def PullRequestForm(repo_id): org_ref = v.UnicodeString(strip=True, required=True) other_repo = v.UnicodeString(strip=True, required=True) other_ref = v.UnicodeString(strip=True, required=True) - review_members = v.Set() pullrequest_title = v.UnicodeString(strip=True, required=True) pullrequest_desc = v.UnicodeString(strip=True, required=False) diff --git a/kallithea/tests/functional/test_pullrequests.py b/kallithea/tests/functional/test_pullrequests.py --- a/kallithea/tests/functional/test_pullrequests.py +++ b/kallithea/tests/functional/test_pullrequests.py @@ -33,43 +33,6 @@ class TestPullrequestsController(TestCon assert response.status == '200 OK' response.mustcontain('This pull request has already been merged to default.') - def test_create_with_existing_reviewer(self): - self.log_user() - response = self.app.post(url(controller='pullrequests', action='create', - repo_name=HG_REPO), - {'org_repo': HG_REPO, - 'org_ref': 'branch:default:default', - 'other_repo': HG_REPO, - 'other_ref': 'branch:default:default', - 'pullrequest_title': 'title', - 'pullrequest_desc': 'description', - '_authentication_token': self.authentication_token(), - 'review_members': TEST_USER_ADMIN_LOGIN, - } - ) - assert response.status == '302 Found' - response = response.follow() - assert response.status == '200 OK' - response.mustcontain('This pull request has already been merged to default.') - - def test_create_with_invalid_reviewer(self): - invalid_user_name = 'invalid_user' - self.log_user() - response = self.app.post(url(controller='pullrequests', action='create', - repo_name=HG_REPO), - { - 'org_repo': HG_REPO, - 'org_ref': 'branch:default:default', - 'other_repo': HG_REPO, - 'other_ref': 'branch:default:default', - 'pullrequest_title': 'title', - 'pullrequest_desc': 'description', - '_authentication_token': self.authentication_token(), - 'review_members': invalid_user_name, - }, - status=400) - response.mustcontain('Invalid reviewer "%s" specified' % invalid_user_name) - def test_update_with_invalid_reviewer(self): invalid_user_id = 99999 self.log_user()