[all-commits] [llvm/llvm-project] 8bed75: [lldb][test] Add `pexpect` category for tests that...

[Jordan Rupprecht via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 8bed754c2f965c8cbbb050be6f650b78f7fd78a6
      https://github.com/llvm/llvm-project/commit/8bed754c2f965c8cbbb050be6f650b78f7fd78a6
  Author: Jordan Rupprecht <rupprecht at google.com>
  Date:   2024-03-13 (Wed, 13 Mar 2024)

  Changed paths:
    M lldb/packages/Python/lldbsuite/test/decorators.py
    M lldb/packages/Python/lldbsuite/test/dotest.py
    M lldb/packages/Python/lldbsuite/test/lldbpexpect.py
    M lldb/packages/Python/lldbsuite/test/test_categories.py
    M lldb/packages/Python/lldbsuite/test/test_result.py
    M lldb/test/API/benchmarks/expression/TestExpressionCmd.py
    M lldb/test/API/benchmarks/expression/TestRepeatedExprs.py
    M lldb/test/API/benchmarks/frame_variable/TestFrameVariableResponse.py
    M lldb/test/API/benchmarks/startup/TestStartupDelays.py
    M lldb/test/API/benchmarks/stepping/TestSteppingSpeed.py
    M lldb/test/API/benchmarks/turnaround/TestCompileRunToBreakpointTurnaround.py
    M lldb/test/API/terminal/TestSTTYBeforeAndAfter.py

  Log Message:
  -----------
  [lldb][test] Add `pexpect` category for tests that `import pexpect` (#84860)

Instead of directly annotating pexpect-based tests with
`@skipIfWindows`, we can tag them with a new `pexpect` category. We
still automatically skip windows behavior by adding `pexpect` to the
skip category list if the platform is windows, but also allow
non-Windows users to skip them by configuring cmake with
`-DLLDB_TEST_USER_ARGS=--skip-category=pexpect`

As a prerequisite, remove the restriction that `@add_test_categories`
can only apply to test cases, and we make the test runner look for
categories on both the class and the test method.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications