[Lldb-commits] [PATCH] D67073: [dotest] Add @skipIfCursesSupportMissing and annotate the new gui test

[Jan Kratochvil via Phabricator via lldb-commits]

jankratochvil accepted this revision.
jankratochvil added inline comments.


================
Comment at: packages/Python/lldbsuite/test/decorators.py:786
 
-def skipIfXmlSupportMissing(func):
+def _get_bool_config_decorator(key):
     config = lldb.SBDebugger.GetBuildConfiguration()
----------------
I find a bit confusing the function returns opposite of `key` existence but its name does not indicate that.  The `skipIf` could stay in its name.



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

https://reviews.llvm.org/D67073