From c28d68173647e8f068c9e12c3102ffc2cb86b624 Mon Sep 17 00:00:00 2001 From: Luca Benci Date: Thu, 2 Nov 2017 19:42:33 +0100 Subject: [PATCH] Change test to avoid calling private functions --- tests/unit/completion/test_completer.py | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/tests/unit/completion/test_completer.py b/tests/unit/completion/test_completer.py index 934872f2b..a32241621 100644 --- a/tests/unit/completion/test_completer.py +++ b/tests/unit/completion/test_completer.py @@ -303,12 +303,11 @@ def test_quickcomplete_flicker(status_command_stub, completer_obj, config_stub.val.completion.quick = True _set_cmd_prompt(status_command_stub, ':open |') + completer_obj.schedule_completion_update() + assert completion_widget_stub.set_model.called + completion_widget_stub.set_model.reset_mock() - url = 'http://example.com' - completer_obj._change_completed_part = unittest.mock.Mock() - completer_obj.on_selection_changed(url) - - # no immediate (default is false) - completer_obj._change_completed_part.assert_called_with(url, # text - ['open'], # before - []) # after + # selecting a completion should not re-set the model + completer_obj.on_selection_changed('http://example.com') + completer_obj.schedule_completion_update() + assert not completion_widget_stub.set_model.called