[Lldb-commits] [PATCH] D42656: [testsuite] Remove flakey test relying on `pexpect`

Davide Italiano via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Jan 29 12:45:41 PST 2018


davide added a comment.

In https://reviews.llvm.org/D42656#991065, @jingham wrote:

> There are a whole bunch of other tests that test completion in this file that use the exact same mechanism but don't seem to be flakey.  Why is this one test flakey?
>
> If for instance it's because "Fo" ends up being ambiguous because we chose too common a start string, then you could trivially fix the test by choosing a more uncommon name to complete on.  But I'd want to know why this test is flakey first.


OK, I'll take a look at this further.


https://reviews.llvm.org/D42656





More information about the lldb-commits mailing list