Clean up pylint failures
This commit is contained in:
@@ -746,7 +746,7 @@ class AcceptanceTest(WebAppTest):
|
||||
os.environ.get('BOK_CHOY_HOSTNAME', '127.0.0.1'),
|
||||
os.environ.get('BOK_CHOY_LMS_PORT', 8003),
|
||||
))
|
||||
except:
|
||||
except: # pylint: disable=bare-except
|
||||
self.browser.get('http://{}:{}'.format(
|
||||
os.environ.get('BOK_CHOY_HOSTNAME', '127.0.0.1'),
|
||||
os.environ.get('BOK_CHOY_CMS_PORT', 8031),
|
||||
|
||||
@@ -143,7 +143,11 @@ class CMSVideoBaseTest(UniqueCourseTest):
|
||||
xblock_index: number starting from 1 (0th entry is the unit page itself)
|
||||
"""
|
||||
self.unit_page.xblocks[xblock_index].edit()
|
||||
EmptyPromise(lambda: self.video.q(css='div.basic_metadata_edit').visible, "Wait for the basic editor to be open", timeout=5).fulfill()
|
||||
EmptyPromise(
|
||||
lambda: self.video.q(css='div.basic_metadata_edit').visible,
|
||||
"Wait for the basic editor to be open",
|
||||
timeout=5
|
||||
).fulfill()
|
||||
|
||||
def open_advanced_tab(self):
|
||||
"""
|
||||
|
||||
@@ -120,7 +120,8 @@ class TestPaverJavaScriptTestTasks(PaverTestCase):
|
||||
for suite in suites:
|
||||
# Karma test command
|
||||
karma_config_file = Env.KARMA_CONFIG_FILES[Env.JS_TEST_ID_KEYS.index(suite)]
|
||||
expected_test_tool_command = u'nodejs --max_old_space_size=4096 node_modules/.bin/karma start {options}'.format(
|
||||
command_template = u'nodejs --max_old_space_size=4096 node_modules/.bin/karma start {options}'
|
||||
expected_test_tool_command = command_template.format(
|
||||
options=self.EXPECTED_KARMA_OPTIONS.format(
|
||||
config_file=karma_config_file,
|
||||
single_run='false' if dev_mode else 'true',
|
||||
|
||||
@@ -323,7 +323,8 @@ def run_eslint(options):
|
||||
violations_limit = int(getattr(options, 'limit', -1))
|
||||
|
||||
sh(
|
||||
"nodejs --max_old_space_size=4096 node_modules/.bin/eslint --ext .js --ext .jsx --format=compact . | tee {eslint_report}".format(
|
||||
"nodejs --max_old_space_size=4096 node_modules/.bin/eslint "
|
||||
"--ext .js --ext .jsx --format=compact . | tee {eslint_report}".format(
|
||||
eslint_report=eslint_report
|
||||
),
|
||||
ignore_error=True
|
||||
|
||||
Reference in New Issue
Block a user