diff --git a/common/djangoapps/course_modes/tests/factories.py b/common/djangoapps/course_modes/tests/factories.py index 5ca0153db1..bbe1ca5c2b 100644 --- a/common/djangoapps/course_modes/tests/factories.py +++ b/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 diff --git a/common/djangoapps/reverification/tests/factories.py b/common/djangoapps/reverification/tests/factories.py index 58684d97f0..4416875243 100644 --- a/common/djangoapps/reverification/tests/factories.py +++ b/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 diff --git a/common/djangoapps/student/tests/factories.py b/common/djangoapps/student/tests/factories.py index 1c99eead81..880beac7da 100644 --- a/common/djangoapps/student/tests/factories.py +++ b/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): diff --git a/common/djangoapps/user_api/tests/factories.py b/common/djangoapps/user_api/tests/factories.py index 25be6bf188..60ee7c2ddc 100644 --- a/common/djangoapps/user_api/tests/factories.py +++ b/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 diff --git a/common/djangoapps/util/tests/test_date_utils.py b/common/djangoapps/util/tests/test_date_utils.py index 579c476c04..de3308658e 100644 --- a/common/djangoapps/util/tests/test_date_utils.py +++ b/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""" diff --git a/common/lib/xmodule/xmodule/modulestore/tests/persistent_factories.py b/common/lib/xmodule/xmodule/modulestore/tests/persistent_factories.py index 9fe356a20a..2eee09740f 100644 --- a/common/lib/xmodule/xmodule/modulestore/tests/persistent_factories.py +++ b/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): diff --git a/lms/djangoapps/certificates/tests/factories.py b/lms/djangoapps/certificates/tests/factories.py index 38527f92c4..d800032f66 100644 --- a/lms/djangoapps/certificates/tests/factories.py +++ b/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 diff --git a/lms/djangoapps/courseware/tests/factories.py b/lms/djangoapps/courseware/tests/factories.py index 7770d13db3..c47fc54f41 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, missing-docstring +# Factories are self documenting +# pylint: disable=missing-docstring import json from functools import partial import factory