You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
If I specify output_style: 'raw' on scan then xcpretty is ripped out from the xcodebuild pipe, as expected (per docs, it disables xcpretty). Because I'm not interested in xcpretty's functionality, I use trainer to generate my junit report. However, when the tests finish, I see this printed:
Generating test results. This may take a while for large projects.
Looking at the code that prints that message, the allegedly expensive operation (per "may take a while") running under the hood is xcpretty, something I've opted out and the reason why this temp_junit_report wasn't found.
Furthermore, the printed test results table says 0 tests due to an xcpretty issue so this final run isn't beneficial anyway.
+--------------------+---+
| Test Results |
+--------------------+---+
| Number of tests | 0 |
| Number of failures | 0 |
+--------------------+---+
Possible solution
For my case specifically, I don't need xcpretty to run at all because trainer will create the junit (faster) and report what I need, so skipping that test_results method when output_style='raw' works for me. Apparently there are people who "disable xcpretty" (output_style=raw) but still want it to run at the end in order to get the test results table as there's a unit test verifying just that. So we can make this a parameter; I actually prefer it to be because it would also allow for clearer documentation.
As of today, gym uses disable_xcpretty for the same purpose that scan uses output_style='raw'. Maybe scan could bring that option in, then force output_style to raw if disable_xcpretty. People who still want xcpretty to run at the end would still use output_style='raw', and people like me who don't, would migrate to disable_xcpretty
Thank you for reading 🙏
The text was updated successfully, but these errors were encountered:
It seems like you have not included the output of fastlane env
To make it easier for us help you resolve this issue, please update the issue to include the output of fastlane env 👍
The pull request #16375 that closed this issue was merged and released as part of fastlane 2.147.0 🚀
Please let us know if the functionality works as expected as a reply here. If it does not, please open a new issue. Thanks!
New Issue Checklist
Issue Description
If I specify
output_style: 'raw'
onscan
then xcpretty is ripped out from the xcodebuild pipe, as expected (per docs, it disables xcpretty). Because I'm not interested in xcpretty's functionality, I usetrainer
to generate my junit report. However, when the tests finish, I see this printed:Looking at the code that prints that message, the allegedly expensive operation (per "may take a while") running under the hood is xcpretty, something I've opted out and the reason why this
temp_junit_report
wasn't found.Furthermore, the printed test results table says 0 tests due to an xcpretty issue so this final run isn't beneficial anyway.
Possible solution
For my case specifically, I don't need xcpretty to run at all because trainer will create the junit (faster) and report what I need, so skipping that
test_results
method whenoutput_style='raw'
works for me. Apparently there are people who "disable xcpretty" (output_style=raw) but still want it to run at the end in order to get the test results table as there's a unit test verifying just that. So we can make this a parameter; I actually prefer it to be because it would also allow for clearer documentation.As of today, gym uses
disable_xcpretty
for the same purpose that scan usesoutput_style='raw'
. Maybe scan could bring that option in, then force output_style toraw
ifdisable_xcpretty
. People who still want xcpretty to run at the end would still useoutput_style='raw'
, and people like me who don't, would migrate todisable_xcpretty
Thank you for reading 🙏
The text was updated successfully, but these errors were encountered: