From ed83f2b1ab063fd8d18539ca6bc413bf4aea5a9e 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 | 1 + zulipterminal/ui_tools/views.py | 4 ++++ 2 files changed, 5 insertions(+) diff --git a/tests/ui/test_ui_tools.py b/tests/ui/test_ui_tools.py index 0a4d9ec030..d6c68577be 100644 --- a/tests/ui/test_ui_tools.py +++ b/tests/ui/test_ui_tools.py @@ -1078,6 +1078,7 @@ 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) + 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..4134f7b1b1 100644 --- a/zulipterminal/ui_tools/views.py +++ b/zulipterminal/ui_tools/views.py @@ -726,6 +726,10 @@ def users_view(self, users: Any = None) -> Any: users_btn_list = list() for user in users: + with open("tx.txt", "a") as f: + f.write(str(self.view.model.is_muted_user(user["user_id"]))) + 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():