maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[lldb] Fix TestCompletion by using SIGPIPE instead of SIGINT as test signal 

The test I added in commit 078003482e was using
SIGINT for testing the tab completion. The idea is to have a signal that only
has one possible completion and I ended up picking SIGIN -> SIGINT for the test.
However on non-Linux systems there is SIGINFO which is a valid completion for
`SIGIN' and so the test fails there.

This replaces SIGIN -> SIGINT with SIGPIP -> SIGPIPE completion which according
to LLDB's signal list in Host.cpp is the only valid completion.
This commit is contained in:
Raphael Isemann 2021-07-22 15:31:12 +02:00
parent f6413d8aaa
commit eb61ffbcb2
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lldb/test/API/functionalities/completion/TestCompletion.py
View File

@ -164,8 +164,8 @@ class CommandLineCompletionTestCase(TestBase):
self.complete_from_to('process signal ',
'process signal SIG')
self.complete_from_to('process signal SIGIN',
'process signal SIGINT')
self.complete_from_to('process signal SIGPIP',
'process signal SIGPIPE')
self.complete_from_to('process signal SIGA',
['SIGABRT',
'SIGALRM'])