[Lldb-commits] [PATCH] D50525: [WIP] Move lldb-mi interpreter tests to LIT

[Alexander Polyakov via Phabricator via lldb-commits]

apolyakov created this revision.
apolyakov added reviewers: aprantl, clayborg, jingham, labath.
Herald added a subscriber: ki.stfu.

In this patch I move some of interpreter tests from python to LIT(I will move all interpreter test if these are OK). It's a WIP since I want to get your opinion about tests like target-list.test. As you may see, in this test we must run lldb-mi without passing an executable, it means that we can't pass it to lldb-mi through LIT. As a solution, I hardcoded executable name as a.exe. What do you think about this approach? We already have one test with hardcoded executable name - lit/tools/lldb-mi/breakpoint/break-insert.test, but I want to be sure that it's OK.


Repository:
  rLLDB LLDB

https://reviews.llvm.org/D50525

Files:
  lit/tools/lldb-mi/interpreter/cli-support/breakpoint-set.test
  lit/tools/lldb-mi/interpreter/cli-support/inputs/main.cpp
  lit/tools/lldb-mi/interpreter/cli-support/lit.local.cfg
  lit/tools/lldb-mi/interpreter/cli-support/settings-set-target-run-after.test
  lit/tools/lldb-mi/interpreter/cli-support/settings-set-target-run-before.test
  lit/tools/lldb-mi/interpreter/cli-support/target-create.test
  lit/tools/lldb-mi/interpreter/cli-support/target-list.test

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D50525.159969.patch
Type: text/x-patch
Size: 4751 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20180809/0afc27b6/attachment.bin>