Remove 'pylint: disable=W0232' because we ignore this warning in pylintrc

common/djangoapps/course_modes/tests/factories.py

@@ -3,8 +3,8 @@ from factory.django import DjangoModelFactory
 from opaque_keys.edx.locations import SlashSeparatedCourseKey
 
 
-# Factories don't have __init__ methods, and are self documenting
-# pylint: disable=W0232
+# Factories are self documenting
+# pylint: disable=missing-docstring
 class CourseModeFactory(DjangoModelFactory):
     FACTORY_FOR = CourseMode
 

common/djangoapps/reverification/tests/factories.py

@@ -8,8 +8,8 @@ from datetime import timedelta, datetime
 from opaque_keys.edx.locations import SlashSeparatedCourseKey
 
 
-# Factories don't have __init__ methods, and are self documenting
-# pylint: disable=W0232
+# Factories are self documenting
+# pylint: disable=missing-docstring
 class MidcourseReverificationWindowFactory(DjangoModelFactory):
     """ Creates a generic MidcourseReverificationWindow. """
     FACTORY_FOR = MidcourseReverificationWindow

common/djangoapps/student/tests/factories.py

@@ -12,8 +12,8 @@ from uuid import uuid4
 from pytz import UTC
 from opaque_keys.edx.locations import SlashSeparatedCourseKey
 
-# Factories don't have __init__ methods, and are self documenting
-# pylint: disable=W0232, missing-docstring
+# Factories are self documenting
+# pylint: disable=missing-docstring
 
 
 class GroupFactory(DjangoModelFactory):

common/djangoapps/user_api/tests/factories.py

@@ -6,8 +6,8 @@ from user_api.models import UserPreference, UserCourseTag, UserOrgTag
 from opaque_keys.edx.locations import SlashSeparatedCourseKey
 
 
-# Factories don't have __init__ methods, and are self documenting
-# pylint: disable=W0232, missing-docstring
+# Factories are self documenting
+# pylint: disable=missing-docstring
 class UserPreferenceFactory(DjangoModelFactory):
     FACTORY_FOR = UserPreference
 

common/djangoapps/util/tests/test_date_utils.py

@@ -70,7 +70,6 @@ def test_get_time_display_coerce():
                   get_time_display(test_time_daylight, '%b %d %H:%M', coerce_tz="US/Pacific"))
 
 
-# pylint: disable=W0232
 class NamelessTZ(tzinfo):
     """Static timezone for testing"""
 

common/lib/xmodule/xmodule/modulestore/tests/persistent_factories.py

@@ -5,8 +5,8 @@ from xmodule.x_module import XModuleDescriptor
 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, missing-docstring
+# Factories are self documenting
+# pylint: disable=missing-docstring
 
 
 class SplitFactory(factory.Factory):

lms/djangoapps/certificates/tests/factories.py

@@ -5,8 +5,8 @@ from opaque_keys.edx.locations import SlashSeparatedCourseKey
 from certificates.models import GeneratedCertificate, CertificateStatuses
 
 
-# Factories don't have __init__ methods, and are self documenting
-# pylint: disable=W0232
+# Factories are self documenting
+# pylint: disable=missing-docstring
 class GeneratedCertificateFactory(DjangoModelFactory):
 
     FACTORY_FOR = GeneratedCertificate

lms/djangoapps/courseware/tests/factories.py

@@ -1,5 +1,5 @@
-# Factories don't have __init__ methods, and are self documenting
-# pylint: disable=W0232, missing-docstring
+# Factories are self documenting
+# pylint: disable=missing-docstring
 import json
 from functools import partial
 import factory