Have pylint check for unused imports.

circle.yml

@@ -15,7 +15,7 @@ test:
   override:
     - "if [ $CIRCLE_NODE_INDEX = '0' ]; then pep8 problem_builder --max-line-length=120; fi":
         parallel: true
-    - "if [ $CIRCLE_NODE_INDEX = '1' ]; then pylint problem_builder --disable=all --enable=function-redefined,undefined-variable,unused-variable; fi":
+    - "if [ $CIRCLE_NODE_INDEX = '1' ]; then pylint problem_builder --disable=all --enable=function-redefined,undefined-variable,unused-import,unused-variable; fi":
         parallel: true
     - "python run_tests.py":
         parallel: true

problem_builder/answer.py

@@ -26,7 +26,7 @@ from lazy import lazy
 from .models import Answer
 
 from xblock.core import XBlock
-from xblock.fields import Scope, Float, Integer, String
+from xblock.fields import Scope, Integer, String
 from xblock.fragment import Fragment
 from xblock.validation import ValidationMessage
 from xblockutils.resources import ResourceLoader

problem_builder/south_migrations/0003_auto__add_share__add_unique_share_shared_by_shared_with_block_id.py

 # -*- coding: utf-8 -*-
-from south.utils import datetime_utils as datetime
 from south.db import db
 from south.v2 import SchemaMigration
-from django.db import models
 
 
 class Migration(SchemaMigration):

problem_builder/table.py

@@ -26,7 +26,7 @@ from django.contrib.auth.models import User
 
 from xblock.core import XBlock
 from xblock.exceptions import JsonHandlerError
-from xblock.fields import Scope, String, Boolean, Dict
+from xblock.fields import Scope, String, Boolean
 from xblock.fragment import Fragment
 
 from xblockutils.resources import ResourceLoader

problem_builder/tests/integration/test_assessment.py

@@ -17,7 +17,7 @@
 # along with this program in a file in the toplevel directory called
 # "AGPLv3".  If not, see <http://www.gnu.org/licenses/>.
 #
-from ddt import ddt, unpack, data
+from ddt import ddt, data
 from .base_test import CORRECT, INCORRECT, PARTIAL, MentoringAssessmentBaseTest, GetChoices
 
 

problem_builder/tests/integration/test_author_changes.py

@@ -3,8 +3,8 @@ If an author makes changes to the block after students have started using it, wi
 happen?
 """
 import time
+
 from .base_test import ProblemBuilderBaseTest, MentoringAssessmentBaseTest
-import re
 
 
 class AuthorChangesTest(ProblemBuilderBaseTest):

problem_builder/tests/integration/test_dashboard.py

@@ -21,7 +21,6 @@ from textwrap import dedent
 from mock import Mock, patch
 from .base_test import ProblemBuilderBaseTest
 from selenium.common.exceptions import NoSuchElementException
-from xblockutils.resources import ResourceLoader
 
 
 class MockSubmissionsAPI(object):

problem_builder/tests/unit/test_dashboard_visual.py

 """
 Unit tests for DashboardVisualData
 """
-from problem_builder.dashboard_visual import DashboardVisualData
-from mock import MagicMock, Mock
 import unittest
-from xblock.field_data import DictFieldData
+
+from problem_builder.dashboard_visual import DashboardVisualData
 
 
 class TestDashboardVisualData(unittest.TestCase):