diff --git a/common/djangoapps/third_party_auth/tests/test_change_enrollment.py b/common/djangoapps/third_party_auth/tests/test_change_enrollment.py index 73ecd1022b..db76ff6904 100644 --- a/common/djangoapps/third_party_auth/tests/test_change_enrollment.py +++ b/common/djangoapps/third_party_auth/tests/test_change_enrollment.py @@ -62,7 +62,7 @@ class PipelineEnrollmentTest(ModuleStoreTestCase): strategy = self._fake_strategy() strategy.session_set('enroll_course_id', unicode(self.course.id)) - result = pipeline.change_enrollment(strategy, 1, user=self.user) # pylint: disable=E1111,E1124 + result = pipeline.change_enrollment(strategy, 1, user=self.user) # pylint: disable=assignment-from-no-return,redundant-keyword-arg self.assertEqual(result, {}) # Check that the user was or was not enrolled @@ -86,7 +86,7 @@ class PipelineEnrollmentTest(ModuleStoreTestCase): # Simulate the pipeline step for enrolling in this course strategy = self._fake_strategy() strategy.session_set('enroll_course_id', unicode(self.course.id)) - result = pipeline.change_enrollment(strategy, 1, user=self.user) # pylint: disable=E1111,E1124 + result = pipeline.change_enrollment(strategy, 1, user=self.user) # pylint: disable=assignment-from-no-return,redundant-keyword-arg self.assertEqual(result, {}) # Expect that the uesr is NOT enrolled in the course @@ -109,7 +109,7 @@ class PipelineEnrollmentTest(ModuleStoreTestCase): # This should fail silently, with no exception strategy = self._fake_strategy() strategy.session_set('enroll_course_id', unicode(self.course.id)) - result = pipeline.change_enrollment(strategy, 1, user=self.user) # pylint: disable=E1111,E1124 + result = pipeline.change_enrollment(strategy, 1, user=self.user) # pylint: disable=assignment-from-no-return,redundant-keyword-arg self.assertEqual(result, {}) # Verify that we were NOT enrolled @@ -117,7 +117,7 @@ class PipelineEnrollmentTest(ModuleStoreTestCase): def test_no_course_id_skips_enroll(self): strategy = self._fake_strategy() - result = pipeline.change_enrollment(strategy, 1, user=self.user) # pylint: disable=E1111,E1124 + result = pipeline.change_enrollment(strategy, 1, user=self.user) # pylint: disable=assignment-from-no-return,redundant-keyword-arg self.assertEqual(result, {}) self.assertFalse(CourseEnrollment.is_enrolled(self.user, self.course.id)) diff --git a/lms/djangoapps/instructor/tests/test_api.py b/lms/djangoapps/instructor/tests/test_api.py index 65fca3c5ae..55c7ad3e57 100644 --- a/lms/djangoapps/instructor/tests/test_api.py +++ b/lms/djangoapps/instructor/tests/test_api.py @@ -2,7 +2,7 @@ """ Unit tests for instructor.api methods. """ -# pylint: disable=E1111 + import unittest import json import requests @@ -111,26 +111,26 @@ class TestCommonExceptions400(unittest.TestCase): def test_user_doesnotexist(self): self.request.is_ajax.return_value = False - resp = view_user_doesnotexist(self.request) + resp = view_user_doesnotexist(self.request) # pylint: disable=assignment-from-no-return self.assertEqual(resp.status_code, 400) self.assertIn("User does not exist", resp.content) def test_user_doesnotexist_ajax(self): self.request.is_ajax.return_value = True - resp = view_user_doesnotexist(self.request) + resp = view_user_doesnotexist(self.request) # pylint: disable=assignment-from-no-return self.assertEqual(resp.status_code, 400) result = json.loads(resp.content) self.assertIn("User does not exist", result["error"]) def test_alreadyrunningerror(self): self.request.is_ajax.return_value = False - resp = view_alreadyrunningerror(self.request) + resp = view_alreadyrunningerror(self.request) # pylint: disable=assignment-from-no-return self.assertEqual(resp.status_code, 400) self.assertIn("Task is already running", resp.content) def test_alreadyrunningerror_ajax(self): self.request.is_ajax.return_value = True - resp = view_alreadyrunningerror(self.request) + resp = view_alreadyrunningerror(self.request) # pylint: disable=assignment-from-no-return self.assertEqual(resp.status_code, 400) result = json.loads(resp.content) self.assertIn("Task is already running", result["error"])