From 006d22de508d18af93899c5ab32651681f816575 Mon Sep 17 00:00:00 2001 From: Raphael Isemann Date: Wed, 21 Aug 2019 12:57:06 +0000 Subject: [PATCH] [lldb] Add tests for setting completions and enable 'settings remove' completion llvm-svn: 369521 --- .../completion/TestCompletion.py | 18 ++++++++++++++++++ lldb/source/Commands/CommandObjectSettings.cpp | 2 ++ 2 files changed, 20 insertions(+) diff --git a/lldb/packages/Python/lldbsuite/test/functionalities/completion/TestCompletion.py b/lldb/packages/Python/lldbsuite/test/functionalities/completion/TestCompletion.py index d41666a1866a..b95d490c8872 100644 --- a/lldb/packages/Python/lldbsuite/test/functionalities/completion/TestCompletion.py +++ b/lldb/packages/Python/lldbsuite/test/functionalities/completion/TestCompletion.py @@ -215,6 +215,24 @@ class CommandLineCompletionTestCase(TestBase): 'settings replace target.ru', 'settings replace target.run-args') + @skipIfFreeBSD # timing out on the FreeBSD buildbot + def test_settings_show_term(self): + self.complete_from_to( + 'settings show term-', + 'settings show term-width') + + @skipIfFreeBSD # timing out on the FreeBSD buildbot + def test_settings_list_term(self): + self.complete_from_to( + 'settings list term-', + 'settings list term-width') + + @skipIfFreeBSD # timing out on the FreeBSD buildbot + def test_settings_remove_term(self): + self.complete_from_to( + 'settings remove term-', + 'settings remove term-width') + @skipIfFreeBSD # timing out on the FreeBSD buildbot def test_settings_s(self): """Test that 'settings s' completes to ['set', 'show'].""" diff --git a/lldb/source/Commands/CommandObjectSettings.cpp b/lldb/source/Commands/CommandObjectSettings.cpp index d756e031de58..200ef34d8a12 100644 --- a/lldb/source/Commands/CommandObjectSettings.cpp +++ b/lldb/source/Commands/CommandObjectSettings.cpp @@ -613,6 +613,8 @@ public: ~CommandObjectSettingsRemove() override = default; + bool WantsCompletion() override { return true; } + int HandleArgumentCompletion( CompletionRequest &request, OptionElementVector &opt_element_vector) override {