diff options
author | Greg Clayton <gclayton@apple.com> | 2012-08-22 17:17:09 +0000 |
---|---|---|
committer | Greg Clayton <gclayton@apple.com> | 2012-08-22 17:17:09 +0000 |
commit | 67cc06366cdb20795d21c31f89624d81e97af799 (patch) | |
tree | 43edfb98f1f542853282474c9685a5e071d2e1d9 /lldb/test/help/TestHelp.py | |
parent | 40dd4d9bf396f17b28fe823160f3e103d2943cfb (diff) | |
download | bcm5719-llvm-67cc06366cdb20795d21c31f89624d81e97af799.tar.gz bcm5719-llvm-67cc06366cdb20795d21c31f89624d81e97af799.zip |
Reimplemented the code that backed the "settings" in lldb. There were many issues with the previous implementation:
- no setting auto completion
- very manual and error prone way of getting/setting variables
- tons of code duplication
- useless instance names for processes, threads
Now settings can easily be defined like option values. The new settings makes use of the "OptionValue" classes so we can re-use the option value code that we use to set settings in command options. No more instances, just "does the right thing".
llvm-svn: 162366
Diffstat (limited to 'lldb/test/help/TestHelp.py')
-rw-r--r-- | lldb/test/help/TestHelp.py | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/lldb/test/help/TestHelp.py b/lldb/test/help/TestHelp.py index 736a92f1f82..839874d1973 100644 --- a/lldb/test/help/TestHelp.py +++ b/lldb/test/help/TestHelp.py @@ -87,7 +87,10 @@ class HelpCommandTestCase(TestBase): def test_help_should_not_hang_emacsshell(self): """Command 'settings set term-width 0' should not hang the help command.""" - self.runCmd("settings set term-width 0") + self.expect("settings set term-width 0", + COMMAND_FAILED_AS_EXPECTED, error=True, + substrs = ['error: 0 is out of range, valid values must be between']) + # self.runCmd("settings set term-width 0") self.expect("help", startstr = 'The following is a list of built-in, permanent debugger commands') |