Skip to content
This repository was archived by the owner on Feb 1, 2018. It is now read-only.

Make bwoken more configurable and resilient to spaces#35

Closed
alecgorge wants to merge 8 commits intobendyworks:masterfrom
hulu:configurability
Closed

Make bwoken more configurable and resilient to spaces#35
alecgorge wants to merge 8 commits intobendyworks:masterfrom
hulu:configurability

Conversation

@alecgorge
Copy link
Contributor

This PR adds two new configuration options to bwoken test:

--product-name          Specify a custom product name (e.g. --product-name="My Product"). Default is the name of of the xcodeproj file
--integration-path      Specify a custom directory to store your test scripts in (e.g. --integration-path=uiautomation/path/dir). Note that this folder still expects the same directory structure as the one create by `bwoken init`.

One new configuration option is added to bwoken init: --integration-path.

These options are important for large existing projects where the product name is different than the workspace name and where a folder called integration already exists.

This PR also allows bwoken to work even when there are spaces in the project path.

@alecgorge
Copy link
Contributor Author

I noticed that Travis is now failing. I will fix this.

@listrophy
Copy link
Contributor

@alecgorge Thanks for writing this up! I've got some suggestions/modifications that I'll provide in the next day or so.

@RestlessThinker
Copy link

When is this going to get accepted? I ran into the same exact issue and was going to fork this repo to make the same exact change until I saw this.

@jonathanpenn
Copy link

Apparently, bwoken is using an older version of my script. You might want to look into the latest version that I've concocted here:

https://gist.github.com/jonathanpenn/1402258

@listrophy
Copy link
Contributor

The functionality in this PR was pulled into #43, which is now on master and released as 2.0.0.beta.2.

@listrophy listrophy closed this Nov 1, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants