rnhmjoj/qutebrowser
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Fix lint

Browse Source
This commit is contained in:
Florian Bruhin 2018-09-08 13:59:11 +02:00
parent 7cb32dca07
commit 9c67239847
3 changed files with 10 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
scripts/dev/run_pylint_on_tests.py
View File

@ -65,9 +65,12 @@ def main():
toxinidir, toxinidir,
] ]
args = (['--disable={}'.format(','.join(disabled)), args = [
'--ignored-modules=helpers,pytest,PyQt5'] + '--disable={}'.format(','.join(disabled)),
sys.argv[2:] + files) '--ignored-modules=helpers,pytest,PyQt5',
r'--ignore-long-lines=(<?https?://|^# Copyright 201\d)|^ *def [a-z]',
r'--method-rgx=[a-z_][A-Za-z0-9_]{1,100}$',
] + sys.argv[2:] + files
env = os.environ.copy() env = os.environ.copy()
env['PYTHONPATH'] = os.pathsep.join(pythonpath) env['PYTHONPATH'] = os.pathsep.join(pythonpath)

2
tests/helpers/fixtures.py
View File

@ -166,9 +166,7 @@ def webkit_tab(qtbot, tab_registry, cookiejar_and_cache, mode_manager,
private=False) private=False)
vbox.addWidget(tab) vbox.addWidget(tab)
# to make sure container isn't GCed # to make sure container isn't GCed
# pylint: disable=attribute-defined-outside-init
tab.container = container tab.container = container
# pylint: enable=attribute-defined-outside-init
with qtbot.waitExposed(container): with qtbot.waitExposed(container):
container.show() container.show()

12
tests/unit/browser/test_caret.py
View File

@ -135,8 +135,7 @@ class TestBlock:
caret.move_to_prev_word() caret.move_to_prev_word()
selection.check("drei") selection.check("drei")
def test_moving_back_to_the_start_of_prev_block_with_sel(self, def test_moving_back_to_the_start_of_prev_block_with_sel(self, caret, selection):
caret, selection):
caret.move_to_end_of_next_block(2) caret.move_to_end_of_next_block(2)
selection.toggle() selection.toggle()
caret.move_to_start_of_prev_block() caret.move_to_start_of_prev_block()
@ -153,8 +152,7 @@ class TestBlock:
caret.move_to_next_word() caret.move_to_next_word()
selection.check("eins ") selection.check("eins ")
def test_moving_to_the_start_of_next_block_with_sel(self, def test_moving_to_the_start_of_next_block_with_sel(self, caret, selection):
caret, selection):
selection.toggle() selection.toggle()
caret.move_to_start_of_next_block() caret.move_to_start_of_next_block()
selection.check("one two three\n") selection.check("one two three\n")
@ -290,8 +288,7 @@ class TestSearch:
@pytest.mark.qtbug60673 @pytest.mark.qtbug60673
@pytest.mark.no_xvfb @pytest.mark.no_xvfb
def test_yanking_a_searched_line( def test_yanking_a_searched_line(self, caret, selection, mode_manager, callback_checker, web_tab, qtbot):
self, caret, selection, mode_manager, callback_checker, web_tab, qtbot):
web_tab.show() web_tab.show()
mode_manager.leave(usertypes.KeyMode.caret) mode_manager.leave(usertypes.KeyMode.caret)
@ -304,8 +301,7 @@ class TestSearch:
@pytest.mark.qtbug60673 @pytest.mark.qtbug60673
@pytest.mark.no_xvfb @pytest.mark.no_xvfb
def test_yanking_a_searched_line_with_multiple_matches( def test_yanking_a_searched_line_with_multiple_matches(self, caret, selection, mode_manager, callback_checker, web_tab, qtbot):
self, caret, selection, mode_manager, callback_checker, web_tab, qtbot):
web_tab.show() web_tab.show()
mode_manager.leave(usertypes.KeyMode.caret) mode_manager.leave(usertypes.KeyMode.caret)