From 10ad48b8c737e83a2861aab8542c71ff160d7588 Mon Sep 17 00:00:00 2001 From: rsashank Date: Fri, 20 Sep 2024 19:29:54 +0530 Subject: [PATCH] views: Remove muted_users from user_list. Skip user from showing in user list if user is muted. Test updated. Co-authored by: Subhasish-Behera --- tests/ui/test_ui_tools.py | 2 ++ zulipterminal/ui_tools/views.py | 2 ++ 2 files changed, 4 insertions(+) diff --git a/tests/ui/test_ui_tools.py b/tests/ui/test_ui_tools.py index 0a4d9ec030..419838d2dd 100644 --- a/tests/ui/test_ui_tools.py +++ b/tests/ui/test_ui_tools.py @@ -1078,6 +1078,8 @@ def test_users_view(self, users, users_btn_len, editor_mode, status, mocker): user_btn = mocker.patch(VIEWS + ".UserButton") users_view = mocker.patch(VIEWS + ".UsersView") right_col_view = RightColumnView(self.view) + mocker.patch("zulipterminal.model.Model.is_muted_user", return_value=False) + self.view.model.is_muted_user.return_value = False if status != "inactive": user_btn.assert_called_once_with( user=self.view.users[0], diff --git a/zulipterminal/ui_tools/views.py b/zulipterminal/ui_tools/views.py index 6d01a82566..2285f72c1f 100644 --- a/zulipterminal/ui_tools/views.py +++ b/zulipterminal/ui_tools/views.py @@ -726,6 +726,8 @@ def users_view(self, users: Any = None) -> Any: users_btn_list = list() for user in users: + if self.view.model.is_muted_user(user["user_id"]): + continue status = user["status"] # Only include `inactive` users in search result. if status == "inactive" and not self.view.controller.is_in_editor_mode():