Adjust :bind completion tests

This commit is contained in:
Panagiotis Ktistakis 2017-11-26 14:07:41 +02:00
parent e05dabefdf
commit 2bb8d404d2

View File

@ -73,6 +73,9 @@ def cmdutils_stub(monkeypatch, stubs):
name='scroll', name='scroll',
desc='Scroll the current tab in the given direction.', desc='Scroll the current tab in the given direction.',
modes=()), modes=()),
'tab-close': stubs.FakeCommand(
name='tab-close',
desc='Close the current tab.'),
}) })
@ -101,9 +104,10 @@ def configdata_stub(monkeypatch, configdata_init):
), ),
), ),
default={ default={
'normal': { 'normal': collections.OrderedDict([
'<ctrl+q>': 'quit' ('<ctrl+q>', 'quit'),
} ('d', 'tab-close'),
])
}, },
backends=[], backends=[],
raw_backends=None)), raw_backends=None)),
@ -122,6 +126,7 @@ def configdata_stub(monkeypatch, configdata_init):
('<ctrl+q>', 'quit'), ('<ctrl+q>', 'quit'),
('ZQ', 'quit'), ('ZQ', 'quit'),
('I', 'invalid'), ('I', 'invalid'),
('d', 'scroll down'),
]) ])
}, },
backends=[], backends=[],
@ -209,6 +214,7 @@ def test_command_completion(qtmodeltester, cmdutils_stub, configdata_stub,
('open', 'open a url', ''), ('open', 'open a url', ''),
('q', "Alias for 'quit'", ''), ('q', "Alias for 'quit'", ''),
('quit', 'quit qutebrowser', 'ZQ, <ctrl+q>'), ('quit', 'quit qutebrowser', 'ZQ, <ctrl+q>'),
('tab-close', 'Close the current tab.', ''),
] ]
}) })
@ -233,7 +239,8 @@ def test_help_completion(qtmodeltester, cmdutils_stub, key_config_stub,
"Commands": [ "Commands": [
(':open', 'open a url', ''), (':open', 'open a url', ''),
(':quit', 'quit qutebrowser', 'ZQ, <ctrl+q>'), (':quit', 'quit qutebrowser', 'ZQ, <ctrl+q>'),
(':scroll', 'Scroll the current tab in the given direction.', '') (':scroll', 'Scroll the current tab in the given direction.', ''),
(':tab-close', 'Close the current tab.', ''),
], ],
"Settings": [ "Settings": [
('aliases', 'Aliases for commands.', None), ('aliases', 'Aliases for commands.', None),
@ -566,9 +573,9 @@ def test_setting_option_completion(qtmodeltester, config_stub,
('aliases', 'Aliases for commands.', '{"q": "quit"}'), ('aliases', 'Aliases for commands.', '{"q": "quit"}'),
('bindings.commands', 'Default keybindings', ('bindings.commands', 'Default keybindings',
'{"normal": {"<ctrl+q>": "quit", "ZQ": "quit", ' '{"normal": {"<ctrl+q>": "quit", "ZQ": "quit", '
'"I": "invalid"}}'), '"I": "invalid", "d": "scroll down"}}'),
('bindings.default', 'Default keybindings', ('bindings.default', 'Default keybindings',
'{"normal": {"<ctrl+q>": "quit"}}'), '{"normal": {"<ctrl+q>": "quit", "d": "tab-close"}}'),
] ]
}) })
@ -589,14 +596,15 @@ def test_bind_completion(qtmodeltester, cmdutils_stub, config_stub,
qtmodeltester.check(model) qtmodeltester.check(model)
_check_completions(model, { _check_completions(model, {
"Current": [ "Current/Default": [
('quit', 'quit qutebrowser', 'ZQ'), ('quit', '(Current) quit qutebrowser', 'ZQ'),
], ],
"Commands": [ "Commands": [
('open', 'open a url', ''), ('open', 'open a url', ''),
('q', "Alias for 'quit'", ''), ('q', "Alias for 'quit'", ''),
('quit', 'quit qutebrowser', 'ZQ, <ctrl+q>'), ('quit', 'quit qutebrowser', 'ZQ, <ctrl+q>'),
('scroll', 'Scroll the current tab in the given direction.', '') ('scroll', 'Scroll the current tab in the given direction.', ''),
('tab-close', 'Close the current tab.', ''),
], ],
}) })
@ -608,21 +616,22 @@ def test_bind_completion_invalid(cmdutils_stub, config_stub, key_config_stub,
model.set_pattern('') model.set_pattern('')
_check_completions(model, { _check_completions(model, {
"Current": [ "Current/Default": [
('invalid', 'Invalid command!', 'I'), ('invalid', '(Current) Invalid command!', 'I'),
], ],
"Commands": [ "Commands": [
('open', 'open a url', ''), ('open', 'open a url', ''),
('q', "Alias for 'quit'", ''), ('q', "Alias for 'quit'", ''),
('quit', 'quit qutebrowser', 'ZQ, <ctrl+q>'), ('quit', 'quit qutebrowser', 'ZQ, <ctrl+q>'),
('scroll', 'Scroll the current tab in the given direction.', '') ('scroll', 'Scroll the current tab in the given direction.', ''),
('tab-close', 'Close the current tab.', ''),
], ],
}) })
def test_bind_completion_no_current(qtmodeltester, cmdutils_stub, config_stub, def test_bind_completion_no_binding(qtmodeltester, cmdutils_stub, config_stub,
key_config_stub, configdata_stub, info): key_config_stub, configdata_stub, info):
"""Test keybinding completion with no current binding.""" """Test keybinding completion with no current or default binding."""
model = configmodel.bind('x', info=info) model = configmodel.bind('x', info=info)
model.set_pattern('') model.set_pattern('')
qtmodeltester.data_display_may_return_none = True qtmodeltester.data_display_may_return_none = True
@ -633,7 +642,30 @@ def test_bind_completion_no_current(qtmodeltester, cmdutils_stub, config_stub,
('open', 'open a url', ''), ('open', 'open a url', ''),
('q', "Alias for 'quit'", ''), ('q', "Alias for 'quit'", ''),
('quit', 'quit qutebrowser', 'ZQ, <ctrl+q>'), ('quit', 'quit qutebrowser', 'ZQ, <ctrl+q>'),
('scroll', 'Scroll the current tab in the given direction.', '') ('scroll', 'Scroll the current tab in the given direction.', ''),
('tab-close', 'Close the current tab.', ''),
],
})
def test_bind_completion_changed(cmdutils_stub, config_stub, key_config_stub,
configdata_stub, info):
"""Test command completion with a non-default command bound."""
model = configmodel.bind('d', info=info)
model.set_pattern('')
_check_completions(model, {
"Current/Default": [
('scroll down',
'(Current) Scroll the current tab in the given direction.', 'd'),
('tab-close', '(Default) Close the current tab.', 'd'),
],
"Commands": [
('open', 'open a url', ''),
('q', "Alias for 'quit'", ''),
('quit', 'quit qutebrowser', 'ZQ, <ctrl+q>'),
('scroll', 'Scroll the current tab in the given direction.', ''),
('tab-close', 'Close the current tab.', ''),
], ],
}) })