Merge pull request #26534 from edx/jawayria/bom-2351-7

BOM-2351: Removed unused imports from common/test

common/test/acceptance/fixtures/base.py

@@ -6,7 +6,6 @@ Common code shared by course and library fixtures.
 import json
 
 import requests
-import six  # lint-amnesty, pylint: disable=unused-import
 from lazy import lazy
 
 from common.test.acceptance.fixtures import STUDIO_BASE_URL

common/test/acceptance/pages/lms/video/video.py

@@ -7,7 +7,7 @@ import logging
 
 from bok_choy.javascript import js_defined, wait_for_js
 from bok_choy.page_object import PageObject
-from bok_choy.promise import EmptyPromise, Promise  # lint-amnesty, pylint: disable=unused-import
+from bok_choy.promise import EmptyPromise  # lint-amnesty, pylint: disable=unused-import
 
 log = logging.getLogger('VideoPage')
 

common/test/acceptance/pages/studio/overview.py

@@ -3,7 +3,7 @@ Course Outline page in Studio.
 """
 
 
-from bok_choy.javascript import js_defined, wait_for_js  # lint-amnesty, pylint: disable=unused-import
+from bok_choy.javascript import js_defined  # lint-amnesty, pylint: disable=unused-import
 from bok_choy.page_object import PageObject
 from bok_choy.promise import EmptyPromise
 from selenium.webdriver.support.ui import Select

common/test/acceptance/tests/discussion/test_discussion.py

@@ -7,17 +7,19 @@ from uuid import uuid4
 
 import pytest
 
-from common.test.acceptance.fixtures.course import CourseFixture, XBlockFixtureDesc  # lint-amnesty, pylint: disable=unused-import
+from common.test.acceptance.fixtures.course import CourseFixture  # lint-amnesty, pylint: disable=unused-import
 from common.test.acceptance.fixtures.discussion import (
     Comment,
     Response,
     SingleThreadViewFixture,
     Thread,
+
 )
 from common.test.acceptance.pages.common.auto_auth import AutoAuthPage
 from common.test.acceptance.pages.lms.discussion import (
     DiscussionTabHomePage,
     DiscussionTabSingleThreadPage,
+
 )
 from common.test.acceptance.tests.discussion.helpers import BaseDiscussionMixin, BaseDiscussionTestCase
 from common.test.acceptance.tests.helpers import UniqueCourseTest

common/test/acceptance/tests/helpers.py

@@ -19,12 +19,12 @@ from bok_choy.promise import EmptyPromise, Promise
 from bok_choy.web_app_test import WebAppTest
 from opaque_keys.edx.locator import CourseLocator
 from path import Path as path
-from pymongo import ASCENDING, MongoClient  # lint-amnesty, pylint: disable=unused-import
+from pymongo import MongoClient  # lint-amnesty, pylint: disable=unused-import
 from selenium.common.exceptions import StaleElementReferenceException
 from selenium.webdriver.common.keys import Keys
 from selenium.webdriver.support.select import Select
 from selenium.webdriver.support.ui import WebDriverWait
-from six.moves import range, zip  # lint-amnesty, pylint: disable=unused-import
+from six.moves import range  # lint-amnesty, pylint: disable=unused-import
 
 from capa.tests.response_xml_factory import MultipleChoiceResponseXMLFactory
 from common.test.acceptance.fixtures.course import XBlockFixtureDesc

common/test/conftest.py

@@ -1,9 +1,4 @@
 """Code run by pylint before running any tests."""
-
-# Patch the xml libs before anything else.
-
-
-from openedx.core.pytest_hooks import pytest_configure  # pylint: disable=unused-import
 from safe_lxml import defuse_xml_libs
 
 defuse_xml_libs()

common/test/utils.py

@@ -6,8 +6,6 @@ General testing utilities.
 import functools
 import sys
 from contextlib import contextmanager
-
-import pytest  # lint-amnesty, pylint: disable=unused-import
 from django.dispatch import Signal
 from markupsafe import escape
 from mock import Mock, patch