From e0f1d3f3f3e40fc110d36d5109a56a328b9559e9 Mon Sep 17 00:00:00 2001 From: Sarina Canelake Date: Sat, 29 Nov 2014 23:01:00 -0500 Subject: [PATCH] s/pylint: disable=C0111/pylint: disable=missing-docstring/ --- .../contentstore/features/advanced_settings.py | 2 +- cms/djangoapps/contentstore/features/checklists.py | 2 +- cms/djangoapps/contentstore/features/common.py | 2 +- cms/djangoapps/contentstore/features/component.py | 2 +- .../features/component_settings_editor_helpers.py | 2 +- .../contentstore/features/course-export.py | 2 +- .../contentstore/features/course-outline.py | 2 +- .../contentstore/features/course-settings.py | 2 +- .../contentstore/features/course-team.py | 2 +- .../contentstore/features/course-updates.py | 2 +- .../contentstore/features/course_import.py | 2 +- cms/djangoapps/contentstore/features/courses.py | 2 +- .../contentstore/features/discussion-editor.py | 2 +- cms/djangoapps/contentstore/features/grading.py | 2 +- cms/djangoapps/contentstore/features/help.py | 2 +- .../contentstore/features/html-editor.py | 2 +- cms/djangoapps/contentstore/features/pages.py | 2 +- .../contentstore/features/problem-editor.py | 2 +- cms/djangoapps/contentstore/features/signup.py | 2 +- cms/djangoapps/contentstore/features/textbooks.py | 2 +- .../contentstore/features/transcripts.py | 2 +- cms/djangoapps/contentstore/features/upload.py | 2 +- .../contentstore/features/video_editor.py | 2 +- .../contentstore/features/video_handout.py | 2 +- cms/djangoapps/contentstore/views/error.py | 2 +- common/djangoapps/enrollment/serializers.py | 2 +- common/djangoapps/student/tests/factories.py | 2 +- common/djangoapps/terrain/course_helpers.py | 2 +- common/djangoapps/terrain/steps.py | 2 +- common/djangoapps/terrain/ui_helpers.py | 2 +- common/djangoapps/user_api/tests/factories.py | 2 +- .../modulestore/tests/persistent_factories.py | 2 +- .../lib/xmodule/xmodule/tests/test_capa_module.py | 2 +- .../lib/xmodule/xmodule/tests/test_xml_module.py | 2 +- .../xmodule/video_module/transcripts_utils.py | 8 ++++---- lms/djangoapps/bulk_email/forms.py | 4 ++-- lms/djangoapps/bulk_email/models.py | 4 ++-- lms/djangoapps/courseware/features/certificates.py | 2 +- lms/djangoapps/courseware/features/common.py | 2 +- lms/djangoapps/courseware/features/conditional.py | 2 +- lms/djangoapps/courseware/features/courseware.py | 2 +- .../courseware/features/courseware_common.py | 2 +- lms/djangoapps/courseware/features/events.py | 2 +- lms/djangoapps/courseware/features/homepage.py | 2 +- lms/djangoapps/courseware/features/login.py | 2 +- lms/djangoapps/courseware/features/lti.py | 2 +- lms/djangoapps/courseware/features/navigation.py | 2 +- lms/djangoapps/courseware/features/openended.py | 2 +- lms/djangoapps/courseware/features/problems.py | 2 +- .../courseware/features/problems_setup.py | 2 +- lms/djangoapps/courseware/features/registration.py | 2 +- lms/djangoapps/courseware/features/signup.py | 2 +- lms/djangoapps/courseware/features/video.py | 2 +- lms/djangoapps/courseware/features/word_cloud.py | 2 +- lms/djangoapps/courseware/tests/factories.py | 2 +- lms/djangoapps/courseware/tests/test_access.py | 2 +- lms/djangoapps/dashboard/tests/test_support.py | 2 +- lms/djangoapps/django_comment_client/base/tests.py | 2 +- .../django_comment_client/forum/tests.py | 2 +- lms/djangoapps/instructor/features/bulk_email.py | 2 +- lms/djangoapps/instructor/features/common.py | 2 +- .../instructor/features/data_download.py | 2 +- .../instructor/tests/test_hint_manager.py | 2 +- lms/djangoapps/instructor/tests/test_legacy_xss.py | 2 +- lms/djangoapps/instructor/views/api.py | 14 +++++++------- lms/djangoapps/mobile_api/users/serializers.py | 4 ++-- lms/djangoapps/shoppingcart/exceptions.py | 2 +- lms/djangoapps/survey/admin.py | 2 +- lms/djangoapps/verify_student/exceptions.py | 2 +- 69 files changed, 81 insertions(+), 81 deletions(-) diff --git a/cms/djangoapps/contentstore/features/advanced_settings.py b/cms/djangoapps/contentstore/features/advanced_settings.py index 87b00610b5..242fde629a 100644 --- a/cms/djangoapps/contentstore/features/advanced_settings.py +++ b/cms/djangoapps/contentstore/features/advanced_settings.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from lettuce import world, step diff --git a/cms/djangoapps/contentstore/features/checklists.py b/cms/djangoapps/contentstore/features/checklists.py index 92c39cc0cd..9b1d427674 100644 --- a/cms/djangoapps/contentstore/features/checklists.py +++ b/cms/djangoapps/contentstore/features/checklists.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from lettuce import world, step diff --git a/cms/djangoapps/contentstore/features/common.py b/cms/djangoapps/contentstore/features/common.py index 023a03f1f3..b0681433b0 100644 --- a/cms/djangoapps/contentstore/features/common.py +++ b/cms/djangoapps/contentstore/features/common.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name import os diff --git a/cms/djangoapps/contentstore/features/component.py b/cms/djangoapps/contentstore/features/component.py index 93df2be2ec..04d5680ef4 100644 --- a/cms/djangoapps/contentstore/features/component.py +++ b/cms/djangoapps/contentstore/features/component.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name # Lettuce formats proposed definitions for unimplemented steps with the diff --git a/cms/djangoapps/contentstore/features/component_settings_editor_helpers.py b/cms/djangoapps/contentstore/features/component_settings_editor_helpers.py index 749f611469..4cd814acd8 100644 --- a/cms/djangoapps/contentstore/features/component_settings_editor_helpers.py +++ b/cms/djangoapps/contentstore/features/component_settings_editor_helpers.py @@ -1,5 +1,5 @@ # disable missing docstring -# pylint: disable=C0111 +# pylint: disable=missing-docstring from lettuce import world from nose.tools import assert_equal, assert_in # pylint: disable=E0611 diff --git a/cms/djangoapps/contentstore/features/course-export.py b/cms/djangoapps/contentstore/features/course-export.py index 0077c17bfa..9c8b09f5ba 100644 --- a/cms/djangoapps/contentstore/features/course-export.py +++ b/cms/djangoapps/contentstore/features/course-export.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name # pylint: disable=W0613 diff --git a/cms/djangoapps/contentstore/features/course-outline.py b/cms/djangoapps/contentstore/features/course-outline.py index 94e41b33e5..141aaa1724 100644 --- a/cms/djangoapps/contentstore/features/course-outline.py +++ b/cms/djangoapps/contentstore/features/course-outline.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from lettuce import world, step diff --git a/cms/djangoapps/contentstore/features/course-settings.py b/cms/djangoapps/contentstore/features/course-settings.py index 44c2b69c45..bc307e5072 100644 --- a/cms/djangoapps/contentstore/features/course-settings.py +++ b/cms/djangoapps/contentstore/features/course-settings.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from lettuce import world, step diff --git a/cms/djangoapps/contentstore/features/course-team.py b/cms/djangoapps/contentstore/features/course-team.py index 2c7abd40b0..855a6d143a 100644 --- a/cms/djangoapps/contentstore/features/course-team.py +++ b/cms/djangoapps/contentstore/features/course-team.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from lettuce import world, step diff --git a/cms/djangoapps/contentstore/features/course-updates.py b/cms/djangoapps/contentstore/features/course-updates.py index 064b4eb5a5..6285c61678 100644 --- a/cms/djangoapps/contentstore/features/course-updates.py +++ b/cms/djangoapps/contentstore/features/course-updates.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from lettuce import world, step diff --git a/cms/djangoapps/contentstore/features/course_import.py b/cms/djangoapps/contentstore/features/course_import.py index f3090a12ee..80365d3884 100644 --- a/cms/djangoapps/contentstore/features/course_import.py +++ b/cms/djangoapps/contentstore/features/course_import.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name # pylint: disable=W0613 diff --git a/cms/djangoapps/contentstore/features/courses.py b/cms/djangoapps/contentstore/features/courses.py index 2f2f47b30c..7b3ccbcbd2 100644 --- a/cms/djangoapps/contentstore/features/courses.py +++ b/cms/djangoapps/contentstore/features/courses.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from lettuce import world, step diff --git a/cms/djangoapps/contentstore/features/discussion-editor.py b/cms/djangoapps/contentstore/features/discussion-editor.py index bca91f1a74..1a04ec0e09 100644 --- a/cms/djangoapps/contentstore/features/discussion-editor.py +++ b/cms/djangoapps/contentstore/features/discussion-editor.py @@ -1,5 +1,5 @@ # disable missing docstring -# pylint: disable=C0111 +# pylint: disable=missing-docstring from lettuce import world, step diff --git a/cms/djangoapps/contentstore/features/grading.py b/cms/djangoapps/contentstore/features/grading.py index b953b66e24..e10fb39a4f 100644 --- a/cms/djangoapps/contentstore/features/grading.py +++ b/cms/djangoapps/contentstore/features/grading.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from lettuce import world, step diff --git a/cms/djangoapps/contentstore/features/help.py b/cms/djangoapps/contentstore/features/help.py index ffa2cbd3d0..25ad14aa99 100644 --- a/cms/djangoapps/contentstore/features/help.py +++ b/cms/djangoapps/contentstore/features/help.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name # pylint: disable=W0613 diff --git a/cms/djangoapps/contentstore/features/html-editor.py b/cms/djangoapps/contentstore/features/html-editor.py index 978487533e..98d57f477d 100644 --- a/cms/djangoapps/contentstore/features/html-editor.py +++ b/cms/djangoapps/contentstore/features/html-editor.py @@ -1,5 +1,5 @@ # disable missing docstring -# pylint: disable=C0111 +# pylint: disable=missing-docstring from collections import OrderedDict diff --git a/cms/djangoapps/contentstore/features/pages.py b/cms/djangoapps/contentstore/features/pages.py index 983c005192..36d3a85df1 100644 --- a/cms/djangoapps/contentstore/features/pages.py +++ b/cms/djangoapps/contentstore/features/pages.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name # pylint: disable=W0613 diff --git a/cms/djangoapps/contentstore/features/problem-editor.py b/cms/djangoapps/contentstore/features/problem-editor.py index 7ad4675f6f..c7b3dd3069 100644 --- a/cms/djangoapps/contentstore/features/problem-editor.py +++ b/cms/djangoapps/contentstore/features/problem-editor.py @@ -1,5 +1,5 @@ # disable missing docstring -# pylint: disable=C0111 +# pylint: disable=missing-docstring import json from lettuce import world, step diff --git a/cms/djangoapps/contentstore/features/signup.py b/cms/djangoapps/contentstore/features/signup.py index 5e4fc6c3d6..26374fe963 100644 --- a/cms/djangoapps/contentstore/features/signup.py +++ b/cms/djangoapps/contentstore/features/signup.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from lettuce import world, step diff --git a/cms/djangoapps/contentstore/features/textbooks.py b/cms/djangoapps/contentstore/features/textbooks.py index 4be767d776..4cfba5dca5 100644 --- a/cms/djangoapps/contentstore/features/textbooks.py +++ b/cms/djangoapps/contentstore/features/textbooks.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from lettuce import world, step diff --git a/cms/djangoapps/contentstore/features/transcripts.py b/cms/djangoapps/contentstore/features/transcripts.py index 11e815c155..8047252417 100644 --- a/cms/djangoapps/contentstore/features/transcripts.py +++ b/cms/djangoapps/contentstore/features/transcripts.py @@ -1,5 +1,5 @@ # disable missing docstring -# pylint: disable=C0111 +# pylint: disable=missing-docstring import os from lettuce import world, step diff --git a/cms/djangoapps/contentstore/features/upload.py b/cms/djangoapps/contentstore/features/upload.py index 84ec55c956..bd483ca3a7 100644 --- a/cms/djangoapps/contentstore/features/upload.py +++ b/cms/djangoapps/contentstore/features/upload.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from lettuce import world, step diff --git a/cms/djangoapps/contentstore/features/video_editor.py b/cms/djangoapps/contentstore/features/video_editor.py index a515f63ec6..687f65e898 100644 --- a/cms/djangoapps/contentstore/features/video_editor.py +++ b/cms/djangoapps/contentstore/features/video_editor.py @@ -1,6 +1,6 @@ # -*- coding: utf-8 -*- # disable missing docstring -# pylint: disable=C0111 +# pylint: disable=missing-docstring import requests from lettuce import world, step diff --git a/cms/djangoapps/contentstore/features/video_handout.py b/cms/djangoapps/contentstore/features/video_handout.py index 63a9e42c9e..9f3aa2c090 100644 --- a/cms/djangoapps/contentstore/features/video_handout.py +++ b/cms/djangoapps/contentstore/features/video_handout.py @@ -1,6 +1,6 @@ # -*- coding: utf-8 -*- # disable missing docstring -# pylint: disable=C0111 +# pylint: disable=missing-docstring from lettuce import world, step from nose.tools import assert_true # pylint: disable=E0611 diff --git a/cms/djangoapps/contentstore/views/error.py b/cms/djangoapps/contentstore/views/error.py index 40e117fc18..2400d63b66 100644 --- a/cms/djangoapps/contentstore/views/error.py +++ b/cms/djangoapps/contentstore/views/error.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111,W0613 +# pylint: disable=missing-docstring,W0613 from django.http import (HttpResponse, HttpResponseServerError, HttpResponseNotFound) diff --git a/common/djangoapps/enrollment/serializers.py b/common/djangoapps/enrollment/serializers.py index 7ec8b6bc16..e95df0943a 100644 --- a/common/djangoapps/enrollment/serializers.py +++ b/common/djangoapps/enrollment/serializers.py @@ -60,7 +60,7 @@ class CourseEnrollmentSerializer(serializers.ModelSerializer): """Retrieves the username from the associated model.""" return model.username - class Meta: # pylint: disable=C0111 + class Meta: # pylint: disable=missing-docstring model = CourseEnrollment fields = ('created', 'mode', 'is_active', 'course', 'student') lookup_field = 'username' diff --git a/common/djangoapps/student/tests/factories.py b/common/djangoapps/student/tests/factories.py index e419b56ea3..1c99eead81 100644 --- a/common/djangoapps/student/tests/factories.py +++ b/common/djangoapps/student/tests/factories.py @@ -13,7 +13,7 @@ from pytz import UTC from opaque_keys.edx.locations import SlashSeparatedCourseKey # Factories don't have __init__ methods, and are self documenting -# pylint: disable=W0232, C0111 +# pylint: disable=W0232, missing-docstring class GroupFactory(DjangoModelFactory): diff --git a/common/djangoapps/terrain/course_helpers.py b/common/djangoapps/terrain/course_helpers.py index 5edabf2e37..f13f10b0b9 100644 --- a/common/djangoapps/terrain/course_helpers.py +++ b/common/djangoapps/terrain/course_helpers.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name import urllib diff --git a/common/djangoapps/terrain/steps.py b/common/djangoapps/terrain/steps.py index e44f36de5b..0a5db23754 100644 --- a/common/djangoapps/terrain/steps.py +++ b/common/djangoapps/terrain/steps.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name # Disable the "wildcard import" warning so we can bring in all methods from diff --git a/common/djangoapps/terrain/ui_helpers.py b/common/djangoapps/terrain/ui_helpers.py index 891e6bf6af..b26b311326 100644 --- a/common/djangoapps/terrain/ui_helpers.py +++ b/common/djangoapps/terrain/ui_helpers.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from lettuce import world diff --git a/common/djangoapps/user_api/tests/factories.py b/common/djangoapps/user_api/tests/factories.py index 3308c2f2ab..25be6bf188 100644 --- a/common/djangoapps/user_api/tests/factories.py +++ b/common/djangoapps/user_api/tests/factories.py @@ -7,7 +7,7 @@ from opaque_keys.edx.locations import SlashSeparatedCourseKey # Factories don't have __init__ methods, and are self documenting -# pylint: disable=W0232, C0111 +# pylint: disable=W0232, missing-docstring class UserPreferenceFactory(DjangoModelFactory): FACTORY_FOR = UserPreference diff --git a/common/lib/xmodule/xmodule/modulestore/tests/persistent_factories.py b/common/lib/xmodule/xmodule/modulestore/tests/persistent_factories.py index a292212a1a..a98400ee4a 100644 --- a/common/lib/xmodule/xmodule/modulestore/tests/persistent_factories.py +++ b/common/lib/xmodule/xmodule/modulestore/tests/persistent_factories.py @@ -6,7 +6,7 @@ import factory from factory.helpers import lazy_attribute from opaque_keys.edx.keys import UsageKey # Factories don't have __init__ methods, and are self documenting -# pylint: disable=W0232, C0111 +# pylint: disable=W0232, missing-docstring class SplitFactory(factory.Factory): diff --git a/common/lib/xmodule/xmodule/tests/test_capa_module.py b/common/lib/xmodule/xmodule/tests/test_capa_module.py index 045af6d390..eb7a01864d 100644 --- a/common/lib/xmodule/xmodule/tests/test_capa_module.py +++ b/common/lib/xmodule/xmodule/tests/test_capa_module.py @@ -2,7 +2,7 @@ """ Tests of the Capa XModule """ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=R0904 # pylint: disable=C0103 # pylint: disable=C0302 diff --git a/common/lib/xmodule/xmodule/tests/test_xml_module.py b/common/lib/xmodule/xmodule/tests/test_xml_module.py index dcdabcf433..36d22c0fba 100644 --- a/common/lib/xmodule/xmodule/tests/test_xml_module.py +++ b/common/lib/xmodule/xmodule/tests/test_xml_module.py @@ -1,5 +1,5 @@ # disable missing docstring -# pylint: disable=C0111 +# pylint: disable=missing-docstring import unittest diff --git a/common/lib/xmodule/xmodule/video_module/transcripts_utils.py b/common/lib/xmodule/xmodule/video_module/transcripts_utils.py index 857cc6b5d2..e38dcd9fec 100644 --- a/common/lib/xmodule/xmodule/video_module/transcripts_utils.py +++ b/common/lib/xmodule/xmodule/video_module/transcripts_utils.py @@ -19,19 +19,19 @@ from xmodule.contentstore.django import contentstore log = logging.getLogger(__name__) -class TranscriptException(Exception): # pylint disable=C0111 +class TranscriptException(Exception): # pylint: disable=missing-docstring pass -class TranscriptsGenerationException(Exception): # pylint disable=C0111 +class TranscriptsGenerationException(Exception): # pylint: disable=missing-docstring pass -class GetTranscriptsFromYouTubeException(Exception): # pylint disable=C0111 +class GetTranscriptsFromYouTubeException(Exception): # pylint: disable=missing-docstring pass -class TranscriptsRequestValidationException(Exception): # pylint disable=C0111 +class TranscriptsRequestValidationException(Exception): # pylint: disable=missing-docstring pass diff --git a/lms/djangoapps/bulk_email/forms.py b/lms/djangoapps/bulk_email/forms.py index 815df5d3a6..3556b274c0 100644 --- a/lms/djangoapps/bulk_email/forms.py +++ b/lms/djangoapps/bulk_email/forms.py @@ -22,7 +22,7 @@ class CourseEmailTemplateForm(forms.ModelForm): # pylint: disable=R0924 name = forms.CharField(required=False) - class Meta: # pylint: disable=C0111 + class Meta: # pylint: disable=missing-docstring model = CourseEmailTemplate fields = ('html_template', 'plain_template', 'name') @@ -76,7 +76,7 @@ class CourseEmailTemplateForm(forms.ModelForm): # pylint: disable=R0924 class CourseAuthorizationAdminForm(forms.ModelForm): # pylint: disable=R0924 """Input form for email enabling, allowing us to verify data.""" - class Meta: # pylint: disable=C0111 + class Meta: # pylint: disable=missing-docstring model = CourseAuthorization def clean_course_id(self): diff --git a/lms/djangoapps/bulk_email/models.py b/lms/djangoapps/bulk_email/models.py index d3ce60d750..045edd99bb 100644 --- a/lms/djangoapps/bulk_email/models.py +++ b/lms/djangoapps/bulk_email/models.py @@ -43,7 +43,7 @@ class Email(models.Model): created = models.DateTimeField(auto_now_add=True) modified = models.DateTimeField(auto_now=True) - class Meta: # pylint: disable=C0111 + class Meta: # pylint: disable=missing-docstring abstract = True @@ -141,7 +141,7 @@ class Optout(models.Model): user = models.ForeignKey(User, db_index=True, null=True) course_id = CourseKeyField(max_length=255, db_index=True) - class Meta: # pylint: disable=C0111 + class Meta: # pylint: disable=missing-docstring unique_together = ('user', 'course_id') diff --git a/lms/djangoapps/courseware/features/certificates.py b/lms/djangoapps/courseware/features/certificates.py index 8d54f7c258..5c65225b14 100644 --- a/lms/djangoapps/courseware/features/certificates.py +++ b/lms/djangoapps/courseware/features/certificates.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from lettuce import world, step diff --git a/lms/djangoapps/courseware/features/common.py b/lms/djangoapps/courseware/features/common.py index 7787244a33..b6eebe7b9e 100644 --- a/lms/djangoapps/courseware/features/common.py +++ b/lms/djangoapps/courseware/features/common.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from __future__ import absolute_import diff --git a/lms/djangoapps/courseware/features/conditional.py b/lms/djangoapps/courseware/features/conditional.py index f6b22d5ff5..2fab88f253 100644 --- a/lms/djangoapps/courseware/features/conditional.py +++ b/lms/djangoapps/courseware/features/conditional.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from lettuce import world, steps diff --git a/lms/djangoapps/courseware/features/courseware.py b/lms/djangoapps/courseware/features/courseware.py index 644c063064..91de6ffe70 100644 --- a/lms/djangoapps/courseware/features/courseware.py +++ b/lms/djangoapps/courseware/features/courseware.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from lettuce import world, step diff --git a/lms/djangoapps/courseware/features/courseware_common.py b/lms/djangoapps/courseware/features/courseware_common.py index 6c54ef3159..fb6dff6aa6 100644 --- a/lms/djangoapps/courseware/features/courseware_common.py +++ b/lms/djangoapps/courseware/features/courseware_common.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name # pylint: disable=unused-argument diff --git a/lms/djangoapps/courseware/features/events.py b/lms/djangoapps/courseware/features/events.py index 4f0d9adccf..eefd0e1b92 100644 --- a/lms/djangoapps/courseware/features/events.py +++ b/lms/djangoapps/courseware/features/events.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring from lettuce import step from lettuce import world diff --git a/lms/djangoapps/courseware/features/homepage.py b/lms/djangoapps/courseware/features/homepage.py index 0741551dc5..37858a132b 100644 --- a/lms/djangoapps/courseware/features/homepage.py +++ b/lms/djangoapps/courseware/features/homepage.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from lettuce import world, step diff --git a/lms/djangoapps/courseware/features/login.py b/lms/djangoapps/courseware/features/login.py index 42f6cb19c2..0881a81eea 100644 --- a/lms/djangoapps/courseware/features/login.py +++ b/lms/djangoapps/courseware/features/login.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from lettuce import step, world diff --git a/lms/djangoapps/courseware/features/lti.py b/lms/djangoapps/courseware/features/lti.py index d64f734bc9..2df7ea3de1 100644 --- a/lms/djangoapps/courseware/features/lti.py +++ b/lms/djangoapps/courseware/features/lti.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring import datetime import os import pytz diff --git a/lms/djangoapps/courseware/features/navigation.py b/lms/djangoapps/courseware/features/navigation.py index e766e1d496..72eb109a7a 100644 --- a/lms/djangoapps/courseware/features/navigation.py +++ b/lms/djangoapps/courseware/features/navigation.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from lettuce import world, step diff --git a/lms/djangoapps/courseware/features/openended.py b/lms/djangoapps/courseware/features/openended.py index cf43cdacd1..9b48aed195 100644 --- a/lms/djangoapps/courseware/features/openended.py +++ b/lms/djangoapps/courseware/features/openended.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from lettuce import world, step diff --git a/lms/djangoapps/courseware/features/problems.py b/lms/djangoapps/courseware/features/problems.py index 4175e64cd3..40ac5f790e 100644 --- a/lms/djangoapps/courseware/features/problems.py +++ b/lms/djangoapps/courseware/features/problems.py @@ -2,7 +2,7 @@ Steps for problem.feature lettuce tests ''' -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from lettuce import world, step diff --git a/lms/djangoapps/courseware/features/problems_setup.py b/lms/djangoapps/courseware/features/problems_setup.py index 407c5068d8..1036eaa2c1 100644 --- a/lms/djangoapps/courseware/features/problems_setup.py +++ b/lms/djangoapps/courseware/features/problems_setup.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name # EVERY PROBLEM TYPE MUST HAVE THE FOLLOWING: diff --git a/lms/djangoapps/courseware/features/registration.py b/lms/djangoapps/courseware/features/registration.py index 1b1bb94ed1..5b8f3c6250 100644 --- a/lms/djangoapps/courseware/features/registration.py +++ b/lms/djangoapps/courseware/features/registration.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from lettuce import world, step diff --git a/lms/djangoapps/courseware/features/signup.py b/lms/djangoapps/courseware/features/signup.py index 10009cf0d7..5f35a40e9a 100644 --- a/lms/djangoapps/courseware/features/signup.py +++ b/lms/djangoapps/courseware/features/signup.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from lettuce import world, step diff --git a/lms/djangoapps/courseware/features/video.py b/lms/djangoapps/courseware/features/video.py index 889a77b9d1..672c6648cb 100644 --- a/lms/djangoapps/courseware/features/video.py +++ b/lms/djangoapps/courseware/features/video.py @@ -1,5 +1,5 @@ # -*- coding: utf-8 -*- -# pylint: disable=C0111 +# pylint: disable=missing-docstring from lettuce import world, step, before, after import json diff --git a/lms/djangoapps/courseware/features/word_cloud.py b/lms/djangoapps/courseware/features/word_cloud.py index 90f6f6e8b5..fa0ea081a8 100644 --- a/lms/djangoapps/courseware/features/word_cloud.py +++ b/lms/djangoapps/courseware/features/word_cloud.py @@ -1,4 +1,4 @@ -# pylint: disable=C0111 +# pylint: disable=missing-docstring from time import sleep diff --git a/lms/djangoapps/courseware/tests/factories.py b/lms/djangoapps/courseware/tests/factories.py index 073d0085c8..7770d13db3 100644 --- a/lms/djangoapps/courseware/tests/factories.py +++ b/lms/djangoapps/courseware/tests/factories.py @@ -1,5 +1,5 @@ # Factories don't have __init__ methods, and are self documenting -# pylint: disable=W0232, C0111 +# pylint: disable=W0232, missing-docstring import json from functools import partial import factory diff --git a/lms/djangoapps/courseware/tests/test_access.py b/lms/djangoapps/courseware/tests/test_access.py index c2f6ea2b81..bd2a8217a7 100644 --- a/lms/djangoapps/courseware/tests/test_access.py +++ b/lms/djangoapps/courseware/tests/test_access.py @@ -15,7 +15,7 @@ from xmodule.course_module import ( CATALOG_VISIBILITY_CATALOG_AND_ABOUT, CATALOG_VISIBILITY_ABOUT, CATALOG_VISIBILITY_NONE) -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=W0212 diff --git a/lms/djangoapps/dashboard/tests/test_support.py b/lms/djangoapps/dashboard/tests/test_support.py index 24a2a78890..b008822dac 100644 --- a/lms/djangoapps/dashboard/tests/test_support.py +++ b/lms/djangoapps/dashboard/tests/test_support.py @@ -49,7 +49,7 @@ class RefundTests(ModuleStoreTestCase): Order.objects.filter(user=self.student).delete() def _enroll(self, purchase=True): - # pylint: disable=C0111 + # pylint: disable=missing-docstring CourseEnrollment.enroll(self.student, self.course_id, self.course_mode.mode_slug) if purchase: self.order = Order.get_cart_for_user(self.student) diff --git a/lms/djangoapps/django_comment_client/base/tests.py b/lms/djangoapps/django_comment_client/base/tests.py index bf4c0fe872..67b1eb91a4 100644 --- a/lms/djangoapps/django_comment_client/base/tests.py +++ b/lms/djangoapps/django_comment_client/base/tests.py @@ -27,7 +27,7 @@ log = logging.getLogger(__name__) CS_PREFIX = "http://localhost:4567/api/v1" -# pylint: disable=C0111 +# pylint: disable=missing-docstring class MockRequestSetupMixin(object): diff --git a/lms/djangoapps/django_comment_client/forum/tests.py b/lms/djangoapps/django_comment_client/forum/tests.py index 2bec1572ed..c435b2dc86 100644 --- a/lms/djangoapps/django_comment_client/forum/tests.py +++ b/lms/djangoapps/django_comment_client/forum/tests.py @@ -30,7 +30,7 @@ TEST_DATA_MONGO_MODULESTORE = mixed_store_config(TEST_DATA_DIR, {}, include_xml= log = logging.getLogger(__name__) -# pylint: disable=C0111 +# pylint: disable=missing-docstring @override_settings(MODULESTORE=TEST_DATA_MONGO_MODULESTORE) diff --git a/lms/djangoapps/instructor/features/bulk_email.py b/lms/djangoapps/instructor/features/bulk_email.py index 782cfdeb95..654de88ded 100644 --- a/lms/djangoapps/instructor/features/bulk_email.py +++ b/lms/djangoapps/instructor/features/bulk_email.py @@ -2,7 +2,7 @@ Define steps for bulk email acceptance test. """ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from lettuce import world, step diff --git a/lms/djangoapps/instructor/features/common.py b/lms/djangoapps/instructor/features/common.py index 3952140d75..c3736b49dd 100644 --- a/lms/djangoapps/instructor/features/common.py +++ b/lms/djangoapps/instructor/features/common.py @@ -2,7 +2,7 @@ Define common steps for instructor dashboard acceptance tests. """ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from __future__ import absolute_import diff --git a/lms/djangoapps/instructor/features/data_download.py b/lms/djangoapps/instructor/features/data_download.py index 4e3164742c..4a072b78fe 100644 --- a/lms/djangoapps/instructor/features/data_download.py +++ b/lms/djangoapps/instructor/features/data_download.py @@ -3,7 +3,7 @@ Define steps for instructor dashboard - data download tab acceptance tests. """ -# pylint: disable=C0111 +# pylint: disable=missing-docstring # pylint: disable=redefined-outer-name from lettuce import world, step diff --git a/lms/djangoapps/instructor/tests/test_hint_manager.py b/lms/djangoapps/instructor/tests/test_hint_manager.py index a3446646f7..fb35f75ab9 100644 --- a/lms/djangoapps/instructor/tests/test_hint_manager.py +++ b/lms/djangoapps/instructor/tests/test_hint_manager.py @@ -12,7 +12,7 @@ from student.tests.factories import UserFactory from xmodule.modulestore.tests.django_utils import ModuleStoreTestCase from xmodule.modulestore.tests.factories import CourseFactory -# pylint: disable=C0111 +# pylint: disable=missing-docstring @override_settings(MODULESTORE=TEST_DATA_MONGO_MODULESTORE) diff --git a/lms/djangoapps/instructor/tests/test_legacy_xss.py b/lms/djangoapps/instructor/tests/test_legacy_xss.py index 310e9c3eb9..58bbf90765 100644 --- a/lms/djangoapps/instructor/tests/test_legacy_xss.py +++ b/lms/djangoapps/instructor/tests/test_legacy_xss.py @@ -15,7 +15,7 @@ from xmodule.modulestore.tests.factories import CourseFactory from instructor.views import legacy -# pylint: disable=C0111 +# pylint: disable=missing-docstring @override_settings(MODULESTORE=TEST_DATA_MONGO_MODULESTORE) diff --git a/lms/djangoapps/instructor/views/api.py b/lms/djangoapps/instructor/views/api.py index 608409457a..0846a9ff2d 100644 --- a/lms/djangoapps/instructor/views/api.py +++ b/lms/djangoapps/instructor/views/api.py @@ -95,7 +95,7 @@ def common_exceptions_400(func): Catches common exceptions and renders matching 400 errors. (decorator without arguments) """ - def wrapped(request, *args, **kwargs): # pylint: disable=C0111 + def wrapped(request, *args, **kwargs): # pylint: disable=missing-docstring use_json = (request.is_ajax() or request.META.get("HTTP_ACCEPT", "").startswith("application/json")) try: @@ -129,8 +129,8 @@ def require_query_params(*args, **kwargs): required_params += [(key, kwargs[key]) for key in kwargs] # required_params = e.g. [('action', 'enroll or unenroll'), ['emails', None]] - def decorator(func): # pylint: disable=C0111 - def wrapped(*args, **kwargs): # pylint: disable=C0111 + def decorator(func): # pylint: disable=missing-docstring + def wrapped(*args, **kwargs): # pylint: disable=missing-docstring request = args[0] error_response_data = { @@ -166,8 +166,8 @@ def require_post_params(*args, **kwargs): required_params += [(key, kwargs[key]) for key in kwargs] # required_params = e.g. [('action', 'enroll or unenroll'), ['emails', None]] - def decorator(func): # pylint: disable=C0111 - def wrapped(*args, **kwargs): # pylint: disable=C0111 + def decorator(func): # pylint: disable=missing-docstring + def wrapped(*args, **kwargs): # pylint: disable=missing-docstring request = args[0] error_response_data = { @@ -206,8 +206,8 @@ def require_level(level): if level not in ['instructor', 'staff']: raise ValueError("unrecognized level '{}'".format(level)) - def decorator(func): # pylint: disable=C0111 - def wrapped(*args, **kwargs): # pylint: disable=C0111 + def decorator(func): # pylint: disable=missing-docstring + def wrapped(*args, **kwargs): # pylint: disable=missing-docstring request = args[0] course = get_course_by_id(SlashSeparatedCourseKey.from_deprecated_string(kwargs['course_id'])) diff --git a/lms/djangoapps/mobile_api/users/serializers.py b/lms/djangoapps/mobile_api/users/serializers.py index b0ae321a3c..267378f674 100644 --- a/lms/djangoapps/mobile_api/users/serializers.py +++ b/lms/djangoapps/mobile_api/users/serializers.py @@ -65,7 +65,7 @@ class CourseEnrollmentSerializer(serializers.ModelSerializer): """ course = CourseField() - class Meta: # pylint: disable=C0111 + class Meta: # pylint: disable=missing-docstring model = CourseEnrollment fields = ('created', 'mode', 'is_active', 'course') lookup_field = 'username' @@ -81,7 +81,7 @@ class UserSerializer(serializers.HyperlinkedModelSerializer): lookup_field='username' ) - class Meta: # pylint: disable=C0111 + class Meta: # pylint: disable=missing-docstring model = User fields = ('id', 'username', 'email', 'name', 'course_enrollments') lookup_field = 'username' diff --git a/lms/djangoapps/shoppingcart/exceptions.py b/lms/djangoapps/shoppingcart/exceptions.py index 6253c3bc46..12358deef6 100644 --- a/lms/djangoapps/shoppingcart/exceptions.py +++ b/lms/djangoapps/shoppingcart/exceptions.py @@ -2,7 +2,7 @@ Exceptions for the shoppingcart app """ # (Exception Class Names are sort of self-explanatory, so skipping docstring requirement) -# pylint: disable=C0111 +# pylint: disable=missing-docstring class PaymentException(Exception): diff --git a/lms/djangoapps/survey/admin.py b/lms/djangoapps/survey/admin.py index 3095c97001..45f30e89e3 100644 --- a/lms/djangoapps/survey/admin.py +++ b/lms/djangoapps/survey/admin.py @@ -10,7 +10,7 @@ from survey.models import SurveyForm class SurveyFormAdminForm(forms.ModelForm): # pylint: disable=R0924 """Form providing validation of SurveyForm content.""" - class Meta: # pylint: disable=C0111 + class Meta: # pylint: disable=missing-docstring model = SurveyForm fields = ('name', 'form') diff --git a/lms/djangoapps/verify_student/exceptions.py b/lms/djangoapps/verify_student/exceptions.py index d31fdb6a6d..d8634987ef 100644 --- a/lms/djangoapps/verify_student/exceptions.py +++ b/lms/djangoapps/verify_student/exceptions.py @@ -2,7 +2,7 @@ Exceptions for the verify student app """ # (Exception Class Names are sort of self-explanatory, so skipping docstring requirement) -# pylint: disable=C0111 +# pylint: disable=missing-docstring class WindowExpiredException(Exception):