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 SauceLabs video config options to default ini #47

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

carolmirakove
Copy link

SauceLabs support, in their tips for speedy tests (http://support.saucelabs.com/entries/23388772-Tips-for-lean-speedy-tests), advises that users consider disabling video options. Here we add both configs to the saunter.ini.default for users to easily toggle.

…celabs.com/entries/23388772-Tips-for-lean-speedy-tests), advises that users consider disabling video options. Here we add both configs to the saunter.ini.default for users to easily toggle.
@carolmirakove
Copy link
Author

To note: I added these configs to testcase/webdriver.py but not remotecontrol.py, mostly because I never work with RC and don't have a great way to test it. but, I'm guessing it would just be this:
https://gist.github.com/carolmirakove/5797052
and I can make those changes if yes.

@adamgoucher
Copy link
Member

I wouldn't worry about that. Over the next couple weeks the great
yamlfication of the configs will be done (starting this weekend) and
i'll be sure to add these in too.

To note: I added these configs to testcase/webdriver.py but not
remotecontrol.py, mostly because I never work with RC and don't have a
great way to test it. but, I'm guessing it would just be this:
https://gist.github.com/carolmirakove/5797052
and I can make those changes if yes.


Reply to this email directly or view it on GitHub
#47 (comment).

@carolmirakove
Copy link
Author

sounds good, thanks Adam.

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.

2 participants