From d651cc75b0c6c2f3b9954c1ca5532c332b9c3f92 Mon Sep 17 00:00:00 2001 From: Ryan Roden-Corrent Date: Sun, 11 Sep 2016 18:09:25 -0400 Subject: [PATCH] Fix flake8/pylint errors. --- qutebrowser/completion/completer.py | 2 +- tests/unit/completion/test_completer.py | 4 +++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/qutebrowser/completion/completer.py b/qutebrowser/completion/completer.py index a3a6b5765..993c7cb83 100644 --- a/qutebrowser/completion/completer.py +++ b/qutebrowser/completion/completer.py @@ -276,7 +276,7 @@ class Completer(QObject): including a trailing space and we shouldn't continue completing the current item. """ - text = self._cmd.prefix() + ' '.join([*before, newtext]) + text = self._cmd.prefix() + ' '.join(before + [newtext]) pos = len(text) + (1 if immediate else 0) if after: text += ' ' + ' '.join(after) diff --git a/tests/unit/completion/test_completer.py b/tests/unit/completion/test_completer.py index 16e9cea02..8ed466fa0 100644 --- a/tests/unit/completion/test_completer.py +++ b/tests/unit/completion/test_completer.py @@ -261,6 +261,7 @@ def test_on_selection_changed(before, newtxt, after, completer_obj, after_txt += ' ' check(True, 1, after_txt, after_pos) + def test_quickcomplete_flicker(status_command_stub, completer_obj, completion_widget_stub, config_stub): """Validate fix for #1519: bookmark-load background highlighting quirk. @@ -271,7 +272,8 @@ def test_quickcomplete_flicker(status_command_stub, completer_obj, one completion available, it keeps the completionmenu open. This test validates that the completion model is not re-set after we - quick-complete an entry after maxsplit.""" + quick-complete an entry after maxsplit. + """ model = unittest.mock.Mock() model.data = unittest.mock.Mock(return_value='http://example.com') indexes = [unittest.mock.Mock()]