mdn / content

The content behind MDN Web Docs
https://developer.mozilla.org
Other
9.24k stars 22.49k forks source link

Cross browser testing - updateJob() in Sauce Labs test details #32401

Open hamishwillee opened 9 months ago

hamishwillee commented 9 months ago

MDN URL

https://developer.mozilla.org/en-US/docs/Learn/Tools_and_testing/Cross_browser_testing/Your_own_automation_environment#filling_in_sauce_labs_test_details_programmatically

What specific section or headline is this issue about?

Filling in Sauce Labs test details programmatically

What information was incorrect, unhelpful, or incomplete?

From https://github.com/mdn/content/pull/31092#issuecomment-1947640396

sauce_google_test.js example in the "Filling in Sauce Labs test details programmatically" section, but it looks like it's missing the username in the saucelabs' updateJob() method call and the driver.sessionID is undefined.

What did you expect to see?

updateJob() with correct details.

Definition for driver.sessionID

Do you have any supporting links, references, or citations?

There is a bit more context on where this report came from here: https://github.com/mdn/content/pull/31092#issuecomment-1960505332

Do you have anything more you want to share?

@ShamarYarde couldn't fill this in/test this because they don't have a subscription to Sauce Labs. So that would be needed to complete this and verify it.

I'm not particularly familiar with this area. Creating this so task does not disappear

bsmth commented 7 months ago

Thanks for recording this. I'm removing needs triage because we're expecting a content update for this. Does the following perf example help for Selenium tests?

https://github.com/saucelabs/performance-js-examples/blob/main/Selenium-Mocha/tests/performance.js

hamishwillee commented 7 months ago

@bsmth I think the problem is that you need a license to do anything with Selenium. The original author ran out of free minutes, and hence can't progress.