Remove Pylint violations deprecated-pragma, bad-option-value

common/djangoapps/embargo/forms.py

@@ -55,7 +55,7 @@ class RestrictedCourseForm(forms.ModelForm):
         return course_key
 
 
-class IPFilterForm(forms.ModelForm):  # pylint: disable=incomplete-protocol
+class IPFilterForm(forms.ModelForm):
     """Form validating entry of IP addresses"""
 
     class Meta:  # pylint: disable=missing-docstring

common/djangoapps/student/views.py

@@ -1043,7 +1043,7 @@ def accounts_login(request):
 
 # Need different levels of logging
 @ensure_csrf_cookie
-def login_user(request, error=""):  # pylint: disable-msg=too-many-statements,unused-argument
+def login_user(request, error=""):  # pylint: disable=too-many-statements,unused-argument
     """AJAX request to log in the user."""
 
     backend_name = None

common/djangoapps/third_party_auth/pipeline.py

@@ -58,7 +58,7 @@ See http://psa.matiasaguirre.net/docs/pipeline.html for more docs.
 """
 
 import random
-import string  # pylint: disable-msg=deprecated-module
+import string  # pylint: disable=deprecated-module
 from collections import OrderedDict
 import urllib
 import analytics
@@ -431,7 +431,7 @@ def running(request):
 # Pipeline functions.
 # Signatures are set by python-social-auth; prepending 'unused_' causes
 # TypeError on dispatch to the auth backend's authenticate().
-# pylint: disable-msg=unused-argument
+# pylint: disable=unused-argument
 
 
 def parse_query_params(strategy, response, *args, **kwargs):

common/djangoapps/third_party_auth/tests/test_pipeline.py

@@ -7,8 +7,8 @@ from third_party_auth.tests import testutil
 import unittest
 
 
-# Allow tests access to protected methods (or module-protected methods) under
-# test. pylint: disable-msg=protected-access
+# Allow tests access to protected methods (or module-protected methods) under test.
+# pylint: disable=protected-access
 
 
 class MakeRandomPasswordTest(testutil.TestCase):

common/djangoapps/third_party_auth/tests/test_pipeline_integration.py

@@ -13,7 +13,7 @@ from social.apps.django_app.default import models as social_models
 
 # Get Django User model by reference from python-social-auth. Not a type
 # constant, pylint.
-User = social_models.DjangoStorage.user.user_model()  # pylint: disable-msg=invalid-name
+User = social_models.DjangoStorage.user.user_model()  # pylint: disable=invalid-name
 
 
 class TestCase(testutil.TestCase, test.TestCase):

common/djangoapps/third_party_auth/tests/test_settings.py

@@ -21,10 +21,10 @@ _SETTINGS_MAP = {
 class SettingsUnitTest(testutil.TestCase):
     """Unit tests for settings management code."""
 
-    # Allow access to protected methods (or module-protected methods) under
-    # test. pylint: disable-msg=protected-access
+    # Allow access to protected methods (or module-protected methods) under test.
+    # pylint: disable=protected-access
     # Suppress sprurious no-member warning on fakes.
-    # pylint: disable-msg=no-member
+    # pylint: disable=no-member
 
     def setUp(self):
         super(SettingsUnitTest, self).setUp()

lms/lib/xblock/test/test_mixin.py

@@ -230,7 +230,7 @@ class XBlockGetParentTest(LmsXBlockMixinTestCase):
                 )
 
 
-class RenamedTuple(tuple):  # pylint: disable=incomplete-protocol
+class RenamedTuple(tuple):
     """
     This class is only used to allow overriding __name__ on the tuples passed
     through ddt, in order to have the generated test names make sense.

pavelib/acceptance_test.py

@@ -8,7 +8,7 @@ from optparse import make_option
 try:
     from pygments.console import colorize
 except ImportError:
-    colorize = lambda color, text: text  # pylint: disable-msg=invalid-name
+    colorize = lambda color, text: text  # pylint: disable=invalid-name
 
 __test__ = False  # do not collect
 

pavelib/bok_choy.py

@@ -12,7 +12,7 @@ import os
 try:
     from pygments.console import colorize
 except ImportError:
-    colorize = lambda color, text: text  # pylint: disable-msg=invalid-name
+    colorize = lambda color, text: text  # pylint: disable=invalid-name
 
 __test__ = False  # do not collect
 

pavelib/i18n.py

@@ -9,7 +9,7 @@ from paver.easy import task, cmdopts, needs, sh
 try:
     from pygments.console import colorize
 except ImportError:
-    colorize = lambda color, text: text  # pylint: disable-msg=invalid-name
+    colorize = lambda color, text: text  # pylint: disable=invalid-name
 
 
 @task

pavelib/tests.py

@@ -11,7 +11,7 @@ from optparse import make_option
 try:
     from pygments.console import colorize
 except ImportError:
-    colorize = lambda color, text: text  # pylint: disable-msg=invalid-name
+    colorize = lambda color, text: text  # pylint: disable=invalid-name
 
 __test__ = False  # do not collect
 

pavelib/utils/test/bokchoy_utils.py

@@ -13,7 +13,7 @@ from pavelib.utils.process import run_background_process
 try:
     from pygments.console import colorize
 except ImportError:
-    colorize = lambda color, text: text  # pylint: disable-msg=invalid-name
+    colorize = lambda color, text: text  # pylint: disable=invalid-name
 
 __test__ = False  # do not collect
 
@@ -80,7 +80,7 @@ def wait_for_server(server, port):
             if int(response.status) == 200:
                 server_ok = True
                 break
-        except:  # pylint: disable-msg=bare-except
+        except:  # pylint: disable=bare-except
             pass
 
         attempts += 1

pavelib/utils/test/suites/bokchoy_suite.py

@@ -12,7 +12,7 @@ from pavelib.utils.test import utils as test_utils
 try:
     from pygments.console import colorize
 except ImportError:
-    colorize = lambda color, text: text  # pylint: disable-msg=invalid-name
+    colorize = lambda color, text: text  # pylint: disable=invalid-name
 
 __test__ = False  # do not collect
 

pavelib/utils/test/suites/suite.py

@@ -10,7 +10,7 @@ from pavelib.utils.process import kill_process
 try:
     from pygments.console import colorize
 except ImportError:
-    colorize = lambda color, text: text  # pylint: disable-msg=invalid-name
+    colorize = lambda color, text: text  # pylint: disable=invalid-name
 
 __test__ = False  # do not collect