[Lldb-commits] [PATCH] D134515: [lldb] Fix completion of 'settings set' values

[David Spickett via Phabricator via lldb-commits]

DavidSpickett added a reviewer: DavidSpickett.
DavidSpickett added inline comments.


================
Comment at: lldb/test/API/functionalities/completion/TestCompletion.py:409
+    def test_settings_set_stop_disassembly_display_value(self):
+        """Test that 'settins set stop-disassembly-display ' completes to [
+        'never', 'always', 'no-debuginfo', 'no-source']."""
----------------
settings


================
Comment at: lldb/test/API/functionalities/completion/TestCompletion.py:410
+        """Test that 'settins set stop-disassembly-display ' completes to [
+        'never', 'always', 'no-debuginfo', 'no-source']."""
+        self.complete_from_to('settings set stop-disassembly-display ',
----------------
Could you add to the comment why this one was chosen in particular? Something like:
```
Checks that we can complete a setting that has enum values.
```

Since there is a `settings set` test above but clearly that wasn't enough coverage.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D134515/new/

https://reviews.llvm.org/D134515