incr-285 (#20592)

* run python modernize

* run isort

* quality

lms/djangoapps/course_api/tests/mixins.py

@@ -2,6 +2,8 @@
 Common mixins for Course API Tests
 """
 
+from __future__ import absolute_import
+
 from datetime import datetime
 
 from student.tests.factories import UserFactory

lms/djangoapps/course_api/tests/test_api.py

@@ -1,6 +1,8 @@
 """
 Test for course API
 """
+from __future__ import absolute_import
+
 from hashlib import md5
 
 from django.contrib.auth.models import AnonymousUser

lms/djangoapps/course_api/tests/test_forms.py

@@ -2,10 +2,13 @@
 Tests for Course API forms.
 """
 
+from __future__ import absolute_import
+
 from itertools import product
-from urllib import urlencode
 
 import ddt
+import six
+from six.moves.urllib.parse import urlencode  # pylint: disable=import-error
 from django.contrib.auth.models import AnonymousUser
 from django.http import QueryDict
 
@@ -125,7 +128,7 @@ class TestCourseDetailGetForm(FormTestMixin, UsernameTestMixin, SharedModuleStor
         self.form_data = QueryDict(
             urlencode({
                 'username': user.username,
-                'course_key': unicode(self.course.id),
+                'course_key': six.text_type(self.course.id),
             }),
             mutable=True,
         )

lms/djangoapps/course_api/tests/test_permissions.py

@@ -2,6 +2,8 @@
 Test authorization functions
 """
 
+from __future__ import absolute_import
+
 from django.contrib.auth.models import AnonymousUser
 from django.test import TestCase
 

lms/djangoapps/course_api/tests/test_serializers.py

@@ -2,7 +2,7 @@
 Test data created by CourseSerializer and CourseDetailSerializer
 """
 
-from __future__ import unicode_literals
+from __future__ import absolute_import, unicode_literals
 
 from datetime import datetime
 

lms/djangoapps/course_api/tests/test_views.py

@@ -1,26 +1,30 @@
 """
 Tests for Course API views.
 """
+from __future__ import absolute_import
+
 from datetime import datetime
-import ddt
 from hashlib import md5
 
+import ddt
+import six
+from six.moves import range
 from django.core.exceptions import ImproperlyConfigured
-from django.urls import reverse
 from django.test import RequestFactory
 from django.test.utils import override_settings
+from django.urls import reverse
+from edx_django_utils.cache import RequestCache
 from search.tests.test_course_discovery import DemoCourse
 from search.tests.tests import TEST_INDEX_NAME
 from search.tests.utils import SearcherMixin
+from waffle.testutils import override_switch
 
 from course_modes.models import CourseMode
 from course_modes.tests.factories import CourseModeFactory
-from edx_django_utils.cache import RequestCache
 from openedx.features.content_type_gating.models import ContentTypeGatingConfig
 from openedx.features.course_duration_limits.models import CourseDurationLimitConfig
 from xmodule.modulestore.tests.django_utils import ModuleStoreTestCase, SharedModuleStoreTestCase
 from xmodule.modulestore.tests.factories import CourseFactory
-from waffle.testutils import override_switch
 
 from ..views import CourseDetailView, CourseListUserThrottle
 from .mixins import TEST_PASSWORD, CourseApiFactoryMixin
@@ -198,7 +202,7 @@ class CourseListViewTestCaseMultipleCourses(CourseApiTestViewMixin, ModuleStoreT
             response = self.verify_response(params=params)
             self.assertEquals(
                 {course['course_id'] for course in response.data['results']},
-                {unicode(course.id) for course in expected_courses},
+                {six.text_type(course.id) for course in expected_courses},
                 u"testing course_api.views.CourseListView with filter_={}".format(filter_),
             )