ChargePoint / xcparse

Command line tool & Swift framework for parsing Xcode 11+ xcresult
MIT License
393 stars 47 forks source link

[feature request] Option to remove UUID in the images #42

Open tedgonzalez opened 4 years ago

tedgonzalez commented 4 years ago

Describe Hi thanks for the awesome tool! I'm just wondering if there's an easy way to remove the UUID from the image filenames?

Desktop (please complete the following information):

abotkin-cpi commented 4 years ago

Not via xcparse right now. The screenshot file names do follow a standard format so it should be possible to remove the UUIDs from the names (right now we just take the name of the attachment in the xcresult and output as that name which includes the UUID).

It’s a feature we should probably do and allow for some customization of attachment names like I mentioned here. Likely won’t get to this for another 2 weeks as engulfed in another project, so if someone wants to take a crack, feel free to comment here.

twink1e commented 4 years ago

I also find the UUID a bit unwieldy. How about renaming the screenshots to reflect the test name that it comes from? It will be easier to debug and track. e.g. {classname}{methodname}{idx}.{file_type}

If it sounds good I can take up the task

AbhayVarshney commented 4 years ago

@abotkin-cpi this is really great. Is there a timeline for the suggestion in the PR for adding flag "--ordered" so we know which screenshots were taken first?

abotkin-cpi commented 4 years ago

I'm looking to bring the ordered into xcparse in the next week or so.

With the file naming, I want to make an extensible format string so folks can pick what they want in there rather than hard-defining it & ensure we handle the 255 character limit for filenames & de-duplication resolution. That's likely tied to a major release before iOS 14 launch.

ryanpato commented 4 years ago

Hey @abotkin-cpi, is the work for ordered still ongoing? Really interested in this one!

abotkin-cpi commented 4 years ago

We've been a bit distracted recently, but expect to have this rolled up by end of this week.