[all-commits] [llvm/llvm-project] 8be41c: [test] Allow skipTestIfFn to apply to entire class...

Jordan Rupprecht via All-commits all-commits at lists.llvm.org
Thu Nov 17 08:17:26 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 8be41c787f9e12a4eb101e2dcad67ddc2da7ec1f
      https://github.com/llvm/llvm-project/commit/8be41c787f9e12a4eb101e2dcad67ddc2da7ec1f
  Author: Jordan Rupprecht <rupprecht at google.com>
  Date:   2022-11-17 (Thu, 17 Nov 2022)

  Changed paths:
    M lldb/packages/Python/lldbsuite/test/decorators.py
    M lldb/test/API/api/check_public_api_headers/TestPublicAPIHeaders.py
    M lldb/test/API/api/multithreaded/TestMultithreaded.py
    M lldb/test/API/functionalities/plugins/command_plugin/TestPluginCommands.py

  Log Message:
  -----------
  [test] Allow skipTestIfFn to apply to entire classes for skipIfNoSBHeaders

Some test cases are already marked @skipIfNoSBHeaders, but they make use of SBAPI headers in test setup. The setup will fail if the headers are missing, so it is too late to wait until the test case to apply the skip annotation.

In addition to allowing this to apply to entire classes, I also changed all the existing annotations from test cases to test classes where necessary/appropriate.

Reviewed By: DavidSpickett

Differential Revision: https://reviews.llvm.org/D138181




More information about the All-commits mailing list