From e02d809f3bc0f140d301e24e6bb4da07a510e5ca Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Michal=20Kone=C4=8Dn=C3=BD?= Date: Thu, 25 Aug 2022 12:39:53 +0200 Subject: [PATCH] Use correct HTTP method when updating pagure project MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The correct method is PATCH not POST. Signed-off-by: Michal Konečný --- tests/utils/test_pagure.py | 8 ++++---- toddlers/utils/pagure.py | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/utils/test_pagure.py b/tests/utils/test_pagure.py index e89d22f..bb5e25b 100644 --- a/tests/utils/test_pagure.py +++ b/tests/utils/test_pagure.py @@ -886,7 +886,7 @@ class TestPagureChangeProjectMainAdmin: response_mock = Mock() response_mock.status_code = 200 - self.pagure._requests_session.post.return_value = response_mock + self.pagure._requests_session.patch.return_value = response_mock namespace = "namespace" repo = "repo" @@ -894,7 +894,7 @@ class TestPagureChangeProjectMainAdmin: self.pagure.change_project_main_admin(namespace, repo, main_admin) - self.pagure._requests_session.post.assert_called_with( + self.pagure._requests_session.patch.assert_called_with( "https://pagure.io/api/0/{0}/{1}".format(namespace, repo), data=json.dumps({"main_admin": main_admin}), headers=self.pagure.get_auth_header(), @@ -907,7 +907,7 @@ class TestPagureChangeProjectMainAdmin: response_mock = Mock() response_mock.status_code = 500 - self.pagure._requests_session.post.side_effect = response_mock + self.pagure._requests_session.patch.side_effect = response_mock namespace = "namespace" repo = "repo" @@ -920,7 +920,7 @@ class TestPagureChangeProjectMainAdmin: with pytest.raises(PagureError, match=expected_error): self.pagure.change_project_main_admin(namespace, repo, main_admin) - self.pagure._requests_session.post.assert_called_with( + self.pagure._requests_session.patch.assert_called_with( "https://pagure.io/api/0/{0}/{1}".format(namespace, repo), data=json.dumps({"main_admin": main_admin}), headers=self.pagure.get_auth_header(), diff --git a/toddlers/utils/pagure.py b/toddlers/utils/pagure.py index e3e55ee..6ccb99e 100644 --- a/toddlers/utils/pagure.py +++ b/toddlers/utils/pagure.py @@ -443,7 +443,7 @@ class Pagure: new_main_admin, namespace, repo ) ) - response = self._requests_session.post( + response = self._requests_session.patch( admin_api_url, data=json.dumps(payload), headers=headers )