[lldb/test] Change base class of lldb-server tests
authorPavel Labath <pavel@labath.sk>
Thu, 10 Dec 2020 14:25:55 +0000 (15:25 +0100)
committerPavel Labath <pavel@labath.sk>
Thu, 10 Dec 2020 15:21:28 +0000 (16:21 +0100)
commitb505142fa5d301238796318d2d092d6fb3bd2d31
tree42d8ddb5ccb332c1767b38d78cddb6174509625a
parentdb8420825073371ddc077b020634e71e315e38a1
[lldb/test] Change base class of lldb-server tests

lldb-server tests are a very special subclass of "api" tests. As they
communicate with lldb-server directly, they don't actually need most of
facilities provided by our TestBase class. In particular, they don't
need the ability to fork debug info flavours of tests (but they could
use debug server flavours).

This makes them inherit from "Base" instead. This avoids the need to
explicitly mark these tests as NO_DEBUG_INFO_TEST_CASE. Two additional
necessary tweaks were:
- move run_platform_command to the base (Base) class. This is used in
  one test, and can be generally useful when running tests remotely.
- add a "build" method, forwarding to buildDefault. This is to avoid
  updating each test case to use buildDefault (also, "build" sounds
  better). It might be interesting to refactor the (Test)Base classes so
  that all debug info flavour handling happens in TestBase, and the Base
  class provides a simple build method automatically.
lldb/packages/Python/lldbsuite/test/lldbtest.py
lldb/packages/Python/lldbsuite/test/tools/lldb-server/gdbremote_testcase.py