s/pylint: disable=W0621/pylint: disable=redefined-outer-name/

This commit is contained in:
Sarina Canelake
2014-11-29 22:50:05 -05:00
parent eacd52568c
commit 0dad9da5e4
96 changed files with 107 additions and 107 deletions

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from lettuce import world, step
from nose.tools import assert_false, assert_equal, assert_regexp_matches # pylint: disable=E0611

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from lettuce import world, step
from nose.tools import assert_true, assert_equal # pylint: disable=E0611

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
import os
from lettuce import world, step

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
# Lettuce formats proposed definitions for unimplemented steps with the
# argument name "step" instead of "_step" and pylint does not like that.

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
# pylint: disable=W0613
from lettuce import world, step

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from lettuce import world, step
from common import *

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from lettuce import world, step
from terrain.steps import reload_the_page

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from lettuce import world, step
from nose.tools import assert_in # pylint: disable=E0611

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from lettuce import world, step
from selenium.webdriver.common.keys import Keys

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
# pylint: disable=W0613
import os

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from lettuce import world, step
from common import *

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from lettuce import world, step
from common import *

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
# pylint: disable=W0613
from nose.tools import assert_false # pylint: disable=no-name-in-module

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
# pylint: disable=W0613
from lettuce import world, step

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from lettuce import world, step

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from lettuce import world, step
from django.conf import settings

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from lettuce import world, step
from lettuce.django import django_url

View File

@@ -71,4 +71,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['course_creators']
complete_apps = ['course_creators']

View File

@@ -89,4 +89,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['course_action_state']
complete_apps = ['course_action_state']

View File

@@ -37,4 +37,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['course_modes']
complete_apps = ['course_modes']

View File

@@ -32,4 +32,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['course_modes']
complete_apps = ['course_modes']

View File

@@ -30,4 +30,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['course_modes']
complete_apps = ['course_modes']

View File

@@ -33,4 +33,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['course_modes']
complete_apps = ['course_modes']

View File

@@ -34,4 +34,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['course_modes']
complete_apps = ['course_modes']

View File

@@ -41,4 +41,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['course_modes']
complete_apps = ['course_modes']

View File

@@ -61,4 +61,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['course_modes']
complete_apps = ['course_modes']

View File

@@ -71,4 +71,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['dark_lang']
complete_apps = ['dark_lang']

View File

@@ -111,4 +111,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['embargo']
complete_apps = ['embargo']

View File

@@ -87,4 +87,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['external_auth']
complete_apps = ['external_auth']

View File

@@ -33,4 +33,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['reverification']
complete_apps = ['reverification']

View File

@@ -170,4 +170,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['student']
complete_apps = ['student']

View File

@@ -180,4 +180,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['student']
complete_apps = ['student']

View File

@@ -185,4 +185,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['student']
complete_apps = ['student']

View File

@@ -191,4 +191,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['student']
complete_apps = ['student']

View File

@@ -182,4 +182,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['student']
complete_apps = ['student']

View File

@@ -143,4 +143,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['student']
complete_apps = ['student']

View File

@@ -153,4 +153,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['student']
complete_apps = ['student']

View File

@@ -186,4 +186,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['student']
complete_apps = ['student']

View File

@@ -176,4 +176,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['student']
complete_apps = ['student']

View File

@@ -177,4 +177,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['student']
complete_apps = ['student']

View File

@@ -171,4 +171,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['student']
complete_apps = ['student']

View File

@@ -169,4 +169,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['student']
complete_apps = ['student']

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
import urllib
from lettuce import world

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
# Disable the "wildcard import" warning so we can bring in all methods from
# course helpers and ui helpers

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from lettuce import world

View File

@@ -75,4 +75,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['user_api']
complete_apps = ['user_api']

View File

@@ -84,4 +84,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['user_api']
complete_apps = ['user_api']

View File

@@ -69,4 +69,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['user_api']
complete_apps = ['user_api']

View File

@@ -92,4 +92,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['bulk_email']
complete_apps = ['bulk_email']

View File

@@ -87,4 +87,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['bulk_email']
complete_apps = ['bulk_email']

View File

@@ -83,4 +83,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['certificates']
complete_apps = ['certificates']

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from lettuce import world, step
from lettuce.django import django_url

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from __future__ import absolute_import

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from lettuce import world, steps
from nose.tools import assert_in, assert_true # pylint: disable=no-name-in-module

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from lettuce import world, step
from lettuce.django import django_url

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
# pylint: disable=unused-argument
from lettuce import world, step

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from lettuce import world, step
from nose.tools import assert_equals, assert_greater # pylint: disable=E0611

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from lettuce import step, world
from django.contrib.auth.models import User

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from lettuce import world, step
from common import course_location

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from lettuce import world, step
from lettuce.django import django_url

View File

