diff --git a/lms/djangoapps/email_marketing/tests/test_signals.py b/lms/djangoapps/email_marketing/tests/test_signals.py index aef30e561a..45a5305a48 100644 --- a/lms/djangoapps/email_marketing/tests/test_signals.py +++ b/lms/djangoapps/email_marketing/tests/test_signals.py @@ -29,7 +29,6 @@ from lms.djangoapps.email_marketing.tasks import ( # lint-amnesty, pylint: disa _get_list_from_email_marketing_provider, _get_or_create_user_list, get_email_cookies_via_sailthru, - update_course_enrollment, update_user, update_user_email ) diff --git a/lms/djangoapps/experiments/tests/test_views.py b/lms/djangoapps/experiments/tests/test_views.py index b82cb1fca2..b1d6624947 100644 --- a/lms/djangoapps/experiments/tests/test_views.py +++ b/lms/djangoapps/experiments/tests/test_views.py @@ -4,12 +4,10 @@ Tests for experimentation views import unittest -from datetime import timedelta from unittest.mock import patch -import six.moves.urllib.error import six.moves.urllib.parse -import six.moves.urllib.request +from datetime import timedelta from django.conf import settings from django.core.handlers.wsgi import WSGIRequest from django.test.utils import override_settings @@ -20,10 +18,7 @@ from rest_framework.test import APITestCase from common.djangoapps.student.tests.factories import UserFactory from lms.djangoapps.course_blocks.transformers.tests.helpers import ModuleStoreTestCase from lms.djangoapps.experiments.factories import ExperimentDataFactory, ExperimentKeyValueFactory -from lms.djangoapps.experiments.models import ( # lint-amnesty, pylint: disable=unused-import - ExperimentData, - ExperimentKeyValue -) +from lms.djangoapps.experiments.models import ExperimentData # lint-amnesty, pylint: disable=unused-import from lms.djangoapps.experiments.serializers import ExperimentDataSerializer from xmodule.modulestore.tests.factories import CourseFactory diff --git a/lms/djangoapps/experiments/utils.py b/lms/djangoapps/experiments/utils.py index 76a0ed5732..e6e657ff4c 100644 --- a/lms/djangoapps/experiments/utils.py +++ b/lms/djangoapps/experiments/utils.py @@ -23,9 +23,6 @@ from openedx.core.djangoapps.schedules.models import Schedule from openedx.features.course_duration_limits.access import get_user_course_duration, get_user_course_expiration_date from xmodule.partitions.partitions_service import get_all_partitions_for_course, get_user_partition_groups -# Import this for backwards compatibility (so that anyone importing this function from here doesn't break) -from .stable_bucketing import stable_bucketing_hash_group # pylint: disable=unused-import - logger = logging.getLogger(__name__) diff --git a/lms/djangoapps/experiments/views.py b/lms/djangoapps/experiments/views.py index e25c593904..6382fccd7d 100644 --- a/lms/djangoapps/experiments/views.py +++ b/lms/djangoapps/experiments/views.py @@ -4,15 +4,15 @@ Experimentation views from django.contrib.auth import get_user_model -from django.db import transaction # lint-amnesty, pylint: disable=unused-import from django.http import Http404 from django_filters.rest_framework import DjangoFilterBackend from edx_rest_framework_extensions.auth.jwt.authentication import JwtAuthentication from edx_rest_framework_extensions.auth.session.authentication import SessionAuthenticationAllowInactiveUser -from opaque_keys.edx.keys import CourseKey -from rest_framework import permissions, viewsets -from rest_framework.response import Response # lint-amnesty, pylint: disable=unused-import, wrong-import-order -from rest_framework.views import APIView +from lms.djangoapps.courseware import courses +from opaque_keys.edx.keys import CourseKey # lint-amnesty, pylint: disable=wrong-import-order +from rest_framework import permissions, viewsets # lint-amnesty, pylint: disable=wrong-import-order +from rest_framework.views import APIView # lint-amnesty, pylint: disable=wrong-import-order +from common.djangoapps.util.json_request import JsonResponse from common.djangoapps.student.models import get_user_by_username_or_email from common.djangoapps.util.json_request import JsonResponse