[Lldb-commits] [PATCH] D85106: Move TestGuiBasicDebug.py to lldb/test and update it

Jim Ingham via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Aug 4 16:19:08 PDT 2020


jingham added a comment.

In D85106#2194840 <https://reviews.llvm.org/D85106#2194840>, @clayborg wrote:

> lldb/test/API is usually for testing the lldb::SB* interfaces IIRC. So not sure this move makes sense?

It's really for any test that wants to use the lldbtest test harness.  There are other tests in the commands sub-directory that use lldbutil methods to get the test up and going, then a bunch of self.expects to test various command line output.  Oddly enough there are also some tests in there (like the disassemble/basic test) that don't seem to test command line commands at all.  But that's a separate issue.


Repository:
  rLLDB LLDB

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

https://reviews.llvm.org/D85106



More information about the lldb-commits mailing list