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

Fixes the sauce labs tests by bumping karma-sauce-launcher #3813

Merged
merged 1 commit into from May 27, 2021

Conversation

@anglinb
Copy link
Contributor

@anglinb anglinb commented May 26, 2021

I was unable to run the sauce labs tests on master so I tracked down some discussion here karma-runner/karma-sauce-launcher#230 (comment) and after testing locally, I believe this should make the sauce labs tests pass. Here are the errors I was seeing

25 05 2021 21:51:40.455:ERROR [SaucelabsLauncher]: Could not quit the Saucelabs selenium connection. Failure message:
25 05 2021 21:51:40.455:ERROR [SaucelabsLauncher]: TypeError: Cannot read property 'deleteSession' of undefined
    at SaucelabsLauncher.<anonymous> (/Users/brian/staffbar/axios/node_modules/karma-sauce-launcher/dist/launcher/launcher.js:149:26)
    at Generator.next (<anonymous>)
    at /Users/brian/staffbar/axios/node_modules/karma-sauce-launcher/dist/launcher/launcher.js:8:71
    at new Promise (<anonymous>)
    at __awaiter (/Users/brian/staffbar/axios/node_modules/karma-sauce-launcher/dist/launcher/launcher.js:4:12)
    at SaucelabsLauncher.<anonymous> (/Users/brian/staffbar/axios/node_modules/karma-sauce-launcher/dist/launcher/launcher.js:141:31)
    at SaucelabsLauncher.emit (node:events:365:28)
    at SaucelabsLauncher.emit (node:domain:470:12)
    at SaucelabsLauncher.emitAsync (/Users/brian/staffbar/axios/node_modules/karma/lib/events.js:52:10)
    at SaucelabsLauncher.BaseLauncher.kill (/Users/brian/staffbar/axios/node_modules/karma/lib/launchers/base.js:61:27)
25 05 2021 21:51:40.455:ERROR [SaucelabsLauncher]: Could not quit the Saucelabs selenium connection. Failure message:
25 05 2021 21:51:40.455:ERROR [SaucelabsLauncher]: TypeError: Cannot read property 'deleteSession' of undefined

Screen Shot 2021-05-25 at 11 22 00 PM

Copy link
Member

@jasonsaayman jasonsaayman left a comment

Cool thanks lets see if this helps 👍

Loading

@jasonsaayman jasonsaayman merged commit e9965bf into axios:master May 27, 2021
1 check passed
Loading
@anglinb
Copy link
Contributor Author

@anglinb anglinb commented May 27, 2021

Well at least we have a new error 🙃 I pinged someone from Sauce Labs on the thread I referenced, we'll see if they have any ideas.

Here's where it failed 👇

2021-05-27T05:54:16.769Z ERROR webdriver: Request failed with status 500 due to Error: Sauce could not start your job. For more information on what happened, please visit https://app.saucelabs.com/tests/90f27026f00a4ae496f5ff7a435e8f28
2021-05-27T05:54:16.771Z ERROR webdriver: Error: Sauce could not start your job. For more information on what happened, please visit https://app.saucelabs.com/tests/90f27026f00a4ae496f5ff7a435e8f28
    at Object.getErrorFromResponseBody (/home/travis/build/[secure]/[secure]/node_modules/webdriver/build/utils.js:94:12)
    at WebDriverRequest._request (/home/travis/build/[secure]/[secure]/node_modules/webdriver/build/request.js:134:31)
    at process._tickCallback (internal/process/next_tick.js:68:7)
27 05 2021 05:54:16.771:ERROR [SaucelabsLauncher]: Error: Failed to create session.
Sauce could not start your job. For more information on what happened, please visit https://app.saucelabs.com/tests/90f27026f00a4ae496f5ff7a435e8f28
    at Object.startWebDriverSession (/home/travis/build/[secure]/[secure]/node_modules/webdriver/build/utils.js:34:15)
    at process._tickCallback (internal/process/next_tick.js:68:7)

Maybe a re-run might just make it work? Since it was a 500 response seems like a problem on their end (that could be transient 🤞 ).

Loading

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants