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

Update docs to avoid deprecated config options #208

Open
garex opened this issue Mar 20, 2016 · 3 comments
Open

Update docs to avoid deprecated config options #208

garex opened this issue Mar 20, 2016 · 3 comments

Comments

@garex
Copy link

garex commented Mar 20, 2016

As I understand video-upload-on-pass: false is extra-deprecated and now we should use recordVideo: false.

It would be good to update all docs and examples with actual config params.

@eugenet8k
Copy link

I got a problem recently, not sure if it's related to this issue. Basically when I set:

'saucelabs-mocha'
  all: 
    options: 
      recordVideo: false 
      videoUploadOnPass: false 
      recordScreenshots: false 

It seems these are being ignored, and saucelabs still record video. I created a ticket on saucelabs for that and they say:

You're right, it looks like we're not being sent those config values by your tests at all.

We're not able to provide more then cursory support for external libraries and tools but I think I might have an idea what's happening here anyway; Assuming you're using https://github.com/axemclion/grunt-saucelabs it looks like you need to add a sauceConfig key to Options, setting it to a map of the additional options to pass, eg

options: {
username: # and so on
sauceConfig: {
'recordVideo': true,
'recordScreenshots': false
}
}

Give that a try, and if it doesn't work, I'd suggest raising a bug against the tool you're using and asking the author for assistance.

Therefore I am wondering, where I actually should define this recordVideo param? Thanks for all help.

@garex
Copy link
Author

garex commented Apr 2, 2016

@eugenet8k
Copy link

@garex thanks. that worked well!

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

No branches or pull requests

2 participants