@@ -3,7 +3,7 @@ Steps for problem.feature lettuce tests
'''
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from lettuce import world, step
from common import i_am_registered_for_the_course, visit_scenario_item

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
# EVERY PROBLEM TYPE MUST HAVE THE FOLLOWING:
# -Section in Dictionary containing:

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from lettuce import world, step
from lettuce.django import django_url

View File

@@ -1,5 +1,5 @@
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from lettuce import world, step

View File

@@ -106,4 +106,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['courseware']
complete_apps = ['courseware']

View File

@@ -97,4 +97,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['courseware']
complete_apps = ['courseware']

View File

@@ -182,4 +182,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['courseware']
complete_apps = ['courseware']

View File

@@ -125,4 +125,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['courseware']
complete_apps = ['courseware']

View File

@@ -145,4 +145,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['courseware']
complete_apps = ['courseware']

View File

@@ -108,4 +108,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['foldit']
complete_apps = ['foldit']

View File

@@ -3,7 +3,7 @@ Define steps for bulk email acceptance test.
"""
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from lettuce import world, step
from lettuce.django import mail

View File

@@ -3,7 +3,7 @@ Define common steps for instructor dashboard acceptance tests.
"""
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from __future__ import absolute_import

View File

@@ -4,7 +4,7 @@ acceptance tests.
"""
# pylint: disable=C0111
# pylint: disable=W0621
# pylint: disable=redefined-outer-name
from lettuce import world, step
from nose.tools import assert_in, assert_regexp_matches # pylint: disable=E0611

View File

@@ -747,7 +747,7 @@ def get_grading_config(request, course_id):
@ensure_csrf_cookie
@cache_control(no_cache=True, no_store=True, must_revalidate=True)
@require_level('staff')
def get_sale_records(request, course_id, csv=False): # pylint: disable=W0613, W0621
def get_sale_records(request, course_id, csv=False): # pylint: disable=W0613, redefined-outer-name
"""
return the summary of all sales records for a particular course
"""
@@ -778,7 +778,7 @@ def get_sale_records(request, course_id, csv=False): # pylint: disable=W0613, W
@ensure_csrf_cookie
@cache_control(no_cache=True, no_store=True, must_revalidate=True)
@require_level('staff')
def get_sale_order_records(request, course_id): # pylint: disable=W0613, W0621
def get_sale_order_records(request, course_id): # pylint: disable=W0613, redefined-outer-name
"""
return the summary of all sales records for a particular course
"""
@@ -878,7 +878,7 @@ def re_validate_invoice(obj_invoice):
@ensure_csrf_cookie
@cache_control(no_cache=True, no_store=True, must_revalidate=True)
@require_level('staff')
def get_purchase_transaction(request, course_id, csv=False): # pylint: disable=W0613, W0621
def get_purchase_transaction(request, course_id, csv=False): # pylint: disable=W0613, redefined-outer-name
"""
return the summary of all purchased transactions for a particular course
"""
@@ -906,7 +906,7 @@ def get_purchase_transaction(request, course_id, csv=False): # pylint: disable=
@ensure_csrf_cookie
@cache_control(no_cache=True, no_store=True, must_revalidate=True)
@require_level('staff')
def get_students_features(request, course_id, csv=False): # pylint: disable=W0613, W0621
def get_students_features(request, course_id, csv=False): # pylint: disable=W0613, redefined-outer-name
"""
Respond with json which contains a summary of all enrolled students profile information.
@@ -1107,7 +1107,7 @@ def generate_registration_codes(request, course_id):
internal_reference=internal_reference, customer_reference_number=customer_reference_number
)
registration_codes = []
for _ in range(course_code_number): # pylint: disable=W0621
for _ in range(course_code_number): # pylint: disable=redefined-outer-name
generated_registration_code = save_registration_code(request.user, course_id, sale_invoice, order=None)
registration_codes.append(generated_registration_code)

View File

@@ -83,4 +83,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['instructor_task']
complete_apps = ['instructor_task']

View File

@@ -73,4 +73,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['instructor_task']
complete_apps = ['instructor_task']

View File

@@ -67,4 +67,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['linkedin']
complete_apps = ['linkedin']

View File

@@ -87,4 +87,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['notes']
complete_apps = ['notes']

View File

@@ -111,4 +111,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['shoppingcart']
complete_apps = ['shoppingcart']

View File

@@ -129,4 +129,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['shoppingcart']
complete_apps = ['shoppingcart']

View File

@@ -128,4 +128,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['shoppingcart']
complete_apps = ['shoppingcart']

View File

@@ -139,4 +139,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['shoppingcart']
complete_apps = ['shoppingcart']

View File

@@ -186,4 +186,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['shoppingcart']
complete_apps = ['shoppingcart']

View File

@@ -213,4 +213,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['shoppingcart']
complete_apps = ['shoppingcart']

View File

@@ -184,4 +184,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['shoppingcart']
complete_apps = ['shoppingcart']

View File

@@ -183,4 +183,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['shoppingcart']
complete_apps = ['shoppingcart']

View File

@@ -249,4 +249,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['shoppingcart']
complete_apps = ['shoppingcart']

View File

@@ -170,4 +170,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['shoppingcart']
complete_apps = ['shoppingcart']

View File

@@ -241,4 +241,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['shoppingcart']
complete_apps = ['shoppingcart']

View File

@@ -226,4 +226,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['shoppingcart']
complete_apps = ['shoppingcart']

View File

@@ -202,4 +202,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['shoppingcart']
complete_apps = ['shoppingcart']

View File

@@ -185,4 +185,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['shoppingcart']
complete_apps = ['shoppingcart']

View File

@@ -151,7 +151,7 @@ class ShoppingCartViewsTests(ModuleStoreTestCase):
resp = self.client.get(billing_url)
self.assertEqual(resp.status_code, 200)
((template, context), _) = render_mock.call_args # pylint: disable=W0621
((template, context), _) = render_mock.call_args # pylint: disable=redefined-outer-name
self.assertEqual(template, 'shoppingcart/billing_details.html')
# check for the default currency in the context
self.assertEqual(context['currency'], 'usd')
@@ -179,7 +179,7 @@ class ShoppingCartViewsTests(ModuleStoreTestCase):
resp = self.client.get(billing_url)
self.assertEqual(resp.status_code, 200)
((template, context), _) = render_mock.call_args # pylint: disable=W0621
((template, context), _) = render_mock.call_args # pylint: disable=redefined-outer-name
self.assertEqual(template, 'shoppingcart/billing_details.html')
# check for the override currency settings in the context
self.assertEqual(context['currency'], 'PKR')
@@ -669,7 +669,7 @@ class ShoppingCartViewsTests(ModuleStoreTestCase):
resp = self.client.get(reverse('shoppingcart.views.show_cart', args=[]))
self.assertEqual(resp.status_code, 200)
((purchase_form_arg_cart,), _) = form_mock.call_args # pylint: disable=W0621
((purchase_form_arg_cart,), _) = form_mock.call_args # pylint: disable=redefined-outer-name
purchase_form_arg_cart_items = purchase_form_arg_cart.orderitem_set.all().select_subclasses()
self.assertIn(reg_item, purchase_form_arg_cart_items)
self.assertIn(cert_item, purchase_form_arg_cart_items)
@@ -693,7 +693,7 @@ class ShoppingCartViewsTests(ModuleStoreTestCase):
resp = self.client.get(reverse('shoppingcart.views.show_cart', args=[]))
self.assertEqual(resp.status_code, 200)
((purchase_form_arg_cart,), _) = form_mock.call_args # pylint: disable=W0621
((purchase_form_arg_cart,), _) = form_mock.call_args # pylint: disable=redefined-outer-name
purchase_form_arg_cart_items = purchase_form_arg_cart.orderitem_set.all().select_subclasses()
self.assertIn(reg_item, purchase_form_arg_cart_items)
@@ -849,7 +849,7 @@ class ShoppingCartViewsTests(ModuleStoreTestCase):
resp = self.client.get(reverse('shoppingcart.views.show_receipt', args=[self.cart.id]))
self.assertEqual(resp.status_code, 200)
((template, context), _) = render_mock.call_args # pylint: disable=W0621
((template, context), _) = render_mock.call_args # pylint: disable=redefined-outer-name
self.assertEqual(template, 'shoppingcart/receipt.html')
self.assertEqual(context['order'], self.cart)
self.assertEqual(context['order'].total_cost, self.testing_cost)
@@ -882,7 +882,7 @@ class ShoppingCartViewsTests(ModuleStoreTestCase):
self.assertIn('FirstNameTesting123', resp.content)
self.assertIn('80.00', resp.content)
((template, context), _) = render_mock.call_args # pylint: disable=W0621
((template, context), _) = render_mock.call_args # pylint: disable=redefined-outer-name
self.assertEqual(template, 'shoppingcart/receipt.html')
self.assertEqual(context['order'], self.cart)
self.assertIn(reg_item, context['shoppingcart_items'][0])
@@ -903,7 +903,7 @@ class ShoppingCartViewsTests(ModuleStoreTestCase):
resp = self.client.get(reverse('shoppingcart.views.show_receipt', args=[self.cart.id]))
self.assertEqual(resp.status_code, 200)
((template, context), _) = render_mock.call_args # pylint: disable=W0621
((template, context), _) = render_mock.call_args # pylint: disable=redefined-outer-name
self.assertEqual(template, 'shoppingcart/receipt.html')
self.assertIn(reg_item, context['shoppingcart_items'][0])
self.assertIn(cert_item, context['shoppingcart_items'][1])
@@ -944,7 +944,7 @@ class ShoppingCartViewsTests(ModuleStoreTestCase):
# check for the enrollment codes content
self.assertIn('Please send each professional one of these unique registration codes to enroll into the course.', resp.content)
((template, context), _) = render_mock.call_args # pylint: disable=W0621
((template, context), _) = render_mock.call_args # pylint: disable=redefined-outer-name
self.assertEqual(template, 'shoppingcart/receipt.html')
self.assertEqual(context['order'], self.cart)
self.assertIn(reg_item, context['shoppingcart_items'][0])

View File

@@ -93,4 +93,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['verify_student']
complete_apps = ['verify_student']

View File

@@ -85,4 +85,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['verify_student']
complete_apps = ['verify_student']

View File

@@ -86,4 +86,4 @@ class Migration(SchemaMigration):
}
}
complete_apps = ['verify_student']
complete_apps = ['verify_student']