iOS UIAutomation Shell Script Inspect Option #6

Merged
merged 2 commits into from Jan 19, 2012

Conversation

Projects
None yet
2 participants
Contributor

ohhorob commented Jan 11, 2012

Slightly more substance than a typo fix :)


Allow an option to automatically open the instruments trace after completion.

@ohhorob ohhorob iOS UIAutomation Shell Script Inspect Option
Allow an option to automatically open the instruments trace after completion.
4247809
Owner

baalexander commented Jan 11, 2012

I'm going to think this over tonight to see if there should be an option that returns the file in a way that the script can be piped into open or any other command or just use this inspect option and default to opening.

Contributor

ohhorob commented Jan 11, 2012

Cool. No worries at all.

I went with inspect to keep it all self contained.

Perhaps -v should output to stderr and the main script emit a \n delimited list of the trace files to stdout?

Another thought is using the - output option to mean piping the trace file contents to stdout instead of moving them to an output directory? That would let a script use them as input for detecting errors and hooking into something like Jenkins.

@ohhorob ohhorob Bugfix iOS Mother JS Exception Handler
Typo caused `ReferenceError: Can't find variable: UAITarget` within the exception handler.
9a3b6b0
Contributor

ohhorob commented Jan 12, 2012

Sorry.. not sure why GitHub put these two changes together. I was hoping they would be different pull requests.

Owner

baalexander commented Jan 19, 2012

As I'm still undecided on alternatives to the inspection option, I'd like to merge your inspection option and the exception fix in.

I'll try to make it clear if the inspection option is ever replaced with something liking piping contents to stdout.

Thanks again @ohhorob!

@baalexander baalexander added a commit that referenced this pull request Jan 19, 2012

@baalexander baalexander Merge pull request #6 from ohhorob/master
iOS UIAutomation Shell Script Inspect Option
c7563c0

@baalexander baalexander merged commit c7563c0 into baalexander:master Jan 19, 2012

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