[lldb-dev] [Bug 28939] New: test infra: convert pexpect errors into failures

via lldb-dev lldb-dev at lists.llvm.org
Thu Aug 11 09:04:47 PDT 2016


            Bug ID: 28939
           Summary: test infra: convert pexpect errors into failures
           Product: lldb
           Version: unspecified
          Hardware: PC
                OS: All
            Status: NEW
          Severity: normal
          Priority: P
         Component: All Bugs
          Assignee: lldb-dev at lists.llvm.org
          Reporter: todd.fiala at gmail.com
                CC: llvm-bugs at lists.llvm.org
    Classification: Unclassified

When LLDB runs pexpect tests, and the pexpect condition does not match
(typically due to a pexpect timeout), this shows up as a test error.  This is
because unittest sees an exception thrown that isn't one of its assert

In reality, the pexpect timeout is a failure - something didn't match.  It
isn't an error, which is reserved to indicate something totally off the rails
went wrong.  A non-match is a failure, not an off-the-rails error.

We should be able to catch pexpect timeouts and adapt them to show up as

You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20160811/f87a5dfc/attachment-0001.html>

More information about the lldb-dev mailing list