[Lldb-commits] [PATCH] D71151: [lldb][test] Remove symlink for API tests.

[Jordan Rupprecht via Phabricator via lldb-commits]

rupprecht added a comment.
Herald added subscribers: wuzish, dexonsmith.

Since the diff is large, I'll paste the only non-mechanical part here:

  % git diff HEAD^1 -- lldb/packages/Python/lldbsuite/__init__.py lldb/test/API/lit.cfg.py
  diff --git a/lldb/packages/Python/lldbsuite/__init__.py b/lldb/packages/Python/lldbsuite/__init__.py
  index c98c668ed6b..188f6ed1532 100644
  --- a/lldb/packages/Python/lldbsuite/__init__.py
  +++ b/lldb/packages/Python/lldbsuite/__init__.py
  @@ -22,11 +22,4 @@ lldb_root = find_lldb_root()
   
   # lldbsuite.lldb_test_src_root refers to the root of the python test case tree
   # (i.e. the actual unit tests).
  -lldb_test_root = os.path.join(
  -    lldb_root,
  -    "packages",
  -    "Python",
  -    "lldbsuite",
  -    "test")
  -# TODO(rupprecht): update the above definition after moving test cases:
  -# lldb_test_root = os.path.join(lldb_root, "test", "API", "test")
  +lldb_test_root = os.path.join(lldb_root, "test", "API", "test")
  diff --git a/lldb/test/API/lit.cfg.py b/lldb/test/API/lit.cfg.py
  index 9b1c3c12f17..90d8a141e9a 100644
  --- a/lldb/test/API/lit.cfg.py
  +++ b/lldb/test/API/lit.cfg.py
  @@ -17,8 +17,7 @@ config.suffixes = ['.py']
   
   # test_source_root: The root path where tests are located.
   # test_exec_root: The root path where tests should be run.
  -config.test_source_root = os.path.join(config.lldb_src_root, 'packages',
  -                                       'Python', 'lldbsuite', 'test')
  +config.test_source_root = os.path.dirname(__file__)
   config.test_exec_root = config.test_source_root
   
   if 'Address' in config.llvm_use_sanitizer:


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71151