From b4d24f36c839ac3c045214759137fa8a2a7db6d5 Mon Sep 17 00:00:00 2001 From: Sarina Canelake Date: Sun, 30 Nov 2014 00:24:43 -0500 Subject: [PATCH] s/pylint: disable=E1103/pylint: disable=maybe-no-member/ --- common/djangoapps/contentserver/tests/test.py | 10 +++++----- .../student/management/tests/test_transfer_students.py | 2 +- common/djangoapps/student/tests/test_auto_auth.py | 4 ++-- lms/djangoapps/dashboard/tests/test_support.py | 2 +- lms/djangoapps/instructor/views/api.py | 2 +- lms/djangoapps/mobile_api/course_info/tests.py | 4 ++-- lms/djangoapps/mobile_api/video_outlines/tests.py | 2 +- lms/djangoapps/shoppingcart/tests/test_models.py | 2 +- 8 files changed, 14 insertions(+), 14 deletions(-) diff --git a/common/djangoapps/contentserver/tests/test.py b/common/djangoapps/contentserver/tests/test.py index 872dea3af8..9aa5204920 100644 --- a/common/djangoapps/contentserver/tests/test.py +++ b/common/djangoapps/contentserver/tests/test.py @@ -69,7 +69,7 @@ class ContentStoreToyCourseTest(ModuleStoreTestCase): """ self.client.logout() resp = self.client.get(self.url_unlocked) - self.assertEqual(resp.status_code, 200) # pylint: disable=E1103 + self.assertEqual(resp.status_code, 200) def test_locked_asset_not_logged_in(self): """ @@ -78,7 +78,7 @@ class ContentStoreToyCourseTest(ModuleStoreTestCase): """ self.client.logout() resp = self.client.get(self.url_locked) - self.assertEqual(resp.status_code, 403) # pylint: disable=E1103 + self.assertEqual(resp.status_code, 403) def test_locked_asset_not_registered(self): """ @@ -87,7 +87,7 @@ class ContentStoreToyCourseTest(ModuleStoreTestCase): """ self.client.login(username=self.non_staff_usr, password=self.non_staff_pwd) resp = self.client.get(self.url_locked) - self.assertEqual(resp.status_code, 403) # pylint: disable=E1103 + self.assertEqual(resp.status_code, 403) def test_locked_asset_registered(self): """ @@ -99,7 +99,7 @@ class ContentStoreToyCourseTest(ModuleStoreTestCase): self.client.login(username=self.non_staff_usr, password=self.non_staff_pwd) resp = self.client.get(self.url_locked) - self.assertEqual(resp.status_code, 200) # pylint: disable=E1103 + self.assertEqual(resp.status_code, 200) def test_locked_asset_staff(self): """ @@ -107,7 +107,7 @@ class ContentStoreToyCourseTest(ModuleStoreTestCase): """ self.client.login(username=self.staff_usr, password=self.staff_pwd) resp = self.client.get(self.url_locked) - self.assertEqual(resp.status_code, 200) # pylint: disable=E1103 + self.assertEqual(resp.status_code, 200) def test_range_request_full_file(self): """ diff --git a/common/djangoapps/student/management/tests/test_transfer_students.py b/common/djangoapps/student/management/tests/test_transfer_students.py index f9847c037d..a0d14a93aa 100644 --- a/common/djangoapps/student/management/tests/test_transfer_students.py +++ b/common/djangoapps/student/management/tests/test_transfer_students.py @@ -75,7 +75,7 @@ class TestTransferStudents(ModuleStoreTestCase): self.assertTrue(self.signal_fired) # Confirm the analytics event was emitted. - self.mock_tracker.emit.assert_has_calls( # pylint: disable=E1103 + self.mock_tracker.emit.assert_has_calls( # pylint: disable=maybe-no-member [ call( EVENT_NAME_ENROLLMENT_ACTIVATED, diff --git a/common/djangoapps/student/tests/test_auto_auth.py b/common/djangoapps/student/tests/test_auto_auth.py index fd6a8b4092..d94e89d097 100644 --- a/common/djangoapps/student/tests/test_auto_auth.py +++ b/common/djangoapps/student/tests/test_auto_auth.py @@ -163,8 +163,8 @@ class AutoAuthEnabledTestCase(UrlResetMixin, TestCase): # Check that session and CSRF are set in the response for cookie in ['csrftoken', 'sessionid']: - self.assertIn(cookie, response.cookies) # pylint: disable=E1103 - self.assertTrue(response.cookies[cookie].value) # pylint: disable=E1103 + self.assertIn(cookie, response.cookies) # pylint: disable=maybe-no-member + self.assertTrue(response.cookies[cookie].value) # pylint: disable=maybe-no-member class AutoAuthDisabledTestCase(UrlResetMixin, TestCase): diff --git a/lms/djangoapps/dashboard/tests/test_support.py b/lms/djangoapps/dashboard/tests/test_support.py index b008822dac..a2173879cf 100644 --- a/lms/djangoapps/dashboard/tests/test_support.py +++ b/lms/djangoapps/dashboard/tests/test_support.py @@ -106,7 +106,7 @@ class RefundTests(ModuleStoreTestCase): pars['confirmed'] = 'true' response = self.client.post('/support/refund/', pars) self.assertTrue(response.status_code, 302) - response = self.client.get(response.get('location')) # pylint: disable=E1103 + response = self.client.get(response.get('location')) # pylint: disable=maybe-no-member self.assertContains(response, "Unenrolled %s from" % self.student) self.assertContains(response, "Refunded 1 for order id") diff --git a/lms/djangoapps/instructor/views/api.py b/lms/djangoapps/instructor/views/api.py index 3a5a47b18a..7327a43642 100644 --- a/lms/djangoapps/instructor/views/api.py +++ b/lms/djangoapps/instructor/views/api.py @@ -1229,7 +1229,7 @@ def spent_registration_codes(request, course_id): # pylint: disable=W0613 company_name = request.POST['spent_company_name'] if company_name: - spent_codes_list = spent_codes_list.filter(invoice__company_name=company_name) # pylint: disable=E1103 + spent_codes_list = spent_codes_list.filter(invoice__company_name=company_name) # pylint: disable=maybe-no-member csv_type = 'spent' return registration_codes_csv("Spent_Registration_Codes.csv", spent_codes_list, csv_type) diff --git a/lms/djangoapps/mobile_api/course_info/tests.py b/lms/djangoapps/mobile_api/course_info/tests.py index a470e7df75..e46d8f0ee6 100644 --- a/lms/djangoapps/mobile_api/course_info/tests.py +++ b/lms/djangoapps/mobile_api/course_info/tests.py @@ -25,7 +25,7 @@ class TestVideoOutline(ModuleStoreTestCase, APITestCase): url = reverse('course-about-detail', kwargs={'course_id': unicode(self.course.id)}) response = self.client.get(url) self.assertEqual(response.status_code, 200) - self.assertTrue('overview' in response.data) # pylint: disable=E1103 + self.assertTrue('overview' in response.data) # pylint: disable=maybe-no-member def test_handouts(self): url = reverse('course-handouts-list', kwargs={'course_id': unicode(self.course.id)}) @@ -36,5 +36,5 @@ class TestVideoOutline(ModuleStoreTestCase, APITestCase): url = reverse('course-updates-list', kwargs={'course_id': unicode(self.course.id)}) response = self.client.get(url) self.assertEqual(response.status_code, 200) - self.assertEqual(response.data, []) # pylint: disable=E1103 + self.assertEqual(response.data, []) # pylint: disable=maybe-no-member # TODO: add handouts and updates, somehow diff --git a/lms/djangoapps/mobile_api/video_outlines/tests.py b/lms/djangoapps/mobile_api/video_outlines/tests.py index 15ab7b4ab8..ddbc547c5e 100644 --- a/lms/djangoapps/mobile_api/video_outlines/tests.py +++ b/lms/djangoapps/mobile_api/video_outlines/tests.py @@ -127,7 +127,7 @@ class TestVideoOutline(ModuleStoreTestCase, APITestCase): url = reverse('video-summary-list', kwargs={'course_id': unicode(self.course.id)}) response = self.client.get(url) self.assertEqual(response.status_code, 200) - return response.data # pylint: disable=E1103 + return response.data # pylint: disable=maybe-no-member def _create_video_with_subs(self): """ diff --git a/lms/djangoapps/shoppingcart/tests/test_models.py b/lms/djangoapps/shoppingcart/tests/test_models.py index 809c717961..0fa9f6cf72 100644 --- a/lms/djangoapps/shoppingcart/tests/test_models.py +++ b/lms/djangoapps/shoppingcart/tests/test_models.py @@ -180,7 +180,7 @@ class OrderTest(ModuleStoreTestCase): self.assertIn(item.additional_instruction_text, mail.outbox[0].body) # Assert Google Analytics event fired for purchase. - self.mock_tracker.track.assert_called_once_with( # pylint: disable=E1103 + self.mock_tracker.track.assert_called_once_with( # pylint: disable=maybe-no-member 1, 'Completed Order', {