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

Add test summary #1

Merged
merged 3 commits into from
Jul 7, 2021
Merged

Add test summary #1

merged 3 commits into from
Jul 7, 2021

Conversation

tahirmt
Copy link
Owner

@tahirmt tahirmt commented Jul 7, 2021

Adds changes from fastlane-community#40 and adds the total number of tests and failures to environment variables.
Also fixes the format of the junit file to include line number

tahirmt and others added 3 commits July 1, 2021 00:30
* Fix xcpretty_naming for xcresult
* Add test result object hierarchy
* Refactored group and name parsing function

# Conflicts:
#	Gemfile.lock
#	lib/trainer/test_parser.rb
@tahirmt tahirmt merged commit 3adc805 into master Jul 7, 2021
@tahirmt tahirmt deleted the add-test-summary branch July 7, 2021 21:55
@kikettas
Copy link

Any plan on merging these fixes in the original repo?

@tahirmt
Copy link
Owner Author

tahirmt commented Sep 27, 2021

Any plan on merging these fixes in the original repo?

I didn’t open a PR there because I merged an open PR from the original repo that was open for two years and added some stuff to it here.

@kikettas
Copy link

But it fixes the missing failed tests, right? I'm wondering if the original repo can take advantage of them.

@tahirmt
Copy link
Owner Author

tahirmt commented Sep 27, 2021

@kikettas yeah but it doesn’t look like the original repo is maintained.

@kikettas
Copy link

I see. It's a pity then, as it's the probably the best extracting tool for xcresult files.

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

Successfully merging this pull request may close these issues.

4 participants