Recommit [lldb/test] Make category-skipping logic "platform"-independent
authorPavel Labath <pavel@labath.sk>
Mon, 14 Mar 2022 15:32:51 +0000 (16:32 +0100)
committerPavel Labath <pavel@labath.sk>
Wed, 30 Mar 2022 15:16:37 +0000 (17:16 +0200)
commit21c5bb0a636c23ec75b13681c0a6fdb03ecd9c0d
treee36c3e5282768e9e21ebc8154c7ac9f2ba2a19eb
parentc531171d995728f55e5775d354a21dceed03edce
Recommit [lldb/test] Make category-skipping logic "platform"-independent

This recommits dddf4ce03, which was reverted because of a couple of test
failures on macos. The reason behind the failures was that the patch
inadvertenly changed the value returned by the host platform from
"macosx" to "darwin". The new version fixes that.

Original commit message was:

The decision which categories are relevant for a particular test run
happen very early in the test setup process. They use the SBPlatform
object to determine which categories should be skipped. The platform
object created for this purpose transcends individual test runs.

This setup is not compatible with the direction discussed in
<https://discourse.llvm.org/t/multiple-platforms-with-the-same-name/59594>
-- when platform objects are tied to a specific (SB)Debugger, they need
to be created alongside it, which currently happens in the test setUp
method.

This patch is the first step in that direction -- it rewrites the
category skipping logic to avoid depending on a global SBPlatform
object. Fortunately, the skipping logic is fairly simple (and I believe
it outght to stay that way) and mainly consists of comparing the
platform name against some hardcoded lists. This patch bases this
comparison on the platform name instead of the os part of the triple (as
reported by the platform).

Differential Revision: https://reviews.llvm.org/D121605
lldb/packages/Python/lldbsuite/test/dotest.py
lldb/packages/Python/lldbsuite/test/lldbplatformutil.py
lldb/test/API/functionalities/paths/TestPaths.py