[all-commits] [llvm/llvm-project] d77b04: [lldb/test] Move and improve TestPlatformProcessCo...

Pavel Labath via All-commits all-commits at lists.llvm.org
Wed Feb 10 12:01:54 PST 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d77b04e4edc00da3be2e0d2c8a0c80ae3ce0d829
      https://github.com/llvm/llvm-project/commit/d77b04e4edc00da3be2e0d2c8a0c80ae3ce0d829
  Author: Pavel Labath <pavel at labath.sk>
  Date:   2021-02-10 (Wed, 10 Feb 2021)

  Changed paths:
    A lldb/test/API/commands/platform/connect/Makefile
    A lldb/test/API/commands/platform/connect/TestPlatformConnect.py
    A lldb/test/API/commands/platform/connect/main.cpp
    R lldb/test/API/tools/lldb-server/platform-process-connect/Makefile
    R lldb/test/API/tools/lldb-server/platform-process-connect/TestPlatformProcessConnect.py
    R lldb/test/API/tools/lldb-server/platform-process-connect/main.cpp

  Log Message:
  -----------
  [lldb/test] Move and improve TestPlatformProcessConnect.py

Although it is located under tools/lldb-server, this test is very
different that other lldb-server tests. The most important distinction
is that it does not test lldb-server directly, but rather interacts with
it through the lldb client. It also tests the relevant client
functionality (the platform connect command, which is even admitted in
the test name). The fact that this test is structured as a lldb-server
test means it cannot access most of the goodies available to the
"normal" lldb tests (the runCmd function, which it reimplements; the
run_break_set_by_symbol utility function; etc.).

This patch makes it a full-fledged lldb this, and rewrites the relevant
bits to make use of the standard features. I also move the test into the
"commands" subtree to better reflect its new status.




More information about the All-commits mailing list