From 42e9e7b5d0c6cc9a63ad512f299c764b65d14414 Mon Sep 17 00:00:00 2001 From: Lenka Segura Date: Nov 11 2022 15:08:14 +0000 Subject: Add test for give_orphan: not in releng group --- diff --git a/pagure_distgit_tests/test_plugin.py b/pagure_distgit_tests/test_plugin.py index 75e0240..ec1d1a3 100644 --- a/pagure_distgit_tests/test_plugin.py +++ b/pagure_distgit_tests/test_plugin.py @@ -529,3 +529,34 @@ class PagureFlaskApiTakeOrphanEndpointTests(tests.Modeltests): data = json.loads(output.get_data(as_text=True)) assert data["error"] == "You are not allowed to modify this project" assert data["error_code"] == "EMODIFYPROJECTNOTALLOWED" + + @patch("pagure_distgit.plugin._is_active_in_pdc") + @patch("pagure_distgit.plugin.pagure.lib.notify.log") + def test_give_orphan_user_not_in_releng_or_admin_group(self, mock_log, mock_pdc): + """Assert that point of contact is correctly updated.""" + mock_pdc.return_value = True + headers = {"Authorization": "token aaabbbcccddd"} + repo = pagure.lib.query.get_authorized_project( + self.session, + "test4", + namespace="rpms", + ) + pagure.lib.query.delete_user_of_group( + self.session, + "pingou", + "rel-eng", + "pingou", + False, + force=True, + ) + self.session.commit() + + output = self.app.post( + "/_dg/give_orphan/rpms/test4?user=foo", + headers=headers, + ) + + data = json.loads(output.get_data(as_text=True)) + assert output.status_code == 403 + assert data["errors"] == "You must be in rel-eng or admin group to assign a package." + assert data["error_code"] == "ENOTHIGHENOUGH"