From 181b1e979b22a119220c4e211f86f7f9d4cdf38c Mon Sep 17 00:00:00 2001
From: Ned Batchelder <ned@nedbatchelder.com>
Date: Fri, 14 Jun 2013 13:30:47 -0400
Subject: [PATCH] Remove unused imports from common, as reported by pylint.
---
common/djangoapps/cache_toolbox/core.py | 1 -
common/djangoapps/course_groups/cohorts.py | 3 +--
common/djangoapps/course_groups/views.py | 10 ++--------
common/djangoapps/mitxmako/makoloader.py | 1 -
common/djangoapps/status/status.py | 1 -
.../student/management/commands/6002exportusers.py | 5 -----
.../student/management/commands/6002importusers.py | 6 ------
.../student/management/commands/assigngroups.py | 5 -----
.../management/commands/create_random_users.py | 4 +---
.../student/management/commands/emaillist.py | 5 -----
.../student/management/commands/massemail.py | 5 -----
.../student/management/commands/massemailtxt.py | 3 ---
.../student/management/commands/pearson_dump.py | 2 +-
.../management/commands/pearson_import_conf_zip.py | 5 +----
.../management/commands/tests/test_pearson.py | 2 +-
.../student/management/commands/userinfo.py | 5 -----
common/djangoapps/student/views.py | 8 ++------
common/djangoapps/terrain/browser.py | 6 +++---
common/djangoapps/terrain/course_helpers.py | 3 +--
common/djangoapps/terrain/steps.py | 2 +-
common/djangoapps/track/middleware.py | 2 --
common/djangoapps/util/models.py | 2 --
common/djangoapps/util/tests/test_memcache.py | 1 -
.../djangoapps/util/tests/test_submit_feedback.py | 1 -
common/djangoapps/util/views.py | 12 ++----------
common/lib/capa/capa/checker.py | 1 -
common/lib/capa/capa/customrender.py | 2 --
common/lib/capa/capa/responsetypes.py | 1 -
common/lib/capa/capa/tests/test_html_render.py | 1 -
common/lib/capa/capa/util.py | 2 +-
common/lib/chem/chem/chemcalc.py | 13 ++-----------
common/lib/symmath/symmath/formula.py | 4 +---
common/lib/symmath/symmath/symmath_check.py | 4 ----
.../xmodule/xmodule/modulestore/tests/factories.py | 1 -
.../xmodule/modulestore/tests/test_modulestore.py | 4 ++--
.../xmodule/xmodule/modulestore/tests/test_mongo.py | 1 -
.../combined_open_ended_modulev1.py | 3 ---
.../grading_service_module.py | 1 -
.../open_ended_image_submission.py | 2 --
.../open_ended_grading_classes/open_ended_module.py | 2 --
.../open_ended_grading_classes/openendedchild.py | 6 ------
common/lib/xmodule/xmodule/progress.py | 1 -
common/lib/xmodule/xmodule/schematic_module.py | 1 -
common/lib/xmodule/xmodule/template_module.py | 1 -
.../lib/xmodule/xmodule/tests/test_html_module.py | 1 -
.../lib/xmodule/xmodule/tests/test_peer_grading.py | 4 ----
.../xmodule/xmodule/tests/test_randomize_module.py | 10 +---------
common/lib/xmodule/xmodule/tests/test_stringify.py | 2 +-
.../xmodule/xmodule/tests/test_util_open_ended.py | 2 +-
common/lib/xmodule/xmodule/timelimit_module.py | 1 -
50 files changed, 25 insertions(+), 146 deletions(-)
diff --git a/common/djangoapps/cache_toolbox/core.py b/common/djangoapps/cache_toolbox/core.py
index a9c7002aa6b..9a7be940b89 100644
--- a/common/djangoapps/cache_toolbox/core.py
+++ b/common/djangoapps/cache_toolbox/core.py
@@ -12,7 +12,6 @@ from django.core.cache import cache
from django.db import DEFAULT_DB_ALIAS
from . import app_settings
-from xmodule.contentstore.content import StaticContent
def get_instance(model, instance_or_pk, timeout=None, using=None):
diff --git a/common/djangoapps/course_groups/cohorts.py b/common/djangoapps/course_groups/cohorts.py
index 7924012bfe2..d2c7e3a782f 100644
--- a/common/djangoapps/course_groups/cohorts.py
+++ b/common/djangoapps/course_groups/cohorts.py
@@ -3,7 +3,6 @@ This file contains the logic for cohort groups, as exposed internally to the
forums, and to the cohort admin views.
"""
-from django.contrib.auth.models import User
from django.http import Http404
import logging
import random
@@ -27,7 +26,7 @@ def local_random():
"""
# ironic, isn't it?
global _local_random
-
+
if _local_random is None:
_local_random = random.Random()
diff --git a/common/djangoapps/course_groups/views.py b/common/djangoapps/course_groups/views.py
index 6d5ac43fb06..764f6c301d1 100644
--- a/common/djangoapps/course_groups/views.py
+++ b/common/djangoapps/course_groups/views.py
@@ -1,24 +1,18 @@
from django_future.csrf import ensure_csrf_cookie
-from django.contrib.auth.decorators import login_required
from django.views.decorators.http import require_POST
from django.contrib.auth.models import User
-from django.core.context_processors import csrf
from django.core.paginator import Paginator, EmptyPage, PageNotAnInteger
from django.core.urlresolvers import reverse
-from django.http import HttpResponse, HttpResponseForbidden, Http404
-from django.shortcuts import redirect
+from django.http import HttpResponse
import json
import logging
import re
from courseware.courses import get_course_with_access
-from mitxmako.shortcuts import render_to_response, render_to_string
+from mitxmako.shortcuts import render_to_response
-from .models import CourseUserGroup
from . import cohorts
-import track.views
-
log = logging.getLogger(__name__)
diff --git a/common/djangoapps/mitxmako/makoloader.py b/common/djangoapps/mitxmako/makoloader.py
index 6b6b31d4642..06ae2219e64 100644
--- a/common/djangoapps/mitxmako/makoloader.py
+++ b/common/djangoapps/mitxmako/makoloader.py
@@ -7,7 +7,6 @@ from django.template.loaders.filesystem import Loader as FilesystemLoader
from django.template.loaders.app_directories import Loader as AppDirectoriesLoader
from mitxmako.template import Template
-import mitxmako.middleware
import tempdir
diff --git a/common/djangoapps/status/status.py b/common/djangoapps/status/status.py
index deacd9c6319..b3ffd6a84c6 100644
--- a/common/djangoapps/status/status.py
+++ b/common/djangoapps/status/status.py
@@ -6,7 +6,6 @@ from django.conf import settings
import json
import logging
import os
-import sys
log = logging.getLogger(__name__)
diff --git a/common/djangoapps/student/management/commands/6002exportusers.py b/common/djangoapps/student/management/commands/6002exportusers.py
index 31d8092d3f8..a92bb0a60c3 100644
--- a/common/djangoapps/student/management/commands/6002exportusers.py
+++ b/common/djangoapps/student/management/commands/6002exportusers.py
@@ -11,12 +11,7 @@
import datetime
import json
-import os.path
-
-from lxml import etree
-
from django.core.management.base import BaseCommand
-from django.conf import settings
from django.contrib.auth.models import User
from student.models import UserProfile
diff --git a/common/djangoapps/student/management/commands/6002importusers.py b/common/djangoapps/student/management/commands/6002importusers.py
index 64be84d9101..1f98bd75225 100644
--- a/common/djangoapps/student/management/commands/6002importusers.py
+++ b/common/djangoapps/student/management/commands/6002importusers.py
@@ -3,17 +3,11 @@
## See export for more info
-import datetime
import json
import dateutil.parser
-import os.path
-
-from lxml import etree
-
from django.core.management.base import BaseCommand
-from django.conf import settings
from django.contrib.auth.models import User
from student.models import UserProfile
diff --git a/common/djangoapps/student/management/commands/assigngroups.py b/common/djangoapps/student/management/commands/assigngroups.py
index 5269c8690eb..3e36bf3129a 100644
--- a/common/djangoapps/student/management/commands/assigngroups.py
+++ b/common/djangoapps/student/management/commands/assigngroups.py
@@ -1,9 +1,4 @@
-import os.path
-
-from lxml import etree
-
from django.core.management.base import BaseCommand
-from django.conf import settings
from django.contrib.auth.models import User
import mitxmako.middleware as middleware
diff --git a/common/djangoapps/student/management/commands/create_random_users.py b/common/djangoapps/student/management/commands/create_random_users.py
index 70374d02f2f..3000c866015 100644
--- a/common/djangoapps/student/management/commands/create_random_users.py
+++ b/common/djangoapps/student/management/commands/create_random_users.py
@@ -2,9 +2,7 @@
## A script to create some dummy users
from django.core.management.base import BaseCommand
-from django.conf import settings
-from django.contrib.auth.models import User
-from student.models import UserProfile, CourseEnrollment
+from student.models import CourseEnrollment
from student.views import _do_create_account, get_random_post_override
diff --git a/common/djangoapps/student/management/commands/emaillist.py b/common/djangoapps/student/management/commands/emaillist.py
index 4011c41bd2f..d3911927acc 100644
--- a/common/djangoapps/student/management/commands/emaillist.py
+++ b/common/djangoapps/student/management/commands/emaillist.py
@@ -1,9 +1,4 @@
-import os.path
-
-from lxml import etree
-
from django.core.management.base import BaseCommand
-from django.conf import settings
from django.contrib.auth.models import User
import mitxmako.middleware as middleware
diff --git a/common/djangoapps/student/management/commands/massemail.py b/common/djangoapps/student/management/commands/massemail.py
index c6f6e5f6d44..1bb65fd169d 100644
--- a/common/djangoapps/student/management/commands/massemail.py
+++ b/common/djangoapps/student/management/commands/massemail.py
@@ -1,9 +1,4 @@
-import os.path
-
-from lxml import etree
-
from django.core.management.base import BaseCommand
-from django.conf import settings
from django.contrib.auth.models import User
import mitxmako.middleware as middleware
diff --git a/common/djangoapps/student/management/commands/massemailtxt.py b/common/djangoapps/student/management/commands/massemailtxt.py
index 4ea75f972b0..fec354e974b 100644
--- a/common/djangoapps/student/management/commands/massemailtxt.py
+++ b/common/djangoapps/student/management/commands/massemailtxt.py
@@ -1,11 +1,8 @@
import os.path
import time
-from lxml import etree
-
from django.core.management.base import BaseCommand
from django.conf import settings
-from django.contrib.auth.models import User
import mitxmako.middleware as middleware
diff --git a/common/djangoapps/student/management/commands/pearson_dump.py b/common/djangoapps/student/management/commands/pearson_dump.py
index 2aade8cf5f2..0c9e215f77f 100644
--- a/common/djangoapps/student/management/commands/pearson_dump.py
+++ b/common/djangoapps/student/management/commands/pearson_dump.py
@@ -2,7 +2,7 @@ from optparse import make_option
from json import dump
from datetime import datetime
-from django.core.management.base import BaseCommand, CommandError
+from django.core.management.base import BaseCommand
from student.models import TestCenterRegistration
diff --git a/common/djangoapps/student/management/commands/pearson_import_conf_zip.py b/common/djangoapps/student/management/commands/pearson_import_conf_zip.py
index 23393837195..1e06a0931ac 100644
--- a/common/djangoapps/student/management/commands/pearson_import_conf_zip.py
+++ b/common/djangoapps/student/management/commands/pearson_import_conf_zip.py
@@ -3,11 +3,8 @@ import csv
from zipfile import ZipFile, is_zipfile
from time import strptime, strftime
-from collections import OrderedDict
from datetime import datetime
-from os.path import isdir
-from optparse import make_option
-from dogapi import dog_http_api, dog_stats_api
+from dogapi import dog_http_api
from django.core.management.base import BaseCommand, CommandError
from django.conf import settings
diff --git a/common/djangoapps/student/management/commands/tests/test_pearson.py b/common/djangoapps/student/management/commands/tests/test_pearson.py
index 65d628fba04..ca6e20673bf 100644
--- a/common/djangoapps/student/management/commands/tests/test_pearson.py
+++ b/common/djangoapps/student/management/commands/tests/test_pearson.py
@@ -14,7 +14,7 @@ from django.test import TestCase
from django.core.management import call_command
from nose.plugins.skip import SkipTest
-from student.models import User, TestCenterRegistration, TestCenterUser, get_testcenter_registration
+from student.models import User, TestCenterUser, get_testcenter_registration
log = logging.getLogger(__name__)
diff --git a/common/djangoapps/student/management/commands/userinfo.py b/common/djangoapps/student/management/commands/userinfo.py
index e4589952846..5467db1733a 100644
--- a/common/djangoapps/student/management/commands/userinfo.py
+++ b/common/djangoapps/student/management/commands/userinfo.py
@@ -1,9 +1,4 @@
-import os.path
-
-from lxml import etree
-
from django.core.management.base import BaseCommand
-from django.conf import settings
from django.contrib.auth.models import User
import mitxmako.middleware as middleware
diff --git a/common/djangoapps/student/views.py b/common/djangoapps/student/views.py
index f129f1b4b1d..de3e52b0800 100644
--- a/common/djangoapps/student/views.py
+++ b/common/djangoapps/student/views.py
@@ -4,7 +4,6 @@ import json
import logging
import random
import string
-import sys
import urllib
import uuid
import time
@@ -20,9 +19,9 @@ from django.core.mail import send_mail
from django.core.urlresolvers import reverse
from django.core.validators import validate_email, validate_slug, ValidationError
from django.db import IntegrityError, transaction
-from django.http import HttpResponse, HttpResponseBadRequest, HttpResponseForbidden, HttpResponseNotAllowed, HttpResponseRedirect, Http404
+from django.http import HttpResponse, HttpResponseBadRequest, HttpResponseForbidden, HttpResponseNotAllowed, Http404
from django.shortcuts import redirect
-from django_future.csrf import ensure_csrf_cookie, csrf_exempt
+from django_future.csrf import ensure_csrf_cookie
from django.utils.http import cookie_date
from mitxmako.shortcuts import render_to_response, render_to_string
@@ -39,14 +38,11 @@ from certificates.models import CertificateStatuses, certificate_status_for_stud
from xmodule.course_module import CourseDescriptor
from xmodule.modulestore.exceptions import ItemNotFoundError
from xmodule.modulestore.django import modulestore
-from xmodule.modulestore import Location
from collections import namedtuple
from courseware.courses import get_courses, sort_by_announcement
from courseware.access import has_access
-from courseware.views import get_module_for_descriptor, jump_to
-from courseware.model_data import ModelDataCache
from statsd import statsd
from pytz import UTC
diff --git a/common/djangoapps/terrain/browser.py b/common/djangoapps/terrain/browser.py
index d2a9480b352..d237edc4b77 100644
--- a/common/djangoapps/terrain/browser.py
+++ b/common/djangoapps/terrain/browser.py
@@ -4,7 +4,6 @@ Browser set up for acceptance tests.
#pylint: disable=E1101
#pylint: disable=W0613
-#pylint: disable=W0611
from lettuce import before, after, world
from splinter.browser import Browser
@@ -15,8 +14,9 @@ from selenium.common.exceptions import WebDriverException
# Let the LMS and CMS do their one-time setup
# For example, setting up mongo caches
-from lms import one_time_startup
-from cms import one_time_startup
+# These names aren't used, but do important work on import.
+from lms import one_time_startup # pylint: disable=W0611
+from cms import one_time_startup # pylint: disable=W0611
# There is an import issue when using django-staticfiles with lettuce
# Lettuce assumes that we are using django.contrib.staticfiles,
diff --git a/common/djangoapps/terrain/course_helpers.py b/common/djangoapps/terrain/course_helpers.py
index fc666d79043..fbc9409e7b4 100644
--- a/common/djangoapps/terrain/course_helpers.py
+++ b/common/djangoapps/terrain/course_helpers.py
@@ -1,7 +1,7 @@
# pylint: disable=C0111
# pylint: disable=W0621
-from lettuce import world, step
+from lettuce import world
from .factories import *
from django.conf import settings
from django.http import HttpRequest
@@ -15,7 +15,6 @@ from xmodule.templates import update_templates
from bs4 import BeautifulSoup
import os.path
from urllib import quote_plus
-from lettuce.django import django_url
@world.absorb
diff --git a/common/djangoapps/terrain/steps.py b/common/djangoapps/terrain/steps.py
index 6e512982b7a..f31be894f9a 100644
--- a/common/djangoapps/terrain/steps.py
+++ b/common/djangoapps/terrain/steps.py
@@ -15,7 +15,7 @@ from lettuce import world, step
from .course_helpers import *
from .ui_helpers import *
from lettuce.django import django_url
-from nose.tools import assert_equals, assert_in
+from nose.tools import assert_equals
from logging import getLogger
logger = getLogger(__name__)
diff --git a/common/djangoapps/track/middleware.py b/common/djangoapps/track/middleware.py
index 52d914aeef7..7fc02d9969f 100644
--- a/common/djangoapps/track/middleware.py
+++ b/common/djangoapps/track/middleware.py
@@ -1,7 +1,5 @@
import json
-from django.conf import settings
-
import views
diff --git a/common/djangoapps/util/models.py b/common/djangoapps/util/models.py
index 71a83623907..6b202199939 100644
--- a/common/djangoapps/util/models.py
+++ b/common/djangoapps/util/models.py
@@ -1,3 +1 @@
-from django.db import models
-
# Create your models here.
diff --git a/common/djangoapps/util/tests/test_memcache.py b/common/djangoapps/util/tests/test_memcache.py
index de8d352c389..60b3a0d0cc2 100644
--- a/common/djangoapps/util/tests/test_memcache.py
+++ b/common/djangoapps/util/tests/test_memcache.py
@@ -4,7 +4,6 @@ Tests for memcache in util app
from django.test import TestCase
from django.core.cache import get_cache
-from django.conf import settings
from util.memcache import safe_key
diff --git a/common/djangoapps/util/tests/test_submit_feedback.py b/common/djangoapps/util/tests/test_submit_feedback.py
index b66d3d642b2..6461ffa8b78 100644
--- a/common/djangoapps/util/tests/test_submit_feedback.py
+++ b/common/djangoapps/util/tests/test_submit_feedback.py
@@ -1,6 +1,5 @@
"""Tests for the Zendesk"""
-from django.conf import settings
from django.contrib.auth.models import AnonymousUser
from django.http import Http404
from django.test import TestCase
diff --git a/common/djangoapps/util/views.py b/common/djangoapps/util/views.py
index aa592d25e82..851202caec5 100644
--- a/common/djangoapps/util/views.py
+++ b/common/djangoapps/util/views.py
@@ -1,20 +1,12 @@
-import datetime
import json
import logging
-import pprint
import sys
from django.conf import settings
-from django.contrib.auth.models import User
-from django.core.context_processors import csrf
-from django.core.mail import send_mail
from django.core.validators import ValidationError, validate_email
-from django.http import Http404, HttpResponse, HttpResponseBadRequest, HttpResponseNotAllowed, HttpResponseServerError
-from django.shortcuts import redirect
-from django_future.csrf import ensure_csrf_cookie
+from django.http import Http404, HttpResponse, HttpResponseNotAllowed
from dogapi import dog_stats_api
-from mitxmako.shortcuts import render_to_response, render_to_string
-from urllib import urlencode
+from mitxmako.shortcuts import render_to_response
import zendesk
import calc
diff --git a/common/lib/capa/capa/checker.py b/common/lib/capa/capa/checker.py
index 15358aac9e4..87cf68d2306 100755
--- a/common/lib/capa/capa/checker.py
+++ b/common/lib/capa/capa/checker.py
@@ -10,7 +10,6 @@ import sys
from path import path
from cStringIO import StringIO
-from collections import defaultdict
from .calc import UndefinedVariable
from .capa_problem import LoncapaProblem
diff --git a/common/lib/capa/capa/customrender.py b/common/lib/capa/capa/customrender.py
index 60d3ce578b9..9d7ff719acc 100644
--- a/common/lib/capa/capa/customrender.py
+++ b/common/lib/capa/capa/customrender.py
@@ -10,8 +10,6 @@ from .registry import TagRegistry
import logging
import re
-import shlex # for splitting quoted strings
-import json
from lxml import etree
import xml.sax.saxutils as saxutils
diff --git a/common/lib/capa/capa/responsetypes.py b/common/lib/capa/capa/responsetypes.py
index 80227490dad..be70e3866cd 100644
--- a/common/lib/capa/capa/responsetypes.py
+++ b/common/lib/capa/capa/responsetypes.py
@@ -11,7 +11,6 @@ Used by capa_problem.py
# standard library imports
import abc
import cgi
-import hashlib
import inspect
import json
import logging
diff --git a/common/lib/capa/capa/tests/test_html_render.py b/common/lib/capa/capa/tests/test_html_render.py
index 62605b48f59..9bc326d7b92 100644
--- a/common/lib/capa/capa/tests/test_html_render.py
+++ b/common/lib/capa/capa/tests/test_html_render.py
@@ -2,7 +2,6 @@ import unittest
from lxml import etree
import os
import textwrap
-import json
import mock
diff --git a/common/lib/capa/capa/util.py b/common/lib/capa/capa/util.py
index ec43da6093f..433e99171d0 100644
--- a/common/lib/capa/capa/util.py
+++ b/common/lib/capa/capa/util.py
@@ -1,4 +1,4 @@
-from calc import evaluator, UndefinedVariable
+from calc import evaluator
from cmath import isinf
#-----------------------------------------------------------------------------
diff --git a/common/lib/chem/chem/chemcalc.py b/common/lib/chem/chem/chemcalc.py
index 5b800050449..612e63c0f0d 100644
--- a/common/lib/chem/chem/chemcalc.py
+++ b/common/lib/chem/chem/chemcalc.py
@@ -1,16 +1,7 @@
from __future__ import division
-import copy
from fractions import Fraction
-import logging
-import math
-import operator
-import re
-import numpy
-import numbers
-import scipy.constants
-
-from pyparsing import (Literal, Keyword, Word, nums, StringEnd, Optional,
- Forward, OneOrMore, ParseException)
+
+from pyparsing import (Literal, StringEnd, OneOrMore, ParseException)
import nltk
from nltk.tree import Tree
diff --git a/common/lib/symmath/symmath/formula.py b/common/lib/symmath/symmath/formula.py
index 8369baa27cf..a926d9ae459 100644
--- a/common/lib/symmath/symmath/formula.py
+++ b/common/lib/symmath/symmath/formula.py
@@ -10,7 +10,6 @@
# Provides sympy representation.
import os
-import sys
import string
import re
import logging
@@ -25,8 +24,7 @@ from sympy.physics.quantum.state import *
# from sympy.core.operations import LatticeOp
# import sympy.physics.quantum.qubit
-import urllib
-from xml.sax.saxutils import escape, unescape
+from xml.sax.saxutils import unescape
import sympy
import unicodedata
from lxml import etree
diff --git a/common/lib/symmath/symmath/symmath_check.py b/common/lib/symmath/symmath/symmath_check.py
index 65a17883f5e..3f09ebf659a 100644
--- a/common/lib/symmath/symmath/symmath_check.py
+++ b/common/lib/symmath/symmath/symmath_check.py
@@ -8,10 +8,6 @@
#
# Takes in math expressions given as Presentation MathML (from ASCIIMathML), converts to Content MathML using SnuggleTeX
-import os
-import sys
-import string
-import re
import traceback
from .formula import *
import logging
diff --git a/common/lib/xmodule/xmodule/modulestore/tests/factories.py b/common/lib/xmodule/xmodule/modulestore/tests/factories.py
index 99c5ec2c910..0a62849d8d0 100644
--- a/common/lib/xmodule/xmodule/modulestore/tests/factories.py
+++ b/common/lib/xmodule/xmodule/modulestore/tests/factories.py
@@ -1,5 +1,4 @@
from factory import Factory, lazy_attribute_sequence, lazy_attribute
-from time import gmtime
from uuid import uuid4
from xmodule.modulestore import Location
from xmodule.modulestore.django import modulestore
diff --git a/common/lib/xmodule/xmodule/modulestore/tests/test_modulestore.py b/common/lib/xmodule/xmodule/modulestore/tests/test_modulestore.py
index 469eedac051..1e2035075a8 100644
--- a/common/lib/xmodule/xmodule/modulestore/tests/test_modulestore.py
+++ b/common/lib/xmodule/xmodule/modulestore/tests/test_modulestore.py
@@ -1,6 +1,6 @@
-from nose.tools import assert_equals, assert_raises, assert_not_equals, with_setup
+from nose.tools import assert_equals, assert_raises
-from xmodule.modulestore.exceptions import ItemNotFoundError, NoPathToItem
+from xmodule.modulestore.exceptions import ItemNotFoundError
from xmodule.modulestore.search import path_to_location
diff --git a/common/lib/xmodule/xmodule/modulestore/tests/test_mongo.py b/common/lib/xmodule/xmodule/modulestore/tests/test_mongo.py
index 07e61245378..c5ef0d751af 100644
--- a/common/lib/xmodule/xmodule/modulestore/tests/test_mongo.py
+++ b/common/lib/xmodule/xmodule/modulestore/tests/test_mongo.py
@@ -1,6 +1,5 @@
import pymongo
-from mock import Mock
from nose.tools import assert_equals, assert_raises, assert_not_equals, assert_false
from pprint import pprint
diff --git a/common/lib/xmodule/xmodule/open_ended_grading_classes/combined_open_ended_modulev1.py b/common/lib/xmodule/xmodule/open_ended_grading_classes/combined_open_ended_modulev1.py
index 01be4c61ab0..9fc438d4c0b 100644
--- a/common/lib/xmodule/xmodule/open_ended_grading_classes/combined_open_ended_modulev1.py
+++ b/common/lib/xmodule/xmodule/open_ended_grading_classes/combined_open_ended_modulev1.py
@@ -1,13 +1,10 @@
import json
import logging
from lxml import etree
-from lxml.html import rewrite_links
from xmodule.timeinfo import TimeInfo
from xmodule.capa_module import ComplexEncoder
-from xmodule.editing_module import EditingDescriptor
from xmodule.progress import Progress
from xmodule.stringify import stringify_children
-from xmodule.xml_module import XmlDescriptor
import self_assessment_module
import open_ended_module
from .combined_open_ended_rubric import CombinedOpenEndedRubric, GRADER_TYPE_IMAGE_DICT, HUMAN_GRADER_TYPE, LEGEND_LIST
diff --git a/common/lib/xmodule/xmodule/open_ended_grading_classes/grading_service_module.py b/common/lib/xmodule/xmodule/open_ended_grading_classes/grading_service_module.py
index 3e3f943cd73..6857876703e 100644
--- a/common/lib/xmodule/xmodule/open_ended_grading_classes/grading_service_module.py
+++ b/common/lib/xmodule/xmodule/open_ended_grading_classes/grading_service_module.py
@@ -3,7 +3,6 @@ import json
import logging
import requests
from requests.exceptions import RequestException, ConnectionError, HTTPError
-import sys
from .combined_open_ended_rubric import CombinedOpenEndedRubric
from lxml import etree
diff --git a/common/lib/xmodule/xmodule/open_ended_grading_classes/open_ended_image_submission.py b/common/lib/xmodule/xmodule/open_ended_grading_classes/open_ended_image_submission.py
index 2eb95022690..ea5c3b35277 100644
--- a/common/lib/xmodule/xmodule/open_ended_grading_classes/open_ended_image_submission.py
+++ b/common/lib/xmodule/xmodule/open_ended_grading_classes/open_ended_image_submission.py
@@ -14,9 +14,7 @@ from urlparse import urlparse
import requests
from boto.s3.connection import S3Connection
from boto.s3.key import Key
-import pickle
import logging
-import re
log = logging.getLogger(__name__)
diff --git a/common/lib/xmodule/xmodule/open_ended_grading_classes/open_ended_module.py b/common/lib/xmodule/xmodule/open_ended_grading_classes/open_ended_module.py
index 1e5b1b233b5..2ac55a83187 100644
--- a/common/lib/xmodule/xmodule/open_ended_grading_classes/open_ended_module.py
+++ b/common/lib/xmodule/xmodule/open_ended_grading_classes/open_ended_module.py
@@ -11,10 +11,8 @@ from lxml import etree
import capa.xqueue_interface as xqueue_interface
from xmodule.capa_module import ComplexEncoder
-from xmodule.editing_module import EditingDescriptor
from xmodule.progress import Progress
from xmodule.stringify import stringify_children
-from xmodule.xml_module import XmlDescriptor
from capa.util import *
import openendedchild
diff --git a/common/lib/xmodule/xmodule/open_ended_grading_classes/openendedchild.py b/common/lib/xmodule/xmodule/open_ended_grading_classes/openendedchild.py
index b5d4e1b676e..4f524d2cd7f 100644
--- a/common/lib/xmodule/xmodule/open_ended_grading_classes/openendedchild.py
+++ b/common/lib/xmodule/xmodule/open_ended_grading_classes/openendedchild.py
@@ -3,14 +3,8 @@ import logging
from lxml.html.clean import Cleaner, autolink_html
import re
-from xmodule.capa_module import ComplexEncoder
import open_ended_image_submission
-from xmodule.editing_module import EditingDescriptor
-from xmodule.html_checker import check_html
from xmodule.progress import Progress
-from xmodule.stringify import stringify_children
-from xmodule.xml_module import XmlDescriptor
-from xmodule.modulestore import Location
from capa.util import *
from .peer_grading_service import PeerGradingService, MockPeerGradingService
import controller_query_service
diff --git a/common/lib/xmodule/xmodule/progress.py b/common/lib/xmodule/xmodule/progress.py
index 7adbb02646e..bad5105fd0f 100644
--- a/common/lib/xmodule/xmodule/progress.py
+++ b/common/lib/xmodule/xmodule/progress.py
@@ -13,7 +13,6 @@ For most subclassing needs, you should only need to reimplement
frac() and __str__().
'''
-from collections import namedtuple
import numbers
diff --git a/common/lib/xmodule/xmodule/schematic_module.py b/common/lib/xmodule/xmodule/schematic_module.py
index d15d629c24d..83bcc5351d6 100644
--- a/common/lib/xmodule/xmodule/schematic_module.py
+++ b/common/lib/xmodule/xmodule/schematic_module.py
@@ -1,4 +1,3 @@
-import json
from .x_module import XModule, XModuleDescriptor
diff --git a/common/lib/xmodule/xmodule/template_module.py b/common/lib/xmodule/xmodule/template_module.py
index 9a9666c0b6f..bf8f616913c 100644
--- a/common/lib/xmodule/xmodule/template_module.py
+++ b/common/lib/xmodule/xmodule/template_module.py
@@ -3,7 +3,6 @@ from xmodule.raw_module import RawDescriptor
from lxml import etree
from mako.template import Template
from xmodule.modulestore.django import modulestore
-import logging
class CustomTagModule(XModule):
diff --git a/common/lib/xmodule/xmodule/tests/test_html_module.py b/common/lib/xmodule/xmodule/tests/test_html_module.py
index e0a49ed98f6..4fe02423787 100644
--- a/common/lib/xmodule/xmodule/tests/test_html_module.py
+++ b/common/lib/xmodule/xmodule/tests/test_html_module.py
@@ -3,7 +3,6 @@ import unittest
from mock import Mock
from xmodule.html_module import HtmlModule
-from xmodule.modulestore import Location
from . import get_test_system
diff --git a/common/lib/xmodule/xmodule/tests/test_peer_grading.py b/common/lib/xmodule/xmodule/tests/test_peer_grading.py
index 3e1a5781187..c386f77e9ba 100644
--- a/common/lib/xmodule/xmodule/tests/test_peer_grading.py
+++ b/common/lib/xmodule/xmodule/tests/test_peer_grading.py
@@ -2,10 +2,6 @@ import unittest
from xmodule.modulestore import Location
from .import get_test_system
from test_util_open_ended import MockQueryDict, DummyModulestore
-import json
-
-from xmodule.peer_grading_module import PeerGradingModule, PeerGradingDescriptor
-from xmodule.open_ended_grading_classes.grading_service_module import GradingServiceError
import logging
diff --git a/common/lib/xmodule/xmodule/tests/test_randomize_module.py b/common/lib/xmodule/xmodule/tests/test_randomize_module.py
index 81935c4013e..81ba45b56c0 100644
--- a/common/lib/xmodule/xmodule/tests/test_randomize_module.py
+++ b/common/lib/xmodule/xmodule/tests/test_randomize_module.py
@@ -1,11 +1,6 @@
import unittest
-from time import strptime
-from fs.memoryfs import MemoryFS
-
-from mock import Mock, patch
-
-from xmodule.modulestore.xml import ImportSystem, XMLModuleStore
+from .test_course_module import DummySystem as DummyImportSystem
ORG = 'test_org'
COURSE = 'test_course'
@@ -13,9 +8,6 @@ COURSE = 'test_course'
START = '2013-01-01T01:00:00'
-from .test_course_module import DummySystem as DummyImportSystem
-
-
class RandomizeModuleTestCase(unittest.TestCase):
"""Make sure the randomize module works"""
@staticmethod
diff --git a/common/lib/xmodule/xmodule/tests/test_stringify.py b/common/lib/xmodule/xmodule/tests/test_stringify.py
index e44b93b0b8d..6c2e44eed51 100644
--- a/common/lib/xmodule/xmodule/tests/test_stringify.py
+++ b/common/lib/xmodule/xmodule/tests/test_stringify.py
@@ -1,4 +1,4 @@
-from nose.tools import assert_equals, assert_true, assert_false
+from nose.tools import assert_equals
from lxml import etree
from xmodule.stringify import stringify_children
diff --git a/common/lib/xmodule/xmodule/tests/test_util_open_ended.py b/common/lib/xmodule/xmodule/tests/test_util_open_ended.py
index 9dbb17ae2f8..63fb4631c98 100644
--- a/common/lib/xmodule/xmodule/tests/test_util_open_ended.py
+++ b/common/lib/xmodule/xmodule/tests/test_util_open_ended.py
@@ -1,6 +1,6 @@
from .import get_test_system
from xmodule.modulestore import Location
-from xmodule.modulestore.xml import ImportSystem, XMLModuleStore
+from xmodule.modulestore.xml import XMLModuleStore
from xmodule.tests.test_export import DATA_DIR
OPEN_ENDED_GRADING_INTERFACE = {
diff --git a/common/lib/xmodule/xmodule/timelimit_module.py b/common/lib/xmodule/xmodule/timelimit_module.py
index 6be14e7574c..9446176f01e 100644
--- a/common/lib/xmodule/xmodule/timelimit_module.py
+++ b/common/lib/xmodule/xmodule/timelimit_module.py
@@ -1,4 +1,3 @@
-import json
import logging
from lxml import etree
--
GitLab