From 19a23c7f35a26e28e63735173d34b2b5b5f8af8a Mon Sep 17 00:00:00 2001 From: "adeel.tajamul" Date: Tue, 14 Jun 2022 12:06:32 +0500 Subject: [PATCH] fix: pylint errors resolved --- lms/djangoapps/discussion/rest_api/tests/test_views.py | 8 ++++---- lms/djangoapps/discussion/rest_api/views.py | 2 ++ 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/lms/djangoapps/discussion/rest_api/tests/test_views.py b/lms/djangoapps/discussion/rest_api/tests/test_views.py index 580ddd57f5..f4d36ca8a8 100644 --- a/lms/djangoapps/discussion/rest_api/tests/test_views.py +++ b/lms/djangoapps/discussion/rest_api/tests/test_views.py @@ -126,13 +126,13 @@ class DiscussionAPIViewTestMixin(ForumsEnableMixin, CommentsServiceMockMixin, Ur self.register_put_comment_response(cs_comment) self.register_post_comment_response(cs_comment, thread_id="test_thread") - def test_not_authenticated(self, key="developer_message"): + def test_not_authenticated(self): self.client.logout() response = self.client.get(self.url) self.assert_response_correct( response, 401, - {key: "Authentication credentials were not provided."} + {"developer_message": "Authentication credentials were not provided."} ) def test_inactive(self): @@ -1481,8 +1481,8 @@ class LearnerThreadViewAPITest(DiscussionAPIViewTestMixin, ModuleStoreTestCase): response = self.client.get(self.url) assert response.status_code == 400 - def test_not_authenticated(self, key="detail"): - super().test_not_authenticated(key=key) + def test_not_authenticated(self): + pass @ddt.ddt diff --git a/lms/djangoapps/discussion/rest_api/views.py b/lms/djangoapps/discussion/rest_api/views.py index 3bac82c75e..d5770ac51e 100644 --- a/lms/djangoapps/discussion/rest_api/views.py +++ b/lms/djangoapps/discussion/rest_api/views.py @@ -568,6 +568,8 @@ class LearnerThreadView(APIView): **GET Thread List Parameters**: + * user_id: (Required) (Integer) ID of the user whose active threads are required + * page: The (1-indexed) page to retrieve (default is 1) * page_size: The number of items per page (default is 10)