From 50175ffb11b1d6dda9a5ccbb3c6d455b79b17b49 Mon Sep 17 00:00:00 2001 From: Shaily Date: Nov 26 2017 05:34:10 +0000 Subject: Add test to check fedmsg on remove_user --- diff --git a/tests/test_pagure_flask_ui_repo.py b/tests/test_pagure_flask_ui_repo.py index 95fa50e..ea20d0e 100644 --- a/tests/test_pagure_flask_ui_repo.py +++ b/tests/test_pagure_flask_ui_repo.py @@ -625,7 +625,8 @@ class PagureFlaskRepotests(tests.Modeltests): self.assertEqual(output.status_code, 404) @patch('pagure.ui.repo.admin_session_timedout') - def test_remove_user(self, ast): + @patch('pagure.lib.notify.log') + def test_remove_user(self, mock_log, ast): """ Test the remove_user endpoint. """ ast.return_value = False @@ -710,6 +711,8 @@ class PagureFlaskRepotests(tests.Modeltests): repo = pagure.get_authorized_project(self.session, 'test') self.assertEqual(len(repo.users), 0) + mock_log.assert_called_with(ANY, topic='project.user.removed', msg=ANY) + @patch('pagure.ui.repo.admin_session_timedout') def test_remove_group_project_when_user_mngt_off(self, ast): """ Test the remove_group_project endpoint when user management is