Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: xcresult reader (WIP) #84

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

feat: xcresult reader (WIP) #84

wants to merge 10 commits into from

Conversation

delatrie
Copy link
Collaborator

@delatrie delatrie commented Jan 31, 2025

The PR adds support for xcresult files to Allure 3.

This is still WIP. Remaining work:

  • Tidy up the tests, disable for non-MacOS
  • Cover activities -> steps/attachments with more tests
  • Cover different combinations of args/retries/devices with tests
  • Check well-known activities -> Allure entities conversion
  • Error traces
  • Legacy API to bypass the new xcresulttool API bugs/limitations
    • Bug URLs
    • Better stack traces & source references
  • Get rid of extra attachment-only steps

Improvements compared to allure-xcresults:

  • Parametrized tests
  • Destinations (a.k.a., devices)
  • Attachments are chronological
  • Suites
  • Packages (bundles, which were suites previously)
  • Tags
  • Bugs

In this PR I experiment a little with sort of typed unknowns to move parsing checks from tests (which are tricky to write in case of ill-formed xcresult files) to the type system. I'm still in the process of figuring out if it is good enough to stay in the code base.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant