File tree Expand file tree Collapse file tree 2 files changed +8
-6
lines changed
lms/djangoapps/discussion/rest_api Expand file tree Collapse file tree 2 files changed +8
-6
lines changed Original file line number Diff line number Diff line change @@ -152,14 +152,12 @@ def get_muted_user_ids(request_user, course_key):
152152 set: Set of user IDs that are muted (personal + course-wide)
153153 """
154154 try :
155- # Import forum integration here to avoid circular imports
156155 muted_ids = ForumIntegrationService .get_muted_user_ids_for_course (
157156 course_id = str (course_key ),
158157 viewer_id = request_user .id
159158 )
160- return muted_ids
161-
162- except Exception as e : # pylint: disable=broad-except
159+ return set (muted_ids ) if muted_ids else set ()
160+ except Exception as e : # pylint: disable=broad-exception-caught
163161 return set ()
164162
165163
Original file line number Diff line number Diff line change 1212import json
1313from datetime import datetime
1414from unittest import mock
15+ from unittest .mock import patch
1516
1617import ddt
1718from forum .backends .mongodb .comments import Comment
@@ -216,7 +217,6 @@ def test_basic(self):
216217 "closed" : False ,
217218 "pinned" : False ,
218219 "editing_user_id" : str (self .user .id ),
219- "include_muted" : None ,
220220 }
221221 self .check_mock_called_with ("update_thread" , - 1 , ** params )
222222
@@ -524,7 +524,11 @@ def test_404(self):
524524 {"developer_message" : "Course not found." }
525525 )
526526
527- def test_basic (self ):
527+ @patch (
528+ "lms.djangoapps.discussion.rest_api.api.ForumIntegrationService.get_muted_user_ids_for_course" ,
529+ return_value = set ()
530+ )
531+ def test_basic (self , mock_get_muted_user_ids ):
528532 self .register_get_user_response (self .user , upvoted_ids = ["test_thread" ])
529533 source_threads = [
530534 self .create_source_thread (
You can’t perform that action at this time.
0 commit comments