Page MenuHomePhabricator

[lldb] [test] Skip ObjC-based tests via 'objc' category

Authored by mgorny on Nov 9 2020, 1:52 AM.



Replace the plethora of ObjC-implied 'skipUnlessDarwin' decorators
with marking tests as 'objc' category (whenever missing), and skip all
ObjC tests on non-Darwin platforms. I have used '.categories' file
wherever it was present already or all (>1) tests were relying on ObjC,
and explicit add_test_categories() where there was only one test.

Event Timeline

mgorny created this revision.Nov 9 2020, 1:52 AM
mgorny requested review of this revision.Nov 9 2020, 1:52 AM
mgorny added inline comments.Nov 9 2020, 1:55 AM
17 ↗(On Diff #304122)

(this whole category has objc in .categories)


This one's curious. I'm wondering why it's explicitly marked for all the platforms while it.s clearly a .m file ;-).

11 ↗(On Diff #304122)


mgorny updated this revision to Diff 303800.Nov 9 2020, 2:42 AM

Failed to git add one of the .categories files.

teemperor accepted this revision.Nov 10 2020, 2:02 AM

LGTM. Thanks for cleaning this up, it's really appreciated!


nit: we do from lldbsuite.test import lldbplatformutil in the surrounding code, so that seems more consistent.


For the record: I think those XFails were just copied to all sanitizers tests that were failing on Linux (independently of whether the issue was lacking sanitizer support on Linux or as here Obj-C support missing).

This revision is now accepted and ready to land.Nov 10 2020, 2:02 AM
mgorny marked an inline comment as done.Nov 10 2020, 2:25 AM
mgorny added inline comments.

Thanks, will fix it in place.

This revision was automatically updated to reflect the committed changes.
mgorny marked an inline comment as done.
Herald added a project: Restricted Project. · View Herald TranscriptNov 10 2020, 3:02 AM